Expect Status with `done()` isn't working

.expect(404, done) doesn’t seem to be failing if the response is actually 200. Not sure if I’m doing something wrong, or if I’ve found a bug?

http://stackoverflow.com/questions/37729206/supertest-expect-not-asserting-status-codes-correctly

[bradwbradw]

I’m running supertest 2.0.0 and copying the exact code pasted here: https://github.com/visionmedia/supertest/issues/345#issuecomment-229496425

Both tests pass, so it seems like a breaking change was introduced in 2.0.0, maybe i’m missing something in the docs, but i’m not sure what to do to test the status code

so to work around this i add an end function:

      .end(function (err, res) {
        err ? done.fail(err) : done();
      });

[rpadovani]

I have the same problem:

This doesn’t fail:

request(app)
				.get('/labels')
				.expect(202, [validLabel, anotherValidLabel], done);

while this fails:

request(app)
				.get('/labels')
				.expect(202, [validLabel, anotherValidLabel])
				.end(err => err ? done.fail(err) : done());