multi_logger 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 7781fb7ec45c05fd258ffd68a4fa97e5a892133f
-  data.tar.gz: 8b735244380af2fcd0b781ddafea50d05be2a183
+SHA256:
+  metadata.gz: 53105078b9d403fe26d8adcb8544c7c0c181d0eef00b87412057aab5334c4659
+  data.tar.gz: 386f88badfc3ec2d1bd42237c05784f246e18274627d38584b4d9d0833ee4c49
 SHA512:
-  metadata.gz: 436ffcf55924ee88d289f3b04f32ba698d35499cdb2fd1c9467c1938474eb20480f86dd9248b34ab4877383f072873b0140f453623c1e2287459a6c6148b7fce
-  data.tar.gz: e2501ae314f9c04e8e79f7cd74685e2b2993337c242e88317c935f635d893c2f209ab1694bb8b484bb56a9d7f5d988cef35af355d6311a4e435d9c77122989ea
+  metadata.gz: d9d2c806a7b3d5f209c505f959e99dba69006282be0dbe730122676997dc893776f5b6f16e98f1eb49e2c7b5c9779d1977a7e1a7cc7cbf0fd4d64e548d533cd0
+  data.tar.gz: 6c8383fe6ca11389dd45e17e519a9cf38a568b7d98286ddf36aa3aa4911bebffbd5ee92f9171fb6e019b9da83d66400a5de8939d4352132fde319031cddc5716

data/CHANGES.md

@@ -1,9 +1,13 @@
 # Change log
 
+## 0.2.0 - 2020/07/14
+
+* [Enhancement] Accept `STDOUT` as log path - Konstantin
+
 ## 0.1.0 - 2013/09/13
 
 * [Enhancement] Accept `shift_age` and `shift size` as an argument to the `Logger.new` - Keisuke KITA
 
 ## 0.0.1 - 2013/05/30
 
-* Initial publication
+* Initial publication

data/README.md

@@ -44,6 +44,12 @@ You can assign formatter to loggers directly, or pass the formatter during setup
     MultiLogger.add_logger('mail', formatter:formatter)
     MultiLogger.add_logger('user', formatter:formatter)
 
+To set a different path to the log file, use `:path` option.
+For example, for setting a different file to each environment use:
+
+    MultiLogger.add_logger('mail', formatter:formatter, path:"mail_#{Rails.env}")
+
+
 ## Contributing
 
 1. Fork it

data/Rakefile

@@ -1 +1,9 @@
 require "bundler/gem_tasks"
+
+require 'rake/testtask'
+Rake::TestTask.new do |t|
+  t.libs.push "lib"
+  t.libs.push "spec"
+  t.test_files = FileList['spec/**/*_spec.rb']
+  t.verbose = true
+end

data/lib/multi_logger.rb

@@ -6,24 +6,23 @@ module MultiLogger
       name = name.to_s
       rails_logger_class = get_rails_logger_class()
 
-      if rails_logger_class.method_defined?(name)
-        raise "'#{name}' is reserved in #{rails_logger_class} and can not be used as a log accessor name."
-      else
-        logger = Logger.new(*extract_options(name, options))
-        rails_logger_class.class_eval do
-          define_method name.to_sym do
-            logger
-          end
-        end
-        if options[:formatter]
-          logger.formatter = options[:formatter]
+      raise "'#{name}' is reserved in #{rails_logger_class} and can not be used as a log accessor name." if rails_logger_class.method_defined?(name)
+
+      logger = Logger.new(*extract_options(name, options))
+      rails_logger_class.class_eval do
+        define_method name.to_sym do
+          logger
         end
-        logger
       end
+
+      logger.formatter = options[:formatter] if options[:formatter]
+      logger
     end
 
     # Computes log file path
     def get_path(name, path=nil)
+      return path if path == STDOUT
+      
       if path.nil?
         path = name.underscore
       end
@@ -47,13 +46,15 @@ module MultiLogger
     end
 
     def extract_options(name, options)
+      path = get_path(name, options[:path])
+
       if options[:shift_age] && options[:shift_size]
-        [get_path(name, options[:path]), options[:shift_age], options[:shift_size]]
+        [path, options[:shift_age], options[:shift_size]]
       elsif options[:shift_age]
         # options[:shift_age] => 'daily', 'weekly'
-        [get_path(name, options[:path]), options[:shift_age]]
+        [path, options[:shift_age]]
       else
-        [get_path(name, options[:path])]
+        [path]
       end
     end
   end

data/lib/multi_logger/version.rb

@@ -1,3 +1,3 @@
 module MultiLogger
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

data/lib/railties/lib/rails.rb

@@ -0,0 +1,7 @@
+module Rails
+  class << self
+    def loggers
+      MultiLogger
+    end
+  end
+end

