semantic_logger 2.19.0 → 2.20.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8edd7638dee098976eeee0f818ea64c213da2a1f
-  data.tar.gz: a03fac478e80e0784e765a7c2f28c8c6b929b94b
+  metadata.gz: 141016c8a502a158974b0dfe589a90b5cd650826
+  data.tar.gz: 09cb7afdf37afe6280098b01f7257c8a21b5995e
 SHA512:
-  metadata.gz: 9f85f11748ef85c23dbd912fa5a690d57cd50af051edaeb37014ba30ef987e9f22866b0632f457327c78f6243e88b242c2fe3cc4c46135b8b2f64c298de9961e
-  data.tar.gz: 1b5ac351139f57b94743022465f86030a8959df41e696fdcb048b4f3f07adfa1736bc21d9f4de74c6b8e0434a5d32ee44c79b7c26df4272a0201a78da8bb017a
+  metadata.gz: 3f734075af2a306b37560903beb0d0003f912706f6d68446914832cbb76a404c7017dd536c587eb155309a1128ebf328719f33c727b0ca129d03d2bea8f6fa21
+  data.tar.gz: a96ffd5c36c47e4e7787dc53970dafe092203f6e717f6aba4289217b45e14b1e36317151dcbe2af60178e78d3266ed92f23bd3f7ba6c39d1592d5fbd37794458

data/lib/semantic_logger/loggable.rb

@@ -29,65 +29,21 @@ module SemanticLogger
 
     def self.included(base)
       base.class_eval do
+        # Returns [SemanticLogger::Logger] class level logger
         def self.logger
           @semantic_logger ||= SemanticLogger[self]
         end
-      end
-    end
 
-    # Also make the logger available as an instance method MixIn
-    # The class logger can be replaced using an instance specific #logger= below
-    def logger
-      @semantic_logger ||= self.class.logger
-    end
+        # Returns [SemanticLogger::Logger] instance level logger
+        def logger
+          @semantic_logger ||= self.class.logger
+        end
 
-    # Set instance specific logger
-    #
-    # By default instances of the class will use the class logger. Sometimes it
-    # is useful to be able to add instance specific logging data to the class name.
-    #
-    # For example, server or host_name that the class instance is using.
-    #
-    # Example:
-    #   require 'semantic_logger'
-    #   SemanticLogger.default_level = :debug
-    #   SemanticLogger.add_appender(STDOUT)
-    #
-    #   class MyClass
-    #     include SemanticLogger::Loggable
-    #
-    #     def self.my_name=(my_name)
-    #       # Use class level logger that only logs class name
-    #       logger.info "My name is changed to #{my_name}"
-    #
-    #       @@my_name = my_name
-    #     end
-    #
-    #     def initialize(host_name)
-    #       # Add host_name to every log entry in this logging instance
-    #       self.logger = SemanticLogger["#{self.class.name} [#{host_name}]"]
-    #
-    #       logger.info "Started server"
-    #     end
-    #
-    #     def check
-    #       logger.debug "Checking..."
-    #     end
-    #   end
-    #
-    #   MyClass.my_name = "Joe"
-    #
-    #   mine = MyClass.new('server.com')
-    #   mine.check
-    #
-    # # Generates the following log output:
-    #
-    # 2013-04-02 15:08:42.368574 I [37279:70198560687720] MyClass -- My name is changed to Joe
-    # 2013-04-02 15:08:42.369934 I [37279:70198560687720] MyClass [server.com] -- Started server
-    # 2013-04-02 15:08:42.371171 D [37279:70198560687720] MyClass [server.com] -- Checking...
-    #
-    def logger=(logger)
-      @semantic_logger = logger
+        # Replace instance level logger
+        def logger=(logger)
+          @semantic_logger = logger
+        end
+      end
     end
 
   end

data/lib/semantic_logger/version.rb

@@ -1,3 +1,3 @@
 module SemanticLogger #:nodoc
-  VERSION = '2.19.0'
+  VERSION = '2.20.0'
 end

data/test/loggable_test.rb

@@ -8,11 +8,25 @@ end
 # Unit Test for SemanticLogger::Appender::File
 #
 class AppenderFileTest < Minitest::Test
+  module Perform
+    def perform
+      logger.info 'perform'
+    end
+  end
+
   class Base
     include SemanticLogger::Loggable
+    include Perform
+  end
+
+  module Process
+    def process
+      logger.info 'process'
+    end
   end
 
   class Subclass < Base
+    include Process
   end
 
   describe SemanticLogger::Loggable do
@@ -36,6 +50,26 @@ class AppenderFileTest < Minitest::Test
         refute_equal child_logger, base.logger
         assert_equal child_logger.object_id, subclass.logger.object_id
       end
+
+      it 'should allow mixins to call parent logger' do
+        base = Base.new
+        base.perform
+        called = false
+        Base.logger.stub(:info, -> description { called = true if description == 'perform' }) do
+          base.perform
+        end
+        assert called, 'Did not call the correct logger'
+      end
+
+      it 'should allow child mixins to call parent logger' do
+        subclass = Subclass.new
+        subclass.process
+        called = false
+        Subclass.logger.stub(:info, -> description { called = true if description == 'process' }) do
+          subclass.process
+        end
+        assert called, 'Did not call the correct logger'
+      end
     end
 
     describe 'logger' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: semantic_logger
 version: !ruby/object:Gem::Version
-  version: 2.19.0
+  version: 2.20.0
 platform: ruby
 authors:
 - Reid Morrison
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-05 00:00:00.000000000 Z
+date: 2015-12-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby