From a0a09e1e4b757d734368d5e976d33fec320e3cd4 Mon Sep 17 00:00:00 2001
From: Jason T Johnson <git@postpostmodern.com>
Date: Mon, 26 May 2008 16:41:02 -0500
Subject: [PATCH] A few more generator and README fixes

* Plural controller name in named routes
* Named routes in session controller spec instead of restful routes
---
 README                                             |  4 ++--
 .../spec/controllers/sessions_controller_spec.rb   | 14 +++++++-------
 .../spec/controllers/users_controller_spec.rb      |  4 ++--
 3 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/README b/README
index ebac07c..748670e 100644
--- a/README
+++ b/README
@@ -147,8 +147,8 @@ alter to suit. There are additional security minutae in @notes/README-Tradeoffs@
 * Add these familiar login URLs to your @config/routes.rb@ if you like:
 
      map.signup  '/signup', :controller => 'users',   :action => 'new' @
-     map.login   '/login',  :controller => 'session', :action => 'new' @
-     map.logout  '/logout', :controller => 'session', :action => 'destroy' @
+     map.login   '/login',  :controller => 'sessions', :action => 'new' @
+     map.logout  '/logout', :controller => 'sessions', :action => 'destroy' @
     
 * With @--include-activation@, also add to your @config/routes.rb@:
   
diff --git a/generators/authenticated/templates/spec/controllers/sessions_controller_spec.rb b/generators/authenticated/templates/spec/controllers/sessions_controller_spec.rb
index 1456814..9fd2096 100644
--- a/generators/authenticated/templates/spec/controllers/sessions_controller_spec.rb
+++ b/generators/authenticated/templates/spec/controllers/sessions_controller_spec.rb
@@ -101,26 +101,26 @@ def do_destroy
 
 describe <%= controller_class_name %>Controller do
   describe "route generation" do
-    it "should route {:controller => '<%= controller_file_path %>', :action => 'new'} to /<%= controller_file_path %>/new" do
-      route_for(:controller => '<%= controller_file_path %>', :action => 'new').should == "/<%= controller_file_path %>/new"
+    it "should route {:controller => '<%= controller_file_path %>', :action => 'new'} to /login" do
+      route_for(:controller => '<%= controller_file_path %>', :action => 'new').should == "/login"
     end
     it "should route {:controller => '<%= controller_file_path %>', :action => 'create'} to /<%= controller_file_path %>" do
       route_for(:controller => '<%= controller_file_path %>', :action => 'create').should == "/<%= controller_file_path %>"
     end
-    it "should route {:controller => '<%= controller_file_path %>', :action => 'destroy'} to /<%= controller_file_path %>" do
-      route_for(:controller => '<%= controller_file_path %>', :action => 'destroy').should == "/<%= controller_file_path %>/destroy"
+    it "should route {:controller => '<%= controller_file_path %>', :action => 'destroy'} to /logout" do
+      route_for(:controller => '<%= controller_file_path %>', :action => 'destroy').should == "/logout"
     end
   end
   
   describe "route recognition" do
-    it "should generate params {:controller => '<%= controller_file_path %>', :action => 'new'} from GET /<%= controller_file_path %>" do
-      params_from(:get, '/<%= controller_file_path %>/new').should == {:controller => '<%= controller_file_path %>', :action => 'new'}
+    it "should generate params {:controller => '<%= controller_file_path %>', :action => 'new'} from GET /login" do
+      params_from(:get, '/login').should == {:controller => '<%= controller_file_path %>', :action => 'new'}
     end
     it "should generate params {:controller => '<%= controller_file_path %>', :action => 'create'} from POST /<%= controller_file_path %>" do
       params_from(:post, '/<%= controller_file_path %>').should == {:controller => '<%= controller_file_path %>', :action => 'create'}
     end
     it "should generate params {:controller => '<%= controller_file_path %>', :action => 'destroy'} from DELETE /<%= controller_file_path %>" do
-      params_from(:delete, '/<%= controller_file_path %>').should == {:controller => '<%= controller_file_path %>', :action => 'destroy'}
+      params_from(:delete, '/logout').should == {:controller => '<%= controller_file_path %>', :action => 'destroy'}
     end
   end
   
diff --git a/generators/authenticated/templates/spec/controllers/users_controller_spec.rb b/generators/authenticated/templates/spec/controllers/users_controller_spec.rb
index 24ded1d..df0ae72 100644
--- a/generators/authenticated/templates/spec/controllers/users_controller_spec.rb
+++ b/generators/authenticated/templates/spec/controllers/users_controller_spec.rb
@@ -100,8 +100,8 @@ def create_<%= file_name %>(options = {})
       route_for(:controller => '<%= model_controller_file_path %>', :action => 'index').should == "/<%= model_controller_file_path %>"
     end
     
-    it "should route {:controller => '<%= model_controller_file_path %>', :action => 'new'} to /<%= model_controller_file_path %>/new" do
-      route_for(:controller => '<%= model_controller_file_path %>', :action => 'new').should == "/<%= model_controller_file_path %>/new"
+    it "should route {:controller => '<%= model_controller_file_path %>', :action => 'new'} to /signup" do
+      route_for(:controller => '<%= model_controller_file_path %>', :action => 'new').should == "/signup"
     end
     
     it "should route {:controller => '<%= model_controller_file_path %>', :action => 'create'} to /<%= model_controller_file_path %>" do