shingara-devise 0.4.3

data/test/integration/recoverable_test.rb

@@ -0,0 +1,131 @@
+require 'test/test_helper'
+
+class PasswordTest < ActionController::IntegrationTest
+
+  def visit_new_password_path
+    visit new_user_session_path
+    click_link 'Forgot password?'
+  end
+
+  def request_forgot_password(&block)
+    visit_new_password_path
+
+    assert_response :success
+    assert_template 'passwords/new'
+    assert_not warden.authenticated?(:user)
+
+    fill_in 'email', :with => 'user@test.com'
+    yield if block_given?
+    click_button 'Send me reset password instructions'
+  end
+
+  def reset_password(options={}, &block)
+    unless options[:visit] == false
+      visit edit_user_password_path(:reset_password_token => options[:reset_password_token])
+    end
+    assert_response :success
+    assert_template 'passwords/edit'
+
+    fill_in 'Password', :with => '987654321'
+    fill_in 'Password confirmation', :with => '987654321'
+    yield if block_given?
+    click_button 'Change my password'
+  end
+
+  test 'authenticated user should not be able to visit forgot password page' do
+    sign_in_as_user
+    assert warden.authenticated?(:user)
+
+    get new_user_password_path
+
+    assert_response :redirect
+    assert_redirected_to root_path
+  end
+
+  test 'not authenticated user should be able to request a forgot password' do
+    create_user
+    request_forgot_password
+
+    assert_template 'sessions/new'
+    assert_contain 'You will receive an email with instructions about how to reset your password in a few minutes.'
+  end
+
+  test 'not authenticated user with invalid email should receive an error message' do
+    request_forgot_password do
+      fill_in 'email', :with => 'invalid.test@test.com'
+    end
+
+    assert_response :success
+    assert_template 'passwords/new'
+    assert_have_selector 'input[type=text][value=\'invalid.test@test.com\']'
+    assert_contain 'Email not found'
+  end
+
+  test 'authenticated user should not be able to visit edit password page' do
+    sign_in_as_user
+
+    get edit_user_password_path
+
+    assert_response :redirect
+    assert_redirected_to root_path
+    assert warden.authenticated?(:user)
+  end
+
+  test 'not authenticated user with invalid reset password token should not be able to change his password' do
+    user = create_user
+    reset_password :reset_password_token => 'invalid_reset_password'
+
+    assert_response :success
+    assert_template 'passwords/edit'
+    assert_have_selector '#errorExplanation'
+    assert_contain 'Reset password token is invalid'
+    assert_not user.reload.valid_password?('987654321')
+  end
+
+  test 'not authenticated user with valid reset password token but invalid password should not be able to change his password' do
+    user = create_user
+    request_forgot_password
+    reset_password :reset_password_token => user.reload.reset_password_token do
+      fill_in 'Password confirmation', :with => 'other_password'
+    end
+
+    assert_response :success
+    assert_template 'passwords/edit'
+    assert_have_selector '#errorExplanation'
+    assert_contain 'Password doesn\'t match confirmation'
+    assert_not user.reload.valid_password?('987654321')
+  end
+
+  test 'not authenticated user with valid data should be able to change his password' do
+    user = create_user
+    request_forgot_password
+    reset_password :reset_password_token => user.reload.reset_password_token
+
+    assert_template 'home/index'
+    assert_contain 'Your password was changed successfully.'
+    assert user.reload.valid_password?('987654321')
+  end
+
+  test 'after entering invalid data user should still be able to change his password' do
+    user = create_user
+    request_forgot_password
+    reset_password :reset_password_token => user.reload.reset_password_token do
+      fill_in 'Password confirmation', :with => 'other_password'
+    end
+    assert_response :success
+    assert_have_selector '#errorExplanation'
+    assert_not user.reload.valid_password?('987654321')
+
+    reset_password :reset_password_token => user.reload.reset_password_token, :visit => false
+    assert_contain 'Your password was changed successfully.'
+    assert user.reload.valid_password?('987654321')
+  end
+
+  test 'sign in user automatically after changing it\'s password' do
+    user = create_user
+    request_forgot_password
+    reset_password :reset_password_token => user.reload.reset_password_token
+
+    assert warden.authenticated?(:user)
+  end
+end

data/test/integration/rememberable_test.rb

