yeti_logger 3.0.0 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 3e5e9a8554d3123f83bd27c2cefe1663025fa9b9
-  data.tar.gz: 8f79e414571cbc0c5815eb5ce888892be8114231
+SHA256:
+  metadata.gz: a7ebf769a9b7ffda8af122c0f39ccf931af0bf2a91f5009755f441ec4df9e3d1
+  data.tar.gz: 5230ebd2a81295a96370c7e2915f59e9c45d75873a4cdf035f00b185d17f31be
 SHA512:
-  metadata.gz: 3fa2b8fedb91ee5f6cb0d0dcf41b4219cc16a5c98092c574ee55576d2c1ad9e0ce702c2d6e05c87af6445fbf80016e8f4e9134afc0d210a7554728fc8e99432a
-  data.tar.gz: 61e988b8d18bc89954b49cbd49a6d4d288280bc53312f9507f15e7404ac3e828f0fd433548599005ca49f8136c41caf02c47db2bdafb12f230dc65ae0823ebf4
+  metadata.gz: b6d144db8cfccda8ecc46ede21c1b7b5838cb31a208d667cfefa0a00c6696a4908fda101458235cd128f76ab276cbf5623e93159aba2b1ca7197bbdcc3baaa54
+  data.tar.gz: 384158be6d4621ce55e5347dd69ace02b25444c4ff3f0e3e7cb389ea7fb736b478d6ff8d984c26c78e58ef9ecc9c9e4d603f2776f814d840b908f3e876bf0321

data/.ruby-gemset

@@ -0,0 +1 @@
+yesware

data/.ruby-version

@@ -1 +1 @@
-ruby-2.2.2
+ruby-2.7.3

data/.travis.yml

@@ -1,11 +1,11 @@
 language: ruby
 
 rvm:
-  - 1.9.3
   - 2.0
   - 2.1
-  - 2.2
-  - jruby-19mode
+  - 2.2.2
+  - 2.3.1
+  - jruby-20mode
 
 script:
   - bundle exec rake ci

data/CHANGELOG.md

@@ -1,4 +1,11 @@
 # yeti_logger changelog
 
+## v3.2.0
+- Added configuration to override debug logging for specific users
+
+## v3.1.0
+- Added `YetiLogger::TestHelper.expect_to_not_see_log_message[s]` for testing
+  that given messages were not logged at the given log level.
+
 ## v3.0.0
 - First public release

data/Gemfile

@@ -1,4 +1,8 @@
 source 'https://rubygems.org'
 
+if RUBY_VERSION < '2.2.2'
+  gem 'activesupport', '< 5'
+end
+
 # Specify your gem's dependencies in yeti_logger.gemspec
 gemspec

data/README.md

@@ -118,6 +118,25 @@ this behavior, pass in the serialized format for the hash, such as:
     log_info { hash.to_s }
     log_info { hash.to_my_log_format }
 
+### Override Debug Logging Level
+
+
+You can configure conditions under which to override the app-wide log level for `log_debug` calls, so that it emit logs from `log_debug` calls even if the app's log level is configured as `info` or higher. For example, using  the [settings-in-redis gem](https://github.com/Yesware/settings-in-redis), you can specify user IDs in `Settings.extra_logging_user_ids` so that if the log hash contains those user IDs in its `user_id` key, we'll log at `info` level instead:
+
+```ruby
+YetiLogger.configure do |config|
+  def config.promote_debug_to_info?(log_hash)
+    Settings.extra_logging_user_ids.is_a?(Array) &&
+    log_hash.is_a?(Hash) &&
+    log_hash.with_indifferent_access[:user_id].to_i.in?(Settings.extra_logging_user_ids)
+  end
+end
+```
+
+In this example, the user ID is taken from the `user_id` key of the log payload, so this won't work with the block syntax.
+
+This feature enables us to get more logging for (and hence more insight into) specific users who are experiencing problems, without hardcoding user IDs or increasing the overall log volume with logs we're not interested in. Depending on your configuration, you may avoid setting environment variables and restarting the app too.
+
 ## Test Support
 
 There are a couple helpers provided to support testing of YetiLogger calls. All

data/lib/yeti_logger.rb

@@ -74,14 +74,18 @@ module YetiLogger
   # passed to the log method.
   # Define these methods explicitly allows the use of yield.
   module LogMethods
+    # See usage at https://github.com/Yesware/yeti_logger/blob/master/README.md#user-based-logging
     def log_debug(obj = nil, ex = nil)
