pliny 0.17.0 → 0.17.1
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/lib/pliny/version.rb +1 -1
- data/spec/commands/generator_spec.rb +1 -3
- data/spec/commands/updater_spec.rb +1 -1
- data/spec/error_reporter_spec.rb +3 -3
- data/spec/helpers/encode_spec.rb +2 -2
- data/spec/log_spec.rb +11 -11
- data/spec/middleware/instruments_spec.rb +4 -4
- data/spec/middleware/request_store_spec.rb +2 -2
- data/spec/middleware/rescue_errors_spec.rb +1 -1
- data/spec/middleware/versioning_spec.rb +0 -1
- data/spec/rollbar_logger_spec.rb +1 -1
- data/spec/spec_helper.rb +1 -1
- metadata +2 -22
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: 56651d8e3b670e7954a475c9d8cb04cc0124099f
|
4
|
+
data.tar.gz: ad7701bdf942d068994be11da8b3ceb7bc231082
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: d73cbdd0b3dcde8423dc2743af931738bc41a5d608281a031f29a160ac9cee73d8e40a7dac34274e0b619e9b695487c464286efcd412c20e79f8e61f4b03d09d
|
7
|
+
data.tar.gz: cf7bc5313022abc17ebee414f53ad004d66cd3177bee895daf8b7ba755451ee93602f41a0b128c9b1f9658cdeda1bebbf291a67a8ed2f424f8867246421978e0
|
data/lib/pliny/version.rb
CHANGED
@@ -9,9 +9,7 @@ describe Pliny::Commands::Generator do
|
|
9
9
|
before do
|
10
10
|
Timecop.freeze(@t = Time.now)
|
11
11
|
|
12
|
-
|
13
|
-
stub(klass).display
|
14
|
-
end
|
12
|
+
allow_any_instance_of(Pliny::Commands::Generator::Base).to receive(:display)
|
15
13
|
end
|
16
14
|
|
17
15
|
around do |example|
|
data/spec/error_reporter_spec.rb
CHANGED
@@ -14,9 +14,9 @@ describe Pliny::ErrorReporter do
|
|
14
14
|
end
|
15
15
|
|
16
16
|
it "notifies rollbar" do
|
17
|
-
|
18
|
-
|
19
|
-
|
17
|
+
expect_any_instance_of(Pliny::ErrorReporter::RollbarReporter).
|
18
|
+
to receive(:notify).
|
19
|
+
with(exception, context: context, rack_env: rack_env)
|
20
20
|
|
21
21
|
notify_reporter
|
22
22
|
end
|
data/spec/helpers/encode_spec.rb
CHANGED
@@ -11,7 +11,7 @@ describe Pliny::Helpers::Encode do
|
|
11
11
|
end
|
12
12
|
|
13
13
|
before do
|
14
|
-
|
14
|
+
allow(Config).to receive(:pretty_json) { false }
|
15
15
|
end
|
16
16
|
|
17
17
|
it "sets the Content-Type" do
|
@@ -33,7 +33,7 @@ describe Pliny::Helpers::Encode do
|
|
33
33
|
end
|
34
34
|
|
35
35
|
it "encodes in pretty mode when set by config" do
|
36
|
-
|
36
|
+
allow(Config).to receive(:pretty_json) { true }
|
37
37
|
payload = { "foo" => "bar" }
|
38
38
|
post "/", payload
|
39
39
|
assert_equal MultiJson.encode(payload, pretty: true), last_response.body
|
data/spec/log_spec.rb
CHANGED
@@ -5,7 +5,7 @@ describe Pliny::Log do
|
|
5
5
|
@io = StringIO.new
|
6
6
|
Pliny.stdout = @io
|
7
7
|
Pliny.stderr = @io
|
8
|
-
|
8
|
+
allow(@io).to receive(:print)
|
9
9
|
end
|
10
10
|
|
11
11
|
after do
|
@@ -13,7 +13,7 @@ describe Pliny::Log do
|
|
13
13
|
end
|
14
14
|
|
15
15
|
it "logs in structured format" do
|
16
|
-
|
16
|
+
expect(@io).to receive(:print).with("foo=bar baz=42\n")
|
17
17
|
Pliny.log(foo: "bar", baz: 42)
|
18
18
|
end
|
19
19
|
|
@@ -26,26 +26,26 @@ describe Pliny::Log do
|
|
26
26
|
end
|
27
27
|
|
28
28
|
it "supports blocks to log stages and elapsed" do
|
29
|
-
|
30
|
-
|
29
|
+
expect(@io).to receive(:print).with("foo=bar at=start\n")
|
30
|
+
expect(@io).to receive(:print).with("foo=bar at=finish elapsed=0.000\n")
|
31
31
|
Pliny.log(foo: "bar") do
|
32
32
|
end
|
33
33
|
end
|
34
34
|
|
35
35
|
it "merges default context" do
|
36
36
|
Pliny.default_context = { app: "pliny" }
|
37
|
-
|
37
|
+
expect(@io).to receive(:print).with("app=pliny foo=bar\n")
|
38
38
|
Pliny.log(foo: "bar")
|
39
39
|
end
|
40
40
|
|
41
41
|
it "merges context from RequestStore" do
|
42
42
|
Pliny::RequestStore.store[:log_context] = { app: "pliny" }
|
43
|
-
|
43
|
+
expect(@io).to receive(:print).with("app=pliny foo=bar\n")
|
44
44
|
Pliny.log(foo: "bar")
|
45
45
|
end
|
46
46
|
|
47
47
|
it "supports a context" do
|
48
|
-
|
48
|
+
expect(@io).to receive(:print).with("app=pliny foo=bar\n")
|
49
49
|
Pliny.context(app: "pliny") do
|
50
50
|
Pliny.log(foo: "bar")
|
51
51
|
end
|
@@ -53,14 +53,14 @@ describe Pliny::Log do
|
|
53
53
|
|
54
54
|
it "local context does not overwrite default context" do
|
55
55
|
Pliny.default_context = { app: "pliny" }
|
56
|
-
|
56
|
+
expect(@io).to receive(:print).with("app=not_pliny foo=bar\n")
|
57
57
|
Pliny.log(app: 'not_pliny', foo: "bar")
|
58
58
|
assert Pliny.default_context[:app] == "pliny"
|
59
59
|
end
|
60
60
|
|
61
61
|
it "local context does not overwrite request context" do
|
62
62
|
Pliny::RequestStore.store[:log_context] = { app: "pliny" }
|
63
|
-
|
63
|
+
expect(@io).to receive(:print).with("app=not_pliny foo=bar\n")
|
64
64
|
Pliny.context(app: "not_pliny") do
|
65
65
|
Pliny.log(foo: "bar")
|
66
66
|
end
|
@@ -69,7 +69,7 @@ describe Pliny::Log do
|
|
69
69
|
|
70
70
|
it "local context does not propagate outside" do
|
71
71
|
Pliny::RequestStore.store[:log_context] = { app: "pliny" }
|
72
|
-
|
72
|
+
expect(@io).to receive(:print).with("app=pliny foo=bar\n")
|
73
73
|
Pliny.context(app: "not_pliny", test: 123) do
|
74
74
|
end
|
75
75
|
Pliny.log(foo: "bar")
|
@@ -78,7 +78,7 @@ describe Pliny::Log do
|
|
78
78
|
it "logs exceptions" do
|
79
79
|
Pliny::RequestStore.store[:log_context] = { app: "pliny" }
|
80
80
|
e = RuntimeError.new
|
81
|
-
|
81
|
+
expect(@io).to receive(:print).with("app=pliny exception class=RuntimeError message=RuntimeError exception_id=#{e.object_id}\n")
|
82
82
|
Pliny.log_exception(e)
|
83
83
|
end
|
84
84
|
end
|
@@ -23,13 +23,13 @@ describe Pliny::Middleware::Instruments do
|
|
23
23
|
end
|
24
24
|
|
25
25
|
it "performs logging" do
|
26
|
-
|
26
|
+
expect(Pliny).to receive(:log).with(hash_including(
|
27
27
|
instrumentation: true,
|
28
28
|
at: "start",
|
29
29
|
method: "GET",
|
30
30
|
path: "/apps/123",
|
31
31
|
))
|
32
|
-
|
32
|
+
expect(Pliny).to receive(:log).with(hash_including(
|
33
33
|
instrumentation: true,
|
34
34
|
at: "finish",
|
35
35
|
method: "GET",
|
@@ -41,8 +41,8 @@ describe Pliny::Middleware::Instruments do
|
|
41
41
|
end
|
42
42
|
|
43
43
|
it "respects Pliny error status codes" do
|
44
|
-
|
45
|
-
|
44
|
+
expect(Pliny).to receive(:log)
|
45
|
+
expect(Pliny).to receive(:log).with(hash_including(
|
46
46
|
status: 404
|
47
47
|
))
|
48
48
|
get "/error"
|
@@ -14,12 +14,12 @@ describe Pliny::Middleware::RequestStore do
|
|
14
14
|
end
|
15
15
|
|
16
16
|
it "clears the store" do
|
17
|
-
|
17
|
+
expect(Pliny::RequestStore).to receive(:clear!)
|
18
18
|
get "/"
|
19
19
|
end
|
20
20
|
|
21
21
|
it "seeds the store" do
|
22
|
-
|
22
|
+
expect(Pliny::RequestStore).to receive(:seed)
|
23
23
|
get "/"
|
24
24
|
end
|
25
25
|
end
|
@@ -28,7 +28,7 @@ describe Pliny::Middleware::RescueErrors do
|
|
28
28
|
|
29
29
|
it "intercepts exceptions and renders" do
|
30
30
|
@app = new_rack_app
|
31
|
-
|
31
|
+
expect(Pliny::ErrorReporter).to receive(:notify)
|
32
32
|
get "/"
|
33
33
|
assert_equal 500, last_response.status
|
34
34
|
error_json = MultiJson.decode(last_response.body)
|
data/spec/rollbar_logger_spec.rb
CHANGED
data/spec/spec_helper.rb
CHANGED
metadata
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: pliny
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.17.
|
4
|
+
version: 0.17.1
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Brandur Leach
|
@@ -9,7 +9,7 @@ authors:
|
|
9
9
|
autorequire:
|
10
10
|
bindir: bin
|
11
11
|
cert_chain: []
|
12
|
-
date: 2016-
|
12
|
+
date: 2016-06-20 00:00:00.000000000 Z
|
13
13
|
dependencies:
|
14
14
|
- !ruby/object:Gem::Dependency
|
15
15
|
name: activesupport
|
@@ -191,26 +191,6 @@ dependencies:
|
|
191
191
|
- - ">="
|
192
192
|
- !ruby/object:Gem::Version
|
193
193
|
version: 0.6.2
|
194
|
-
- !ruby/object:Gem::Dependency
|
195
|
-
name: rr
|
196
|
-
requirement: !ruby/object:Gem::Requirement
|
197
|
-
requirements:
|
198
|
-
- - "~>"
|
199
|
-
- !ruby/object:Gem::Version
|
200
|
-
version: '1.1'
|
201
|
-
- - ">="
|
202
|
-
- !ruby/object:Gem::Version
|
203
|
-
version: 1.1.2
|
204
|
-
type: :development
|
205
|
-
prerelease: false
|
206
|
-
version_requirements: !ruby/object:Gem::Requirement
|
207
|
-
requirements:
|
208
|
-
- - "~>"
|
209
|
-
- !ruby/object:Gem::Version
|
210
|
-
version: '1.1'
|
211
|
-
- - ">="
|
212
|
-
- !ruby/object:Gem::Version
|
213
|
-
version: 1.1.2
|
214
194
|
- !ruby/object:Gem::Dependency
|
215
195
|
name: rspec
|
216
196
|
requirement: !ruby/object:Gem::Requirement
|