.expect(status[, fn]) expect undefined response body !?! about supertest

[cjroebuck]

Yep. Just came across this too. I wrote these tests to demonstrate the issue - first one is taken from the readme:

it('should assert status only 1', function(done){ 
  var app = express();

  app.get('/user', function(req, res){
    res.send(201, { name: 'tobi' });
  });

  request(app)
    .get('/user')
    .expect('Content-Type', /json/)
    .expect('Content-Length', '20')
    .expect(201)
    .end(function(err, res){
      if (err) throw err;
    });
})

it('should assert the status only 2', function(done){
  var app = express();

  app.set('json spaces', 0);

  app.get('/', function(req, res){
    res.send({ foo: 'bar' });
  });

  request(app)
  .get('/')
  .expect(200)
  .end(function(err, res){
    err.message.should.equal('expected { foo: \'baz\' } response body, got { foo: \'bar\' }');

    request(app)
    .get('/')
    .expect({ foo: 'bar' })
    .end(done);
  });
})


 1) request(app) .expect(body[, fn]) should assert status only 1:
  Error: Error: expected undefined response body, got '{\n  "name": "tobi"\n}' (undefined:undefined)
   at process.uncaught (/Users/cjroebuck/Dev/supertest/node_modules/mocha/lib/runner.js:492:19)
   at process.EventEmitter.emit (events.js:126:20)

2) request(app) .expect(body[, fn]) should assert the status only 2:
  Error: AssertionError: expected 'expected undefined response body, got \'{"foo":"bar"}\'' to equal 'expected { foo:    \'baz\' } response body, got { foo: \'bar\' }' (undefined:undefined)
   at process.uncaught (/Users/cjroebuck/Dev/supertest/node_modules/mocha/lib/runner.js:492:19)
   at process.EventEmitter.emit (events.js:126:20)

It must be the change to support multiple body assertions that has caused this bug. The workaround for now is to either don't use expect and do your assertion in the end function callback. So instead of .expect(200) it would be .end(function(err,res) { res.status.should.equal(200) }, or if you do use expect.. you need to make sure you specify a body aswell as just a status..

[tj]

a pull-requested test would be more helpful, then I can take a look at fixing it

[fengmk2]

I reappear this bug and also fixed it on #41

[ragulka]

When will this be put to npm? Almost all my tests are failing at the moment

[tj]

0.5.1