feat(router): add support for nested controller before/after model hooks

src/router.js

@@ -198,6 +198,14 @@ class Router {
       if (actionHooks && actionHooks.after) {
         handlers.push(actionHooks.after.bind(controller));
       }
+    } else if (controller.beforeModel || controller.afterModel) {
+      if (controller.beforeModel) {
+        handlers.unshift(controller.beforeModel.bind(controller));
+      }
+
+      if (controller.afterModel) {
+        handlers.unshift(controller.afterModel.bind(controller));
+      }
     }
 
     return handlers;

test/fixtures/controllers/comments/create.js

@@ -3,8 +3,18 @@
 import Controller from "../../../../src/controller";
 
 export default class CreateComment extends Controller {
+  beforeModel(req, res, next) {
+    this.beforeModel.called = true;
+    next();
+  }
+
   model(req, res, next) {
     res.send(201);
     next();
   }
+
+  afterModel(req, res, next) {
+    this.afterModel.called = true;
+    next();
+  }
 }

test/router_tests.js

@@ -286,6 +286,36 @@ describe("Router", function () {
           done();
         });
     });
+
+    it("binds a beforeModel hook", function (done) {
+      router.resource("comment");
+      client = supertest(app);
+      controller = registry.lookup("controller:comment.create");
+
+      client.post("/comments").send({
+        name: "foo"
+      }).end(function (err, res) {
+        if (err) { return done(err); }
+
+        expect(controller.beforeModel.called).to.be.true;
+        done();
+      });
+    });
+
+    it("binds an afterModel hook", function (done) {
+      router.resource("comment");
+      client = supertest(app);
+      controller = registry.lookup("controller:comment.create");
+
+      client.post("/comments").send({
+        name: "foo"
+      }).end(function (err, res) {
+        if (err) { return done(err); }
+
+        expect(controller.afterModel.called).to.be.true;
+        done();
+      });
+    });
   });
 
   describe("#route", function () {