devise-security 0.12.0 → 0.18.0

data/test/test_database_authenticatable_patch.rb

@@ -0,0 +1,146 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class TestDatabaseAuthenticatablePatch < ActiveSupport::TestCase
+  def create_user
+    User.create(
+      email: 'bob@microsoft.com',
+      password: 'Password1!',
+      password_confirmation: 'Password1!'
+    ) do |user|
+      user.extend(Devise::Models::DatabaseAuthenticatablePatch)
+    end
+  end
+
+  test 'updates if all params are present and valid' do
+    user = create_user
+
+    assert(
+      user.update_with_password(
+        {
+          current_password: 'Password1!',
+          password: 'Password2!',
+          password_confirmation: 'Password2!'
+        }
+      )
+    )
+  end
+
+  test 'does not update if current_password is missing' do
+    user = create_user
+
+    user.update_with_password(
+      {
+        password: 'Password2!',
+        password_confirmation: 'Password2!'
+      }
+    )
+
+    assert_equal(["Current password can't be blank"], user.errors.full_messages)
+  end
+
+  test 'does not update if current_password is incorrect' do
+    user = create_user
+
+    user.update_with_password(
+      {
+        current_password: 'Password2!',
+        password: 'Password2!',
+        password_confirmation: 'Password2!'
+      }
+    )
+
+    assert_equal(['Current password is invalid'], user.errors.full_messages)
+  end
+
+  test 'does not update if password is missing' do
+    user = create_user
+
+    user.update_with_password(
+      {
+        current_password: 'Password1!',
+        password: '',
+        password_confirmation: 'Password2!'
+      }
+    )
+
+    assert_equal(["Password can't be blank"], user.errors.full_messages)
+  end
+
+  test 'does not update if password is invalid and mismatches confirmation' do
+    user = create_user
+
+    user.update_with_password(
+      {
+        current_password: 'Password1!',
+        password: 'f',
+        password_confirmation: 'Password2!'
+      }
+    )
+
+    assert_equal(
+      [
+        "Password confirmation doesn't match Password",
+        'Password is too short (minimum is 7 characters)',
+        'Password must contain at least one digit',
+        'Password must contain at least one upper-case letter'
+      ],
+      user.errors.full_messages
+    )
+  end
+
+  test 'does not update if password is invalid and matches confirmation' do
+    user = create_user
+
+    user.update_with_password(
+      {
+        current_password: 'Password1!',
+        password: 'f',
+        password_confirmation: 'f'
+      }
+    )
+
+    assert_equal(
+      [
+        'Password is too short (minimum is 7 characters)',
+        'Password must contain at least one digit',
+        'Password must contain at least one upper-case letter'
+      ],
+      user.errors.full_messages
+    )
+  end
+
+  test 'does not update if password_confirmation is missing' do
+    user = create_user
+
+    user.update_with_password(
+      {
+        current_password: 'Password1!',
+        password: 'Password2!',
+        password_confirmation: ''
+      }
+    )
+
+    assert_equal(
+      ["Password confirmation can't be blank"], user.errors.full_messages
+    )
+  end
+
+  test 'does not update if password_confirmation is mismatched' do
+    user = create_user
+
+    user.update_with_password(
+      {
+        current_password: 'Password1!',
+        password: 'Password2!',
+        password_confirmation: 'Password3!'
+      }
+    )
+
+    assert_equal(
+      ["Password confirmation doesn't match Password"],
+      user.errors.full_messages
+    )
+  end
+end

data/test/test_helper.rb

@@ -1,20 +1,52 @@
+# frozen_string_literal: true
+
 ENV['RAILS_ENV'] ||= 'test'
 
 require 'simplecov'
-require 'coveralls'
-Coveralls.wear!
 
