deviseOne 1.0.0

data/test/models/omniauthable_test.rb

@@ -0,0 +1,7 @@
+require 'test_helper'
+
+class OmniauthableTest < ActiveSupport::TestCase
+  test 'required_fields should contain the fields that Devise uses' do
+    assert_same_content Devise::Models::Omniauthable.required_fields(User), []
+  end
+end

data/test/models/recoverable_test.rb

@@ -0,0 +1,205 @@
+require 'test_helper'
+
+class RecoverableTest < ActiveSupport::TestCase
+
+  def setup
+    setup_mailer
+  end
+
+  test 'should not generate reset password token after creating a record' do
+    assert_nil new_user.reset_password_token
+  end
+
+  test 'should never generate the same reset password token for different users' do
+    reset_password_tokens = []
+    3.times do
+      user = create_user
+      user.send_reset_password_instructions
+      token = user.reset_password_token
+      assert !reset_password_tokens.include?(token)
+      reset_password_tokens << token
+    end
+  end
+
+  test 'should reset password and password confirmation from params' do
+    user = create_user
+    user.reset_password!('123456789', '987654321')
+    assert_equal '123456789', user.password
+    assert_equal '987654321', user.password_confirmation
+  end
+
+  test 'should reset password and save the record' do
+    assert create_user.reset_password!('123456789', '123456789')
+  end
+
+  test 'should clear reset password token while reseting the password' do
+    user = create_user
+    assert_nil user.reset_password_token
+
+    user.send_reset_password_instructions
+    assert_present user.reset_password_token
+    assert user.reset_password!('123456789', '123456789')
+    assert_nil user.reset_password_token
+  end
+
+  test 'should not clear reset password token if record is invalid' do
+    user = create_user
+    user.send_reset_password_instructions
+    assert_present user.reset_password_token
+    assert_not user.reset_password!('123456789', '987654321')
+    assert_present user.reset_password_token
+  end
+
+  test 'should not reset password with invalid data' do
+    user = create_user
+    user.stubs(:valid?).returns(false)
+    assert_not user.reset_password!('123456789', '987654321')
+  end
+
+  test 'should reset reset password token and send instructions by email' do
+    user = create_user
+    assert_email_sent do
+      token = user.reset_password_token
+      user.send_reset_password_instructions
+      assert_not_equal token, user.reset_password_token
+    end
+  end
+
+  test 'should find a user to send instructions by email' do
+    user = create_user
+    reset_password_user = User.send_reset_password_instructions(email: user.email)
+    assert_equal reset_password_user, user
+  end
+
+  test 'should return a new record with errors if user was not found by e-mail' do
+    reset_password_user = User.send_reset_password_instructions(email: "invalid@example.com")
+    assert_not reset_password_user.persisted?
+    assert_equal "not found", reset_password_user.errors[:email].join
+  end
+
+  test 'should find a user to send instructions by authentication_keys' do
+    swap Devise, authentication_keys: [:username, :email] do
+      user = create_user
+      reset_password_user = User.send_reset_password_instructions(email: user.email, username: user.username)
+      assert_equal reset_password_user, user
+    end
+  end
+
+  test 'should require all reset_password_keys' do
+      swap Devise, reset_password_keys: [:username, :email] do
+          user = create_user
+          reset_password_user = User.send_reset_password_instructions(email: user.email)
+          assert_not reset_password_user.persisted?
+          assert_equal "can't be blank", reset_password_user.errors[:username].join
+      end
+  end
+
+  test 'should reset reset_password_token before send the reset instructions email' do
+    user = create_user
+    token = user.reset_password_token
+    User.send_reset_password_instructions(email: user.email)
+    assert_not_equal token, user.reload.reset_password_token
+  end
+
+  test 'should send email instructions to the user reset their password' do
+    user = create_user
+    assert_email_sent do
+      User.send_reset_password_instructions(email: user.email)
+    end
+  end
+
+  test 'should find a user to reset their password based on the raw token' do
+    user = create_user
+    raw  = user.send_reset_password_instructions
+
+    reset_password_user = User.reset_password_by_token(reset_password_token: raw)
+    assert_equal reset_password_user, user
+  end
+
+  test 'should return a new record with errors if no reset_password_token is found' do
+    reset_password_user = User.reset_password_by_token(reset_password_token: 'invalid_token')
+    assert_not reset_password_user.persisted?
+    assert_equal "is invalid", reset_password_user.errors[:reset_password_token].join
+  end
+
+  test 'should return a new record with errors if reset_password_token is blank' do
+    reset_password_user = User.reset_password_by_token(reset_password_token: '')
+    assert_not reset_password_user.persisted?
+    assert_match "can't be blank", reset_password_user.errors[:reset_password_token].join
+  end
+
+  test 'should return a new record with errors if password is blank' do
+    user = create_user
+    raw  = user.send_reset_password_instructions
+
+    reset_password_user = User.reset_password_by_token(reset_password_token: raw, password: '')
+    assert_not reset_password_user.errors.empty?
+    assert_match "can't be blank", reset_password_user.errors[:password].join
+    assert_equal raw, reset_password_user.reset_password_token
+  end
+
+  test 'should reset successfully user password given the new password and confirmation' do
+    user = create_user
+    old_password = user.password
+    raw  = user.send_reset_password_instructions
+
+    reset_password_user = User.reset_password_by_token(
+      reset_password_token: raw,
+      password: 'new_password',
+      password_confirmation: 'new_password'
+    )
+    assert_nil reset_password_user.reset_password_token
+
+    user.reload
+    assert_not user.valid_password?(old_password)
+    assert user.valid_password?('new_password')
+    assert_nil user.reset_password_token
+  end
+
+  test 'should not reset password after reset_password_within time' do
+    swap Devise, reset_password_within: 1.hour do
+      user = create_user
+      raw  = user.send_reset_password_instructions
+
+      old_password = user.password
+      user.reset_password_sent_at = 2.days.ago
+      user.save!
+
+      reset_password_user = User.reset_password_by_token(
+        reset_password_token: raw,
+        password: 'new_password',
+        password_confirmation: 'new_password'
+      )
+      user.reload
+
+      assert user.valid_password?(old_password)
+      assert_not user.valid_password?('new_password')
+      assert_equal "has expired, please request a new one", reset_password_user.errors[:reset_password_token].join
+    end
+  end
+
+  test 'required_fields should contain the fields that Devise uses' do
+    assert_same_content Devise::Models::Recoverable.required_fields(User), [
+      :reset_password_sent_at,
+      :reset_password_token
+    ]
+  end
+
+  test 'should return a user based on the raw token' do
+    user = create_user
+    raw  = user.send_reset_password_instructions
+
+    assert_equal User.with_reset_password_token(raw), user
+  end
+
+  test 'should return the same reset password token as generated' do
+    user = create_user
+    raw  = user.send_reset_password_instructions
+    assert_equal Devise.token_generator.digest(self.class, :reset_password_token, raw), user.reset_password_token
+  end
+
+  test 'should return nil if a user based on the raw token is not found' do
+    assert_equal User.with_reset_password_token('random-token'), nil
+  end
+
+end

data/test/models/registerable_test.rb

@@ -0,0 +1,7 @@
+require 'test_helper'
+
+class RegisterableTest < ActiveSupport::TestCase
+  test 'required_fields should contain the fields that Devise uses' do
+    assert_same_content Devise::Models::Registerable.required_fields(User), []
+  end
+end

data/test/models/rememberable_test.rb

@@ -0,0 +1,198 @@
+require 'test_helper'
+
+class RememberableTest < ActiveSupport::TestCase
+  def resource_class
+    User
+  end
+
+  def create_resource
+    create_user
+  end
+
+  test 'remember_me should not generate a new token if using salt' do
+    user = create_user
+    user.expects(:valid?).never
+    user.remember_me!
+  end
+
+  test 'forget_me should not clear remember token if using salt' do
+    user = create_user
+    user.remember_me!
+    user.expects(:valid?).never
+    user.forget_me!
+  end
+
+  test 'can generate remember token' do
+    user = create_user
+    user.singleton_class.send(:attr_accessor, :remember_token)
+    User.to_adapter.expects(:find_first).returns(nil)
+    user.remember_me!
+    assert user.remember_token
+  end
+
+  test 'serialize into cookie' do
+    user = create_user
+    user.remember_me!
+    assert_equal [user.to_key, user.authenticatable_salt], User.serialize_into_cookie(user)
+  end
+
+  test 'serialize from cookie' do
+    user = create_user
+    user.remember_me!
+    assert_equal user, User.serialize_from_cookie(user.to_key, user.authenticatable_salt)
+  end
+
+  test 'raises a RuntimeError if authenticatable_salt is nil' do
+    user = User.new
+    user.encrypted_password = nil
+    assert_raise RuntimeError do
+      user.rememberable_value
+    end
+  end
+
+  test 'should respond to remember_me attribute' do
+    assert resource_class.new.respond_to?(:remember_me)
+    assert resource_class.new.respond_to?(:remember_me=)
+  end
+
+  test 'forget_me should clear remember_created_at if expire_all_remember_me_on_sign_out is true' do
+    swap Devise, expire_all_remember_me_on_sign_out: true do
+      resource = create_resource
+      resource.remember_me!
+      assert_not_nil resource.remember_created_at
+
+      resource.forget_me!
+      assert_nil resource.remember_created_at
+    end
+  end
+
+  test 'forget_me should not clear remember_created_at if expire_all_remember_me_on_sign_out is false' do
+    swap Devise, expire_all_remember_me_on_sign_out: false do
+      resource = create_resource
+      resource.remember_me!
+
+      assert_not_nil resource.remember_created_at
+
+      resource.forget_me!
+      assert_not_nil resource.remember_created_at
+    end
+  end
+
+  test 'forget_me should not try to update resource if it has been destroyed' do
+    resource = create_resource
+    resource.expects(:remember_created_at).never
+    resource.expects(:save).never
+
+    resource.destroy
+    resource.forget_me!
+  end
+
+  test 'remember is expired if not created at timestamp is set' do
+    assert create_resource.remember_expired?
+  end
+
+  test 'serialize should return nil if no resource is found' do
+    assert_nil resource_class.serialize_from_cookie([0], "123")
+  end
+
+  test 'remember me return nil if is a valid resource with invalid token' do
+    resource = create_resource
+    assert_nil resource_class.serialize_from_cookie([resource.id], "123")
+  end
+
+  test 'remember for should fallback to devise remember for default configuration' do
+    swap Devise, remember_for: 1.day do
+      resource = create_resource
+      resource.remember_me!
+      assert_not resource.remember_expired?
+    end
+  end
+
+  test 'remember expires at should sum date of creation with remember for configuration' do
+    swap Devise, remember_for: 3.days do
+      resource = create_resource
+      resource.remember_me!
+      assert_equal 3.days.from_now.to_date, resource.remember_expires_at.to_date
+
+      Devise.remember_for = 5.days
+      assert_equal 5.days.from_now.to_date, resource.remember_expires_at.to_date
+    end
+  end
+
+  test 'remember should be expired if remember_for is zero' do
+    swap Devise, remember_for: 0.days do
+      Devise.remember_for = 0.days
+      resource = create_resource
+      resource.remember_me!
+      assert resource.remember_expired?
+    end
+  end
+
+  test 'remember should be expired if it was created before limit time' do
+    swap Devise, remember_for: 1.day do
+      resource = create_resource
+      resource.remember_me!
+      resource.remember_created_at = 2.days.ago
+      resource.save
+      assert resource.remember_expired?
+    end
+  end
+
+  test 'remember should not be expired if it was created within the limit time' do
+    swap Devise, remember_for: 30.days do
+      resource = create_resource
+      resource.remember_me!
+      resource.remember_created_at = (30.days.ago + 2.minutes)
+      resource.save
+      assert_not resource.remember_expired?
+    end
+  end
+
+  test 'if extend_remember_period is false, remember_me! should generate a new timestamp if expired' do
+    swap Devise, remember_for: 5.minutes do
+      resource = create_resource
+      resource.remember_me!(false)
+      assert resource.remember_created_at
+
+      resource.remember_created_at = old = 10.minutes.ago
+      resource.save
+
+      resource.remember_me!(false)
+      assert_not_equal old.to_i, resource.remember_created_at.to_i
+    end
+  end
+
+  test 'if extend_remember_period is false, remember_me! should not generate a new timestamp' do
+    swap Devise, remember_for: 1.year do
+      resource = create_resource
+      resource.remember_me!(false)
+      assert resource.remember_created_at
+
+      resource.remember_created_at = old = 10.minutes.ago.utc
+      resource.save
+
+      resource.remember_me!(false)
+      assert_equal old.to_i, resource.remember_created_at.to_i
+    end
+  end
+
+  test 'if extend_remember_period is true, remember_me! should always generate a new timestamp' do
+    swap Devise, remember_for: 1.year do
+      resource = create_resource
+      resource.remember_me!(true)
+      assert resource.remember_created_at
+
+      resource.remember_created_at = old = 10.minutes.ago
+      resource.save
+
+      resource.remember_me!(true)
+      assert_not_equal old, resource.remember_created_at
+    end
+  end
+
+  test 'should have the required_fields array' do
+    assert_same_content Devise::Models::Rememberable.required_fields(User), [
+      :remember_created_at
+    ]
+  end
+end

