Move login_required_redirection_url() into rails_helper.rb

Author: mexisme

spec/rails_helper.rb

@@ -82,6 +82,28 @@
 require Rails.root.join('config', 'initializers', 'active_record_observer_hooks')
 # see comments in this file for more info
 
+# Common login-testing strategies in the specs
+#
+if Squash::Configuration.authentication.strategy == 'google'
+  RSpec.shared_context "setup for required logged-in user" do
+    unless Squash::Configuration.authentication.strategy == 'google'
+      def login_required_redirection_url(*args)
+        login_url(*args)
+      end
+
+    else
+      before :each do
+        allow(SecureRandom).to receive(:hex).and_return('1234')
+      end
+
+      def login_required_redirection_url(*args)
+        controller.instance_eval { google_authentication_uri }
+      end
+    end
+
+  end
+end
+
 # Logs a user in for spec purposes.
 #
 # @params [User] user A user.

spec/spec_helper.rb

@@ -24,29 +24,6 @@
 # individual file that may not need all of that loaded. Instead, consider making
 # a separate helper file that requires the additional dependencies and performs
 # the additional setup, and require it from the spec files that actually need it.
-# Common login-testing strategies in the specs
-#
-# @param [*args] Args to pass to #get
-if Squash::Configuration.authentication.strategy == 'google'
-  RSpec.shared_context "setup for required logged-in user" do
-    unless Squash::Configuration.authentication.strategy == 'google'
-      def login_required_redirection_url(*args)
-        login_url(*args)
-      end
-
-    else
-      before :each do
-        allow(SecureRandom).to receive(:hex).and_return('1234')
-      end
-
-      def login_required_redirection_url(*args)
-        controller.instance_eval { google_authentication_uri }
-      end
-    end
-
-  end
-end
-
 #
 # The `.rspec` file also contains a few flags that are not defaults but that
 # users commonly want.