grape-middleware-logger 1.5.0 → 1.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3669f6ad5c3a84da925373a79582e997b707fd3a
-  data.tar.gz: 94f09b4036667d394c5f794eab4782aedf656bb6
+  metadata.gz: efcf5674de9e8fcbcce7aebfda90fa8231e35475
+  data.tar.gz: 856cc2ebb17f174185c02103200457da79841acc
 SHA512:
-  metadata.gz: 80804fc3261ce716e57e458c32381f4bb83c330fd6801f901d394b22a62d68d5b0574b197e9b3bf3467f5bcc633a2e0d1754beef86a239d04cd1dbe63dd4c63a
-  data.tar.gz: 46d9aadf9b6ee7ea7ff9201e2dc8dd596f53d9727d2c25b6c1a8de919fa592ab7dd420ce8a9a4302f406afdf5668c9f150c470cd6d4ee10b31b90f6fba961b34
+  metadata.gz: 98ad10db03c07d57c1264d06fbb3cb54ab312746998873ae1e0b3aa9c73d7e503ca2a5d252d78a5d685bc08f75819c80c8b0a2192541e53b97db083b483b6df8
+  data.tar.gz: 0e1975f5d77664c5cfa05d73d74d45fbf4c4daab48af8c2de17eff7ca51c1744933ff3048121537cb978476cb6dfc1029202fcd982d1b957a5399243c0374ad2

data/README.md

@@ -93,6 +93,7 @@ and would love to see these two consolidated at some point.
 ## Contributing
 
 1. Fork it ( https://github.com/ridiculous/grape-middleware-logger/fork )
+1. Run the test suite with `bin/test`
 2. Create your feature branch (`git checkout -b my-new-feature`)
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)

data/grape-middleware-logger.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |spec|
   spec.name          = 'grape-middleware-logger'
-  spec.version       = '1.5.0'
+  spec.version       = '1.5.1'
   spec.platform      = Gem::Platform::RUBY
   spec.authors       = ['Ryan Buckley']
   spec.email         = ['arebuckley@gmail.com']

data/lib/grape/middleware/logger.rb

@@ -55,13 +55,8 @@ class Grape::Middleware::Logger < Grape::Middleware::Globals
       after_failure(error)
       throw(:error, error)
     else
-      # Usually a rack response object is returned: https://github.com/ruby-grape/grape/blob/master/UPGRADING.md#changes-in-middleware
-      # However, rack/auth/abstract/handler.rb still returns an array instead of a rack response object.
-      if @app_response.is_a?(Array)
-        after(@app_response[0])
-      else
-        after(@app_response.status)
-      end
+      status, _, _ = *@app_response
+      after(status)
     end
     @app_response
   end

data/spec/lib/grape/middleware/logger_spec.rb

@@ -40,20 +40,20 @@ describe Grape::Middleware::Logger do
     end
 
     context 'when calling the app results in an array response' do
-      let(:app_response_array) { [401, {}, []] }
+      let(:app_response) { [401, {}, []] }
 
       it 'calls +after+ with the correct status' do
-        expect(app).to receive(:call).with(env).and_return(app_response_array)
+        expect(app).to receive(:call).with(env).and_return(app_response)
         expect(subject).to receive(:before)
         expect(subject).to receive(:after).with(401)
         subject.call!(env)
       end
 
       it 'returns the @app_response' do
-        expect(app).to receive(:call).with(env).and_return(app_response_array)
+        expect(app).to receive(:call).with(env).and_return(app_response)
         allow(subject).to receive(:before)
         allow(subject).to receive(:after)
-        expect(subject.call!(env)).to eq app_response_array
+        expect(subject.call!(env)).to eq app_response
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: grape-middleware-logger
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.5.1
 platform: ruby
 authors:
 - Ryan Buckley
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-12 00:00:00.000000000 Z
+date: 2015-12-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: grape