rack-action_logger 0.1.9 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 22b077d4a3c2233f2cc7ad125f6327a38097439c
-  data.tar.gz: 7afbcbfe31b62c9062529e271038d432527c271e
+  metadata.gz: decd5af4f47d90252f92d6442c7642b486681d47
+  data.tar.gz: 55096e953ae0a64a26c3e364d64a6579f3c33084
 SHA512:
-  metadata.gz: 294f9871ddd3695e7b1f3bfff2583029172a3e9ac45384901b3120ebb38881be1a97a07ecbf91a2d46f5c10150ac610cbc4784050af5ac2cdb6b930380c9ad59
-  data.tar.gz: 2e90f1d3f4abdb0024203c4fc4041c8c8b629f61f4930d4beb51948fb593a3a01766bdff2a970ea6e09ff44e217c6f693edd93852d40e4c3a3750d2c347b049c
+  metadata.gz: 2e8fff404973516a228d9df8fa9f8b649bde879a200f12d71a6d43249fd77958165167038d87deca3d38c828db353dcf7b8e41f67675eb2fa8862b8e4c1a7509
+  data.tar.gz: cdfcdf11d86bf527f3b66f8e4df80f1b1d6966581e6d599729c7674bcc4dc81b9af29bdefe69051a4a208bd1fbdcd8fe857eebd279f3977c7ae27e1016a9d4f6

data/.simplecov

@@ -0,0 +1,3 @@
+SimpleCov.start do
+  add_group 'lib', 'lib'
+end

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rack-action_logger (0.1.9)
+    rack-action_logger (0.2.0)
       activesupport
       fluent-logger (~> 0.5)
       woothee (~> 1.4)
@@ -16,9 +16,11 @@ GEM
       tzinfo (~> 1.1)
     concurrent-ruby (1.0.2)
     diff-lcs (1.2.5)
+    docile (1.1.5)
     fluent-logger (0.6.1)
       msgpack (>= 0.5.6, < 2)
     i18n (0.7.0)
+    json (2.0.2)
     minitest (5.9.1)
     msgpack (1.0.2)
     rack (2.0.1)
@@ -38,6 +40,11 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.5.0)
     rspec-support (3.5.0)
+    simplecov (0.12.0)
+      docile (~> 1.1.0)
+      json (>= 1.8, < 3)
+      simplecov-html (~> 0.10.0)
+    simplecov-html (0.10.0)
     thread_safe (0.3.5)
     tzinfo (1.2.2)
       thread_safe (~> 0.1)
@@ -52,6 +59,7 @@ DEPENDENCIES
   rack-test (~> 0.6.3)
   rake (~> 10.0)
   rspec (~> 3.0)
+  simplecov
 
 BUNDLED WITH
    1.13.6

data/README.md

@@ -144,7 +144,7 @@ Request can be customized by creating new class for rack_metrics configuration.
 Add the following code to any code on any times.
 
 ```ruby
-Rack::ActionLogger::Container.set_append_log({ value: 'ok' }, 'activities')
+Rack::ActionLogger::Container.add_append_log({ value: 'ok' }, 'activities')
 ```
 
 ### Add Model Logger
@@ -189,7 +189,7 @@ class TestWorker < ApplicationController
 
   def perform(title, context)
     Rack::ActionLogger::Emitter.new.emit(context) do
-      Rack::ActionLogger::Container.set_append_log({ title: title }, 'worker')
+      Rack::ActionLogger::Container.add_append_log({ title: title }, 'worker')
       p 'work: title=' + title
     end
   end

data/lib/rack/action_logger/active_record_extension.rb

@@ -17,7 +17,7 @@ module Rack::ActionLogger
         record["_#{column_name}"] = self.try(column_name)
       end
       record = Rack::ActionLogger::ParameterFiltering.apply_filter(record)
-      Rack::ActionLogger::Container.set_append_log(record, "model_#{self.class.table_name}")
+      Rack::ActionLogger::Container.add_append_log(record, "model_#{self.class.table_name}")
     end
 
     def capture_action_log_update
@@ -31,7 +31,7 @@ module Rack::ActionLogger
         end
       end
       record = Rack::ActionLogger::ParameterFiltering.apply_filter(record)
-      Rack::ActionLogger::Container.set_append_log(record, "model_#{self.class.table_name}")
+      Rack::ActionLogger::Container.add_append_log(record, "model_#{self.class.table_name}")
     end
 
     def capture_action_log_destroy
@@ -42,7 +42,7 @@ module Rack::ActionLogger
         end
       end
       record = Rack::ActionLogger::ParameterFiltering.apply_filter(record)
-      Rack::ActionLogger::Container.set_append_log(record, "model_#{self.class.table_name}")
+      Rack::ActionLogger::Container.add_append_log(record, "model_#{self.class.table_name}")
     end
   end
 end

