graphite_client 0.4.1 → 0.4.2

Sign up to get free protection for your applications and to get access to all the features.
data/VERSION CHANGED
@@ -1 +1 @@
1
- 0.4.1
1
+ 0.4.2
@@ -5,11 +5,11 @@
5
5
 
6
6
  Gem::Specification.new do |s|
7
7
  s.name = "graphite_client"
8
- s.version = "0.4.1"
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-14"
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
@@ -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.1
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-14 00:00:00.000000000 Z
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: 654855312543279234
117
+ hash: -3052936494144076736
118
118
  required_rubygems_version: !ruby/object:Gem::Requirement
119
119
  none: false
120
120
  requirements: