:two_factor_authenticatable should be inserted on top

devise-two-factor · May 21, 2023 · 61ca1a0 · 61ca1a0
1 parent 733d11f
commit 61ca1a0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/devise-two-factor.rb b/lib/devise-two-factor.rb
@@ -31,7 +31,7 @@ module Devise
 end
 
 Devise.add_module(:two_factor_authenticatable, :route => :session, :strategy => true,
-                  :controller => :sessions, :model  => true)
+                  :controller => :sessions, :model  => true, :insert_at => 0)
 
 Devise.add_module(:two_factor_backupable, :route => :session, :strategy => true,
                   :controller => :sessions, :model  => true)
diff --git a/spec/devise/models/two_factor_authenticatable_spec.rb b/spec/devise/models/two_factor_authenticatable_spec.rb
@@ -25,6 +25,10 @@ def save(validate)
 end
 
 describe ::Devise::Models::TwoFactorAuthenticatable do
+  it 'should be inserted prior to other devise modules' do
+    expect(Devise::ALL.first).to eq(:two_factor_authenticatable)
+  end
+
   context 'When included in a class' do
     subject { TwoFactorAuthenticatableDouble.new }