@@ -0,0 +1,65 @@
+require 'test/test_helper'
+
+class RememberMeTest < ActionController::IntegrationTest
+
+  def create_user_and_remember(add_to_token='')
+    Devise.remember_for = 1
+    user = create_user
+    user.remember_me!
+    cookies['remember_token'] = User.serialize_into_cookie(user) + add_to_token
+    user
+  end
+
+  test 'do not remember the user if he has not checked remember me option' do
+    user = sign_in_as_user
+
+    assert_nil user.reload.remember_token
+  end
+
+  test 'generate remember token after sign in' do
+    user = sign_in_as_user :remember_me => true
+
+    assert_not_nil user.reload.remember_token
+  end
+
+  test 'remember the user before sign in' do
+    user = create_user_and_remember
+    get users_path
+    assert_response :success
+    assert warden.authenticated?(:user)
+    assert warden.user(:user) == user
+  end
+
+  test 'do not remember with invalid token' do
+    user = create_user_and_remember('add')
+    get users_path
+    assert_response :success
+    assert_not warden.authenticated?(:user)
+  end
+
+  test 'do not remember with token expired' do
+    user = create_user_and_remember
+    Devise.remember_for = 0
+    get users_path
+    assert_response :success
+    assert_not warden.authenticated?(:user)
+  end
+
+  test 'forget the user before sign out' do
+    user = create_user_and_remember
+    get users_path
+    assert warden.authenticated?(:user)
+    get destroy_user_session_path
+    assert_not warden.authenticated?(:user)
+    assert_nil user.reload.remember_token
+  end
+
+  test 'do not remember the user anymore after forget' do
+    user = create_user_and_remember
+    get users_path
+    assert warden.authenticated?(:user)
+    get destroy_user_session_path
+    get users_path
+    assert_not warden.authenticated?(:user)
+  end
+end

data/test/mailers/confirmation_instructions_test.rb

@@ -0,0 +1,59 @@
+require 'test/test_helper'
+
+class ConfirmationInstructionsTest < ActionMailer::TestCase
+
+  def setup
+    setup_mailer
+    DeviseMailer.sender = 'test@example.com'
+  end
+
+  def user
+    @user ||= create_user
+  end
+
+  def mail
+    @mail ||= begin
+      user
+      ActionMailer::Base.deliveries.first
+    end
+  end
+
+  test 'email sent after creating the user' do
+    assert_not_nil mail
+  end
+
+  test 'content type should be set to html' do
+    assert_equal 'text/html', mail.content_type
+  end
+
+  test 'send confirmation instructions to the user email' do
+    mail
+    assert_equal [user.email], mail.to
+  end
+
+  test 'setup sender from configuration' do
+    assert_equal ['test@example.com'], mail.from
+  end
+
+  test 'setup subject from I18n' do
+    store_translations :en, :devise => { :mailer => { :confirmation_instructions => 'Account Confirmation' } } do
+      assert_equal 'Account Confirmation', mail.subject
+    end
+  end
+
+  test 'subject namespaced by model' do
+    store_translations :en, :devise => { :mailer => { :user => { :confirmation_instructions => 'User Account Confirmation' } } } do
+      assert_equal 'User Account Confirmation', mail.subject
+    end
+  end
+
+  test 'body should have user info' do
+    assert_match /#{user.email}/, mail.body
+  end
+
+  test 'body should have link to confirm the account' do
+    host = ActionMailer::Base.default_url_options[:host]
+    confirmation_url_regexp = %r{<a href=\"http://#{host}/users/confirmation\?confirmation_token=#{user.confirmation_token}">}
+    assert_match confirmation_url_regexp, mail.body
+  end
+end

data/test/mailers/reset_password_instructions_test.rb

@@ -0,0 +1,62 @@
+require 'test/test_helper'
+
+class ResetPasswordInstructionsTest < ActionMailer::TestCase
+
+  def setup
+    setup_mailer
+    DeviseMailer.sender = 'test@example.com'
+  end
+
+  def user
+    @user ||= begin
+      user = create_user
+      user.send_reset_password_instructions
+      user
+    end
+  end
+
+  def mail
+    @mail ||= begin
+      user
+      ActionMailer::Base.deliveries.last
+    end
+  end
+
+  test 'email sent after reseting the user password' do
+    assert_not_nil mail
+  end
+
+  test 'content type should be set to html' do
+    assert_equal 'text/html', mail.content_type
+  end
+
+  test 'send confirmation instructions to the user email' do
+    assert_equal [user.email], mail.to
+  end
+
+  test 'setup sender from configuration' do
+    assert_equal ['test@example.com'], mail.from
+  end
+
+  test 'setup subject from I18n' do
+    store_translations :en, :devise => { :mailer => { :reset_password_instructions => 'Reset instructions' } } do
+      assert_equal 'Reset instructions', mail.subject
+    end
+  end
+
+  test 'subject namespaced by model' do
+    store_translations :en, :devise => { :mailer => { :user => { :reset_password_instructions => 'User Reset Instructions' } } } do
+      assert_equal 'User Reset Instructions', mail.subject
+    end
+  end
+
+  test 'body should have user info' do
+    assert_match /#{user.email}/, mail.body
+  end
+
+  test 'body should have link to confirm the account' do
+    host = ActionMailer::Base.default_url_options[:host]
+    reset_url_regexp = %r{<a href=\"http://#{host}/users/password/edit\?reset_password_token=#{user.reset_password_token}">}
+    assert_match reset_url_regexp, mail.body
+  end
+end

