patient_zero 0.2.2 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 986f8ed157da8c5cfd002539e08ba67b214e5cec
-  data.tar.gz: a89ba560beb80054824ec480edc7ca8681a281fd
+  metadata.gz: a037356146290a626fa56f41f3ea9ac05ef1ba8e
+  data.tar.gz: 140f175a8f2f333105e5ac50b8b4263419ca8bac
 SHA512:
-  metadata.gz: c939f5cc167ead485b684cbf1e3f3aea8a0dc37b6aedeb9b66c2101e8f31346422a9d53a9bec67749dcaa0e54eb704572d8bdb4a5266a4b7022680875d40c361
-  data.tar.gz: 67a76674e62505781e1067a8fc393251b70107db7e179b27f489edb8086fd09f1b5da39ba98796b0f1f48bf314022ce32e2fc2a72e4092b45eb59f1efe8658be
+  metadata.gz: b0e70002f5430e5b3453802ada0c013e9c8154087df702d50b8c21e2c0c310d344a70f32ea89b9822a9a6d2d0de9d3adfdad10115ba7e8eba513cf9acfab3209
+  data.tar.gz: af84b55608d2edfd6f415c883d80eba65f07ffe7f201ad9dd537553f9b131dd1c2abab310d80a9bd6ba10d8c6fd98548d801d33ae59b72b49fe61a84dbecb5e0

data/lib/patient_zero/analytics/base.rb

@@ -1,6 +1,8 @@
 module PatientZero
   module Analytics
-    class Base < Client
+    class Base
+      include Client
+
       attr_accessor :token, :source_id, :start_date, :end_date
 
       def initialize token:, source_id:, start_date: nil, end_date: nil

data/lib/patient_zero/authorization.rb

@@ -1,5 +1,7 @@
 module PatientZero
-  class Authorization < Client
+  class Authorization
+    include Client
+
     def self.token
       response = post '/mobile/api/v1/user/login', email: PatientZero.email, password: PatientZero.password
       response['user_token']

data/lib/patient_zero/client.rb

@@ -1,43 +1,33 @@
 module PatientZero
-  class Client
-    def self.connection
-      @@connection ||= Faraday.new(PatientZero.url) do |faraday|
+  module Client
+    def self.included base
+      base.extend self
+    end
+
+    def connection
+      PatientZero.connection ||= Faraday.new(PatientZero.url) do |faraday|
         faraday.request  :url_encoded
         faraday.response :logger
         faraday.adapter  Faraday.default_adapter
       end
     end
 
-    def self.parse request
+    def parse request
       response = JSON.parse request.body
       raise Error, response['error'] unless response['error'].nil?
       response
     end
 
-    def self.get *args
-      parse connection.get *args
-    end
-
-    def self.post *args
-      parse connection.post *args
-    end
-
-    def self.put *args
-      parse connection.put *args
-    end
-    
-    private
-
     def get *args
-      self.class.get *args
+      parse connection.get *args
     end
 
     def post *args
-      self.class.post *args
+      parse connection.post *args
     end
 
     def put *args
-      self.class.put *args
+      parse connection.put *args
     end
   end
 end

data/lib/patient_zero/configurable.rb

@@ -1,6 +1,6 @@
 module PatientZero
   module Configurable
-    attr_accessor :url, :api_key, :email, :password
+    attr_accessor :url, :api_key, :email, :password, :connection
 
     def self.extended(base)
       base.reset

data/lib/patient_zero/organization.rb

@@ -1,5 +1,6 @@
 module PatientZero
-  class Organization < Client
+  class Organization
+    include Client
     attr_accessor :id, :name, :avatar
 
     def initialize attributes

data/lib/patient_zero/profile.rb

@@ -1,5 +1,7 @@
 module PatientZero
-  class Profile < Client
+  class Profile
+    include Client
+
     CATEGORIES = ['General', 'Brand', 'Television', 'Movies', 'Music', 'Person', 'Sports', 'Politics', 'Topic']
 
     attr_accessor :id, :expression, :name, :category

data/lib/patient_zero/source.rb

@@ -1,5 +1,7 @@
 module PatientZero
-  class Source < Client
+  class Source
+    include Client
+
     attr_accessor :id, :name, :platform, :token
 
     def initialize attributes, token

data/lib/patient_zero/version.rb

@@ -1,3 +1,3 @@
 module PatientZero
