fix: fixed the broken testcases (#1676)

ladjs · Jan 22, 2022 · 1fd4f90 · 1fd4f90
1 parent 6a101c2
commit 1fd4f90
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,4 @@
+.vscode
 build
 lib-cov
 coverage.html

diff --git a/test/basic.js b/test/basic.js
@@ -100,7 +100,7 @@ describe('request', function () {
 
       return Promise.all([request_, request_, request_]).then((results) => {
         for (const item of results) {
-          assert.equal(
+          assert.deepEqual(
             item.body,
             results[0].body,
             'It should keep returning the same result after being called once'

diff --git a/test/node/multipart.js b/test/node/multipart.js
@@ -114,7 +114,7 @@ describe('Multipart', () => {
 
       it('should report ENOENT via the callback', (done) => {
         request
-          .post('http://127.0.0.1:1') // nobody is listening there
+          .post(`${base}/echo`)
           .attach('name', 'file-does-not-exist')
           .end((error, res) => {
             assert.ok(Boolean(error), 'Request should have failed');
@@ -125,7 +125,7 @@ describe('Multipart', () => {
 
       it('should report ENOENT via Promise', () => {
         return request
-          .post(`http://127.0.0.1:1`) // nobody is listening there
+          .post(`${base}/echo`)
           .attach('name', 'file-does-not-exist')
           .then(
             (res) => assert.fail('Request should have failed'),