data/multi_logger.gemspec

@@ -19,4 +19,6 @@ Gem::Specification.new do |gem|
   gem.require_paths = ["lib"]
 
   gem.add_dependency 'railties'
+  gem.add_development_dependency "activesupport"
+  gem.add_development_dependency "mocha"
 end

data/spec/helper.rb

@@ -0,0 +1,21 @@
+require 'rubygems'
+
+begin
+  require 'bundler'
+rescue LoadError => e
+  STDERR.puts e.message
+  STDERR.puts "Run `gem install bundler` to install Bundler."
+  exit e.status_code
+end
+
+begin
+  Bundler.setup(:default, :development, :test)
+rescue Bundler::BundlerError => e
+  STDERR.puts e.message
+  STDERR.puts "Run `bundle install` to install missing gems."
+  exit e.status_code
+end
+
+require 'minitest/autorun'
+require 'minitest/spec'
+require "mocha/mini_test"

data/spec/multi_logger_spec.rb

@@ -0,0 +1,66 @@
+require 'helper'
+require 'multi_logger'
+
+require 'logger'
+require 'active_support/core_ext/string'
+
+# Dummy class to avoid loading Rails
+module Rails
+  def self.root
+    Pathname.new('./spec/')
+  end
+end
+
+describe MultiLogger do
+  it 'has version' do
+    MultiLogger.const_get('VERSION').wont_be_empty
+  end
+
+  describe '.add_logger' do
+    it 'raises error if name already in use as method name' do
+      lambda { MultiLogger.add_logger('object_id')  }.must_raise RuntimeError
+    end
+    it 'defines a instance method in Rails Logger class linking to custom logger' do
+      rails_logger_klass = Class.new
+      MultiLogger.stubs(:get_rails_logger_class).returns(rails_logger_klass)
+
+      logger = Logger.new('test')
+      Logger.stubs(:new).returns(logger)
+
+      MultiLogger.add_logger('payment').must_equal logger
+
+      rails_logger_klass.new.payment.must_equal logger
+    end
+    it 'assigns formatter to custom logger' do
+      rails_logger_klass = Class.new
+      MultiLogger.stubs(:get_rails_logger_class).returns(rails_logger_klass)
+
+      logger = Logger.new('test')
+      Logger.stubs(:new).returns(logger)
+
+      formatter = mock('formatter')
+      MultiLogger.add_logger('fb',{formatter:formatter}).must_equal logger
+
+      rails_logger_klass.new.fb.must_equal logger
+      logger.formatter.must_equal formatter
+    end
+  end
+
+  describe '.get_path' do
+    it '' do
+      MultiLogger.send(:get_path,'parser').to_s.must_equal './spec/log/parser.log'
+    end
+    it 'uses path as file path different to name' do
+      MultiLogger.send(:get_path,'fb','facebook').to_s.must_equal './spec/log/facebook.log'
+    end
+    it 'uses path as file path (relative to Rails root) different to name' do
+      MultiLogger.send(:get_path,'fb','facebook').to_s.must_equal './spec/log/facebook.log'
+    end
+    it 'uses path as file path as is if it contains slash' do
+      MultiLogger.send(:get_path,'fb','./facebook').to_s.must_equal './facebook.log'
+    end
+    it 'does not append .log if path already ends with it' do
+      MultiLogger.send(:get_path,'fb','./facebook.log').to_s.must_equal './facebook.log'
+    end
+  end
+end

metadata

@@ -1,27 +1,55 @@
 --- !ruby/object:Gem::Specification
 name: multi_logger
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - lulalala
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-13 00:00:00.000000000 Z
+date: 2020-07-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  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'
+- !ruby/object:Gem::Dependency
+  name: mocha
+  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: Create multiple loggers in Rails.
@@ -31,7 +59,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
+- ".gitignore"
 - CHANGES.md
 - Gemfile
 - LICENSE.txt
@@ -39,7 +67,11 @@ files:
 - Rakefile
 - lib/multi_logger.rb
 - lib/multi_logger/version.rb
+- lib/railties/lib/rails.rb
 - multi_logger.gemspec
+- spec/helper.rb
+- spec/log/.gitkeep
+- spec/multi_logger_spec.rb
 homepage: https://github.com/lulalala/multi_logger
 licenses: []
 metadata: {}
@@ -49,20 +81,21 @@ 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: []
-rubyforge_project: 
-rubygems_version: 2.0.5
+rubygems_version: 3.1.3
 signing_key: 
 specification_version: 4
 summary: Provide helper to define loggers and access them. Each logger will log into
   a different file under the log  folder.
-test_files: []
-has_rdoc: 
+test_files:
+- spec/helper.rb
+- spec/log/.gitkeep
+- spec/multi_logger_spec.rb