librato-metrics 1.6.2 → 2.0.0.beta

data/spec/unit/metrics/smart_json_spec.rb

@@ -0,0 +1,79 @@
+require "spec_helper"
+
+module Librato
+  module Metrics
+
+    describe SmartJSON do
+
+      describe ".read" do
+        context "with MultiJSON" do
+          before do
+            $".delete_if {|s| s.include?("multi_json")}
+            require "multi_json"
+            load "lib/librato/metrics/smart_json.rb"
+          end
+
+          after do
+            Object.send(:remove_const, :MultiJson)
+            load "lib/librato/metrics/smart_json.rb"
+          end
+
+          it "uses .load or .decode" do
+            actual = SmartJSON.read("{\"abc\":\"def\"}")
+
+            expect(SmartJSON.handler).to eq(:multi_json)
+            expect(actual).to be_a(Hash)
+            expect(actual).to have_key("abc")
+            expect(actual["abc"]).to eq("def")
+          end
+        end
+
+        context "with JSON" do
+          it "uses .parse" do
+            actual = SmartJSON.read("{\"abc\":\"def\"}")
+
+            expect(SmartJSON.handler).to eq(:json)
+            expect(actual).to be_a(Hash)
+            expect(actual).to have_key("abc")
+            expect(actual["abc"]).to eq("def")
+          end
+        end
+      end
+
+      describe ".write" do
+        context "with MultiJSON" do
+          before do
+            $".delete_if {|s| s.include?("multi_json")}
+            require "multi_json"
+            load "lib/librato/metrics/smart_json.rb"
+          end
+
+          after do
+            Object.send(:remove_const, :MultiJson)
+            load "lib/librato/metrics/smart_json.rb"
+          end
+
+          it "uses .dump or .decode" do
+            actual = SmartJSON.write({abc: 'def'})
+
+            expect(SmartJSON.handler).to eq(:multi_json)
+            expect(actual).to be_a(String)
+            expect(actual).to eq("{\"abc\":\"def\"}")
+          end
+        end
+
+        context "with JSON" do
+          it "uses .generate" do
+            actual = SmartJSON.write({abc: 'def'})
+
+            expect(SmartJSON.handler).to eq(:json)
+            expect(actual).to be_a(String)
+            expect(actual).to eq("{\"abc\":\"def\"}")
+          end
+        end
+      end
+
+    end
+
+  end
+end

data/spec/unit/metrics_spec.rb

@@ -6,36 +6,36 @@ module Librato
 
    describe "#authorize" do
      context "when given two arguments" do
-       it "should store them on simple" do
+       it "stores them on simple" do
          Metrics.authenticate 'tester@librato.com', 'api_key'
-         Metrics.client.email.should == 'tester@librato.com'
-         Metrics.client.api_key.should == 'api_key'
+         expect(Metrics.client.email).to eq('tester@librato.com')
+         expect(Metrics.client.api_key).to eq('api_key')
        end
      end
    end
-   
+
    describe "#faraday_adapter" do
-     it "should return current default adapter" do
-       Metrics.faraday_adapter.should_not be nil
+     it "returns current default adapter" do
+       expect(Metrics.faraday_adapter).not_to be_nil
      end
    end
-   
+
    describe "#faraday_adapter=" do
      before(:all) { @current_adapter = Metrics.faraday_adapter }
      after(:all) { Metrics.faraday_adapter = @current_adapter }
-     
-     it "should allow setting of faraday adapter" do
+
+     it "allows setting of faraday adapter" do
        Metrics.faraday_adapter = :excon
-       Metrics.faraday_adapter.should == :excon
+       expect(Metrics.faraday_adapter).to eq(:excon)
        Metrics.faraday_adapter = :patron
-       Metrics.faraday_adapter.should == :patron
+       expect(Metrics.faraday_adapter).to eq(:patron)
      end
    end
 
    describe "#persistence" do
-     it "should allow configuration of persistence method" do
+     it "allows configuration of persistence method" do
        Metrics.persistence = :test
-       Metrics.persistence.should == :test
+       expect(Metrics.persistence).to eq(:test)
      end
    end
 
@@ -46,19 +46,19 @@ module Librato
      end
      after(:all) { Librato::Metrics.client.flush_authentication }
 
-     it "should persist metrics immediately" do
+     it "persists metrics immediately" do
        Metrics.persistence = :test
-       Metrics.submit(:foo => 123).should eql true
-       Metrics.persister.persisted.should == {:gauges => [{:name => 'foo', :value => 123}]}
+       expect(Metrics.submit(foo: 123)).to be true
+       expect(Metrics.persister.persisted).to eq({gauges: [{name: 'foo', value: 123}]})
      end
 
