44

I am testing an express API with supertest.

I couldn't get multiple requests in a test case to work with supertest. Below is what i tried in a test case. But the test case seem to only execute the last call which is the HTTP GET.

it('should respond to GET with added items', function(done) {
    var agent = request(app);
    agent.post('/player').type('json').send({name:"Messi"});
    agent.post('/player').type('json').send({name:"Maradona"});
    agent.get('/player').set("Accept", "application/json")
        .expect(200)
        .end(function(err, res) {
            res.body.should.have.property('items').with.lengthOf(2);
            done();
    });
);

Anything i am missing here, or is there another way to chain http calls with superagent?

4 Answers4

67

Tried to put this in a comment above, formatting wasn't working out.

I'm using async, which is really standard and works really well.

it('should respond to only certain methods', function(done) {
    async.series([
        function(cb) { request(app).get('/').expect(404, cb); },
        function(cb) { request(app).get('/new').expect(200, cb); },
        function(cb) { request(app).post('/').send({prop1: 'new'}).expect(404, cb); },
        function(cb) { request(app).get('/0').expect(200, cb); },
        function(cb) { request(app).get('/0/edit').expect(404, cb); },
        function(cb) { request(app).put('/0').send({prop1: 'new value'}).expect(404, cb); },
        function(cb) { request(app).delete('/0').expect(404, cb); },
    ], done);
});
Tim
  • 978
  • 1
  • 7
  • 12
26

The calls are made asynchronous, so you need to use callback functions to chain them.

it('should respond to GET with added items', function(done) {
    var agent = request(app);
    agent.post('/player').type('json').send({name:"Messi"}).end(function(){
        agent.post('/player').type('json').send({name:"Maradona"}).end(function(){
            agent.get('/player')
                .set("Accept", "application/json")
                .expect(200)
                .end(function(err, res) {
                    res.body.should.have.property('items').with.lengthOf(2);
                    done();
                });
        });
    });
});
harun
  • 1,790
  • 17
  • 18
  • 9
    Have a look at [supertest-as-promised](https://www.npmjs.org/package/supertest-as-promised) to reduce the nesting of callbacks – Luke H Aug 06 '14 at 11:41
  • Is it possible to chain them without using `mocha` or `it` ? – gurvinder372 Mar 23 '16 at 06:59
  • 4
    supertest has native promise support since version [2.0.0](https://github.com/visionmedia/supertest/blob/master/History.md#200--2016-07-29) – ricca May 07 '17 at 07:50
  • I couldn't get this solution to work. However, I could get the "async" solution (by Tim) below to work. – user952342 Mar 23 '18 at 13:33
  • 2
    This answer is correct, however outdated, using async/await can give the same result with a much cleaner implementation – Mazen Elkashef Mar 08 '20 at 03:08
14

This can be most elegantly solved with promises, and there's a really useful library to use promises with supertest: https://www.npmjs.com/package/supertest-as-promised

Their example:

return request(app)
  .get("/user")
  .expect(200)
  .then(function (res) {
    return request(app)
      .post("/kittens")
      .send({ userId: res})
      .expect(201);
  })
  .then(function (res) {
    // ... 
  });
Phil
  • 160
  • 1
  • 5
  • Promises are a good option here. Just need to be weary of situations where you might need to recover from a 404 somewhere in that chain. – backdesk Aug 08 '16 at 13:22
  • 3
    supertest has native promise support since version [2.0.0](https://github.com/visionmedia/supertest/blob/master/History.md#200--2016-07-29) – ricca May 07 '17 at 07:52
9

I built upon Tim’s reply but used async.waterfall instead, to be able to do assert tests on the results (note: I use Tape here instead of Mocha):

test('Test the entire API', function (assert) {
    const app = require('../app/app');
    async.waterfall([
            (cb) => request(app).get('/api/accounts').expect(200, cb),
            (results, cb) => { assert.ok(results.body.length, 'Returned accounts list'); cb(null, results); },
            (results, cb) => { assert.ok(results.body[0].reference, 'account #0 has reference'); cb(null, results); },
            (results, cb) => request(app).get('/api/plans').expect(200, cb),
            (results, cb) => request(app).get('/api/services').expect(200, cb),
            (results, cb) => request(app).get('/api/users').expect(200, cb),
        ],
        (err, results) => {
            app.closeDatabase();
            assert.end();
        }
    );
});
Tom Söderlund
  • 3,972
  • 3
  • 34
  • 53