logput 0.0.2 → 0.0.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bcf996c334e9c8bd54c66b78402180034082c1ea
-  data.tar.gz: b18636e63e3c9b52088c30f5c95c1949f3e72865
+  metadata.gz: 7cbacb1dd5d03161a1b94546de8454b44fd66084
+  data.tar.gz: cbcc16f6c0207a1b9d47f6c2e51d9a166a9520c4
 SHA512:
-  metadata.gz: 2dc24bb10c3ebd667d7c028dbb69e69b7c795fc51b730bbaf22314e4f6cf48bce4ba9d83fa976db92e866e9560f7bdfa6ac10035f7a95110f236c0913e07d3aa
-  data.tar.gz: fd2ed287036550895d81682c43555b60fbaa0e255f7a36fffc2a08e245ac92c089756d15b5a3eb4c6fa4ede6c0cded8bb2b379166b22c260910f16e6ee093c39
+  metadata.gz: 9f2af5f8a8eca97c8e94c465c558087467340feb61aaaa975102ee9949c611f4e7c15464d2ee370ed1e574bcb2167d1684aa10d6b4d0e40a6d4724146fc47962
+  data.tar.gz: 4d9ba7d77cecfcb8e4c52c6237a9dbf73de61a0d4b30b369bf9f4d1437b30bcb10f8cae10af233d7fbaa158a0e553c07a0a18da9176f61578fbd524b16f47076

data/Fudgefile

@@ -15,7 +15,7 @@ task_group :duplication do
 end
 
 task_group :complexity do
-  task :flog, :exclude => '^\.\/spec\/', :max => 14.6, :average => 7.2, :methods => true
+  task :flog, :exclude => '^\.\/spec\/', :max => 7.7, :average => 5.7, :methods => true
 end
 
 build :default do

data/README.md

@@ -1,6 +1,6 @@
 # Logput
 
-[![Build Status](https://travis-ci.org/chrisbarber86/logput.svg?branch=add_travis)](https://travis-ci.org/chrisbarber86/logput)
+[![Build Status](https://api.travis-ci.org/asmega/logput.svg)](https://travis-ci.org/asmega/logput)
 
 Rack middleware to sit in a rails app to put put the current environments log to a webpage. eg /logput
 

data/lib/logput/middleware.rb

@@ -25,17 +25,21 @@ module Logput
     private
 
     def default_path_to_log_file(env)
-      raise Exception, 'Must specify path to log file' unless defined? Rails
-
-      if Rails.version >= "4.0.0"
-        logger(env).instance_variable_get(:@logdev).instance_variable_get(:@dev).path
-      else
-        logger(env).instance_variable_get(:@logger).instance_variable_get(:@log_dest).path
-      end
+      raise Exception, 'Must specify path to Rails log file' unless defined? Rails
+      path(logger(env)) || raise(Exception, "#{logger(env).class} not supported.")
     end
 
     def logger(env)
       env['action_dispatch.logger']
     end
+
+    def path(logger)
+      case logger
+        when ::ActiveSupport::TaggedLogging
+          logger.instance_variable_get(:@logger).instance_variable_get(:@log_dest).path
+        when ::Logger
+          logger.instance_variable_get(:@logdev).filename
+      end
+    end
   end
 end

data/lib/logput/version.rb

@@ -1,5 +1,5 @@
 # Logput
 module Logput
   # Gem version
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/spec/middleware_spec.rb

@@ -1,5 +1,9 @@
 require 'spec_helper'
 
+module ActiveSupport; class TaggedLogging; end; end
+class Logger; end
+class Rails; end
+
 describe Logput::Middleware do
   subject{ described_class.new(app, :path_to_log_file => './spec/support/test.log') }
 
@@ -31,47 +35,37 @@ describe Logput::Middleware do
     end
 
     context 'when rails is defined' do
-      let(:logger) { double }
+      let(:path) { './spec/support/test.log' }
       let(:logvar) { double }
-      let(:logdest) { double(:path => './spec/support/test.log') }
+      let(:logdev) { double(:filename => path) }
+      let(:log_dest) { double(:path => path) }
 
-      before :each do
-        class Rails
-          def self.version
-            @rails_version
-          end
-
-          def self.version=(v)
-            @rails_version = v
-          end
-        end
-      end
+      context 'TaggedLogging' do
+        let(:logger) { ::ActiveSupport::TaggedLogging.new }
 
-      context 'Rails 4' do
         before :each do
-          Rails.version = '4.0.0'
-          allow(logger).to receive(:instance_variable_get).with(:@logdev).and_return(logvar)
-          allow(logvar).to receive(:instance_variable_get).with(:@dev).and_return(logdest)
+          allow(logger).to receive(:instance_variable_get).with(:@logger).and_return(logvar)
+          allow(logvar).to receive(:instance_variable_get).with(:@log_dest).and_return(log_dest)
 
           @request = server.get('/logput', { 'action_dispatch.logger' => logger })
         end
 
         it 'accesses the correct log file' do
-          expect(@request.status).to eq(200)
+          expect(server.get('/logput').status).to eq(200)
         end
       end
 
-      context 'Rails 3' do
+      context 'Logger' do
+        let(:logger) { ::Logger.new }
+
         before :each do
-          Rails.version = '3.0.0'
-          allow(logger).to receive(:instance_variable_get).with(:@logger).and_return(logvar)
-          allow(logvar).to receive(:instance_variable_get).with(:@log_dest).and_return(logdest)
+          allow(logger).to receive(:instance_variable_get).with(:@logdev).and_return(logdev)
 
           @request = server.get('/logput', { 'action_dispatch.logger' => logger })
         end
 
         it 'accesses the correct log file' do
-          expect(server.get('/logput').status).to eq(200)
+          expect(@request.status).to eq(200)
         end
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: logput
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: ruby
 authors:
 - Philip Lee