diff --git a/src/server/router/plural.js b/src/server/router/plural.js index 0c8b117..42a5e03 100644 --- a/src/server/router/plural.js +++ b/src/server/router/plural.js @@ -253,6 +253,7 @@ module.exports = (db, name) => { function update (req, res, next) { const id = req.params.id let chain = db.get(name) + console.log(req.body) chain = req.method === 'PATCH' ? chain.updateById(id, req.body) diff --git a/test/server/plural.js b/test/server/plural.js index fa0b751..d999330 100644 --- a/test/server/plural.js +++ b/test/server/plural.js @@ -551,10 +551,11 @@ describe('Server', () => { }) describe('PUT /:resource/:id', () => { - it('should respond with json and replace resource', (done) => { + it.only('should respond with json and replace resource', (done) => { var post = {id: 1, booleanValue: true, integerValue: 1} request(server) .put('/posts/1') + .set('Accept', 'application/json') // body property omitted to test that the resource is replaced .send(post) .expect('Content-Type', /json/) @@ -562,8 +563,11 @@ describe('Server', () => { .expect(200) .end((err, res) => { if (err) return done(err) + // TODO find a "supertest" way to test this + // https://github.com/typicode/json-server/issues/396 + assert.deepStrictEqual(res.body, post) // assert it was created in database too - assert.deepEqual(db.posts[0], post) + assert.deepStrictEqual(db.posts[0], post) done() }) }) @@ -588,8 +592,9 @@ describe('Server', () => { .expect(200) .end((err, res) => { if (err) return done(err) + assert.deepStrictEqual(res.body, post) // assert it was created in database too - assert.deepEqual(db.posts[0], {id: 1, body: 'bar'}) + assert.deepStrictEqual(db.posts[0], {id: 1, body: 'bar'}) done() }) })