Add fallback helper name for better support "as" option

1602 · Jan 29, 2012 · 08cb1fb · 08cb1fb
1 parent c2fdf6b
commit 08cb1fb
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 3 deletions.
diff --git a/lib/railway_routes.js b/lib/railway_routes.js
@@ -103,20 +103,20 @@ Map.prototype.root = function (handler, middleware, options) {
             action: action
         });
 
-        this.addPath(path, action);
+        this.addPath(path, action, options.as);
 
         this.app[method].apply(this.app, args);
     };
 });
 
-Map.prototype.addPath = function (templatePath, action) {
+Map.prototype.addPath = function (templatePath, action, fallbackHelperName) {
     if (templatePath instanceof RegExp) {
         // TODO: think about adding to `path_to` routes by reg ex
         return;
     }
     var paramsLength = templatePath.match(/\/:/g);
     paramsLength = paramsLength === null ? 0 : paramsLength.length;
-    var helperName = this.urlHelperName(templatePath, action);
+    var helperName = this.urlHelperName(templatePath, action) || fallbackHelperName;
 
     // already defined? not need to redefine
     if (this.pathTo[helperName]) return;

diff --git a/test/railway_routes_test.js b/test/railway_routes_test.js
@@ -135,3 +135,14 @@ it('should handle root url', function (test) {
     test.done();
 });
 
+it('should allow to specify url helper name', function (test) {
+    var paths = [];
+    var map = new routes.Map(fakeApp(paths), fakeBridge());
+    map.get('/p/:id', 'posts#show', {as: 'post'});
+    test.deepEqual(paths, [
+        [ 'GET', '/p/:id', 'posts#show' ]
+    ]);
+    test.equal(map.pathTo.post(1), '/p/1');
+    test.done();
+});
+