clearance 0.9.0.rc1 → 0.9.0.rc2

data/README.md

@@ -20,7 +20,7 @@ Fork away and create a [Github Issue](http://github.com/thoughtbot/clearance/iss
 Installation
 ------------
 
-Clearance is a Rails engine. The latest stable version works with versions of Rails greater than 3.  If you need to run on Rails 2.x, install Clearance Version 0.8.8.
+Clearance is a Rails engine. The latest stable version (0.8.8) works with versions of Rails 2.x.
 
 Install it as a gem however you like to install gems. Also, uninstall old versions:
 
@@ -38,6 +38,10 @@ This:
 * inserts Clearance::Authentication into your ApplicationController
 * created a migration that either creates a users table or adds only missing columns
 
+There is a release candidate which features Rails 3 support. To install this version:
+
+    gem install clearance --prerelease
+
 Usage
 -----
 

data/lib/clearance/engine.rb

@@ -3,6 +3,5 @@ require "rails"
 
 module Clearance
   class Engine < Rails::Engine
-    engine_name :clearance
   end
 end

data/shoulda_macros/clearance.rb

@@ -93,14 +93,10 @@ module Clearance
 
     def should_create_user_successfully
       warn "[DEPRECATION] should_create_user_successfully: not meant to be public, no longer used internally"
-      should_assign_to :user
+      should assign_to(:user)
       should_change 'User.count', :by => 1
 
-      should "send the confirmation email" do
-        assert_sent_email do |email|
-          email.subject =~ /account confirmation/i
-        end
-      end
+      should have_sent_email.with_subject(/account confirmation/i)
 
       should set_the_flash.to(/confirm/i)
       should_redirect_to_url_after_create

data/test/controllers/confirmations_controller_test.rb

@@ -89,8 +89,11 @@ class ConfirmationsControllerTest < ActionController::TestCase
 
     should set_the_flash.to(/already confirmed/i)
     should set_the_flash.to(/sign in/i)
-    should_not_be_signed_in
     should_redirect_to_url_already_confirmed
+
+    should "not be signed in" do
+      assert_nil cookies[:remember_token]
+    end
   end
 
   context "no users" do

data/test/controllers/passwords_controller_test.rb

@@ -4,8 +4,8 @@ class PasswordsControllerTest < ActionController::TestCase
 
   tests Clearance::PasswordsController
 
-  should_route :get, '/users/1/password/edit',
-    :controller => 'clearance/passwords', :action  => 'edit', :user_id => '1'
+  should route(:get, '/users/1/password/edit').
+           to(:controller => 'clearance/passwords', :action  => 'edit', :user_id => '1')
 
   context "a signed up user" do
     setup do
@@ -30,11 +30,7 @@ class PasswordsControllerTest < ActionController::TestCase
           assert_not_nil @user.reload.confirmation_token
         end
 
-        should "send the change your password email" do
-          assert_sent_email do |email|
-            email.subject =~ /change your password/i
-          end
-        end
+        should have_sent_email.with_subject(/change your password/i)
 
         should set_the_flash.to(/password/i)
         should_redirect_to_url_after_create
@@ -87,7 +83,6 @@ class PasswordsControllerTest < ActionController::TestCase
 
       should respond_with(:success)
       should render_template(:edit)
-      should_display_a_password_update_form
     end
 
     should_forbid "on GET to #edit with correct id but blank token" do
@@ -155,12 +150,13 @@ class PasswordsControllerTest < ActionController::TestCase
         assert_not_nil @user.confirmation_token
       end
 
-      should_not_be_signed_in
+      should "not be signed in" do
+        assert_nil cookies[:remember_token]
+      end
+
       should_not set_the_flash
       should respond_with(:success)
       should render_template(:edit)
-
-      should_display_a_password_update_form
     end
 
     should_forbid "on PUT to #update with id but no token" do

data/test/controllers/sessions_controller_test.rb

@@ -11,7 +11,6 @@ class SessionsControllerTest < ActionController::TestCase
     should respond_with(:success)
     should render_template(:new)
     should_not set_the_flash
-    should_display_a_sign_in_form
   end
 
   context "on POST to #create with unconfirmed credentials" do
@@ -144,10 +143,9 @@ class SessionsControllerTest < ActionController::TestCase
     should set_the_flash.to(/bad/i)
     should respond_with(:unauthorized)
     should render_template(:new)
-    should_not_be_signed_in
 
-    should 'not create the cookie' do
-      assert_nil cookies['remember_token']
+    should "not be signed in" do
+      assert_nil cookies[:remember_token]
     end
   end
 

data/test/controllers/users_controller_test.rb

@@ -15,8 +15,6 @@ class UsersControllerTest < ActionController::TestCase
       should respond_with(:success)
       should render_template(:new)
       should_not set_the_flash
-
-      should_display_a_sign_up_form
     end
 
     context "on GET to #new with email" do
@@ -37,24 +35,25 @@ class UsersControllerTest < ActionController::TestCase
         post :create, :user => user_attributes
       end
 
-      should_assign_to :user
+      should assign_to(:user)
 
       should "create a new user" do
         assert_equal @old_user_count + 1, User.count
       end
 
-      should "send the confirmation email" do
-        assert_sent_email do |email|
-          email.subject =~ /account confirmation/i
-        end
-      end
+      should have_sent_email.with_subject(/account confirmation/i)
 
       should set_the_flash.to(/confirm/i)
       should_redirect_to_url_after_create
     end
   end
 
-  signed_in_user_context do
+  context "A signed-in user" do
+    setup do
+      @user = Factory(:email_confirmed_user)
+      sign_in_as @user
+    end
+
     context "GET to new" do
       setup { get :new }
       should redirect_to("the home page") { root_url }

data/test/models/user_test.rb

@@ -10,7 +10,8 @@ class UserTest < ActiveSupport::TestCase
   # signing up
 
   context "When signing up" do
-    should_validate_presence_of :email, :password
+    should validate_presence_of(:email)
+    should validate_presence_of(:password)
     should allow_value("foo@example.com").for(:email)
     should_not allow_value("foo").for(:email)
     should_not allow_value("example.com").for(:email)
@@ -59,11 +60,7 @@ class UserTest < ActiveSupport::TestCase
       assert_equal "John.Doe@example.com", user.email
     end
 
-    should "send the confirmation email" do
-      assert_sent_email do |email|
-        email.subject =~ /account confirmation/i
-      end
-    end
+    should have_sent_email.with_subject(/account confirmation/i)
   end
 
   context "When signing up with email already confirmed" do
@@ -72,14 +69,12 @@ class UserTest < ActiveSupport::TestCase
       Factory(:user, :email_confirmed => true)
     end
 
-    should "not send the confirmation email" do
-      assert_did_not_send_email
-    end
+    should_not have_sent_email
   end
 
   context "When multiple users have signed up" do
     setup { Factory(:user) }
-    should_validate_uniqueness_of :email
+    should validate_uniqueness_of(:email)
   end
 
   # confirming email

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification 
 name: clearance
 version: !ruby/object:Gem::Version 
-  hash: 977940486
+  hash: 977940487
   prerelease: true
   segments: 
   - 0
   - 9
   - 0
-  - rc1
-  version: 0.9.0.rc1
+  - rc2
+  version: 0.9.0.rc2
 platform: ruby
 authors: 
 - Dan Croak