+if ENV['CI']
+  require 'simplecov-lcov'
+  SimpleCov.formatter = SimpleCov::Formatter::LcovFormatter
+  SimpleCov::Formatter::LcovFormatter.config.report_with_single_file = true
+end
+
+SimpleCov.start do
+  add_filter 'gemfiles'
+  add_filter 'test/dummy/db'
+  add_group 'ActiveRecord', 'active_record'
+  add_group 'Expirable', /(?<!password_)expirable/
+  add_group 'Mongoid', 'mongoid'
+  add_group 'Paranoid Verifiable', 'paranoid_verification'
+  add_group 'Password Archivable', /password_archivable|old_password/
+  add_group 'Password Expirable', /password_expirable|password_expired/
+  add_group 'Secure Validateable', 'secure_validatable'
+  add_group 'Security Questionable', 'security_question'
+  add_group 'Session Limitable', 'session_limitable'
+  add_group 'Tests', 'test'
+end
+
+require 'pry'
 require 'dummy/config/environment'
 require 'minitest/autorun'
 require 'rails/test_help'
 require 'devise-security'
+require 'database_cleaner'
+require "orm/#{DEVISE_ORM}"
 
-ActiveRecord::Migration.verbose = false
-ActiveRecord::Base.logger = Logger.new(nil)
-ActiveRecord::Migrator.migrate(File.expand_path('../dummy/db/migrate', __FILE__))
+# Controller testing is the way that Devise itself tests the functionality of
+# controller, even though it has been deprecated in favor of request tests.
+require 'rails-controller-testing'
+Rails::Controller::Testing.install
+require 'support/integration_helpers'
 
-SimpleCov.formatter = Coveralls::SimpleCov::Formatter
-SimpleCov.start do
-  add_filter 'test/dummy'
-  add_filter 'gemfiles'
+class Minitest::Test
+  def before_setup
+    DatabaseCleaner.start
+  end
+
+  def after_teardown
+    DatabaseCleaner.clean
+  end
 end
+
+DatabaseCleaner.clean

data/test/test_install_generator.rb

@@ -1,16 +1,33 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 require 'rails/generators/test_case'
 require 'generators/devise_security/install_generator'
 
 class TestInstallGenerator < Rails::Generators::TestCase
   tests DeviseSecurity::Generators::InstallGenerator
-  destination File.expand_path('../tmp', __FILE__)
+  destination File.expand_path('tmp', __dir__)
   setup :prepare_destination
 
   test 'Assert all files are properly created' do
     run_generator
-    assert_file 'config/initializers/devise-security.rb'
-    assert_file 'config/locales/devise.security_extension.en.yml'
+    assert_file 'config/initializers/devise_security.rb'
+    assert_file 'config/locales/devise.security_extension.by.yml'
+    assert_file 'config/locales/devise.security_extension.cs.yml'
     assert_file 'config/locales/devise.security_extension.de.yml'
+    assert_file 'config/locales/devise.security_extension.en.yml'
+    assert_file 'config/locales/devise.security_extension.es.yml'
+    assert_file 'config/locales/devise.security_extension.fa.yml'
+    assert_file 'config/locales/devise.security_extension.fr.yml'
+    assert_file 'config/locales/devise.security_extension.hi.yml'
+    assert_file 'config/locales/devise.security_extension.it.yml'
+    assert_file 'config/locales/devise.security_extension.ja.yml'
+    assert_file 'config/locales/devise.security_extension.nl.yml'
+    assert_file 'config/locales/devise.security_extension.pt.yml'
+    assert_file 'config/locales/devise.security_extension.ru.yml'
+    assert_file 'config/locales/devise.security_extension.tr.yml'
+    assert_file 'config/locales/devise.security_extension.uk.yml'
+    assert_file 'config/locales/devise.security_extension.zh_CN.yml'
+    assert_file 'config/locales/devise.security_extension.zh_TW.yml'
   end
 end

data/test/test_paranoid_verification.rb

@@ -1,15 +1,17 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class TestParanoidVerification < ActiveSupport::TestCase
   test 'need to paranoid verify if code present' do
     user = User.new
     user.generate_paranoid_code
-    assert_equal(true, user.need_paranoid_verification?)
+    assert(user.need_paranoid_verification?)
   end
 
   test 'no need to paranoid verify if no code' do
     user = User.new
-    assert_equal(false, user.need_paranoid_verification?)
+    assert_not(user.need_paranoid_verification?)
   end
 
   test 'generate code' do
@@ -27,23 +29,23 @@ class TestParanoidVerification < ActiveSupport::TestCase
     user.generate_paranoid_code
     # default generator generates 5 char string
     assert_equal(user.paranoid_verification_code.class, String)
-    assert_equal(user.paranoid_verification_code.length, 5)
+    assert_equal(5, user.paranoid_verification_code.length)
   end
 
   test 'when code match upon verify code, should mark record that it\'s no loger needed to verify' do
     user = User.new(paranoid_verification_code: 'abcde')
 
-    assert_equal(true, user.need_paranoid_verification?)
+    assert(user.need_paranoid_verification?)
     user.verify_code('abcde')
-    assert_equal(false, user.need_paranoid_verification?)
+    assert_not(user.need_paranoid_verification?)
   end
 
   test 'when code match upon verify code, should no longer need verification' do
     user = User.new(paranoid_verification_code: 'abcde')
 
-    assert_equal(true, user.need_paranoid_verification?)
+    assert(user.need_paranoid_verification?)
     user.verify_code('abcde')
-    assert_equal(false, user.need_paranoid_verification?)
+    assert_not(user.need_paranoid_verification?)
   end
 
   test 'when code match upon verification code, should set when verification was accepted' do
@@ -55,7 +57,7 @@ class TestParanoidVerification < ActiveSupport::TestCase
   test 'when code not match upon verify code, should still need verification' do
     user = User.new(paranoid_verification_code: 'abcde')
     user.verify_code('wrong')
-    assert_equal(true, user.need_paranoid_verification?)
+    assert(user.need_paranoid_verification?)
   end
 
   test 'when code not match upon verification code, should not set paranoid_verified_at' do
@@ -92,7 +94,6 @@ class TestParanoidVerification < ActiveSupport::TestCase
     Devise.paranoid_code_regenerate_after_attempt = original_regenerate
   end
 
-
   test 'by default paranoid code regenerate should have 10 attempts' do
     user = User.new(paranoid_verification_code: 'abcde')
     assert_equal 10, user.paranoid_attempts_remaining

data/test/test_password_archivable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class TestPasswordArchivable < ActiveSupport::TestCase
@@ -17,7 +19,7 @@ class TestPasswordArchivable < ActiveSupport::TestCase
 
   test 'cannot use same password' do
     user = User.create email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
-    assert_raises(ActiveRecord::RecordInvalid) { set_password(user,  'Password1') }
+    assert_raises(ORMInvalidRecordException) { set_password(user, 'Password1') }
   end
 
   test 'indirectly saving associated user does not cause deprecation warning' do
@@ -35,26 +37,24 @@ class TestPasswordArchivable < ActiveSupport::TestCase
     assert_equal 0, OldPassword.count
   end
 
-  test 'cannot use archived passwords' do
+  test 'cannot reuse archived passwords' do
     assert_equal 2, Devise.password_archiving_count
 
     user = User.create! email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
     assert_equal 0, OldPassword.count
-
-    set_password(user,  'Password2')
+    set_password(user, 'Password2')
     assert_equal 1, OldPassword.count
 
-    assert_raises(ActiveRecord::RecordInvalid) { set_password(user,  'Password1') }
-
-    set_password(user,  'Password3')
+    assert_raises(ORMInvalidRecordException) { set_password(user, 'Password1') }
+    set_password(user, 'Password3')
     assert_equal 2, OldPassword.count
 
     # rotate first password out of archive
-    assert set_password(user,  'Password4')
+    assert set_password(user, 'Password4')
 
     # archive count was 2, so first password should work again
-    assert set_password(user,  'Password1')
-    assert set_password(user,  'Password2')
+    assert set_password(user, 'Password1')
+    assert set_password(user, 'Password2')
   end
 
   test 'the option should be dynamic during runtime' do
@@ -66,10 +66,34 @@ class TestPasswordArchivable < ActiveSupport::TestCase
 
     user = User.create email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
 
-    assert set_password(user,  'Password2')
+    assert set_password(user, 'Password2')
+
+    assert_raises(ORMInvalidRecordException) { set_password(user, 'Password2') }
+
+    assert_raises(ORMInvalidRecordException) { set_password(user, 'Password1') }
+  end
+
+  test 'default sort orders do not affect archiving' do
+    class ::OldPassword
+      default_scope { order(created_at: :asc) }
+    end
+
+    assert_equal 2, Devise.password_archiving_count
 
-    assert_raises(ActiveRecord::RecordInvalid) { set_password(user,  'Password2') }
+    user = User.create! email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
+    assert_equal 0, OldPassword.count
+    set_password(user, 'Password2')
+    assert_equal 1, OldPassword.count
 
-    assert_raises(ActiveRecord::RecordInvalid) { set_password(user,  'Password1') }
+    assert_raises(ORMInvalidRecordException) { set_password(user, 'Password1') }
+    set_password(user, 'Password3')
+    assert_equal 2, OldPassword.count
+
+    # rotate first password out of archive
+    assert set_password(user, 'Password4')
+
+    # archive count was 2, so first password should work again
+    assert set_password(user, 'Password1')
+    assert set_password(user, 'Password2')
   end
 end

data/test/test_password_expirable.rb

@@ -1,32 +1,95 @@
+# frozen_string_literal: true
+
 require 'test_helper'
 
 class TestPasswordArchivable < ActiveSupport::TestCase
   setup do
-    Devise.expire_password_after = 2.month
+    Devise.expire_password_after = 2.months
   end
 
   teardown do
     Devise.expire_password_after = 90.days
   end
 
-  test 'password expires' do
+  test 'does nothing if disabled' do
+    Devise.expire_password_after = false
     user = User.create email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
-    refute user.need_change_password?
+    assert_not user.need_change_password?
+    assert_not user.password_expired?
+    user.need_change_password!
+    assert_not user.need_change_password?
+    assert_not user.password_expired?
+  end
 
-    user.update(password_changed_at: Time.now.ago(3.month))
+  test 'password change can be requested' do
+    Devise.expire_password_after = true
+    user = User.create email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
+    assert_not user.need_change_password?
+    assert_not user.password_expired?
+    assert_not user.password_change_requested?
+    user.need_change_password!
     assert user.need_change_password?
+    assert_not user.password_expired? # it's not too old because it's not set at all
+    assert user.password_change_requested?
   end
 
-  test 'override expire after at runtime' do
+  test 'password expires' do
+    user = User.create email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
+    assert_not user.need_change_password?
+    assert_not user.password_expired?
+    assert_not user.password_too_old?
+    user.update(password_changed_at: Time.zone.now.ago(3.months))
+    assert user.password_too_old?
+    assert user.need_change_password?
+    assert user.password_expired?
+    assert_not user.password_change_requested?
+  end
+
+  test 'saving a record records the time the password was changed' do
     user = User.new email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
+    assert_nil user.password_changed_at
+    assert_not user.password_change_requested?
+    assert_not user.password_expired?
+    user.save
+    assert user.password_changed_at.present?
+    assert_not user.password_change_requested?
+    assert_not user.password_expired?
+  end
+
+  test 'updating a record updates the time the password was changed if the password is changed' do
+    user = User.create email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
+    user.update(password_changed_at: Time.zone.now.ago(3.months))
+    original_password_changed_at = user.password_changed_at
+    user.expire_password!
+    assert user.password_change_requested?
+    user.password = 'NewPassword1'
+    user.password_confirmation = 'NewPassword1'
+    user.save
+    assert user.password_changed_at > original_password_changed_at
+    assert_not user.password_change_requested?
+  end
+
+  test 'updating a record does not updates the time the password was changed if the password was not changed' do
+    user = User.create email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
+    user.expire_password!
+    assert user.password_change_requested?
+    user.save
+    assert_not user.previous_changes.key?(:password_changed_at)
+    assert user.password_change_requested?
+  end
+
+  test 'override expire after at runtime' do
+    user = User.create email: 'bob@microsoft.com', password: 'Password1', password_confirmation: 'Password1'
     user.instance_eval do
       def expire_password_after
-        4.month
+        4.months
       end
     end
-    user.password_changed_at = Time.now.ago(3.month)
-    refute user.need_change_password?
-    user.password_changed_at = Time.now.ago(5.month)
+    user.password_changed_at = Time.zone.now.ago(3.months)
+    assert_not user.need_change_password?
+    assert_not user.password_expired?
+    user.password_changed_at = Time.zone.now.ago(5.months)
     assert user.need_change_password?
+    assert user.password_expired?
   end
 end