Store previous verbose setting before it can fail in tests.

- thanks to hoisting it can be wrongly set to nil instead of previous value
Nittarab · Jul 31, 2022 · bfdf158 · bfdf158
1 parent a01d6f8
commit bfdf158
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/activerecord/lib/active_record/tasks/database_tasks.rb b/activerecord/lib/active_record/tasks/database_tasks.rb
@@ -252,11 +252,11 @@ def truncate_all(environment = env)
       end
 
       def migrate(version = nil)
-        check_target_version
-
         scope = ENV["SCOPE"]
         verbose_was, Migration.verbose = Migration.verbose, verbose?
 
+        check_target_version
+
         Base.connection.migration_context.migrate(target_version) do |migration|
           if version.blank?
             scope.blank? || scope == migration.scope