Skip to content

Commit f226b37

Browse files
committed
fix: pr feedback
1 parent 5ddd161 commit f226b37

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

lib/oneflow.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ function OneFlowClient(baseUrl, token, secret, options = {}) {
2121
const headers = makeHeaders(url, method, options);
2222
const axiosRetry = options['axios-retry'] || {};
2323
return client
24-
.request({ method, url, data: data || {}, headers, 'axios-retry': axiosRetry })
24+
.request({ method, url, data: data === null ? {} : data, headers, 'axios-retry': axiosRetry })
2525
.then(res => res.data);
2626
}
2727

test/oneflow.test.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,8 @@ describe('Oneflow', function () {
131131
const result = await sdk.request('get', '/order', undefined);
132132

133133
result.method.should.be.equal('get');
134-
result.data.should.be.eql({});
134+
console.log(result.data);
135+
(result.data === undefined).should.equal(true);
135136
});
136137

137138
it('should perform an http request with data=""', async () => {
@@ -140,7 +141,7 @@ describe('Oneflow', function () {
140141
const result = await sdk.request('get', '/order', '');
141142

142143
result.method.should.be.equal('get');
143-
result.data.should.be.eql({});
144+
result.data.should.be.equal('');
144145
});
145146

146147
it('should support service user requests', async () => {

0 commit comments

Comments
 (0)