Single tenant by default & update tests

app/models/concerns/multi_tenant.rb

@@ -3,7 +3,7 @@ module MultiTenant
 
   class_methods do
     def accepting_signups?
-      ENV.fetch("MULTI_TENANT", "true") == "true" || Account.none?
+      ENV.fetch("MULTI_TENANT", "false") == "true" || Account.none?
     end
   end
 end

test/controllers/signups_controller_test.rb

@@ -65,17 +65,14 @@ class SignupsControllerTest < ActionDispatch::IntegrationTest
   end
 
   test "redirects to session#new when single_tenant and user exists" do
-    previous_multi_tenant = ENV["MULTI_TENANT"]
-    ENV["MULTI_TENANT"] = "false"
-
     users(:david)
 
-    untenanted do
-      get new_signup_path
+    in_single_tenant_mode do
+      untenanted do
+        get new_signup_path
 
-      assert_redirected_to new_session_url
+        assert_redirected_to new_session_url
+      end
     end
-  ensure
-    ENV["MULTI_TENANT"] = previous_multi_tenant
   end
 end

test/test_helper.rb

@@ -1,4 +1,6 @@
 ENV["RAILS_ENV"] ||= "test"
+ENV["MULTI_TENANT"] ||= "true"
+
 require_relative "../config/environment"
 
 require "rails/test_help"

test/test_helpers/session_test_helper.rb

@@ -57,4 +57,12 @@ module SessionTestHelper
   ensure
     integration_session.default_url_options[:script_name] = original_script_name
   end
+
+  def in_single_tenant_mode
+    previous_multi_tenant = ENV["MULTI_TENANT"]
+    ENV["MULTI_TENANT"] = "false"
+    yield
+  ensure
+    ENV["MULTI_TENANT"] = previous_multi_tenant
+  end
 end