cta_aggregator_client 0.1.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 409f45de3418b9c9b28d8658dba105cc036f1173
-  data.tar.gz: caff1471ad88c925a46d1ab2037613bd1f435eab
+  metadata.gz: '0296b1a6b5e845c92ff5e651a0845f9e414ebc53'
+  data.tar.gz: ac44dbb3526e56a04b642a0fdfc298ab2af27f4b
 SHA512:
-  metadata.gz: 611d21df72ce9c2c93aedf0aeb9aeb3fb210d7af405882b89c0abc0d69c592eb7a1802b0dc748f8bc768a83476324d48eb25ba4ed9120c37dedc5e5401f8a40f
-  data.tar.gz: eb92d66882035b642d7beba93572aad7607dff1c0175762686a83b4a9b0825c9ee1b8e4f2508dc77562711df19f88291ff751e78df294c23e0aab81d17ee8bc5
+  metadata.gz: 4a323263b16f7661e6398968bcebdc945936807efb20bd30f71a60d24fea94bf529f2e63d73798ea6c49a9c69349881b4fa117f1ebbb023b563577cec4a30860
+  data.tar.gz: c32e4d4ce6caedcf35a50b50219215fc0739eabbd700980c1fe75fe412419473b354181dd09d406124c6a469110a2fef8dd6b7074384e772e40ad43dd0f7d72a

data/lib/cta_aggregator_client/advocacy_campaign.rb

@@ -1,10 +1,10 @@
 require 'cta_aggregator_client/cta_resource'
-
 module CTAAggregatorClient
   module AdvocacyCampaign
     extend CTAResource
 
     RESOURCE_NAME = :advocacy_campaign
+    RELATIONSHIPS = :targets
 
     class << self
 
@@ -12,6 +12,10 @@ module CTAAggregatorClient
         RESOURCE_NAME
       end
 
+      def relationships
+        RELATIONSHIPS
+      end
+
     end
   end
 end

data/lib/cta_aggregator_client/api/client.rb

@@ -36,8 +36,28 @@ module CTAAggregatorClient
           perform_authenticated_request(:post, url, payload)
         end
 
+        def update(resource_name, uuid, attributes)
+          url = "#{base_url}/#{api_version}/#{resource_name}s/#{uuid}"
+          payload = {
+            'data': {
+              'type': "#{resource_name}s",
+              'id': uuid,
+              'attributes': attributes
+            }
+          }.to_json
+
+          perform_authenticated_request(:put, url, payload)
+        end
+
+        def delete(resource_name, uuid)
+          url = "#{base_url}/#{api_version}/#{resource_name}s/#{uuid}"
+
+          raw_response = RestClient.delete(url, headers_with_access_token)
+          translate_response(raw_response)
+        end
+
         def relationship_params(relationships)
-          return {} unless relationships
+          return {} unless relationships && relationships.reject { |k,v| v.nil? }.any?
 
           relationships.each_with_object({}) do |(resource_name, uuid_data), obj|
             if uuid_data.is_a? Array
@@ -73,7 +93,7 @@ module CTAAggregatorClient
           perform_authenticated_request(:put, url, payload)
         end
 
-        def perform_authenticated_request(http_method, url, payload)
+        def perform_authenticated_request(http_method, url, payload={})
           raw_response = RestClient.send(http_method.to_sym, url, payload, headers_with_access_token)
           translate_response(raw_response)
         end

data/lib/cta_aggregator_client/api/utilities.rb

@@ -20,7 +20,7 @@ module CTAAggregatorClient
       end
 
       def auth_url
-        "#{base_url}/#{api_version}/authorize"
+        "#{base_url}/#{api_version}/authentications"
       end
 
       def api_key

data/lib/cta_aggregator_client/cta_resource.rb

@@ -11,13 +11,30 @@ module CTAAggregatorClient
       API::Client.find(resource_name, uuid)
     end
 
-    def create(attributes, relationships = {})
-      API::Client.create(resource_name, attributes, relationships)
+    def create(attributes)
+      if relationships
+        relationship_data = { relationships => attributes.delete(relationships) }
+      end
+
+      API::Client.create(resource_name, attributes, relationship_data)
+    end
+
+    def update(attributes)
+      uuid = attributes.delete(:id)
+      API::Client.update(resource_name, uuid, attributes)
+    end
+
+    def delete(uuid)
+      API::Client.delete(resource_name, uuid)
     end
 
     def resource_name
       raise NotImplementedError
     end
 
+    def relationships
+      nil
+    end
+
   end
 end

data/lib/cta_aggregator_client/event.rb

@@ -5,6 +5,7 @@ module CTAAggregatorClient
     extend CTAResource
 
     RESOURCE_NAME = :event
+    RELATIONSHIPS = :location
 
     class << self
 
@@ -12,6 +13,10 @@ module CTAAggregatorClient
         RESOURCE_NAME
       end
 
+      def relationships
+        RELATIONSHIPS
+      end
+
     end
   end
 end

data/lib/cta_aggregator_client/version.rb

@@ -1,3 +1,3 @@
 module CTAAggregatorClient
-  VERSION = '0.1.0'
+  VERSION = '0.4.0'
 end

data/spec/helpers/test_helpers.rb

@@ -11,7 +11,7 @@ module TestHelpers
   end
 
   def auth_url
-    "#{api_url}/authorize"
+    "#{api_url}/authentications"
   end
 
   def headers_with_auth_creds

data/spec/integration/advocacy_campaign_spec.rb

@@ -4,6 +4,8 @@ describe CTAAggregatorClient::AdvocacyCampaign do
 
   it_behaves_like 'listable resource'
   it_behaves_like 'findable resource'
+  it_behaves_like 'updatable resource'
+  it_behaves_like 'deletable resource'
 
   context 'creation' do
     campaign_attrs = {
@@ -14,9 +16,9 @@ describe CTAAggregatorClient::AdvocacyCampaign do
       featured_image_url: 'http://lorempixel.com/300/300',
       action_type: 'email',
       template: 'Eum itaque et nisi dolores assumenda ipsum. Voluptates qui aut nobis veniam maxime qui. Illum saepe eum corporis vero qui soluta aliquam.\nTemporibus reiciendis velit fugiat. Facere laborum quia ea laboriosam necessitatibus quisquam eligendi. Et est dolorem eligendi id aut sint.\nAnimi qui totam voluptatem nesciunt iure et dolorem. Eos ut dolorum et quo illum fuga atque. Facere a ipsa corrupti assumenda provident commodi facilis. Eligendi officia est et.',
+      targets: ['a2f6f86b-a214-4892-8c06-8caece820fb0', '215ed993-3cd1-4fbc-b8af-7e2082813d06']
     }
-    relationship_attrs = { targets: ['a2f6f86b-a214-4892-8c06-8caece820fb0', '215ed993-3cd1-4fbc-b8af-7e2082813d06'] }
 
-    it_behaves_like 'creatable resource', campaign_attrs, relationship_attrs
+    it_behaves_like 'creatable resource', campaign_attrs, :targets
   end
 end

data/spec/integration/events_spec.rb

@@ -4,6 +4,9 @@ describe CTAAggregatorClient::Event do
 
   it_behaves_like 'listable resource'
   it_behaves_like 'findable resource'
+  it_behaves_like 'updatable resource'
+  it_behaves_like 'deletable resource'
+
   context 'creation' do
     event_attrs = {
       title: 'March on Washington',
@@ -14,10 +17,10 @@ describe CTAAggregatorClient::Event do
       start_date: '2017-07-08T03:58:25.098Z',
       end_date: '2017-07-13T03:58:25.098Z',
       free: false,
+      location: '215ed993-3cd1-4fbc-b8af-7e2082813d06'
     }
-    relationship_attrs = { location: '215ed993-3cd1-4fbc-b8af-7e2082813d06' }
 
-    it_behaves_like 'creatable resource', event_attrs, relationship_attrs
+    it_behaves_like 'creatable resource', event_attrs, :location
   end
 
 end

data/spec/integration/location_spec.rb

@@ -4,6 +4,9 @@ describe CTAAggregatorClient::Location do
 
   it_behaves_like 'listable resource'
   it_behaves_like 'findable resource'
+  it_behaves_like 'updatable resource'
+  it_behaves_like 'deletable resource'
+
   context 'creation' do
     attributes = {
       venue: 'Eastern Kemmer University',

data/spec/integration/target_spec.rb

@@ -4,6 +4,8 @@ describe CTAAggregatorClient::Target do
 
   it_behaves_like 'listable resource'
   it_behaves_like 'findable resource'
+  it_behaves_like 'updatable resource'
+  it_behaves_like 'deletable resource'
 
   context 'creation' do
     attributes = {

data/spec/shared_examples/creatable_resource.rb

@@ -4,7 +4,8 @@ shared_examples_for "creatable resource" do |passed_attrs, passed_relationships,
   let(:token) { Factory.token }
 
   def relationships(relationship_hash)
-    return {} unless relationship_hash
+    return {} unless relationship_hash && relationship_hash.reject { |k,v| v.nil? }.any?
+
     relationship_hash.each_with_object({}) do |(resource_name, uuid_data), obj|
       if uuid_data.is_a? Array
         obj[resource_name.to_sym] = {
@@ -28,12 +29,14 @@ shared_examples_for "creatable resource" do |passed_attrs, passed_relationships,
       resource_name = formatted_resource_name(described_class, resource)
 
       url = "#{api_url}/#{resource_name}"
+      payload_params = passed_attrs.clone
+      relationship_data = { passed_relationships => payload_params.delete(passed_relationships) }
 
       payload = {
         'data': {
           'type': resource_name,
-          'attributes': passed_attrs,
-          'relationships': relationships(passed_relationships)
+          'attributes': payload_params,
+          'relationships': relationships(relationship_data)
         }.reject{ |k,v| v.empty? }
       }
 
@@ -53,7 +56,7 @@ shared_examples_for "creatable resource" do |passed_attrs, passed_relationships,
         headers_with_auth_token
       ).and_return(response)
 
-      expect(described_class.create(passed_attrs, passed_relationships)).to eq response
+      expect(described_class.create(passed_attrs)).to eq response
     end
   end
 
@@ -73,11 +76,14 @@ shared_examples_for "creatable resource" do |passed_attrs, passed_relationships,
       resource_name = formatted_resource_name(described_class, resource)
 
       url = "#{api_url}/#{resource_name}"
+      payload_params = passed_attrs.clone
+      relationship_data = { passed_relationships => payload_params.delete(passed_relationships) }
+
       payload = {
         'data': {
           'type': resource_name,
-          'attributes': passed_attrs,
-          'relationships': relationships(passed_relationships)
+          'attributes': payload_params,
+          'relationships': relationships(relationship_data)
         }.reject{ |k,v| v.empty? }
       }
 
@@ -101,7 +107,7 @@ shared_examples_for "creatable resource" do |passed_attrs, passed_relationships,
         header_sans_auth_token
       ).and_return(Factory.bad_token_response)
 
-      expect(described_class.create(passed_attrs, passed_relationships).code).to eq 401
+      expect(described_class.create(passed_attrs).code).to eq 401
     end
   end
 

data/spec/shared_examples/deletable_resource.rb

@@ -0,0 +1,31 @@
+shared_examples_for "deletable resource" do |resource|
+  let(:response) { Factory.response }
+  let(:auth_response) { Factory.auth_response }
+  let(:token) { Factory.token }
+  let(:uuid) { '123' }
+
+  after do
+    CTAAggregatorClient::API::Authenticator.reset_auth
+  end
+
+  it 'updates record' do
+    url = "#{api_url}/#{formatted_resource_name(described_class, resource)}/#{uuid}"
+
+    allow(RestClient).to receive(:post).with(
+      auth_url,
+      nil,
+      headers_with_auth_creds
+    ).and_return(auth_response)
+
+    headers_with_auth_token = CTAAggregatorClient::API::Client.default_headers.merge(
+      authorization: "Bearer: #{token}"
+    )
+
+    expect(RestClient).to receive(:delete).with(
+      url,
+      headers_with_auth_token
+    ).and_return(response)
+
+    expect(described_class.delete(uuid)).to eq response
+  end
+end

data/spec/shared_examples/updatable_resource.rb

@@ -0,0 +1,42 @@
+shared_examples_for "updatable resource" do |resource|
+  let(:response) { Factory.response }
+  let(:auth_response) { Factory.auth_response }
+  let(:token) { Factory.token }
+  let(:uuid) { '123' }
+  let(:attrs_to_change) { {foo: :bar} }
+  let(:attrs) { {id: uuid}.merge(attrs_to_change) }
+
+  after do
+    CTAAggregatorClient::API::Authenticator.reset_auth
+  end
+
+  it 'updates record' do
+    url = "#{api_url}/#{formatted_resource_name(described_class, resource)}/#{uuid}"
+
+    payload = {
+      'data': {
+        'type': formatted_resource_name(described_class, resource),
+        'id': uuid,
+        'attributes': attrs_to_change,
+      }
+    }
+
+    allow(RestClient).to receive(:post).with(
+      auth_url,
+      nil,
+      headers_with_auth_creds
+    ).and_return(auth_response)
+
+    headers_with_auth_token = CTAAggregatorClient::API::Client.default_headers.merge(
+      authorization: "Bearer: #{token}"
+    )
+
+    expect(RestClient).to receive(:put).with(
+      url,
+      payload.to_json,
+      headers_with_auth_token
+    ).and_return(response)
+
+    expect(described_class.update(attrs)).to eq response
+  end
+end

data/spec/spec_helper.rb

@@ -2,6 +2,8 @@ require 'cta_aggregator_client'
 require 'shared_examples/listable_resource'
 require 'shared_examples/findable_resource'
 require 'shared_examples/creatable_resource'
+require 'shared_examples/updatable_resource'
+require 'shared_examples/deletable_resource'
 require 'helpers/test_helpers'
 require 'factory'
 require 'pry'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cta_aggregator_client
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Ben Downey
@@ -77,8 +77,10 @@ files:
 - spec/integration/location_spec.rb
 - spec/integration/target_spec.rb
 - spec/shared_examples/creatable_resource.rb
+- spec/shared_examples/deletable_resource.rb
 - spec/shared_examples/findable_resource.rb
 - spec/shared_examples/listable_resource.rb
+- spec/shared_examples/updatable_resource.rb
 - spec/spec_helper.rb
 homepage: https://github.com/Ragtagteam/cta-aggregator-client-ruby
 licenses:
@@ -112,6 +114,8 @@ test_files:
 - spec/integration/location_spec.rb
 - spec/integration/target_spec.rb
 - spec/shared_examples/creatable_resource.rb
+- spec/shared_examples/deletable_resource.rb
 - spec/shared_examples/findable_resource.rb
 - spec/shared_examples/listable_resource.rb
+- spec/shared_examples/updatable_resource.rb
 - spec/spec_helper.rb