semantic_logger 4.2.0 → 4.2.1

data/test/measure_test.rb

@@ -46,7 +46,7 @@ class MeasureTest < Minitest::Test
 
         it ':exception' do
           assert_raises RuntimeError do
-            logger.send(measure_level, 'hello world', payload: payload) { raise RuntimeError.new('Test') }
+            logger.send(measure_level, 'hello world', payload: payload) { raise 'Test' }
           end
 
           assert log = log_message
@@ -57,7 +57,7 @@ class MeasureTest < Minitest::Test
 
         it ':on_exception_level' do
           assert_raises RuntimeError do
-            logger.measure(level, 'hello world', payload: payload, on_exception_level: :fatal) { raise RuntimeError.new('Test') }
+            logger.measure(level, 'hello world', payload: payload, on_exception_level: :fatal) { raise 'Test' }
           end
 
           assert log = log_message
@@ -69,7 +69,7 @@ class MeasureTest < Minitest::Test
         describe 'log_exception' do
           it 'default' do
             assert_raises RuntimeError do
-              logger.send(measure_level, 'hello world') { raise RuntimeError.new('Test') }
+              logger.send(measure_level, 'hello world') { raise 'Test' }
             end
 
             assert log = log_message
@@ -79,7 +79,7 @@ class MeasureTest < Minitest::Test
 
           it ':full' do
             assert_raises RuntimeError do
-              logger.send(measure_level, 'hello world', log_exception: :full) { raise RuntimeError.new('Test') }
+              logger.send(measure_level, 'hello world', log_exception: :full) { raise 'Test' }
             end
 
             assert log = log_message
@@ -91,7 +91,7 @@ class MeasureTest < Minitest::Test
 
           it ':partial' do
             assert_raises RuntimeError do
-              logger.send(measure_level, 'hello world', log_exception: :partial) { raise RuntimeError.new('Test') }
+              logger.send(measure_level, 'hello world', log_exception: :partial) { raise 'Test' }
             end
 
             assert log = log_message
@@ -101,7 +101,7 @@ class MeasureTest < Minitest::Test
 
           it ':none' do
             assert_raises RuntimeError do
-              logger.send(measure_level, 'hello world', log_exception: :none) { raise RuntimeError.new('Test') }
+              logger.send(measure_level, 'hello world', log_exception: :none) { raise 'Test' }
             end
 
             assert log = log_message
@@ -124,7 +124,7 @@ class MeasureTest < Minitest::Test
 
             assert log = log_message
             assert log.backtrace
-            assert log.backtrace.size > 0
+            assert log.backtrace.size.positive?
 
             # Extract file name and line number from backtrace
             h = log.to_h
@@ -171,7 +171,7 @@ class MeasureTest < Minitest::Test
 
         it ':exception' do
           assert_raises RuntimeError do
-            logger.measure(level, 'hello world', payload: payload) { raise RuntimeError.new('Test') }
+            logger.measure(level, 'hello world', payload: payload) { raise 'Test' }
           end
 
           assert log = log_message
@@ -182,7 +182,7 @@ class MeasureTest < Minitest::Test
 
         it ':on_exception_level' do
           assert_raises RuntimeError do
-            logger.measure(level, 'hello world', payload: payload, on_exception_level: :fatal) { raise RuntimeError.new('Test') }
+            logger.measure(level, 'hello world', payload: payload, on_exception_level: :fatal) { raise 'Test' }
           end
 
           assert log = log_message
@@ -205,7 +205,7 @@ class MeasureTest < Minitest::Test
 
             assert log = log_message
             assert log.backtrace
-            assert log.backtrace.size > 0
+            assert log.backtrace.size.positive?
 
             # Extract file name and line number from backtrace
             h = log.to_h
@@ -253,7 +253,7 @@ class MeasureTest < Minitest::Test
 
         it ':exception' do
           assert_raises RuntimeError do
-            logger.send(measure_level, message: 'hello world', payload: payload) { raise RuntimeError.new('Test') }
+            logger.send(measure_level, message: 'hello world', payload: payload) { raise 'Test' }
           end
 
           assert log = log_message
@@ -264,7 +264,7 @@ class MeasureTest < Minitest::Test
 
         it ':on_exception_level' do
           assert_raises RuntimeError do
-            logger.send(measure_level, message: 'hello world', payload: payload, on_exception_level: :fatal) { raise RuntimeError.new('Test') }
+            logger.send(measure_level, message: 'hello world', payload: payload, on_exception_level: :fatal) { raise 'Test' }
           end
 
           assert log = log_message
@@ -287,7 +287,7 @@ class MeasureTest < Minitest::Test
 
             assert log = log_message
             assert log.backtrace
