fixup! test: add unit tests for two-factor model, strategy, and registration

Author: santiagorodriguez96

test/strategies/two_factor_test.rb

@@ -7,15 +7,21 @@ class TwoFactorStrategyTest < ActiveSupport::TestCase
     assert defined?(Devise::Strategies::TwoFactor)
   end
 
-  test 'TwoFactor base strategy is never valid' do
-    strategy = Devise::Strategies::TwoFactor.new(nil)
-    assert_equal false, strategy.valid?
+  test 'TwoFactor base strategy is not valid without a pending session' do
+    strategy = Devise::Strategies::TwoFactor.new(env_with_session)
+    assert_not strategy.valid?
   end
 
   test 'verify_two_factor! raises NotImplementedError by default' do
-    strategy = Devise::Strategies::TwoFactor.new(nil)
+    strategy = Devise::Strategies::TwoFactor.new(env_with_session)
     assert_raises(NotImplementedError) do
       strategy.verify_two_factor!(Object.new)
     end
   end
+
+  private
+
+  def env_with_session(session = {})
+    { 'rack.session' => session }
+  end
 end