Skip to content

Commit 298d07e

Browse files
fixup! test: add unit tests for two-factor model, strategy, and registration
1 parent 268858d commit 298d07e

2 files changed

Lines changed: 0 additions & 4 deletions

File tree

test/devise_test.rb

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,6 @@ class DeviseTest < ActiveSupport::TestCase
9090
Devise.register_two_factor_method(:fake_2fa, model: 'devise/models/fake_2fa', strategy: :fake_2fa_strategy)
9191
assert_equal({ model: 'devise/models/fake_2fa', strategy: :fake_2fa_strategy }, Devise.two_factor_method_configs[:fake_2fa])
9292
assert_equal :fake_2fa_strategy, Devise::STRATEGIES[:fake_2fa]
93-
ensure
9493
Devise.two_factor_method_configs.delete(:fake_2fa)
9594
Devise::STRATEGIES.delete(:fake_2fa)
9695
end
@@ -99,7 +98,6 @@ class DeviseTest < ActiveSupport::TestCase
9998
Devise.register_two_factor_method(:fake_rt, model: 'x', route: { fake_rt: [nil, :new] })
10099
assert_equal :fake_rt, Devise::ROUTES[:fake_rt]
101100
assert_equal [nil, :new], Devise::URL_HELPERS[:fake_rt]
102-
ensure
103101
Devise.two_factor_method_configs.delete(:fake_rt)
104102
Devise::ROUTES.delete(:fake_rt)
105103
Devise::URL_HELPERS.delete(:fake_rt)
@@ -115,7 +113,6 @@ class DeviseTest < ActiveSupport::TestCase
115113
assert_raises(ArgumentError) do
116114
Devise.add_module(:test_mod, two_factor: true)
117115
end
118-
ensure
119116
Devise::ALL.delete(:test_mod)
120117
end
121118

test/models/two_factor_authenticatable_test.rb

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,6 @@ def self.include(mod)
9191

9292
klass.two_factor_methods = [:includable_test]
9393
assert_equal [:includable_test], Array(klass.instance_variable_get(:@two_factor_methods))
94-
ensure
9594
Devise.two_factor_method_configs.delete(:includable_test)
9695
Devise::STRATEGIES.delete(:includable_test)
9796
end

0 commit comments

Comments
 (0)