-  VERSION = "0.2.2"
+  VERSION = '0.3.0'
 end

data/spec/patient_zero/analytics/base_spec.rb

@@ -10,11 +10,11 @@ module PatientZero
       let(:token) { 'token-shmoken' }
       let(:analytical_data) { { 'stats' => [ { 'id' => source_id, 'platform' => platform, 'name' => name, 'messages' => messages } ] } }
       subject(:analytics_base) { Base.new token: token, source_id: source_id }
-      before { allow(Client).to receive(:get).with('/mobile/api/v1/analytics', anything).and_return analytical_data }
+      before { allow(analytics_base).to receive(:get).with('/mobile/api/v1/analytics', anything).and_return analytical_data }
 
       describe '#name' do
         it 'calls the analytics API endpoint' do
-          expect(Client).to receive(:get).with('/mobile/api/v1/analytics', anything)
+          expect(analytics_base).to receive(:get).with('/mobile/api/v1/analytics', anything)
           analytics_base.name
         end
         it 'pulls the name from the analytical_data hash' do
@@ -24,7 +24,7 @@ module PatientZero
 
       describe '#platform' do
         it 'calls the analytics API endpoint' do
-          expect(Client).to receive(:get).with('/mobile/api/v1/analytics', anything)
+          expect(analytics_base).to receive(:get).with('/mobile/api/v1/analytics', anything)
           analytics_base.name
         end
         it 'pulls the platform from the analytical_data hash' do
@@ -34,7 +34,7 @@ module PatientZero
 
       describe '#messages' do
         it 'calls the analytics API endpoint' do
-          expect(Client).to receive(:get).with('/mobile/api/v1/analytics', anything)
+          expect(analytics_base).to receive(:get).with('/mobile/api/v1/analytics', anything)
           analytics_base.name
         end
         context 'when there are messages' do

data/spec/patient_zero/authorization_spec.rb

@@ -5,9 +5,9 @@ module PatientZero
     describe '.token' do
       let(:token) { 'token-shmoken' }
       let(:authorization_response) { response_with_body user_token: token }
-      before{ allow(Client.connection).to receive(:post).with('/mobile/api/v1/user/login', anything).and_return authorization_response }
+      before{ allow(Authorization.connection).to receive(:post).with('/mobile/api/v1/user/login', anything).and_return authorization_response }
       it 'calls the login api endpoint' do
-        expect(Client.connection).to receive(:post).with('/mobile/api/v1/user/login', anything)
+        expect(Authorization.connection).to receive(:post).with('/mobile/api/v1/user/login', anything)
         Authorization.token
       end
       context 'when email and password are correct' do

data/spec/patient_zero/client_spec.rb

@@ -1,31 +1,97 @@
 require 'spec_helper'
 
 module PatientZero
+  class FakeClient
+    include Client
+  end
+
   describe Client do
     let(:token) { 'token-shmoken' }
     let(:body) { "{}" }
-    let(:response) { double body: body }
+    let(:response) { double :response, body: body }
+    let(:connection) { double :conn, get: response, post: response, put: response }
 
     describe '.connection' do
       it 'news up a connection to the Viral Heat API' do
-        expect(Client.connection).to be_a Faraday::Connection
+        expect(FakeClient.connection).to be_a Faraday::Connection
+      end
+
+      it 'exists on the instance' do
+        expect { FakeClient.new.connection }.to_not raise_error
       end
     end
 
     describe '.parse' do
-      subject(:parse) { Client.parse response }
+      it 'exists on the instance' do
+        expect { FakeClient.new.parse response }.to_not raise_error
+      end
       context 'when the response contains an error key with nil' do
         let(:body) { "{\"status\":200,\"error\":null,\"user_token\":\"#{token}\"}" }
         it 'returns a hash of data' do
-          expect(parse).to be_a Hash
+          expect(FakeClient.parse response).to be_a Hash
         end
       end
       context 'when the response contains an error key that is not nil' do
         let(:body) { "{\"status\":403,\"error\":\"Invalid login/password\",\"user_token\":null}" }
         it 'raises an error' do
-          expect { parse }.to raise_error Error
+          expect { FakeClient.parse response }.to raise_error Error
         end
       end
     end