-     it "should tolerate multiple metrics" do
-       lambda{ Librato::Metrics.submit :foo => 123, :bar => 456 }.should_not raise_error
-       expected = {:gauges => [{:name => 'foo', :value => 123}, {:name => 'bar', :value => 456}]}
-       Librato::Metrics.persister.persisted.should equal_unordered(expected)
+     it "tolerates multiple metrics" do
+       expect { Librato::Metrics.submit foo: 123, bar: 456 }.not_to raise_error
+       expected = {gauges: [{name: 'foo', value: 123}, {name: 'bar', value: 456}]}
+       expect(Librato::Metrics.persister.persisted).to equal_unordered(expected)
      end
    end
 
   end
 
-end
+end

metadata

@@ -1,62 +1,68 @@
 --- !ruby/object:Gem::Specification
 name: librato-metrics
 version: !ruby/object:Gem::Version
-  version: 1.6.2
+  version: 2.0.0.beta
 platform: ruby
 authors:
 - Matt Sanders
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain:
-- certs/librato-public.pem
-date: 2020-08-19 00:00:00.000000000 Z
+- !binary |-
+  LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tCk1JSURMakNDQWhhZ0F3SUJB
+  Z0lCQURBTkJna3Foa2lHOXcwQkFRVUZBREE5TVEwd0N3WURWUVFEREFSdFlY
+  UjAKTVJjd0ZRWUtDWkltaVpQeUxHUUJHUllIYkdsaWNtRjBiekVUTUJFR0Nn
+  bVNKb21UOGl4a0FSa1dBMk52YlRBZQpGdzB4TlRFd01USXlNakEwTXpWYUZ3
+  MHhOakV3TVRFeU1qQTBNelZhTUQweERUQUxCZ05WQkFNTUJHMWhkSFF4CkZ6
+  QVZCZ29Ka2lhSmsvSXNaQUVaRmdkc2FXSnlZWFJ2TVJNd0VRWUtDWkltaVpQ
+  eUxHUUJHUllEWTI5dE1JSUIKSWpBTkJna3Foa2lHOXcwQkFRRUZBQU9DQVE4
+  QU1JSUJDZ0tDQVFFQXZmTjhSbEFzVXMrSzVrSzQ2U2tocFlleQo1NG01Ylhs
+  R1plcUlndU95Z3BHKzJGSnJVSk1DbmtROVJteUswV1VqZG1BTmV2VzAxY2FR
+  UVN4NVdhdFFac28wCkdmSnlKZktuNGRHZWN6OUxPRVBwYmx3KytxYXBMWHI0
+  dmJ1T2xOWHN1bTJnZFVGYzQvWVY5bDNjc3hDbE1WRXEKK1pVbXJIamQya29P
+  dnpqSytHRFJmM2lJb3pDc2dZMG9lc2VyWjBsaS8wcVJBNVpKZUZ3V3pzOUZP
+  OVNYUHZJRApNazA3V05hWFNSVDM4bGxjUEdYNEw4SytETDd3aFF3eGFGajZJ
+  WmJvYlVFdFJqelY5di9pUDdQaUppcGlqYktCCkVpZWZnVXVvbFJSMzhOeFFF
+  K2Y0TS9sRXF3RXBrWS9mZVlNWUY2UTRoWFZydUczZnN3anJER1d5aUc1bkVR
+  SUQKQVFBQm96a3dOekFKQmdOVkhSTUVBakFBTUIwR0ExVWREZ1FXQkJSOCts
+  MFpTVGJqL1RUNllWYkZGcGRHcGc3cQpVREFMQmdOVkhROEVCQU1DQkxBd0RR
+  WUpLb1pJaHZjTkFRRUZCUUFEZ2dFQkFLdVVpdUcyUEh2NnBhUWpYamFaCkxR
+  T3NGdW5TY0h5aERoUEZoL0NjQ1VEYm1YVkRPSjNXSXlEVlF1VWdLN0kzalBF
+  RGFTRmV6U1BCcGp3WnZnTUsKMC9pbjdwWExSS2pJOEN6c1k4WTR1MTIyRE45
+  dEJ4THVyK0Uva3Y1ZnBVYmY4RkdVU3VSVDVYM3doNlpCVHpzWgpFYU1lM0xw
+  UEZVZU1TaEgvSC9GRVlyZXhSdmFNSG8vNTJCZzB6UDB5U1NBUURvdG0rcWdN
+  ZHJ1MlhSRGpFNU1zCjlCaGVBbm95Z0dIcG9LV3RFeHBza2xDcHBMMXEyWGx6
+  dzJsaG9DTUxDU1NqOVQvWXlSVEwzVVFZb0s1cFBBNzYKaHZqeDBXSlA4cHpa
+  TUpQS0pCUlpRWEpPNWlmRVB5S2paeU1pNVhNSG1ydERjbEhMajNzeDRSQXZF
+  WmpHV2tSUApKU1E9Ci0tLS0tRU5EIENFUlRJRklDQVRFLS0tLS0K
+date: 2016-08-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0.7'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '2.0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0.7'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '2.0'
-- !ruby/object:Gem::Dependency
-  name: multi_json
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: aggregate
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 0.2.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: 0.2.2
 description: An easy to use ruby wrapper for Librato's Metrics API