-      if YetiLogger.logger.level <= Logger::DEBUG
+      should_log_as_info = YetiLogger.try(:promote_debug_to_info?, obj)
+
+      if YetiLogger.logger.level <= Logger::DEBUG ||
+         (should_log_as_info && YetiLogger.logger.level <= Logger::INFO)
         msg = if block_given?
                 MessageFormatters.build_log_message(log_class_name, yield)
               else
                 MessageFormatters.build_log_message(log_class_name, obj, ex)
               end
-        YetiLogger.logger.send(:debug, msg)
+        YetiLogger.logger.send(should_log_as_info ? :info : :debug, msg)
       end
     end
 

data/lib/yeti_logger/test_helper.rb

@@ -24,17 +24,7 @@ module YetiLogger::TestHelper
 
   # Plural version of above
   def expect_to_see_log_messages(messages, level = :debug, &block)
-    log_messages = []
-
-    allow(YetiLogger.logger).to receive(level) do |log_line|
-      log_messages << log_line
-    end
-
-    block.call
-
-    # There is no unstub in rspec 3, but the closest to that would be to
-    # continue to stub it, but defer to the original implementation.
-    allow(YetiLogger.logger).to receive(level).and_call_original
+    log_messages = get_log_messages(level, &block)
 
     # Find each message, removing the first occurrence.
     messages.each do |message|
@@ -54,6 +44,47 @@ module YetiLogger::TestHelper
     end
   end
 
+  # Execute a block and ensure that the supplied log message is not among the
+  # messages logged at the specified level.
+  def expect_to_not_see_log_message(message, level = :debug, &block)
+    expect_to_not_see_log_messages([message], level, &block)
+  end
+
+  # Plural version of above
+  def expect_to_not_see_log_messages(messages, level = :debug, &block)
+    log_messages = get_log_messages(level, &block)
+
+    found = messages.find do |message|
+      if message.is_a?(Regexp)
+        log_messages.find do |log_message|
+          log_message =~ message
+        end
+      else
+        log_messages.include?(message)
+      end
+    end
+
+    if found.present?
+      fail "Should not have found #{found.inspect} amongst #{log_messages.inspect}"
+    end
+  end
+
+  def get_log_messages(level = :debug, &block)
+    log_messages = []
+
+    allow(YetiLogger.logger).to receive(level) do |log_line|
+      log_messages << log_line
+    end
+
+    block.call
+
+    # There is no unstub in rspec 3, but the closest to that would be to
+    # continue to stub it, but defer to the original implementation.
+    allow(YetiLogger.logger).to receive(level).and_call_original
+
+    log_messages
+  end
+
   def should_log(level = :info)
     expect(YetiLogger.logger).to(receive(level))
   end

data/lib/yeti_logger/version.rb

@@ -1,3 +1,3 @@
 module YetiLogger
-  VERSION = "3.0.0"
+  VERSION = "3.2.0"
 end

data/spec/lib/yeti_logger/test_helper_spec.rb

@@ -121,6 +121,95 @@ describe YetiLogger::TestHelper do
 
   end
 