data/lib/rack/action_logger/container.rb

@@ -12,31 +12,31 @@ module Rack::ActionLogger
         Thread.current[THREAD_KEY] = nil
       end
 
-      def set_is_emit_started
-        store[:rack_action_logger_emit_started] = true
+      def is_emit_started=(value)
+        store[:rack_action_logger_emit_started] = value
       end
 
-      def get_is_emit_started
+      def is_emit_started
         store[:rack_action_logger_emit_started] ||= false
       end
 
-      def set_append_log(hash, tag=nil)
+      def add_append_log(hash, tag=nil)
         return unless is_valid_hash hash
         return unless is_valid_tag tag
         hash[:tag] = tag
-        get_append_logs.push(hash)
+        append_logs.push(hash)
       end
 
-      def get_append_logs
+      def append_logs
         store[:rack_action_logger_append_logs] ||= []
       end
 
       def merge_attributes(attributes)
         return unless is_valid_hash attributes
-        get_attributes.merge! attributes
+        self.attributes = self.attributes.merge! attributes
       end
 
-      def get_attributes
+      def attributes
         store[:rack_action_logger_attributes] ||= {}
       end
 
@@ -44,20 +44,18 @@ module Rack::ActionLogger
         return unless is_valid_hash hash
         return unless is_valid_tag tag
         hash[:tag] = tag
-        get_request_log.merge! hash
+        self.request_log = hash
       end
 
-      def get_request_log
+      def request_log
         store[:rack_action_logger_request_log] ||= {}
       end
 
       def export
-        # ここenumerableで置き換える
-        hash = {}
-        EXPORT_KEYS.each do |key|
-          hash[key] = store[key] if store[key]
+        EXPORT_KEYS.inject({}) do |result, key|
+          result[key] = store[key] if store[key]
+          result
         end
-        hash
       end
 
       def import(hash)
@@ -90,6 +88,14 @@ module Rack::ActionLogger
           false
         end
       end
+
+      def attributes=(value)
+        store[:rack_action_logger_attributes] = value
+      end
+
+      def request_log=(value)
+        store[:rack_action_logger_request_log] = value
+      end
     end
 
   end

data/lib/rack/action_logger/emitter.rb

@@ -2,7 +2,7 @@ module Rack::ActionLogger
   class Emitter
 
     def initialize
-      @can_emit = !Container.get_is_emit_started
+      @can_emit = !Container.is_emit_started
       unless @can_emit
         Rack::ActionLogger.logger.error("#{self.class} is already defined.")
         Rack::ActionLogger.logger.error("#{Thread.current.backtrace.join("\n")}")
@@ -12,7 +12,7 @@ module Rack::ActionLogger
     end
 
     def emit(context=nil)
-      @container.set_is_emit_started
+      @container.is_emit_started = true
       @container.import(context) if context
       result = yield
       emit_all_logs  # emit log unless exception raised
@@ -30,16 +30,16 @@ module Rack::ActionLogger
     end
 
     def emit_request_log
-      return unless (@container.get_request_log.is_a?(Hash) && @container.get_request_log != {})
-      hash = @container.get_request_log.merge @container.get_attributes
+      return unless (@container.request_log.is_a?(Hash) && @container.request_log != {})
+      hash = @container.request_log.merge @container.attributes
       hash = format_tag(hash)
       @emit_adapter.emit(hash)
     end
 
     def emit_append_logs
-      @container.get_append_logs.each do |hash|
+      @container.append_logs.each do |hash|
         hash = format_tag(hash)
-        @emit_adapter.emit(@container.get_attributes.merge(hash))
+        @emit_adapter.emit(@container.attributes.merge(hash))
       end
     end
 

data/lib/rack/action_logger/version.rb

@@ -1,5 +1,5 @@
 module Rack
   module ActionLogger
-    VERSION = '0.1.9'
+    VERSION = '0.2.0'
   end
 end

data/rack-action_logger.gemspec

@@ -32,4 +32,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rake', '~> 10.0'
   spec.add_development_dependency 'rspec', '~> 3.0'
   spec.add_development_dependency 'rack-test', '~> 0.6.3'
+  spec.add_development_dependency 'simplecov'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rack-action_logger
 version: !ruby/object:Gem::Version
-  version: 0.1.9
+  version: 0.2.0
 platform: ruby
 authors:
 - Koichi Ishida
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2016-11-22 00:00:00.000000000 Z
+date: 2016-11-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -108,6 +108,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.6.3
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  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: Rack::ActionLogger is a tool to log user activity
 email:
 - wapa5pow@gmail.com
@@ -117,6 +131,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".simplecov"
 - ".travis.yml"
 - CODE_OF_CONDUCT.md
 - Gemfile