-            assert log.backtrace.size > 0
+            assert log.backtrace.size.positive?
 
             # Extract file name and line number from backtrace
             h = log.to_h
@@ -296,7 +296,6 @@ class MeasureTest < Minitest::Test
           end
         end
       end
-
     end
 
     describe 'return' do
@@ -343,6 +342,5 @@ class MeasureTest < Minitest::Test
       end
       'Bad'
     end
-
   end
 end

data/test/metric/new_relic_test.rb

@@ -6,7 +6,6 @@ require_relative '../test_helper'
 module Metric
   class NewRelicTest < Minitest::Test
     describe SemanticLogger::Appender::NewRelic do
-
       before do
         @appender = SemanticLogger::Metric::NewRelic.new
         @message  = 'AppenderNewRelicTest log message'
@@ -14,7 +13,7 @@ module Metric
 
       it 'logs counter metric' do
         name = amount = nil
-        NewRelic::Agent.stub(:increment_metric, -> name_, amount_ { name = name_, amount = amount_ }) do
+        NewRelic::Agent.stub(:increment_metric, ->(name_, amount_) { name = name_, amount = amount_ }) do
           @appender.info(message: @message, metric: 'User/authenticated')
         end
         assert_equal 'Custom/User/authenticated', name.first
@@ -23,7 +22,7 @@ module Metric
 
       it 'logs duration metric' do
         name = duration = nil
-        NewRelic::Agent.stub(:record_metric, -> name_, duration_ { name = name_, duration = duration_ }) do
+        NewRelic::Agent.stub(:record_metric, ->(name_, duration_) { name = name_, duration = duration_ }) do
           @appender.measure_info(message: @message, metric: 'User/authenticate') do
             sleep 0.001
           end

data/test/metric/signalfx_test.rb

@@ -27,7 +27,7 @@ module Appender
           else
             response_mock = Struct.new(:code, :body)
             request       = nil
-            appender.http.stub(:request, -> r { request = r; response_mock.new('200', 'ok') }) do
+            appender.http.stub(:request, ->(r) { request = r; response_mock.new('200', 'ok') }) do
               appender.log(@log)
             end
           end
@@ -50,7 +50,7 @@ module Appender
 
         it 'whitelists dimensions' do
           @log.named_tags               = {user_id: 47, application: 'sample', tracking_number: 7474, session_id: 'hsdhngsd'}
-          appender.formatter.dimensions = [:user_id, :application]
+          appender.formatter.dimensions = %i[user_id application]
           assert response
         end
       end
@@ -62,17 +62,16 @@ module Appender
 
         it 'not logs when no metric' do
           @log.message = 'blah'
-          @log.metric = nil
+          @log.metric  = nil
           refute appender.should_log?(@log)
         end
 
         it 'logs metric only metric with dimensions' do
-          @log.metric = 'Filter/count'
+          @log.metric     = 'Filter/count'
           @log.dimensions = {action: 'hit', user: 'jbloggs', state: 'FL'}
           assert appender.should_log?(@log)
         end
       end
-
     end
   end
 end

data/test/semantic_logger_test.rb

@@ -124,7 +124,7 @@ class SemanticLoggerTest < Minitest::Test
             logger.info('Hello world')
 
             assert log = log_message
-            assert_equal %w(12345 DJHSFK), log.tags
+            assert_equal %w[12345 DJHSFK], log.tags
           end
         end
 
@@ -190,7 +190,7 @@ class SemanticLoggerTest < Minitest::Test
             logger.info('Hello world')
 
             assert log = log_message
-            assert_equal %w(12345), log.tags
+            assert_equal %w[12345], log.tags
           end
         end
       end
@@ -272,7 +272,32 @@ class SemanticLoggerTest < Minitest::Test
         end
       end
 
-    end
+      describe '.on_log' do
+        before do
+          SemanticLogger.default_level = :info
+        end
+
+        after do
+          SemanticLogger::Processor.instance.appender.log_subscribers = nil
+        end
+
+        it 'registers a log listener' do
+          SemanticLogger.on_log do |log|
+            log.set_context(:custom_info, 'test')
+          end
+
+          assert_equal :info, SemanticLogger.default_level
+          assert_equal :info, logger.level
+          logger.silence(:debug) do
+            logger.debug('hello world')
+          end
 
+          assert log = log_message
+          assert_equal :debug, log.level
+          assert_equal 'hello world', log.message
+          assert_equal 'test', log.context[:custom_info]
+        end
+      end
+    end
   end
 end

data/test/test_helper.rb