+  describe '.expect_to_not_see_log_messages' do
+    it 'has a singular form' do
+      expect_to_not_see_log_message('YetiLogger::TestLogger: foo', :warn) do
+        instance.log_warn('bar')
+      end
+    end
+
+    it 'checks for multiple messages' do
+      messages = [
+                  'YetiLogger::TestLogger: one',
+                  'YetiLogger::TestLogger: two'
+                 ]
+      expect_to_not_see_log_messages(messages, :info) do
+        instance.log_info('three')
+        instance.log_info('four')
+      end
+    end
+
+    it 'only stubs the log level you request' do
+      expect(YetiLogger.logger).to receive(:info).with('YetiLogger::TestLogger: msg')
+
+      expect_to_not_see_log_message('YetiLogger::TestLogger: msg', :warn) do
+        instance.log_info { 'msg' }
+        instance.log_error { 'msg' }
+      end
+    end
+
+    it 'supports regexes' do
+      messages = [
+                  'YetiLogger::TestLogger: one',
+                  /two-\d/,
+                  'YetiLogger::TestLogger: three'
+                 ]
+      expect_to_not_see_log_messages(messages, :info) do
+        instance.log_info('four')
+        instance.log_info('five')
+      end
+    end
+
+    it 'fails when it finds the string message' do
+      expect do
+        message = 'YetiLogger::TestLogger: should not be there'
+        expect_to_not_see_log_message(message, :info) do
+          instance.log_info('should not be there')
+        end
+      end.to raise_exception(RuntimeError)
+    end
+
+    it 'fails when it matches the regexp message' do
+      expect do
+        expect_to_not_see_log_message(/bazinga/, :info) do
+          instance.log_info('bazinga, punk!')
+        end
+      end.to raise_exception(RuntimeError)
+    end
+
+    it 'fails when it finds one of the string messages' do
+      expect do
+        messages = [
+                    'YetiLogger::TestLogger: one',
+                    /two-\d/,
+                    'YetiLogger::TestLogger: three',
+                    'YetiLogger::TestLogger: four'
+                   ]
+        expect_to_not_see_log_messages(messages, :info) do
+          instance.log_info('one')
+          instance.log_info('five')
+          instance.log_info('six')
+        end
+      end.to raise_exception(RuntimeError)
+    end
+
+    it 'fails when it finds matches one of the regexps' do
+      expect do
+        messages = [
+                    'YetiLogger::TestLogger: one',
+                    /two-\d/,
+                    /three/,
+                    'YetiLogger::TestLogger: four'
+                   ]
+        expect_to_not_see_log_messages(messages, :info) do
+          instance.log_info('two-2')
+          instance.log_info('five')
+          instance.log_info('six')
+        end
+      end.to raise_exception(RuntimeError)
+    end
+  end
+
   describe '.should_log' do
     it 'verifies a log message came through' do
       should_log(:info).with("YetiLogger::TestLogger: hello!")

data/spec/lib/yeti_logger_spec.rb

@@ -97,10 +97,63 @@ describe YetiLogger do
 
       end
 
-      # There is a lot of debug output, so we can't verify it like we do the other
-      # levels. For now, just validate the method is there.
-      it "can call a #{target_type} debug method" do
-        target.log_debug("#{target_type} debuggin")
+      describe "log_debug" do
+        let(:user_id) { rand(9000) }
+
+        shared_examples_for "it logs at the expected level" do |expected_level|
+          it "logs at #{expected_level} level" do
+            regex = /#{class_name}:.*#{target_type}.*debuggin.*user_id=#{user_id}/
+            expect_to_see_log_message(regex, expected_level) do
+              target.log_debug(msg: "#{target_type} debuggin", user_id: user_id)
+            end
+          end
+        end
+
+        after(:each) do
+          if described_class.respond_to?(:promote_debug_to_info?)
+            class <<described_class
+              undef_method(:promote_debug_to_info?)
+            end
+          end
+        end
+
+        context "when promote_debug_to_info? isn't configured," do
+          before(:each) do
+            if described_class.respond_to?(:promote_debug_to_info?)
+              class <<described_class
+                undef_method(:promote_debug_to_info?)
+              end
+            end
+          end
+
+          it_behaves_like "it logs at the expected level", :debug
+        end
+
+        context "when promote_debug_to_info? is configured," do
+          context "when promote_debug_to_info is true," do
+            before(:each) do
+              described_class.configure do |config|
+                def config.promote_debug_to_info?(log_hash)
+                  true
+                end
+              end
+            end
+
+            it_behaves_like "it logs at the expected level", :info
+          end
+
+          context "when promote_debug_to_info is false," do
+            before(:each) do
+              described_class.configure do |config|
+                def config.promote_debug_to_info?(log_hash)
+                  false
+                end
+              end
+            end
+
+            it_behaves_like "it logs at the expected level", :debug
+          end
+        end
       end
 
       it "can log key value pairs at #{target_type} level" do

data/yeti_logger.gemspec

@@ -25,4 +25,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "yard"
   spec.add_development_dependency "kramdown"
   spec.add_development_dependency "simplecov"
+  spec.add_development_dependency "byebug"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: yeti_logger
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.2.0
 platform: ruby
 authors:
 - Yesware, Inc
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2015-10-27 00:00:00.000000000 Z
+date: 2021-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -94,6 +94,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Provides standardized logging
 email:
 - engineering@yesware.com
@@ -103,6 +117,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".ruby-gemset"
 - ".ruby-version"
 - ".travis.yml"
 - CHANGELOG.md
@@ -127,7 +142,7 @@ homepage: ''
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -142,9 +157,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.6
-signing_key: 
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: Provides standardized logging
 test_files:
@@ -153,4 +167,3 @@ test_files:
 - spec/lib/yeti_logger/wrapped_logger_spec.rb
 - spec/lib/yeti_logger_spec.rb
 - spec/spec_helper.rb
-has_rdoc: