grape-middleware-logger 1.1.0 → 1.1.1
Sign up to get free protection for your applications and to get access to all the features.
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: ef676f453ae5872cc3267134ec2e74ca58d16438
|
4
|
+
data.tar.gz: 36a9bd808bcd222b54c6634abe9946c867473451
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: 28469e0366ee321b7578e3f739cc02b000c5f80edd3933a5ce53f2283ee9eb1856a7947051652f7221332a6a15ce69a50fdc66881fb2bac697d1a06cc7395693
|
7
|
+
data.tar.gz: 884e76d28eecbd5dd5ba39e44e24ef1bc43e3b85de597f894a030041e0eeeccaa2c435d81598667926e0447a2afd361bcbb6e91f573a62a18c257f27a76bd358
|
@@ -31,7 +31,13 @@ class Grape::Middleware::Logger < Grape::Middleware::Globals
|
|
31
31
|
after_failure(error)
|
32
32
|
throw(:error, error)
|
33
33
|
else
|
34
|
-
|
34
|
+
# Usually a rack response object is returned: https://github.com/ruby-grape/grape/blob/master/UPGRADING.md#changes-in-middleware
|
35
|
+
# However, rack/auth/abstract/handler.rb still returns an array instead of a rack response object.
|
36
|
+
if @app_response.is_a?(Array)
|
37
|
+
after(@app_response[0])
|
38
|
+
else
|
39
|
+
after(@app_response.status)
|
40
|
+
end
|
35
41
|
end
|
36
42
|
@app_response
|
37
43
|
end
|
@@ -49,6 +49,24 @@ describe Grape::Middleware::Logger do
|
|
49
49
|
expect(subject.call!(env)).to eq app_response
|
50
50
|
end
|
51
51
|
end
|
52
|
+
|
53
|
+
context 'when calling the app results in an array response' do
|
54
|
+
let(:app_response_array) { [401, {}, []] }
|
55
|
+
|
56
|
+
it 'calls +after+ with the correct status' do
|
57
|
+
expect(app).to receive(:call).with(env).and_return(app_response_array)
|
58
|
+
expect(subject).to receive(:before)
|
59
|
+
expect(subject).to receive(:after).with(401)
|
60
|
+
subject.call!(env)
|
61
|
+
end
|
62
|
+
|
63
|
+
it 'returns the @app_response' do
|
64
|
+
expect(app).to receive(:call).with(env).and_return(app_response_array)
|
65
|
+
allow(subject).to receive(:before)
|
66
|
+
allow(subject).to receive(:after)
|
67
|
+
expect(subject.call!(env)).to eq app_response_array
|
68
|
+
end
|
69
|
+
end
|
52
70
|
end
|
53
71
|
|
54
72
|
describe '#after_failure' do
|
metadata
CHANGED
@@ -1,14 +1,14 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: grape-middleware-logger
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 1.1.
|
4
|
+
version: 1.1.1
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Ryan Buckley
|
8
8
|
autorequire:
|
9
9
|
bindir: bin
|
10
10
|
cert_chain: []
|
11
|
-
date: 2015-
|
11
|
+
date: 2015-11-18 00:00:00.000000000 Z
|
12
12
|
dependencies:
|
13
13
|
- !ruby/object:Gem::Dependency
|
14
14
|
name: grape
|