+
+    describe '.get' do
+      before { allow(PatientZero).to receive(:connection).and_return connection }
+
+      it 'delegates to the connection' do
+        expect(PatientZero.connection).to receive(:get).with('an arg') { response }
+        FakeClient.get 'an arg'
+      end
+
+      it 'calls parse' do
+        expect(FakeClient).to receive(:parse).with response
+        FakeClient.get 'an arg'
+      end
+
+      it 'exists on the instance' do
+        expect { FakeClient.new.get 'an arg' }.to_not raise_error
+      end
+    end
+
+    describe '.post' do
+      before { allow(PatientZero).to receive(:connection).and_return connection }
+
+      it 'delegates to the connection' do
+        expect(PatientZero.connection).to receive(:post).with('an arg') { response }
+        FakeClient.post 'an arg'
+      end
+
+      it 'calls parse' do
+        expect(FakeClient).to receive(:parse).with response
+        FakeClient.post 'an arg'
+      end
+
+      it 'exists on the instance' do
+        expect { FakeClient.new.post 'an arg' }.to_not raise_error
+      end
+    end
+
+    describe '.put' do
+      before { allow(PatientZero).to receive(:connection).and_return connection }
+
+      it 'delegates to the connection' do
+        expect(PatientZero.connection).to receive(:put).with('an arg') { response }
+        FakeClient.put 'an arg'
+      end
+
+      it 'calls parse' do
+        expect(FakeClient).to receive(:parse).with response
+        FakeClient.put 'an arg'
+      end
+
+      it 'exists on the instance' do
+        expect { FakeClient.new.put 'an arg' }.to_not raise_error
+      end
+    end
+
   end
 end

data/spec/patient_zero/organization_spec.rb

@@ -20,10 +20,10 @@ module PatientZero
     describe '.all' do
       let(:organizations) { [organization] }
       let(:organizations_response) { response_with_body organizations: organizations_response_data }
-      before { allow(Client.connection).to receive(:get).with('/mobile/api/v1/organizations', anything) { organizations_response } }
+      before { allow(Organization.connection).to receive(:get).with('/mobile/api/v1/organizations', anything) { organizations_response } }
 
       it 'calls the organizations api endpoint' do
-        expect(Client.connection).to receive(:get).with('/mobile/api/v1/organizations', anything)
+        expect(Organization.connection).to receive(:get).with('/mobile/api/v1/organizations', anything)
         Organization.all
       end
 
@@ -52,10 +52,10 @@ module PatientZero
     describe '#token' do
       let(:organization_specific_token) { 'organization-token' }
       let(:switch_response) { response_with_body user_token: organization_specific_token }
-      before { allow(Client.connection).to receive(:get).with("/mobile/api/v1/organizations/#{id}/switch", anything).and_return switch_response }
+      before { allow(Organization.connection).to receive(:get).with("/mobile/api/v1/organizations/#{id}/switch", anything).and_return switch_response }
 
       it 'calls the organization switch api endpoint' do
-        expect(Client.connection).to receive(:get).with("/mobile/api/v1/organizations/#{id}/switch", anything)
+        expect(Organization.connection).to receive(:get).with("/mobile/api/v1/organizations/#{id}/switch", anything)
         organization.token
       end
 

data/spec/patient_zero/profile_spec.rb

@@ -21,10 +21,10 @@ module PatientZero
 
     describe '.all' do
       let(:profile_response) { response_with_body profiles: profiles }
-      before { allow(Client.connection).to receive(:get).with('/social/api/v2/monitoring/profiles', anything) { profile_response } }
+      before { allow(Profile.connection).to receive(:get).with('/social/api/v2/monitoring/profiles', anything) { profile_response } }
 
       it 'calls the profiles api endpoint' do
-        expect(Client.connection).to receive(:get).with('/social/api/v2/monitoring/profiles', anything)
+        expect(Profile.connection).to receive(:get).with('/social/api/v2/monitoring/profiles', anything)
         Profile.all
       end
 
@@ -35,10 +35,10 @@ module PatientZero
 
     describe '.find' do
       let(:profile_response) { response_with_body profiles: [profile_data] }
-      before { allow(Client.connection).to receive(:get).with('/social/api/v2/monitoring/profiles', anything) { profile_response } }
+      before { allow(Profile.connection).to receive(:get).with('/social/api/v2/monitoring/profiles', anything) { profile_response } }
 
       it 'calls the profiles api endpoint' do
-        expect(Client.connection).to receive(:get).with('/social/api/v2/monitoring/profiles', anything)
+        expect(Profile.connection).to receive(:get).with('/social/api/v2/monitoring/profiles', anything)
         Profile.find id
       end
 
