travis-metrics 2.0.0.rc3 → 2.0.0.rc4
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/Gemfile.lock +1 -1
- data/lib/travis/metrics/reporter/graphite.rb +3 -2
- data/lib/travis/metrics/reporter/librato.rb +3 -2
- data/lib/travis/metrics/version.rb +1 -1
- data/spec/travis/metrics/reporter/graphite_spec.rb +8 -4
- data/spec/travis/metrics/reporter/librato_spec.rb +7 -8
- metadata +1 -1
checksums.yaml
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
---
|
2
2
|
SHA1:
|
3
|
-
metadata.gz:
|
4
|
-
data.tar.gz:
|
3
|
+
metadata.gz: 58b038816eaddc65ee765d986391e931b9c9fc4f
|
4
|
+
data.tar.gz: 9ac884a2d7cb52418d754521336a747ecb1c4797
|
5
5
|
SHA512:
|
6
|
-
metadata.gz:
|
7
|
-
data.tar.gz:
|
6
|
+
metadata.gz: fa36355535b333cc97fa636e69d128c2ab614d2de33cc8ae38ee5582121980fcb7c9fc037e940f2ea95e6932f728f26838e8cd4be283a1d77063aea4ccc22b3b
|
7
|
+
data.tar.gz: 4a34aa214f2c9640c9eac2526518d330704e4d3bd3d35a010235f834c32cf543b161c0ec8be4a3165cff903de8113b548a9a6b062a4ca7a22d293cfc64f9bfe6
|
data/Gemfile.lock
CHANGED
@@ -9,12 +9,13 @@ module Travis
|
|
9
9
|
error: 'Graphite error: %s (%s)'
|
10
10
|
}
|
11
11
|
|
12
|
+
attr_reader :reporter
|
13
|
+
|
12
14
|
def setup
|
13
15
|
return unless host
|
14
16
|
logger.info MSGS[:setup] % [host, port]
|
15
|
-
reporter = Metriks::Reporter::Graphite.new(host, port, interval: interval, on_error: method(:on_error))
|
17
|
+
@reporter = Metriks::Reporter::Graphite.new(host, port, interval: interval, on_error: method(:on_error))
|
16
18
|
reporter.start
|
17
|
-
reporter
|
18
19
|
end
|
19
20
|
|
20
21
|
private
|
@@ -9,12 +9,13 @@ module Travis
|
|
9
9
|
error: 'Librato error: %s (%s)'
|
10
10
|
}
|
11
11
|
|
12
|
+
attr_reader :reporter
|
13
|
+
|
12
14
|
def setup
|
13
15
|
return unless email && token
|
14
16
|
logger.info MSGS[:setup] % [source, email]
|
15
|
-
reporter = Metriks::LibratoMetricsReporter.new(email, token, source: source, on_error: method(:on_error))
|
17
|
+
@reporter = Metriks::LibratoMetricsReporter.new(email, token, source: source, on_error: method(:on_error))
|
16
18
|
reporter.start
|
17
|
-
reporter
|
18
19
|
end
|
19
20
|
|
20
21
|
private
|
@@ -15,8 +15,11 @@ describe Travis::Metrics::Reporter::Graphite do
|
|
15
15
|
it { expect(subject).to be_nil }
|
16
16
|
end
|
17
17
|
|
18
|
-
describe '
|
19
|
-
|
18
|
+
describe 'starts a graphite reporter' do
|
19
|
+
let(:metriks) { double('metriks', start: nil) }
|
20
|
+
before { allow(Metriks::Reporter::Graphite).to receive(:new).and_return(metriks) }
|
21
|
+
before { reporter.setup }
|
22
|
+
it { expect(metriks).to have_received(:start) }
|
20
23
|
it { expect(log).to include "Using Graphite metrics reporter (host: #{host}, port: 1234)" }
|
21
24
|
end
|
22
25
|
|
@@ -26,9 +29,10 @@ describe Travis::Metrics::Reporter::Graphite do
|
|
26
29
|
before { allow_any_instance_of(Metriks::Reporter::Graphite).to receive(:write).and_raise(error) }
|
27
30
|
|
28
31
|
before do
|
29
|
-
|
32
|
+
reporter.setup
|
33
|
+
reporter.reporter.start
|
30
34
|
sleep 0.1 # ugh.
|
31
|
-
|
35
|
+
reporter.reporter.stop
|
32
36
|
end
|
33
37
|
|
34
38
|
it { expect(stdout.string).to include 'Graphite error: message (body)' }
|
@@ -5,11 +5,6 @@ describe Travis::Metrics::Reporter::Librato do
|
|
5
5
|
let(:reporter) { described_class.new(config, logger) }
|
6
6
|
subject { reporter.setup }
|
7
7
|
|
8
|
-
def log
|
9
|
-
reporter.setup
|
10
|
-
super
|
11
|
-
end
|
12
|
-
|
13
8
|
describe 'setup' do
|
14
9
|
describe 'returns nil if email is missing' do
|
15
10
|
let(:email) { nil }
|
@@ -21,8 +16,11 @@ describe Travis::Metrics::Reporter::Librato do
|
|
21
16
|
it { expect(subject).to be_nil }
|
22
17
|
end
|
23
18
|
|
24
|
-
describe '
|
25
|
-
|
19
|
+
describe 'starts a librato reporter' do
|
20
|
+
let(:metriks) { double('metriks', start: nil) }
|
21
|
+
before { allow(Metriks::LibratoMetricsReporter).to receive(:new).and_return(metriks) }
|
22
|
+
before { reporter.setup }
|
23
|
+
it { expect(metriks).to have_received(:start) }
|
26
24
|
it { expect(log).to include 'Using Librato metrics reporter (source: source, account: anja@travis-ci.org)' }
|
27
25
|
end
|
28
26
|
|
@@ -30,7 +28,8 @@ describe Travis::Metrics::Reporter::Librato do
|
|
30
28
|
let(:error) { StandardError.new('message') }
|
31
29
|
before { allow(error).to receive(:response).and_return(double(body: 'body')) }
|
32
30
|
before { allow_any_instance_of(Metriks::LibratoMetricsReporter).to receive(:write).and_raise(error) }
|
33
|
-
before {
|
31
|
+
before { reporter.setup }
|
32
|
+
before { reporter.reporter.flush }
|
34
33
|
it { expect(stdout.string).to include 'Librato error: message (body)' }
|
35
34
|
end
|
36
35
|
end
|