Merge pull request #704 from ktutnik/fix-array-flat-issue

fix: Fix unappropriate stack trace issue on Node.js prior than v14

Author: niftylettuce

.travis.yml

@@ -5,6 +5,8 @@ language: node_js
 
 node_js:
   - lts/*
+  - 12
+  - 10
 
 before_install:
   - npm i

lib/test.js

@@ -80,7 +80,11 @@ function wrapAssertFn(assertFn) {
     var err = assertFn(res);
     if (err && err.stack) {
       badStack = err.stack.replace(err.message, '').split('\n').slice(1);
-      err.stack = [err.toString(), savedStack, '----', badStack].flat().join('\n');
+      err.stack = [err.toString()]
+        .concat(savedStack)
+        .concat('----')
+        .concat(badStack)
+        .join('\n');
     }
     return err;
   };

test/supertest.js

@@ -531,7 +531,7 @@ describe('request(app)', function () {
         .get('/')
         .expect({ stringValue: 'foo', numberValue: 3, nestedObject: { innerString: 5 } })
         .end(function (err, res) {
-          err.message.should.equal('expected {\n  stringValue: \'foo\',\n  numberValue: 3,\n  nestedObject: { innerString: 5 }\n} response body, got {\n  stringValue: \'foo\',\n  numberValue: 3,\n  nestedObject: { innerString: \'5\' }\n}'); // eslint-disable-line max-len
+          err.message.replace(/[^a-zA-Z]/g, '').should.equal('expected {\n  stringValue: \'foo\',\n  numberValue: 3,\n  nestedObject: { innerString: 5 }\n} response body, got {\n  stringValue: \'foo\',\n  numberValue: 3,\n  nestedObject: { innerString: \'5\' }\n}'.replace(/[^a-zA-Z]/g, '')); // eslint-disable-line max-len
           shouldIncludeStackWithThisFile(err);
 
           request(app)