Merge pull request rails#53851 from rails/rm-invalidate_transaction

`all_open_transactions` should not include invalidated transactions

Author: rafaelfranca

activerecord/lib/active_record.rb

@@ -548,8 +548,10 @@ def self.all_open_transactions # :nodoc:
     open_transactions = []
     Base.connection_handler.each_connection_pool do |pool|
       if active_connection = pool.active_connection
-        if active_connection.current_transaction.open? && active_connection.current_transaction.joinable?
-          open_transactions << active_connection.current_transaction
+        current_transaction = active_connection.current_transaction
+
+        if current_transaction.open? && current_transaction.joinable? && !current_transaction.state.invalidated?
+          open_transactions << current_transaction
         end
       end
     end

activerecord/test/cases/transactions_test.rb

@@ -67,6 +67,14 @@ def test_after_all_transactions_commit
       raise ActiveRecord::Rollback
     end
     assert_equal 0, called
+
+    called = 0
+    Topic.transaction do |transaction|
+      transaction.instance_variable_get(:@internal_transaction).invalidate!
+      ActiveRecord.after_all_transactions_commit { called += 1 }
+      assert_equal 1, called
+    end
+    assert_equal 1, called
   end
 
   def test_after_current_transaction_commit_multidb_nested_transactions