Add server response log when status code error

ladjs · Feb 1, 2021 · d1c2c2a · d1c2c2a
1 parent 1bb8c66
commit d1c2c2a
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/.eslintrc b/.eslintrc
@@ -18,7 +18,7 @@
     "no-underscore-dangle": [0],
 
     // IMHO, more sensible overrides to existing airbnb error definitions
-    "max-len": [2, 100, 4, {"ignoreComments": true, "ignoreUrls": true}],
+    "max-len": [2, 115, 4, {"ignoreComments": true, "ignoreUrls": true}],
     "no-unused-expressions": [2, { "allowShortCircuit": true, "allowTernary": true }]
   }
 }
diff --git a/lib/test.js b/lib/test.js
@@ -293,7 +293,9 @@ Test.prototype._assertStatus = function(status, res) {
   if (res.status !== status) {
     a = http.STATUS_CODES[status];
     b = http.STATUS_CODES[res.status];
-    return new Error('expected ' + status + ' "' + a + '", got ' + res.status + ' "' + b + '"');
+    return new Error(
+      `expected ${status} "${a}", got ${res.status} "${b}", with response: ${JSON.stringify(res, null, 2)}`
+    );
   }
 };
 

diff --git a/test/supertest.js b/test/supertest.js
@@ -363,7 +363,9 @@ describe('request(app)', function () {
         .get('/')
         .expect(404)
         .end(function (err, res) {
-          err.message.should.equal('expected 404 "Not Found", got 200 "OK"');
+          err.message.should.equal(
+            `expected 404 "Not Found", got 200 "OK", with response: ${JSON.stringify(res, null, 2)}`
+          );
           shouldIncludeStackWithThisFile(err);
           done();
         });
@@ -466,7 +468,9 @@ describe('request(app)', function () {
         .expect(200)
         .expect('hey')
         .end(function (err, res) {
-          err.message.should.equal('expected 200 "OK", got 500 "Internal Server Error"');
+          err.message.should.equal(
+            `expected 200 "OK", got 500 "Internal Server Error", with response: ${JSON.stringify(res, null, 2)}`
+          );
           shouldIncludeStackWithThisFile(err);
           done();
         });
@@ -1038,7 +1042,9 @@ describe('assert ordering by call order', function () {
       .expect(200)
       .expect('hey')
       .end(function (err, res) {
-        err.message.should.equal('expected 200 "OK", got 500 "Internal Server Error"');
+        err.message.should.equal(
+          `expected 200 "OK", got 500 "Internal Server Error", with response: ${JSON.stringify(res, null, 2)}`
+        );
         shouldIncludeStackWithThisFile(err);
         done();
       });