@@ -38,7 +38,6 @@ module RoleSystem # which inherits from an AdminController that specifies a default role. def skip_role_system self.skipping_role_system = true - @roles_checked = true end # These actions don't require roles at all. This method is helpful for instances @@ -82,7 +81,7 @@ module RoleSystem private def check_roles - return true if self.skipping_role_system + return true if self.skipping_role_system == true return true if no_roles_required_for(binding) raise RoleSystem::RoleRequired unless @role_player user = self.send(@role_player) lib/role_system.rb @@ -223,7 +223,7 @@ describe RoleSystem, "a controller without authenticated system", :type => :cont it "should just return an access denied header when authenticated_system is not available" do login_as('editor') get :index - response.headers['Status'].should == "401 Unauthorized" + response.status.should == "401 Unauthorized" end end spec/role_system_spec.rb 8820e5059a1d52b82581272f5ad98b0393eeda55 Mark Daggett heavysixer@heavysixer.local http://github.com/heavysixer/rolesystem/commit/ac4a453d88636c252f2c3d397736de83a732969c ac4a453d88636c252f2c3d397736de83a732969c 2009-06-25T09:17:19-07:00 2009-06-25T09:17:19-07:00 Explicitly match the skipping_role_system variable and fix the test that checked for return status because Rack doesn't allow you access to the .header['Status']. 708f894a2ca3515eee015b2bcc0d5d2e70376cb1 Mark Daggett heavysixer@heavysixer.local