leafy-logger 0.5.0-java → 0.6.0-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d22064eade496723c9f18d44e38ed9f0b1c65869
-  data.tar.gz: f902ff68c2622a53d20fa8f0f7f067c40d72d142
+  metadata.gz: 82ed92ed106adeed17aa9a89dbc0a4610c591051
+  data.tar.gz: b3713e451293841fcf2ecafa8b39aaea10f00e0c
 SHA512:
-  metadata.gz: 3853809fc106ed7259221e764ee53f77a8586ef6122983d8ca40983b6735fef7bc382c8db89df595e87e753865613f1d06e2635c732b961dd4b7f549ca7c7fe1
-  data.tar.gz: 4465bc3314315cda8d09bf55f79b840ce9956dfdcaabdf5d3f6a79c11ff0711b8ec031a6a8076387eb19886274fbb7b18010fafab67f709770a539de576db4e9
+  metadata.gz: 9f24e22c6645281ca38959272d9059682d71363a3d43372a0b664c9f849b0e594baa53c9b844b5c3d091a3ee6d855bc43e9be426d379aae911d027ddb3e876ab
+  data.tar.gz: 32c13ca48b9a4585dd8777ecd096862ac4a0a6e882b199bad6687e240d5c4ce7e2099dc0fbaedbae3fa393bcb65e5cd6e64198407ca47eaa065cea2c4d6f31d1

data/lib/leafy/logger/factory.rb

@@ -1,5 +1,6 @@
 require 'leafy/logger'
 require 'leafy/logger/appender_factories'
+require 'leafy/logger/proxy_logger'
 require 'yaml'
 require 'stringio'
 require 'jruby/synchronized'
@@ -22,7 +23,7 @@ module Leafy
       end
 
       def self.get_logger( name )
-        org.slf4j.LoggerFactory.get_logger name
+        ProxyLogger.new( org.slf4j.LoggerFactory.get_logger( name ) )
       end
 
       def self.bootstrap

data/lib/leafy/logger/proxy_logger.rb

@@ -0,0 +1,30 @@
+module Leafy
+  module Logger
+    class ProxyLogger
+      def initialize(logger)
+        @logger = logger
+      end
+
+      def method_missing(sym, *args, &block)
+        if @logger.respond_to?(sym)
+          @logger.send(sym, *args)
+        else
+          super
+        end
+      end
+
+      def respond_to?(sym, include_private = false)
+        @logger.respond_to?(sym, include_private)
+      end
+
+      # slf4j does not include the fatal level
+      def fatal(*args)
+        error(*args)
+      end
+
+      # Provided for compatibility with the stdlib Logger interface. Not used, since the logger
+      # that we're proxying has a much better formatter! (Right?)
+      attr_accessor :formatter
+    end
+  end
+end

data/lib/leafy/logger/version.rb

@@ -1,6 +1,6 @@
 module Leafy
   module Logger
-    VERSION = '0.5.0'.freeze
+    VERSION = '0.6.0'.freeze
   end
 end
 

data/spec/factory_spec.rb

@@ -48,7 +48,7 @@ describe Leafy::Logger::Factory do
     subject.level = 'INFO'
     expect( subject.level ).to eq 'INFO'
 
-    subject.level 'DEBUG'    
+    subject.level 'DEBUG'
     expect( subject.level ).to eq 'DEBUG'
   end
 
@@ -61,7 +61,7 @@ describe Leafy::Logger::Factory do
     logger2.debug 'debug2'
     logger2.info 'good'
     logger2.warn 'good'
- 
+
     subject.stop
 
     lines = File.read( log ).split( /\n/ )
@@ -124,6 +124,10 @@ describe Leafy::Logger::Factory do
 
   end
 
+  it 'uses ProxyLoggers' do
+    expect( logger1 ).to be_a Leafy::Logger::ProxyLogger
+  end
+
   it 'fails on missing yaml configuration' do
     expect {Leafy::Logger::Factory.new_from_yaml( yaml + '.gone' ) }.to raise_error( Exception )
   end

data/spec/proxy_logger_spec.rb

@@ -0,0 +1,27 @@
+require_relative 'setup'
+require 'leafy/logger/proxy_logger'
+require 'logger'
+
+describe Leafy::Logger::ProxyLogger do
+  subject(:proxy) { described_class.new(logger) }
+  let(:logger) { instance_double(Logger, debug: debug_result, error: error_result) }
+  let(:debug_result) { 'debug result' }
+  let(:error_result) { 'error result' }
+
+  describe '#method_missing' do
+    it 'should call the logger when given a method that exists' do
+      expect(proxy.debug).to eq debug_result
+    end
+
+    it 'should raise a NameError when given a method that does not exist' do
+      expect { proxy.lol }.to raise_error(NameError)
+    end
+  end
+
+  describe '#fatal' do
+    it 'should call #error' do
+      expect(proxy.fatal(:a)).to eq error_result
+      expect(logger).to have_received(:error).with(:a)
+    end
+  end
+end

metadata

@@ -1,25 +1,25 @@
 --- !ruby/object:Gem::Specification
 name: leafy-logger
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: java
 authors:
 - christian meier
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2015-05-12 00:00:00.000000000 Z
+date: 2015-06-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jar-dependencies
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.1.8
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.1.8
   prerelease: false
@@ -28,26 +28,26 @@ dependencies:
   name: leafy-metrics
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: 0.6.0
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: 0.6.0
   prerelease: false
   type: :runtime
 - !ruby/object:Gem::Dependency
   name: rspec
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3.1'
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3.1'
   prerelease: false
@@ -56,12 +56,12 @@ dependencies:
   name: yard
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.8.7
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.8.7
   prerelease: false
@@ -70,12 +70,12 @@ dependencies:
   name: rake
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '10.2'
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '10.2'
   prerelease: false
@@ -87,7 +87,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".gitignore"
 - Gemfile
 - LICENSE
 - README.md
@@ -98,10 +98,12 @@ files:
 - lib/leafy/logger.rb
 - lib/leafy/logger/appender_factories.rb
 - lib/leafy/logger/factory.rb
+- lib/leafy/logger/proxy_logger.rb
 - lib/leafy/logger/version.rb
 - spec/appender_factories_spec.rb
 - spec/factory_spec.rb
 - spec/logging.yml
+- spec/proxy_logger_spec.rb
 - spec/setup.rb
 homepage: https://github.com/lookout/leafy
 licenses:
@@ -113,12 +115,12 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: