graphite_client 0.4.1 → 0.4.2
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- data/VERSION +1 -1
- data/graphite_client.gemspec +2 -2
- data/lib/graphite_client/event_reporter.rb +2 -0
- data/spec/event_reporter_spec.rb +21 -0
- metadata +3 -3
data/VERSION
CHANGED
@@ -1 +1 @@
|
|
1
|
-
0.4.
|
1
|
+
0.4.2
|
data/graphite_client.gemspec
CHANGED
@@ -5,11 +5,11 @@
|
|
5
5
|
|
6
6
|
Gem::Specification.new do |s|
|
7
7
|
s.name = "graphite_client"
|
8
|
-
s.version = "0.4.
|
8
|
+
s.version = "0.4.2"
|
9
9
|
|
10
10
|
s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
|
11
11
|
s.authors = ["Alex Dean", "Matthew Trost"]
|
12
|
-
s.date = "2012-12-
|
12
|
+
s.date = "2012-12-19"
|
13
13
|
s.description = "Original code by https://github.com/joakimk."
|
14
14
|
s.email = "alex@crackpot.org"
|
15
15
|
s.extra_rdoc_files = [
|
@@ -20,6 +20,8 @@ class GraphiteClient
|
|
20
20
|
|
21
21
|
def report(event={})
|
22
22
|
event[:tags] = Array(event[:tags]).join(',')
|
23
|
+
# data should be a string.
|
24
|
+
event[:data] = event[:data].is_a?(String) ? event[:data] : event[:data].to_json
|
23
25
|
@req.body = event.to_json
|
24
26
|
@http.request(@req)
|
25
27
|
end
|
data/spec/event_reporter_spec.rb
CHANGED
@@ -30,6 +30,27 @@ describe "GraphiteClient::EventReporter" do
|
|
30
30
|
event_reporter.report(@event)
|
31
31
|
end
|
32
32
|
|
33
|
+
it "should encode a non-string data element as json" do
|
34
|
+
Net::HTTP.should_receive(:new).with(@uri.host, @uri.port).and_return(http = mock())
|
35
|
+
Net::HTTP::Post.should_receive(:new).with(@uri.request_uri).and_return(req = mock())
|
36
|
+
|
37
|
+
event = {
|
38
|
+
:what => 'test',
|
39
|
+
:tags => 'test',
|
40
|
+
:data => {:some => 'a', :useful => 'b', :data => 'c'}
|
41
|
+
}
|
42
|
+
|
43
|
+
expected = {}
|
44
|
+
expected[:what] = event[:what]
|
45
|
+
expected[:tags] = event[:tags]
|
46
|
+
expected[:data] = event[:data].to_json
|
47
|
+
|
48
|
+
req.should_receive(:body=).with(expected.to_json)
|
49
|
+
http.should_receive(:request).with(req)
|
50
|
+
event_reporter = GraphiteClient::EventReporter.new(@url)
|
51
|
+
event_reporter.report(event)
|
52
|
+
end
|
53
|
+
|
33
54
|
context "with basic auth" do
|
34
55
|
it "should create an HTTP connection and POST to it, using basic auth" do
|
35
56
|
Net::HTTP.should_receive(:new).with(@uri.host, @uri.port).and_return(http = mock())
|
metadata
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: graphite_client
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.4.
|
4
|
+
version: 0.4.2
|
5
5
|
prerelease:
|
6
6
|
platform: ruby
|
7
7
|
authors:
|
@@ -10,7 +10,7 @@ authors:
|
|
10
10
|
autorequire:
|
11
11
|
bindir: bin
|
12
12
|
cert_chain: []
|
13
|
-
date: 2012-12-
|
13
|
+
date: 2012-12-19 00:00:00.000000000 Z
|
14
14
|
dependencies:
|
15
15
|
- !ruby/object:Gem::Dependency
|
16
16
|
name: rspec
|
@@ -114,7 +114,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
|
|
114
114
|
version: '0'
|
115
115
|
segments:
|
116
116
|
- 0
|
117
|
-
hash:
|
117
|
+
hash: -3052936494144076736
|
118
118
|
required_rubygems_version: !ruby/object:Gem::Requirement
|
119
119
|
none: false
|
120
120
|
requirements:
|