use prefix as base for '/' routes like explorer or non-strict hooks

flatiron · Sep 24, 2012 · d4a6fb1 · d4a6fb1
1 parent 5a7db39
commit d4a6fb1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/lib/restful.js b/lib/restful.js
@@ -169,7 +169,7 @@ exports.extendRouter = function (router, resources, options, respond) {
     //
     // Bind GET / to a generic explorer view of routing map ( using `director-explorer` )
     //
-    router.get('/', function () {
+    router.get(options.prefix + '/', function () {
       var rsp = '';
       //
       // Remark: Output the basic routing map for every resource using https://github.com/flatiron/director-reflector
@@ -178,7 +178,7 @@ exports.extendRouter = function (router, resources, options, respond) {
       this.res.end(rsp);
     });
   } else {
-    router.get('/', function (_id) {
+    router.get(options.prefix + '/', function (_id) {
       var res = this.res,
           req = this.req;
       if (!options.strict) {

diff --git a/test/macros/index.js b/test/macros/index.js
@@ -36,6 +36,8 @@ macros.resourceTest = function (name, options, context) {
   }
 
   return context
+    .get(prefix)
+      .expect(200)
     .get(prefix + '/creature')
       .expect(200)
     .next()
@@ -180,6 +182,8 @@ macros.nonStrictResourceTest = function (options, context) {
   var prefix = options.prefix || '';
 
   return context
+    .get(prefix)
+      .expect(200)
     .get(prefix + '/creature')
       .expect(200)
   .next()
@@ -532,4 +536,4 @@ macros.nonStrictTypeValidationTest = function (options, context) {
   .next()
     .post('/user',   { email: "marak.squires@gmail.com", age: 50 })
       .expect(201)
-};
+};