@@ -66,9 +72,9 @@ extensions: []
 extra_rdoc_files:
 - LICENSE
 files:
-- ".gitignore"
-- ".rspec"
-- ".travis.yml"
+- .gitignore
+- .rspec
+- .travis.yml
 - CHANGELOG.md
 - Gemfile
 - LICENSE
@@ -97,7 +103,6 @@ files:
 - lib/librato/metrics/smart_json.rb
 - lib/librato/metrics/version.rb
 - librato-metrics.gemspec
-- spec/integration/deprecated_methods_spec.rb
 - spec/integration/metrics/annotator_spec.rb
 - spec/integration/metrics/connection_spec.rb
 - spec/integration/metrics/middleware/count_requests_spec.rb
@@ -110,34 +115,34 @@ files:
 - spec/unit/metrics/connection_spec.rb
 - spec/unit/metrics/queue/autosubmission_spec.rb
 - spec/unit/metrics/queue_spec.rb
+- spec/unit/metrics/smart_json_spec.rb
 - spec/unit/metrics_spec.rb
 homepage: https://github.com/librato/librato-metrics
 licenses:
 - BSD 3-clause
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options:
-- "--charset=UTF-8"
+- --charset=UTF-8
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '1.9'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>'
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.3.1
 requirements: []
-rubyforge_project:
-rubygems_version: 2.6.14
-signing_key:
+rubyforge_project: 
+rubygems_version: 2.6.6
+signing_key: 
 specification_version: 2
 summary: Ruby wrapper for Librato's Metrics API
 test_files:
-- spec/integration/deprecated_methods_spec.rb
 - spec/integration/metrics/annotator_spec.rb
 - spec/integration/metrics/connection_spec.rb
 - spec/integration/metrics/middleware/count_requests_spec.rb
@@ -150,4 +155,5 @@ test_files:
 - spec/unit/metrics/connection_spec.rb
 - spec/unit/metrics/queue/autosubmission_spec.rb
 - spec/unit/metrics/queue_spec.rb
+- spec/unit/metrics/smart_json_spec.rb
 - spec/unit/metrics_spec.rb

data/spec/integration/deprecated_methods_spec.rb

@@ -1,85 +0,0 @@
-require 'spec_helper'
-
-DEPRECATED_METHODS = %w[fetch list update delete]
-describe Librato::Metrics do
-
-  DEPRECATED_METHODS.each do |deprecated_method|
-    it { should respond_to(deprecated_method) }
-  end
-
-  describe "Client" do
-    let(:client) { Librato::Metrics.client }
-    subject { client }
-
-    before(:all) { prep_integration_tests }
-
-    before do
-      client.submit :test_metric => 123.0
-    end
-
-    DEPRECATED_METHODS.each do |deprecated_method|
-      it { should respond_to(deprecated_method) }
-    end
-
-    describe "#fetch" do
-      context "with no measurements attributes" do
-        let(:metric) { client.fetch(:test_metric) }
-        subject { metric }
-
-        it { should_not be_nil }
-
-        it "should return a metric" do
-          metric["name"].should == "test_metric"
-        end
-      end
-
-      context "with measurements attributes" do
-        let(:measurements) { client.fetch(:test_metric, :count => 1) }
-        subject { measurements }
-
-        it { should_not be_nil }
-        it { should_not be_empty }
-
-        it "should return the measurements" do
-          measurements.should have_key("unassigned")
-          measurements["unassigned"].should be_an(Array)
-          measurements["unassigned"].first["value"].should == 123.0
-        end
-      end
-    end
-
-    describe "#list" do
-      let(:metrics) { client.list }
-      subject { metrics }
-
-      it { should_not be_nil }
-      it { should_not be_empty }
-
-      it "should return the list of metrics" do
-        metric = metrics.find { |m| m["name"] == "test_metric" }
-        metric.should_not be_nil
-      end
-    end
-
-    describe "#update" do
-      before do
-        client.update("test_metric", :display_name => "Test Deprecated Update")
-      end
-
-      let(:updated_metric) { client.get_metric("test_metric") }
-
-      it "should update the metric" do
-        updated_metric["display_name"].should == "Test Deprecated Update"
-      end
-    end
-
-    describe "#delete" do
-      it "should delete the metric" do
-        client.metrics(:name => "test_metric").should_not be_empty
-        client.delete("test_metric")
-        client.metrics(:name => "test_metric").should be_empty
-      end
-    end
-
-  end
-end