Merge pull request rails#51939 from ElMassimo/fix-broadcast-logger-compatibility

Improve compatibility for `ActiveSupport::BroadcastLogger`

Author: rafaelfranca

activesupport/lib/active_support/broadcast_logger.rb

@@ -229,6 +229,7 @@ def initialize_copy(other)
     private
       def dispatch(&block)
         @broadcasts.each { |logger| block.call(logger) }
+        true
       end
 
       def method_missing(name, ...)

activesupport/test/broadcast_logger_test.rb

@@ -302,6 +302,11 @@ def info(msg, &block)
       assert_same logger, broadcast_logger.broadcasts.sole
     end
 
+    test "logging always returns true" do
+      assert_equal true, @logger.info("Hello")
+      assert_equal true, @logger.error("Hello")
+    end
+
     class CustomLogger
       attr_reader :adds, :closed, :chevrons
       attr_accessor :level, :progname, :formatter, :local_level