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 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: