allscripts_unity_client 2.0.5 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 68c98b1b9d819e56c4c9fde2ad52859c706127b3
-  data.tar.gz: 6ed674a745e5b8a99ee184b15ea7156df79ee2e8
+  metadata.gz: 88cc45a2484b53948ae52a7175a9c633b8cf8108
+  data.tar.gz: c792e3083d2727be7872e47e932348699ceb416f
 SHA512:
-  metadata.gz: 53d768648a047d2460d07f0b9c206248fdbe1cc28397b8a4f73ac72dbd058e23dd406c5a58394d515c741094424e08a5a2d39f59722c1fde5c9dcd50c8127dc3
-  data.tar.gz: 94b884fb135786a92c0b419a47f35983696e02e928d4aead6600fbfb6888d383f17cc982546c2a45c6cc4c99e15fe00f3a5c3ed7bca81655a06080b963cb2b23
+  metadata.gz: 20d968b2549340bd6fc496e95bab512970db94bae8f3637d0609c0859fdf1b33f640164eee30e77da9c9f3e223c0c7de428565285a6f9d5245900e7b8e56be61
+  data.tar.gz: 01714c07e5f1568d03b15e27f653d7c4efa8b3571b053bb5f9ebdb2b6473e651ca9116534b6a319e1106e5035ee66635fcc7d3cc6efe3ad41e1c7eec464d2090

data/README.md