data/test/models/serializable_test.rb

@@ -0,0 +1,49 @@
+require 'test_helper'
+
+class SerializableTest < ActiveSupport::TestCase
+  setup do
+    @user = create_user
+  end
+
+  test 'should not include unsafe keys on XML' do
+    assert_match(/email/, @user.to_xml)
+    assert_no_match(/confirmation-token/, @user.to_xml)
+  end
+
+  test 'should not include unsafe keys on XML even if a new except is provided' do
+    assert_no_match(/email/, @user.to_xml(except: :email))
+    assert_no_match(/confirmation-token/, @user.to_xml(except: :email))
+  end
+
+  test 'should include unsafe keys on XML if a force_except is provided' do
+    assert_no_match(/<email/, @user.to_xml(force_except: :email))
+    assert_match(/confirmation-token/, @user.to_xml(force_except: :email))
+  end
+
+  test 'should not include unsafe keys on JSON' do
+    keys = from_json().keys.select{ |key| !key.include?("id") }
+    assert_equal %w(created_at email facebook_token updated_at username), keys.sort
+  end
+
+  test 'should not include unsafe keys on JSON even if a new except is provided' do
+    assert_no_key "email", from_json(except: :email)
+    assert_no_key "confirmation_token", from_json(except: :email)
+  end
+
+  test 'should include unsafe keys on JSON if a force_except is provided' do
+    assert_no_key "email", from_json(force_except: :email)
+    assert_key "confirmation_token", from_json(force_except: :email)
+  end
+
+  def assert_key(key, subject)
+    assert subject.key?(key), "Expected #{subject.inspect} to have key #{key.inspect}"
+  end
+
+  def assert_no_key(key, subject)
+    assert !subject.key?(key), "Expected #{subject.inspect} to not have key #{key.inspect}"
+  end
+
+  def from_json(options=nil)
+    ActiveSupport::JSON.decode(@user.to_json(options))["user"]
+  end
+end