Add ActiveModelSerializers.logger with default null device

Author: bf4

lib/active_model/serializer/railtie.rb

@@ -1,6 +1,12 @@
 require 'rails/railtie'
 module ActiveModel
   class Railtie < Rails::Railtie
+    initializer 'active_model_serializers.logger' do
+      ActiveSupport.on_load(:action_controller) do
+        ActiveModelSerializers.logger = ActionController::Base.logger
+      end
+    end
+
     initializer 'generators' do |app|
       app.load_generators
       require 'generators/serializer/resource_override'

lib/active_model_serializers.rb

@@ -1,4 +1,12 @@
+require 'logger'
+require 'active_model'
+require "active_support/railtie"
+require 'action_controller'
+require "action_controller/railtie"
 module ActiveModelSerializers
+  mattr_accessor :logger
+  self.logger = Rails.logger || Logger.new(IO::NULL)
+
   module_function
 
   def silence_warnings
@@ -10,22 +18,17 @@ def silence_warnings
   end
 
 end
-require 'active_model'
-require 'active_model/serializer/version'
+
 require 'active_model/serializer'
 require 'active_model/serializable_resource'
+require 'active_model/serializer/version'
 
-begin
-  require 'active_model/serializer/railtie'
-  require 'action_controller'
-  require 'action_controller/serialization'
-
-  ActiveSupport.on_load(:action_controller) do
-    include ::ActionController::Serialization
-    ActionDispatch::Reloader.to_prepare do
-      ActiveModel::Serializer.serializers_cache.clear
-    end
+require 'action_controller/serialization'
+ActiveSupport.on_load(:action_controller) do
+  include ::ActionController::Serialization
+  ActionDispatch::Reloader.to_prepare do
+    ActiveModel::Serializer.serializers_cache.clear
   end
-rescue LoadError
-  # rails not installed, continuing
 end
+
+require 'active_model/serializer/railtie'

test/logger_test.rb

@@ -0,0 +1,19 @@
+require 'test_helper'
+
+class ActiveModelSerializers::LoggerTest < Minitest::Test
+
+  def test_logger_is_set_to_action_controller_logger_when_initializer_runs
+    assert_equal ActiveModelSerializers.logger, ActionController::Base.logger
+  end
+
+  def test_logger_can_be_set
+    original_logger = ActiveModelSerializers.logger
+    logger = Logger.new(STDOUT)
+
+    ActiveModelSerializers.logger = logger
+
+    assert_equal ActiveModelSerializers.logger, logger
+  ensure
+    ActiveModelSerializers.logger = original_logger
+  end
+end