data/test/mapping_test.rb

@@ -0,0 +1,101 @@
+require 'test/test_helper'
+
+class MappingTest < ActiveSupport::TestCase
+
+  test 'store options' do
+    mapping = Devise.mappings[:user]
+
+    assert_equal User,                mapping.to
+    assert_equal User.devise_modules, mapping.for
+    assert_equal :users,              mapping.as
+  end
+
+  test 'allows as to be given' do
+    assert_equal :admin_area, Devise.mappings[:admin].as
+  end
+
+  test 'allow custom singular to be given' do
+    assert_equal :organizers, Devise.mappings[:manager].as
+  end
+
+  test 'allows a controller depending on the mapping' do
+    mapping = Devise.mappings[:user]
+    assert mapping.allows?(:sessions)
+    assert mapping.allows?(:confirmations)
+    assert mapping.allows?(:passwords)
+
+    mapping = Devise.mappings[:admin]
+    assert mapping.allows?(:sessions)
+    assert_not mapping.allows?(:confirmations)
+    assert_not mapping.allows?(:passwords)
+  end
+
+  test 'return mapping by path' do
+    assert_nil   Devise::Mapping.find_by_path("/foo/bar")
+    assert_equal Devise.mappings[:user], Devise::Mapping.find_by_path("/users/session")
+  end
+
+  test 'return mapping by customized path' do
+    assert_equal Devise.mappings[:admin], Devise::Mapping.find_by_path("/admin_area/session")
+  end
+
+  test 'return default path names' do
+    mapping = Devise.mappings[:user]
+    assert_equal 'sign_in', mapping.path_names[:sign_in]
+    assert_equal 'sign_out', mapping.path_names[:sign_out]
+    assert_equal 'password', mapping.path_names[:password]
+    assert_equal 'confirmation', mapping.path_names[:confirmation]
+  end
+
+  test 'allow custom path names to be given' do
+    mapping = Devise.mappings[:account]
+    assert_equal 'login', mapping.path_names[:sign_in]
+    assert_equal 'logout', mapping.path_names[:sign_out]
+    assert_equal 'secret', mapping.path_names[:password]
+    assert_equal 'verification', mapping.path_names[:confirmation]
+  end
+
+  test 'has an empty path as default path prefix' do
+    mapping = Devise.mappings[:account]
+    assert_equal '/', mapping.path_prefix
+  end
+
+  test 'allow path prefix to be configured' do
+    mapping = Devise.mappings[:manager]
+    assert_equal '/:locale/', mapping.path_prefix
+  end
+
+  test 'retrieve as from the proper position' do
+    assert_equal 1, Devise.mappings[:account].as_position
+    assert_equal 2, Devise.mappings[:manager].as_position
+  end
+
+  test 'raw path is returned' do
+    assert_equal '/account', Devise.mappings[:account].raw_path
+    assert_equal '/:locale/organizers', Devise.mappings[:manager].raw_path
+  end
+
+  test 'parsed path is returned' do
+    begin
+      Devise.default_url_options {{ :locale => I18n.locale }}
+      assert_equal '/account', Devise.mappings[:account].parsed_path
+      assert_equal '/en/organizers', Devise.mappings[:manager].parsed_path
+    ensure
+      Devise.default_url_options {{ }}
+    end
+  end
+
+  test 'magic predicates' do
+    mapping = Devise.mappings[:user]
+    assert mapping.authenticatable?
+    assert mapping.confirmable?
+    assert mapping.recoverable?
+    assert mapping.rememberable?
+
+    mapping = Devise.mappings[:admin]
+    assert mapping.authenticatable?
+    assert_not mapping.confirmable?
+    assert_not mapping.recoverable?
+    assert_not mapping.rememberable?
+  end
+end

data/test/models/authenticatable_test.rb

@@ -0,0 +1,130 @@
+require 'test/test_helper'
+require 'digest/sha1'
+
+class AuthenticatableTest < ActiveSupport::TestCase
+
+  def encrypt_password(user, pepper=User.pepper, stretches=User.stretches, encryptor = ::Devise::Encryptors::Sha1)
+    encryptor.digest('123456', stretches, user.password_salt, pepper)
+  end
+
+  test 'should respond to password and password confirmation' do
+    user = new_user
+    assert user.respond_to?(:password)
+    assert user.respond_to?(:password_confirmation)
+  end
+
+  test 'should generate salt while setting password' do
+    assert_present new_user.password_salt
+    assert_present new_user(:password => nil).password_salt
+    assert_present new_user(:password => '').password_salt
+    assert_present create_user.password_salt
+  end
+
+  test 'should not change password salt when updating' do
+    user = create_user
+    salt = user.password_salt
+    user.expects(:password_salt=).never
+    user.save!
+    assert_equal salt, user.password_salt
+  end
+
+  test 'should generate a base64 hash using SecureRandom for password salt' do
+    ActiveSupport::SecureRandom.expects(:base64).with(15).returns('friendly_token')
+    assert_equal 'friendly_token', new_user.password_salt
+  end
+
+  test 'should never generate the same salt for different users' do
+    password_salts = []
+    10.times do
+      salt = create_user.password_salt
+      assert_not password_salts.include?(salt)
+      password_salts << salt
+    end
+  end
+
+  test 'should generate encrypted password while setting password' do
+    assert_present new_user.encrypted_password
+    assert_present new_user(:password => nil).encrypted_password
+    assert_present new_user(:password => '').encrypted_password
+    assert_present create_user.encrypted_password
+  end
+
+  test 'should encrypt password again if password has changed' do
+    user = create_user
+    encrypted_password = user.encrypted_password
+    user.password = user.password_confirmation = 'new_password'
+    user.save!
+    assert_not_equal encrypted_password, user.encrypted_password
+  end
+
+  test 'should fallback to Sha1 as default encryption' do
+    user = new_user
+    assert_equal encrypt_password(user), user.encrypted_password
+  end
+
+  test 'should fallback to devise pepper default configuring' do
+    begin
+      Devise.pepper = ''
+      user = new_user
+      assert_equal encrypt_password(user), user.encrypted_password
+      assert_not_equal encrypt_password(user, 'another_pepper'), user.encrypted_password
+      Devise.pepper = 'new_pepper'
+      user = new_user
+      assert_equal encrypt_password(user, 'new_pepper'), user.encrypted_password
+      assert_not_equal encrypt_password(user, 'another_pepper'), user.encrypted_password
+      Devise.pepper = '123456'
+      user = new_user
+      assert_equal encrypt_password(user, '123456'), user.encrypted_password
+      assert_not_equal encrypt_password(user, 'another_pepper'), user.encrypted_password
+    ensure
+      Devise.pepper = nil
+    end
+  end
+
+  test 'should fallback to devise stretches default configuring' do
+    begin
+      default_stretches = Devise.stretches
+      Devise.stretches = 1
+      user = new_user
+      assert_equal encrypt_password(user, nil, 1), user.encrypted_password
+      assert_not_equal encrypt_password(user, nil, 2), user.encrypted_password
+    ensure
+      Devise.stretches = default_stretches
+    end
+  end
+  
+  test 'should respect encryptor configuration' do
+    begin 
+      Devise.encryptor = ::Devise::Encryptors::Sha512
+      user = create_user
+      assert_equal user.encrypted_password, encrypt_password(user, User.pepper, User.stretches, ::Devise::Encryptors::Sha512)
+    ensure
+      Devise.encryptor = ::Devise::Encryptors::Sha1
+    end
+  end
+
+  test 'should test for a valid password' do
+    user = create_user
+    assert user.valid_password?('123456')
+    assert_not user.valid_password?('654321')
+  end
+
+  test 'should authenticate a valid user with email and password and return it' do
+    user = create_user
+    User.any_instance.stubs(:confirmed?).returns(true)
+    authenticated_user = User.authenticate(:email => user.email, :password => user.password)
+    assert_equal authenticated_user, user
+  end
+
+  test 'should return nil when authenticating an invalid user by email' do
+    user = create_user
+    authenticated_user = User.authenticate(:email => 'another.email@email.com', :password => user.password)
+    assert_nil authenticated_user
+  end
+
+  test 'should return nil when authenticating an invalid user by password' do
+    user = create_user
+    authenticated_user = User.authenticate(:email => user.email, :password => 'another_password')
+    assert_nil authenticated_user
+  end
+end