fixing accounts controller test to meet new should_deny_access_on int…

…erface, changing routes to meet groupies experience.
thoughtbot · Apr 2, 2009 · 0e8d1a6 · 0e8d1a6
1 parent 6da73ca
commit 0e8d1a6
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 4 deletions.
diff --git a/config/routes.rb b/config/routes.rb
@@ -7,7 +7,7 @@
     :controller => 'clearance/sessions',
     :only => [:new, :create, :destroy]
 
-  map.resources :users, :controller => 'clearance/users' do |users|
+  map.resources :users do |users|
     users.resource :password,
       :controller => 'clearance/passwords',
       :only => [:create, :edit, :update]

diff --git a/shoulda_macros/clearance.rb b/shoulda_macros/clearance.rb
@@ -34,7 +34,7 @@ def should_not_be_signed_in
     # Examples:
     #   should_deny_access_on :get, :index, :flash => /not authorized/i
     #   should_deny_access_on :get, :show, :id => '1'
-    def self.should_deny_access_on(http_method, action, opts = {})
+    def should_deny_access_on(http_method, action, opts = {})
       flash_message = opts.delete(:flash)
       context "on #{http_method} to #{action}" do
         setup do

diff --git a/test/rails_root/test/functional/accounts_controller_test.rb b/test/rails_root/test/functional/accounts_controller_test.rb
@@ -1,9 +1,10 @@
 require 'test_helper'
 
 class AccountsControllerTest < ActionController::TestCase
+
   public_context do
-    should_deny_access_on 'get :edit'
-    should_deny_access_on 'put :update'
+    should_deny_access_on :get, :edit
+    should_deny_access_on :put, :update
   end
 
   context "on POST to create" do