Merge pull request #27 from ghermeto/master

Restify middleware: calling next on success

Author: theganyo

lib/restify_middleware.js

@@ -28,6 +28,7 @@ function Middleware(runner) {
             res.statusCode = 404;
             res.end('Not found');
           }
+          next();
         });
       });
     });

package.json

@@ -44,6 +44,7 @@
     "mocha-lcov-reporter": "^1.0.0",
     "restify": "^4.0.3",
     "should": "^7.1.0",
+    "sinon": "^1.17.2",
     "supertest": "^1.1.0"
   },
   "scripts": {

test/assets/project/api/mocks/hello_world.js

@@ -4,6 +4,7 @@ module.exports = {
   hello_mock: hello_mock
 };
 
-function hello_mock(req, res) {
+function hello_mock(req, res, next) {
   res.json({ message: 'mocking from the controller!'});
+  next();
 }

test/lib/restify_middleware.js

@@ -4,6 +4,7 @@ var should = require('should');
 var request = require('supertest');
 var path = require('path');
 var _ = require('lodash');
+var sinon = require('sinon');
 
 var SwaggerRunner = require('../..');
 
@@ -34,9 +35,25 @@ describe('restify_middleware', function() {
   });
 
   describe('mock', function() {
+    var afterEvent;
 
     before(function(done) {
       createServer.call(this, MOCK_CONFIG, done);
+      afterEvent = sinon.spy();
+      this.app.on('after', afterEvent);
+    });
+
+    describe('after event', function () {
+      it('should been emitted', function (done) {
+        request(this.app)
+          .get('/hello')
+          .set('Accept', 'application/json')
+          .expect(200)
+          .end(function(err, res) {
+            sinon.assert.called(afterEvent);
+            done();
+          });
+      });
     });
 
     after(function(done) {