@@ -8,25 +8,24 @@ require 'minitest/autorun'
 require 'minitest/reporters'
 require 'minitest/stub_any_instance'
 require 'semantic_logger'
-#require 'logger'
+# require 'logger'
 require_relative 'in_memory_appender'
 require_relative 'in_memory_batch_appender'
 require_relative 'in_memory_metrics_appender'
 require_relative 'in_memory_appender_helper'
 require 'awesome_print'
 
-#Minitest::Reporters.use! Minitest::Reporters::SpecReporter.new
+# Minitest::Reporters.use! Minitest::Reporters::SpecReporter.new
 class Minitest::Test
   # Use AwesomePrint to display diffs
   define_method :mu_pp, &:awesome_inspect
 
   # Use AwesomePrint to display messages
-  def message msg = nil, ending = nil, &default
+  def message(msg = nil, ending = nil)
     proc {
-      msg = msg.call.chomp(".") if Proc === msg
-      custom_message = "#{msg.ai}.\n" unless msg.nil? or msg.to_s.empty?
-      "#{custom_message}#{default.call}#{ending || "."}"
+      msg            = msg.call.chomp('.') if Proc === msg
+      custom_message = "#{msg.ai}.\n" unless msg.nil? || msg.to_s.empty?
+      "#{custom_message}#{yield}#{ending || '.'}"
     }
   end
-
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: semantic_logger
 version: !ruby/object:Gem::Version
-  version: 4.2.0
+  version: 4.2.1
 platform: ruby
 authors:
 - Reid Morrison
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-28 00:00:00.000000000 Z
+date: 2018-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -24,8 +24,7 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
-description: Next generation logging system for Ruby to support highly concurrent,
-  high throughput, low latency enterprise systems
+description: 
 email:
 - reidmo@gmail.com
 executables: []
@@ -133,7 +132,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.1'
+      version: '2.3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -141,46 +140,47 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.11
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
-summary: Scalable, next generation enterprise logging for Ruby
+summary: Feature rich logging framework, and replacement for existing Ruby & Rails
+  loggers.
 test_files:
-- test/appender/async_batch_test.rb
-- test/appender/async_test.rb
-- test/appender/bugsnag_test.rb
-- test/appender/elasticsearch_http_test.rb
-- test/appender/elasticsearch_test.rb
-- test/appender/file_test.rb
-- test/appender/graylog_test.rb
-- test/appender/honeybadger_test.rb
-- test/appender/http_test.rb
-- test/appender/kafka_test.rb
-- test/appender/mongodb_test.rb
-- test/appender/new_relic_test.rb
-- test/appender/newrelic_rpm.rb
 - test/appender/sentry_test.rb
 - test/appender/splunk_http_test.rb
+- test/appender/honeybadger_test.rb
 - test/appender/splunk_test.rb
+- test/appender/graylog_test.rb
 - test/appender/syslog_test.rb
+- test/appender/async_batch_test.rb
+- test/appender/newrelic_rpm.rb
 - test/appender/tcp_test.rb
-- test/appender/udp_test.rb
+- test/appender/mongodb_test.rb
+- test/appender/elasticsearch_http_test.rb
+- test/appender/new_relic_test.rb
 - test/appender/wrapper_test.rb
-- test/concerns/compatibility_test.rb
-- test/debug_as_trace_logger_test.rb
+- test/appender/file_test.rb
+- test/appender/udp_test.rb
+- test/appender/async_test.rb
+- test/appender/kafka_test.rb
+- test/appender/elasticsearch_test.rb
+- test/appender/bugsnag_test.rb
+- test/appender/http_test.rb
+- test/logger_test.rb
+- test/loggable_test.rb
+- test/measure_test.rb
+- test/in_memory_batch_appender.rb
+- test/in_memory_appender_helper.rb
 - test/formatters/color_test.rb
-- test/formatters/default_test.rb
-- test/formatters/one_line_test.rb
 - test/formatters/signalfx_test.rb
+- test/formatters/one_line_test.rb
+- test/formatters/default_test.rb
+- test/semantic_logger_test.rb
 - test/formatters_test.rb
-- test/in_memory_appender.rb
-- test/in_memory_appender_helper.rb
-- test/in_memory_batch_appender.rb
-- test/in_memory_metrics_appender.rb
-- test/loggable_test.rb
-- test/logger_test.rb
-- test/measure_test.rb
+- test/test_helper.rb
+- test/debug_as_trace_logger_test.rb
 - test/metric/new_relic_test.rb
 - test/metric/signalfx_test.rb
-- test/semantic_logger_test.rb
-- test/test_helper.rb
+- test/in_memory_metrics_appender.rb
+- test/concerns/compatibility_test.rb
+- test/in_memory_appender.rb