@@ -49,10 +49,10 @@ module PatientZero
 
     describe '.create' do
       let(:profile_response) { response_with_body profile: profile_data }
-      before { allow(Client.connection).to receive(:post).with('/social/api/v2/monitoring/profiles', anything) { profile_response } }
+      before { allow(Profile.connection).to receive(:post).with('/social/api/v2/monitoring/profiles', anything) { profile_response } }
 
       it 'calls the profiles api endpoint' do
-        expect(Client.connection).to receive(:post).with('/social/api/v2/monitoring/profiles', anything)
+        expect(Profile.connection).to receive(:post).with('/social/api/v2/monitoring/profiles', anything)
         Profile.create profile_data
       end
 
@@ -60,20 +60,20 @@ module PatientZero
         expect(Profile.create(profile_data)).to be_a Profile
       end
     end
-    
+
     describe '.update' do
-      let(:params) { {id: 1} } 
+      let(:params) { {id: 1} }
       let(:profile_response) { response_with_body profile: profile_data }
-      before { allow(Client.connection).to receive(:put).with("/social/api/v2/monitoring/profiles/#{params[:id]}", anything) { profile_response } }
+      before { allow(Profile.connection).to receive(:put).with("/social/api/v2/monitoring/profiles/#{params[:id]}", anything) { profile_response } }
 
       it 'calls the profiles api endpoint' do
-        expect(Client.connection).to receive(:put).with("/social/api/v2/monitoring/profiles/#{params[:id]}", anything)
+        expect(Profile.connection).to receive(:put).with("/social/api/v2/monitoring/profiles/#{params[:id]}", anything)
         Profile.update params
       end
 
       it 'returns a Profile object' do
         expect(Profile.update(params)).to be_a Profile
       end
-    end    
+    end
   end
 end

data/spec/patient_zero/source_spec.rb

@@ -18,10 +18,10 @@ module PatientZero
     describe '.all' do
       before do
         allow(Authorization).to receive(:token).and_return token
-        allow(Client.connection).to receive(:get).with('/mobile/api/v1/sources/', anything).and_return response_with_body sources: sources_response_data
+        allow(Source.connection).to receive(:get).with('/mobile/api/v1/sources/', anything).and_return response_with_body sources: sources_response_data
       end
       it 'calls the sources index api' do
-        expect(Client.connection).to receive(:get).with('/mobile/api/v1/sources/', anything)
+        expect(Source.connection).to receive(:get).with('/mobile/api/v1/sources/', anything)
         Source.all
       end
       it 'calls Authorization.token if no token is passed in' do
@@ -35,7 +35,7 @@ module PatientZero
       end
       context 'when the associated organization token has no sources' do
         it 'returns an empty array' do
-          allow(Client.connection).to receive(:get).with('/mobile/api/v1/sources/', anything).and_return response_with_body sources: []
+          allow(Source.connection).to receive(:get).with('/mobile/api/v1/sources/', anything).and_return response_with_body sources: []
           expect(Source.all).to be_empty
         end
       end
@@ -44,10 +44,10 @@ module PatientZero
     describe '.find' do
       before do
         allow(Authorization).to receive(:token).and_return token
-        allow(Client.connection).to receive(:get).with('/mobile/api/v1/sources/show/', anything).and_return response_with_body source: source_response_data
+        allow(Source.connection).to receive(:get).with('/mobile/api/v1/sources/show/', anything).and_return response_with_body source: source_response_data
       end
       it 'calls the sources show api' do
-        expect(Client.connection).to receive(:get).with('/mobile/api/v1/sources/show/', anything)
+        expect(Source.connection).to receive(:get).with('/mobile/api/v1/sources/show/', anything)
         Source.find id, token
       end
       context 'when the source exists' do
@@ -57,7 +57,7 @@ module PatientZero
       end
       context 'when no source exists' do
         it 'throws an PatientZero::Error' do
-          allow(Client).to receive(:get).and_raise Error, 'The source could not be found. Please check your inputs.'
+          allow(Source).to receive(:get).and_raise Error, 'The source could not be found. Please check your inputs.'
           expect{ Source.find id, token }.to raise_error PatientZero::NotFoundError
         end
       end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: patient_zero
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.3.0
 platform: ruby
 authors:
 - Adam Zaninovich