Merge pull request #735 from benjosantony/master

Add on to the mitigation in https://github.com/visionmedia/supertest/…

Author: niftylettuce

Diff for: lib/test.js

@@ -128,7 +128,7 @@ Test.prototype.expect = function(a, b, c) {
   }
 
   // multiple statuses
-  if (Array.isArray(a) && a.every(val => typeof val === 'number')) {
+  if (Array.isArray(a) && a.length > 0 && a.every(val => typeof val === 'number')) {
     this._asserts.push(wrapAssertFn(this._assertStatusArray.bind(this, a)));
     return this;
   }

Diff for: test/supertest.js

@@ -584,6 +584,17 @@ describe('request(app)', function () {
         .expect(['a', { id: 1 }], done);
     });
 
+    it('should support empty array responses', function (done) {
+      const app = express();
+      app.get('/', function (req, res) {
+        res.status(200).json([]);
+      });
+
+      request(app)
+        .get('/')
+        .expect([], done);
+    });
+
     it('should support regular expressions', function (done) {
       const app = express();