tests: reorganize json error tests

test/json.js

@@ -15,14 +15,6 @@ describe('bodyParser.json()', function () {
     .expect(200, '{"user":"tobi"}', done)
   })
 
-  it('should fail gracefully', function (done) {
-    request(createServer())
-    .post('/')
-    .set('Content-Type', 'application/json')
-    .send('{"user"')
-    .expect(400, parseError('{"user"'), done)
-  })
-
   it('should handle Content-Length: 0', function (done) {
     request(createServer())
     .get('/')
@@ -47,14 +39,6 @@ describe('bodyParser.json()', function () {
     .expect(200, '{}', done)
   })
 
-  it('should 400 on malformed JSON', function (done) {
-    request(createServer())
-    .post('/')
-    .set('Content-Type', 'application/json')
-    .send('{:')
-    .expect(400, parseError('{:'), done)
-  })
-
   it('should 400 when invalid content-length', function (done) {
     var jsonParser = bodyParser.json()
     var server = createServer(function (req, res, next) {
@@ -85,6 +69,28 @@ describe('bodyParser.json()', function () {
     .expect(200, '{"user":"tobi"}', done)
   })
 
+  describe('when JSON is invalid', function () {
+    before(function () {
+      this.server = createServer()
+    })
+
+    it('should 400 for bad token', function (done) {
+      request(this.server)
+      .post('/')
+      .set('Content-Type', 'application/json')
+      .send('{:')
+      .expect(400, parseError('{:'), done)
+    })
+
+    it('should 400 for incomplete', function (done) {
+      request(this.server)
+      .post('/')
+      .set('Content-Type', 'application/json')
+      .send('{"user"')
+      .expect(400, parseError('{"user"'), done)
+    })
+  })
+
   describe('with limit option', function () {
     it('should 413 when over limit with Content-Length', function (done) {
       var buf = Buffer.alloc(1024, '.')