@@ -244,6 +244,26 @@ unity_client = AllscriptsUnityClient.create({
 This gem uses the [american_date](http://rubygems.org/gems/american_date) gem to force `Date.parse` to
 accept USA locale dates by default. There are currently no plans to support other locales. Pull requests welcome.
 
+### New Relic support
+
+New Relic is supported for tracing Unity HTTP requests and the overall time processing the reqeusts takes. To enable
+New Relic, simply require the gem in your project and enable it on the client:
+
+```ruby
+require 'newrelic_rpm'
+NewRelic::Agent.manual_start
+
+unity_client = AllscriptsUnityClient.create({
+  base_unity_url: "http://unity.base.url",
+  appname: "appname",
+  username: "username",
+  password: "password",
+  new_relic: true
+})
+```
+
+For more information on the New Relic gem, go here: (https://github.com/newrelic/rpm)
+
 ## Examples
 
 ### GetServerInfo SOAP

data/allscripts_unity_client.gemspec

@@ -29,11 +29,12 @@ Gem::Specification.new do |gem|
   gem.add_runtime_dependency 'nokogiri', '< 1.6', '>= 1.4.0'
   gem.add_runtime_dependency 'nori', '~> 2.3.0'
   gem.add_runtime_dependency 'american_date', '~> 1.1.0'
+  gem.add_runtime_dependency 'oj', '~> 2.9.8'
 
   gem.add_development_dependency 'factory_girl', '~> 4.4.0'
   gem.add_development_dependency 'rake', '~> 10.3.1'
   gem.add_development_dependency 'faker', '~> 1.3.0'
-  gem.add_development_dependency 'rspec', '~> 3.0.0.beta2'
+  gem.add_development_dependency 'rspec', '~> 3.0.0'
   gem.add_development_dependency 'simplecov', '~> 0.8.2'
   gem.add_development_dependency 'webmock', '~> 1.17.4'
   gem.add_development_dependency 'coveralls', '~> 0.7.0'

data/lib/allscripts_unity_client.rb

@@ -9,6 +9,7 @@ require 'allscripts_unity_client/client_driver'
 require 'allscripts_unity_client/client_options'
 require 'allscripts_unity_client/soap_client_driver'
 require 'allscripts_unity_client/json_client_driver'
+require 'allscripts_unity_client/new_relic_support'
 
 module AllscriptsUnityClient
   class APIError < RuntimeError

data/lib/allscripts_unity_client/client.rb

@@ -501,26 +501,33 @@ module AllscriptsUnityClient
       # Generate XML structure for rxxml
       builder = Nokogiri::XML::Builder.new do |xml|
         xml.saverx {
-          xml.field('name' => 'transid', 'value' => rxxml[:transid]) unless rxxml[:transid].nil?
-          xml.field('name' => 'PharmID', 'value' => rxxml[:pharmid]) unless rxxml[:pharmid].nil?
-          xml.field('name' => 'DDI', 'value' => rxxml[:ddi]) unless rxxml[:ddi].nil?
-          xml.field('name' => 'GPPCCode', 'value' => rxxml[:gppccode]) unless rxxml[:gppccode].nil?
-          xml.field('name' => 'GPPCText', 'value' => rxxml[:gppctext]) unless rxxml[:gppctext].nil?
-          xml.field('name' => 'GPPCCustom', 'value' => rxxml[:gppccustom]) unless rxxml[:gppccustom].nil?
-          xml.field('name' => 'Sig', 'value' => rxxml[:sig]) unless rxxml[:sig].nil?
-          xml.field('name' => 'QuanPresc', 'value' => rxxml[:quanpresc]) unless rxxml[:quanpresc].nil?
-          xml.field('name' => 'Refills', 'value' => rxxml[:refills]) unless rxxml[:refills].nil?
-          xml.field('name' => 'DAW', 'value' => rxxml[:daw]) unless rxxml[:daw].nil?
-          xml.field('name' => 'DaysSupply', 'value' => rxxml[:dayssupply]) unless rxxml[:dayssupply].nil?
-          xml.field('name' => 'startdate', 'value' => utc_to_local(Date.parse(rxxml[:startdate].to_s))) unless rxxml[:startdate].nil?
-          xml.field('name' => 'historicalflag', 'value' => rxxml[:historicalflag]) unless rxxml[:historicalflag].nil?
-          xml.field('name' => 'rxaction', 'value' => rxxml[:rxaction]) unless rxxml[:rxaction].nil?
-          xml.field('name' => 'delivery', 'value' => rxxml[:delivery]) unless rxxml[:delivery].nil?
-          xml.field('name' => 'ignorepharmzero', 'value' => rxxml[:ignorepharmzero]) unless rxxml[:ignorepharmzero].nil?
-          xml.field('name' => 'orderedbyid', 'value' => rxxml[:orderedbyid]) unless rxxml[:orderedbyid].nil?
-          xml.field('name' => 'newqty', 'value' => rxxml[:newqty]) unless rxxml[:newqty].nil?
-          xml.field('name' => 'newrefills', 'value' => rxxml[:newrefills]) unless rxxml[:newrefills].nil?
-          xml.field('name' => 'comments', 'value' => rxxml[:comments]) unless rxxml[:comments].nil?
+          xml.field('name' => 'transid', 'value' => rxxml[:transid]) unless rxxml[:transid]
+          xml.field('name' => 'PharmID', 'value' => rxxml[:pharmid]) unless rxxml[:pharmid]
+          xml.field('name' => 'DDI', 'value' => rxxml[:ddi]) unless rxxml[:ddi]
+          xml.field('name' => 'GPPCCode', 'value' => rxxml[:gppccode]) unless rxxml[:gppccode]
+          xml.field('name' => 'GPPCText', 'value' => rxxml[:gppctext]) unless rxxml[:gppctext]
+          xml.field('name' => 'GPPCCustom', 'value' => rxxml[:gppccustom]) unless rxxml[:gppccustom]
+          xml.field('name' => 'Sig', 'value' => rxxml[:sig]) unless rxxml[:sig]
+          xml.field('name' => 'QuanPresc', 'value' => rxxml[:quanpresc]) unless rxxml[:quanpresc]
+          xml.field('name' => 'Refills', 'value' => rxxml[:refills]) unless rxxml[:refills]
+          xml.field('name' => 'DAW', 'value' => rxxml[:daw]) unless rxxml[:daw]
+          xml.field('name' => 'DaysSupply', 'value' => rxxml[:dayssupply]) unless rxxml[:dayssupply]
+          xml.field('name' => 'startdate', 'value' => utc_to_local(Date.parse(rxxml[:startdate].to_s))) unless rxxml[:startdate]
+          xml.field('name' => 'historicalflag', 'value' => rxxml[:historicalflag]) unless rxxml[:historicalflag]
+          xml.field('name' => 'rxaction', 'value' => rxxml[:rxaction]) unless rxxml[:rxaction]
+          xml.field('name' => 'delivery', 'value' => rxxml[:delivery]) unless rxxml[:delivery]
+          xml.field('name' => 'ignorepharmzero', 'value' => rxxml[:ignorepharmzero]) unless rxxml[:ignorepharmzero]
+          xml.field('name' => 'orderedbyid', 'value' => rxxml[:orderedbyid]) unless rxxml[:orderedbyid]
+          xml.field('name' => 'newqty', 'value' => rxxml[:newqty]) unless rxxml[:newqty]
+          xml.field('name' => 'newrefills', 'value' => rxxml[:newrefills]) unless rxxml[:newrefills]
+          xml.field('name' => 'comments', 'value' => rxxml[:comments]) unless rxxml[:comments]
+          xml.field('name' => 'orderstatus', 'value' => rxxml[:order_status]) unless rxxml[:order_status]
+
+          if rxxml[:problems]
+            rxxml[:problems].each do |problem|
+              xml.field('name' => 'Problem', 'value' => problem)
+            end
+          end
         }
       end
 

data/lib/allscripts_unity_client/client_driver.rb

@@ -8,6 +8,15 @@ module AllscriptsUnityClient
 
     def initialize(options)
       @options = ClientOptions.new(options)
+
+      # If New Relic support is enabled, setup method tracing
+      if @options.new_relic
+        NewRelicSupport.enable_method_tracer(self)
+
+        class << self
+          add_method_tracer :magic
+        end
+      end
     end
 
     def security_token?

data/lib/allscripts_unity_client/client_options.rb

@@ -1,6 +1,6 @@
 module AllscriptsUnityClient
   class ClientOptions
-    attr_accessor :proxy, :logger, :ca_file, :ca_path, :timeout
+    attr_accessor :proxy, :logger, :ca_file, :ca_path, :timeout, :new_relic
     attr_reader :base_unity_url, :username, :password, :appname, :timezone
 
     def initialize(options = {})
@@ -12,6 +12,7 @@ module AllscriptsUnityClient
       @ca_file = options[:ca_file]
       @ca_path = options[:ca_path]
       @timeout = options[:timeout]
+      @new_relic = options[:new_relic]
 
       self.timezone = options[:timezone]
       self.base_unity_url = options[:base_unity_url]
@@ -78,5 +79,9 @@ module AllscriptsUnityClient
     def timeout?
       !@timeout.to_s.strip.empty?
     end
+
+    def new_relic?
+      !@new_relic.nil?
+    end
   end
 end

data/lib/allscripts_unity_client/json_client_driver.rb

@@ -1,4 +1,4 @@
-require 'json'
+require 'oj'
 require 'faraday'
 require 'em-http-request'
 
@@ -22,17 +22,20 @@ module AllscriptsUnityClient
     def magic(parameters = {})
       request_data = JSONUnityRequest.new(parameters, @options.timezone, @options.appname, @security_token)
 
-      response = @connection.post do |request|
-        request.url "#{UNITY_JSON_ENDPOINT}/MagicJson"
-        request.headers['Content-Type'] = 'application/json'
-        request.body = JSON.generate(request_data.to_hash)
-        set_request_timeout(request)
+      response = nil
+      NewRelicSupport.trace_execution_scoped_if_available(JSONClientDriver, ["Custom/UnityJSON/#{parameters[:action]}"]) do
+        response = @connection.post do |request|
+          request.url "#{UNITY_JSON_ENDPOINT}/MagicJson"
+          request.headers['Content-Type'] = 'application/json'
+          request.body = Oj.dump(request_data.to_hash, mode: :compat)
+          set_request_timeout(request)
 
-        start_timer
+          start_timer
+        end
       end
       end_timer
 
-      response = JSON.parse(response.body)
+      response = Oj.load(response.body, mode: :strict)
 
       raise_if_response_error(response)
       log_magic(request_data)
@@ -55,7 +58,7 @@ module AllscriptsUnityClient
       response = @connection.post do |request|
         request.url "#{UNITY_JSON_ENDPOINT}/GetToken"
         request.headers['Content-Type'] = 'application/json'
-        request.body = JSON.generate(request_data)
+        request.body = Oj.dump(request_data, mode: :compat)
         set_request_timeout(request)
 
         start_timer
@@ -80,7 +83,7 @@ module AllscriptsUnityClient
       response = @connection.post do |request|
         request.url "#{UNITY_JSON_ENDPOINT}/RetireSecurityToken"
         request.headers['Content-Type'] = 'application/json'
-        request.body = JSON.generate(request_data)
+        request.body = Oj.dump(request_data, mode: :compat)
         set_request_timeout(request)
 
         start_timer

data/lib/allscripts_unity_client/json_unity_response.rb

@@ -1,5 +1,3 @@
-require 'json'
-
 module AllscriptsUnityClient
   class JSONUnityResponse < UnityResponse
     def to_hash

data/lib/allscripts_unity_client/new_relic_support.rb

@@ -0,0 +1,19 @@
+module AllscriptsUnityClient
+  module NewRelicSupport
+    def self.enable_method_tracer(instance)
+      class << instance
+        if !respond_to?(:trace_execution_scoped) && !respond_to?(:add_method_tracer)
+          extend ::NewRelic::Agent::MethodTracer
+        end
+      end
+    end
+
+    def self.trace_execution_scoped_if_available(klass, scope)
+      if klass.respond_to?(:trace_execution_scoped)
+        klass.trace_execution_scoped(scope, &Proc.new)
+      else
+        yield if block_given?
+      end
+    end
+  end
+end

data/lib/allscripts_unity_client/soap_client_driver.rb

@@ -59,10 +59,13 @@ module AllscriptsUnityClient
         soap_action: "#{UNITY_ENDPOINT_NAMESPACE}/Magic"
       }
 
+      response = nil
       begin
         start_timer
-        response = @savon_client.call('Magic', call_data)
-        end_timer
+        NewRelicSupport.trace_execution_scoped_if_available(SOAPClientDriver, ["Custom/UnitySOAP/#{parameters[:action]}"]) do
+          response = @savon_client.call('Magic', call_data)
+          end_timer
+        end
       rescue Savon::SOAPFault => e
         raise APIError, e.message
       end

data/lib/allscripts_unity_client/version.rb

@@ -1,3 +1,3 @@
 module AllscriptsUnityClient
-  VERSION = '2.0.5'
+  VERSION = '2.1.0'
 end

data/spec/allscripts_unity_client_spec.rb

@@ -1,9 +1,9 @@
 require 'spec_helper'
 
-describe 'AllscriptsUnityClient' do
+describe AllscriptsUnityClient do
   include Savon::SpecHelper
 
-  subject { AllscriptsUnityClient }
+  subject { described_class }
 
   describe '.create' do
     context 'when given mode: :soap' do

data/spec/client_driver_spec.rb

@@ -1,9 +1,10 @@
 require 'spec_helper'
 
-describe 'ClientDriver' do
+describe AllscriptsUnityClient::ClientDriver do
   it_behaves_like 'a client driver'
 
   subject { build(:client_driver) }
+  let(:new_relic_client_driver) { build(:client_driver, new_relic: true) }
 
   describe '#client_type' do
     it { expect(subject.client_type).to be(:none) }

data/spec/client_options_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'ClientOptions' do
+describe AllscriptsUnityClient::ClientOptions do
   subject { build(:client_options) }
 
   let(:url_with_slash) { 'http://www.example.com/' }
@@ -182,4 +182,20 @@ describe 'ClientOptions' do
       end
     end
   end
+
+  describe '#new_relic?' do
+    context 'when new_relic is nil' do
+      it 'returns false' do
+        subject.new_relic = nil
+        expect(subject.new_relic?).to be_falsey
+      end
+    end
+
+    context 'when timeout is not nil' do
+      it 'returns true' do
+        subject.new_relic = true
+        expect(subject.new_relic?).to be_truthy
+      end
+    end
+  end
 end

data/spec/client_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'Client' do
+describe AllscriptsUnityClient::Client do
   subject { build(:client) }
 
   describe '#initialize' do

data/spec/factories/client_driver_factory.rb

@@ -12,5 +12,9 @@ FactoryGirl.define do
     ca_file nil
     ca_path nil
     timeout nil
+    new_relic nil
+
+    factory :json_client_driver, class: AllscriptsUnityClient::JSONClientDriver
+    factory :soap_client_driver, class: AllscriptsUnityClient::SOAPClientDriver
   end
 end

data/spec/factories/client_options.rb

@@ -12,5 +12,6 @@ FactoryGirl.define do
     ca_file nil
     ca_path nil
     timeout nil
+    new_relic nil
   end
 end

data/spec/factories/unity_request_factory.rb

@@ -6,5 +6,7 @@ FactoryGirl.define do
     timezone { build(:timezone) }
     appname Faker::Name.name
     security_token SecureRandom.uuid
+
+    factory :json_unity_request, class: AllscriptsUnityClient::JSONUnityRequest
   end
 end

data/spec/factories/unity_response_factory.rb

@@ -4,5 +4,7 @@ FactoryGirl.define do
 
     response Hash.new
     timezone { build(:timezone) }
+
+    factory :json_unity_response, class: AllscriptsUnityClient::JSONUnityResponse
   end
 end

data/spec/json_client_driver_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'JSONClientDriver' do
+describe AllscriptsUnityClient::JSONClientDriver do
   it_behaves_like 'a client driver'
 
   subject do
@@ -9,6 +9,7 @@ describe 'JSONClientDriver' do
     client_driver
   end
 
+  let(:new_relic_client_driver) { build(:json_client_driver, new_relic: true) }
   let(:get_server_info) { FixtureLoader.load_file('get_server_info.json') }
   let(:get_security_token) { FixtureLoader.load_file('get_security_token.json') }
   let(:retire_security_token) { FixtureLoader.load_file('retire_security_token.json') }
@@ -23,7 +24,7 @@ describe 'JSONClientDriver' do
   end
 
   let(:json_hash) do
-    JSON.generate(hash)
+    Oj.dump(hash, mode: :strict)
   end
 
   describe '#initialize' do
@@ -51,6 +52,11 @@ describe 'JSONClientDriver' do
       expect(WebMock).to have_requested(:post, 'http://www.example.com/Unity/UnityService.svc/json/MagicJson').with(body: /\{"Action":(null|"[^"]*"),"AppUserID":(null|"[^"]*"),"Appname":(null|"[^"]*"),"PatientID":(null|"[^"]*"),"Token":(null|"[^"]*"),"Parameter1":(null|"[^"]*"),"Parameter2":(null|"[^"]*"),"Parameter3":(null|"[^"]*"),"Parameter4":(null|"[^"]*"),"Parameter5":(null|"[^"]*"),"Parameter6":(null|"[^"]*"),"Data":(null|"[^"]*")\}/, headers: { 'Content-Type' => 'application/json' })
     end
 
+    it 'should serialize DateTime to iso8601 when given' do
+      subject.magic(parameter1: DateTime.now)
+      expect(WebMock).to have_requested(:post, 'http://www.example.com/Unity/UnityService.svc/json/MagicJson').with(body: /\{"Action":(null|"[^"]*"),"AppUserID":(null|"[^"]*"),"Appname":(null|"[^"]*"),"PatientID":(null|"[^"]*"),"Token":(null|"[^"]*"),"Parameter1":"\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2}(-|\+)\d{2}:\d{2}","Parameter2":(null|"[^"]*"),"Parameter3":(null|"[^"]*"),"Parameter4":(null|"[^"]*"),"Parameter5":(null|"[^"]*"),"Parameter6":(null|"[^"]*"),"Data":(null|"[^"]*")\}/, headers: { 'Content-Type' => 'application/json' })
+    end
+
     it 'should call start_timer' do
       subject.magic
       expect(subject).to have_received(:start_timer)

data/spec/json_unity_request_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'JSONUnityRequest' do
+describe AllscriptsUnityClient::JSONUnityRequest do
   it_behaves_like 'a unity request'
 
   subject { build(:json_unity_request) }

data/spec/json_unity_response_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'JSONUnityResponse' do
+describe AllscriptsUnityClient::JSONUnityResponse do
   it_behaves_like 'a unity response'
 
   subject { build(:json_unity_response, response: get_server_info) }

data/spec/new_relic_support_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe AllscriptsUnityClient::NewRelicSupport do
+  subject { described_class }
+  let(:scope) { ['scope'] }
+  let(:block) { lambda {} }
+
+  before :all do
+    # Mock a test class for NewRelic mixin tests.
+    class NewRelicSupportTest
+    end
+  end
+
+  describe '.enable_method_tracer' do
+    context 'when given an object that does not have the NewRelic::Agent::MethodTracer module mixed in' do
+      it 'mixes in the NewRelic::Agent::MethodTracer module' do
+        allow(NewRelicSupportTest).to receive(:extend)
+        subject.enable_method_tracer(NewRelicSupportTest.new)
+        expect(NewRelicSupportTest).to have_received(:extend).with(::NewRelic::Agent::MethodTracer)
+      end
+    end
+
+    context 'when given an object that does have the NewRelic::Agent::MethodTracer module mixed in' do
+      it 'does not mix in the NewRelic::Agent::MethodTracer module' do
+        allow(NewRelicSupportTest).to receive(:extend)
+        allow(NewRelicSupportTest).to receive(:trace_execution_scoped)
+        allow(NewRelicSupportTest).to receive(:add_method_tracer)
+        subject.enable_method_tracer(NewRelicSupportTest.new)
+        expect(NewRelicSupportTest).not_to have_received(:extend).with(::NewRelic::Agent::MethodTracer)
+      end
+    end
+  end
+
+  describe '.trace_execution_scoped_if_available' do
+    context 'when a class does not have trace_execution_scoped and is given block' do
+      it 'will yield to the block' do
+        expect { |b| subject.trace_execution_scoped_if_available(NewRelicSupportTest, scope, &b) }.to yield_control
+      end
+    end
+
+    context 'when a class has trace_execution_scoped and is given a block' do
+      it 'will pass the block to trace_execution_scoped' do
+        allow(NewRelicSupportTest).to receive(:trace_execution_scoped)
+        subject.trace_execution_scoped_if_available(NewRelicSupportTest, scope, &block)
+        expect(NewRelicSupportTest).to have_received(:trace_execution_scoped).with(scope)
+      end
+    end
+  end
+end

data/spec/soap_client_driver_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'SOAPClientDriver' do
+describe AllscriptsUnityClient::SOAPClientDriver do
   include Savon::SpecHelper
   it_behaves_like 'a client driver'
 
@@ -10,6 +10,7 @@ describe 'SOAPClientDriver' do
     client_driver
   end
 
+  let(:new_relic_client_driver) { build(:soap_client_driver, new_relic: true) }
   let(:get_server_info) { FixtureLoader.load_file('get_server_info.xml') }
   let(:get_security_token) { FixtureLoader.load_file('get_security_token.xml') }
   let(:retire_security_token) { FixtureLoader.load_file('retire_security_token.xml') }

data/spec/support/new_relic.rb

@@ -0,0 +1,16 @@
+RSpec.configure do |config|
+  config.before(:suite) do
+    # Mock the NewRelic::Agent::MethodTracer module
+    module NewRelic
+      module Agent
+        module MethodTracer
+          def trace_execution_scoped
+          end
+
+          def add_method_tracer
+          end
+        end
+      end
+    end
+  end
+end

data/spec/support/shared_examples_for_client_driver.rb

@@ -1,4 +1,16 @@
 shared_examples 'a client driver' do
+  describe '#initialize' do
+    context 'when options.new_relic is true' do
+      it 'enables NewRelic tracing' do
+        allow(AllscriptsUnityClient::NewRelicSupport).to receive(:enable_method_tracer)
+        allow(described_class).to receive(:add_method_tracer)
+        new_relic_client_driver
+        expect(AllscriptsUnityClient::NewRelicSupport).to have_received(:enable_method_tracer).with(new_relic_client_driver)
+        expect(described_class).to have_received(:add_method_tracer).with(:magic)
+      end
+    end
+  end
+
   describe '#security_token?' do
     context 'when @security_token is nil' do
       it do

data/spec/timezone_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'Timezone' do
+describe AllscriptsUnityClient::Timezone do
   subject { build(:timezone) }
 
   let(:positive_timezone) { build(:timezone, zone_identifier: 'Asia/Jakarta') }

data/spec/unity_request_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'UnityRequest' do
+describe AllscriptsUnityClient::UnityRequest do
   it_behaves_like 'a unity request'
 
   subject { build(:unity_request) }

data/spec/unity_response_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe 'UnityResponse' do
+describe AllscriptsUnityClient::UnityResponse do
   it_behaves_like 'a unity response'
 
   subject { build(:unity_response, response: get_server_info) }

data/spec/utilities_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
-describe 'Utilities' do
-  subject { AllscriptsUnityClient::Utilities }
+describe AllscriptsUnityClient::Utilities do
+  subject { described_class }
 
   let(:date_string) { '2013-02-15' }
   let(:date) { Date.parse(date_string) }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: allscripts_unity_client
 version: !ruby/object:Gem::Version
-  version: 2.0.5
+  version: 2.1.0
 platform: ruby
 authors:
 - Ash Gupta
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-06 00:00:00.000000000 Z
+date: 2014-07-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: savon
@@ -115,6 +115,20 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 1.1.0
+- !ruby/object:Gem::Dependency
+  name: oj
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.9.8
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 2.9.8
 - !ruby/object:Gem::Dependency
   name: factory_girl
   requirement: !ruby/object:Gem::Requirement
@@ -163,14 +177,14 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 3.0.0.beta2
+        version: 3.0.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 3.0.0.beta2
+        version: 3.0.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
@@ -236,6 +250,7 @@ files:
 - lib/allscripts_unity_client/json_client_driver.rb
 - lib/allscripts_unity_client/json_unity_request.rb
 - lib/allscripts_unity_client/json_unity_response.rb
+- lib/allscripts_unity_client/new_relic_support.rb
 - lib/allscripts_unity_client/soap_client_driver.rb
 - lib/allscripts_unity_client/timezone.rb
 - lib/allscripts_unity_client/unity_request.rb
@@ -250,11 +265,7 @@ files:
 - spec/factories/client_driver_factory.rb
 - spec/factories/client_factory.rb
 - spec/factories/client_options.rb
-- spec/factories/json_client_driver_factory.rb
-- spec/factories/json_unity_request_factory.rb
-- spec/factories/json_unity_response_factory.rb
 - spec/factories/magic_request_factory.rb
-- spec/factories/soap_client_driver_factory.rb
 - spec/factories/timezone_factory.rb
 - spec/factories/unity_request_factory.rb
 - spec/factories/unity_response_factory.rb
@@ -281,10 +292,12 @@ files:
 - spec/json_client_driver_spec.rb
 - spec/json_unity_request_spec.rb
 - spec/json_unity_response_spec.rb
+- spec/new_relic_support_spec.rb
 - spec/soap_client_driver_spec.rb
 - spec/spec_helper.rb
 - spec/support/factory_girl.rb
 - spec/support/fixture_loader.rb
+- spec/support/new_relic.rb
 - spec/support/shared_examples_for_client_driver.rb
 - spec/support/shared_examples_for_unity_request.rb
 - spec/support/shared_examples_for_unity_response.rb

data/spec/factories/json_client_driver_factory.rb

@@ -1,3 +0,0 @@
-FactoryGirl.define do
-  factory :json_client_driver, parent: :client_driver, class: AllscriptsUnityClient::JSONClientDriver
-end

data/spec/factories/json_unity_request_factory.rb

@@ -1,3 +0,0 @@
-FactoryGirl.define do
-  factory :json_unity_request, parent: :unity_request, class: AllscriptsUnityClient::JSONUnityRequest
-end

data/spec/factories/json_unity_response_factory.rb

@@ -1,3 +0,0 @@
-FactoryGirl.define do
-  factory :json_unity_response, parent: :unity_response, class: AllscriptsUnityClient::JSONUnityResponse
-end

data/spec/factories/soap_client_driver_factory.rb

@@ -1,3 +0,0 @@
-FactoryGirl.define do
-  factory :soap_client_driver, parent: :client_driver, class: AllscriptsUnityClient::SOAPClientDriver
-end