Remove deprecated #all_foreign_keys_valid? from database adapters

Author: rafaelfranca

activerecord/CHANGELOG.md

@@ -1,3 +1,7 @@
+*   Remove deprecated `#all_foreign_keys_valid?` from database adapters.
+
+    *Rafael Mendonça França*
+
 *   Remove deprecated support to passing coder and class as second argument to `serialize`.
 
     *Rafael Mendonça França*

activerecord/lib/active_record/connection_adapters/abstract_adapter.rb

@@ -647,15 +647,6 @@ def disable_referential_integrity
         yield
       end
 
-      # Override to check all foreign key constraints in a database.
-      def all_foreign_keys_valid?
-        check_all_foreign_keys_valid!
-        true
-      rescue ActiveRecord::StatementInvalid
-        false
-      end
-      deprecate :all_foreign_keys_valid?, deprecator: ActiveRecord.deprecator
-
       # Override to check all foreign key constraints in a database.
       # The adapter should raise a +ActiveRecord::StatementInvalid+ if foreign key
       # constraints are not met.

activerecord/test/cases/adapter_test.rb

@@ -459,12 +459,6 @@ def test_truncate_tables_with_query_cache
       @connection.disable_query_cache!
     end
 
-    def test_all_foreign_keys_valid_is_deprecated
-      assert_deprecated(ActiveRecord.deprecator) do
-        @connection.all_foreign_keys_valid?
-      end
-    end
-
     # test resetting sequences in odd tables in PostgreSQL
     if ActiveRecord::Base.connection.respond_to?(:reset_pk_sequence!)
       require "models/movie"

guides/source/7_2_release_notes.md

@@ -142,6 +142,8 @@ Please refer to the [Changelog][active-record] for detailed changes.
 
 *   Remove deprecated support to passing coder and class as second argument to `serialize`.
 
+*   Remove deprecated `#all_foreign_keys_valid?` from database adapters.
+
 ### Deprecations
 
 *   Deprecate `Rails.application.config.active_record.allow_deprecated_singular_associations_name`