live_paper 0.0.9 → 0.0.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e0be6ece34f4d51cf32f7ea78ffaae9d6c8543b3
-  data.tar.gz: b3838914f2bbaca46a1af9d87d259612186849f0
+  metadata.gz: 54159788ca4d35d52a44ed05de7e2fdfad88998e
+  data.tar.gz: 4f319ab3b4ad6f7380828d0bc68b06ca1979433a
 SHA512:
-  metadata.gz: 4956d0c6516a9f2778e2e84c2958a4ebc50b981fa9fce39dd2260b8ef00b07016834d13b43f14858c2f72a3ba516501c6058baca22fa50f8564d2b3310291346
-  data.tar.gz: bde0081f9039ec48fb5354cee3fdf5c062967a7f2cd3ca85bd5fab44ea7cda3067e3b68f99305895c4256c1e36e413159983159b7692713f6ec8dbf805545aa8
+  metadata.gz: a8a4256e05ff2fbf38165b9ed6098673d3602b3a387de482cc4a7f911a82961f148bf118867d7bff796f12a0ed88a3e938a30741a0edd1dcac3a4e5436ae3a33
+  data.tar.gz: c9416a9a5d40f8eabb7d3897b775c390a56346cb2b629e92616a364d2af098d4a124778fef1b6dde36c0868741d059885e6e80bce8597c0a696c0a8630c296a4

data/lib/live_paper/base_object.rb

@@ -4,7 +4,7 @@ require 'json'
 module LivePaper
   class BaseObject
     extend HttpClient
-    attr_accessor :id, :name, :date_created, :date_modified
+    attr_accessor :id, :name, :date_created, :date_modified, :link
 
     def assign_attributes(data)
       data.each do |key, value|
@@ -30,7 +30,7 @@ module LivePaper
       self
     end
 
-    def self.find(id)
+    def self.get(id)
       request_handling_auth("#{api_url}/#{id}", 'GET') do |request|
         response = send_request(request, 'application/json')
         parse response.body
@@ -49,6 +49,10 @@ module LivePaper
       raise NotImplementedError
     end
 
+    def rel(key)
+      link.find { |obj| obj[:rel] == key }[:href] rescue nil
+    end
+
     protected
     def all_present?(array)
       array.all? { |e| present? e } rescue false

data/lib/live_paper/image.rb

@@ -3,7 +3,6 @@ require 'rest-client'
 
 module LivePaper
   class Image
-  extend HttpClient
 
     attr_accessor :url
 

data/lib/live_paper/link.rb

@@ -2,22 +2,21 @@ require_relative 'base_object'
 
 module LivePaper
   class Link < BaseObject
-    attr_accessor :payoff_id, :trigger_id, :analytics
+    attr_accessor :payoff_id, :trigger_id
 
-    def parse(data)
-      data = JSON.parse(data, symbolize_names: true)[:link]
+    def parse(jsondata)
+      data = JSON.parse(jsondata, symbolize_names: true)[:link]
       assign_attributes data
-      @analytics = get_link_for data, 'analytics'
       self
     end
 
     def payoff
-      @payoff ||= LivePaper::Payoff.find @payoff_id
+      @payoff ||= LivePaper::Payoff.get @payoff_id
     end
 
     def trigger
       #todo: need to get the right object created here!!!
-      @trigger ||= LivePaper::WmTrigger.find @trigger_id
+      @trigger ||= LivePaper::WmTrigger.get @trigger_id
     end
 
     def self.api_url
@@ -38,9 +37,5 @@ module LivePaper
         }
       }
     end
-
-    def get_link_for(data, rel)
-      data[:link].find { |obj| obj[:rel] == rel }[:href] rescue ''
-    end
   end
 end

data/lib/live_paper/version.rb

@@ -1,3 +1,3 @@
 module LivePaper
-  VERSION = "0.0.9"
+  VERSION = "0.0.10"
 end

data/spec/live_paper/base_object_spec.rb

@@ -101,7 +101,7 @@ describe LivePaper::BaseObject do
     end
   end
 
-  describe '.find' do
+  describe '.get' do
     before do
       allow(LivePaper::BaseObject).to receive(:api_url).and_return(@api_url)
       @data = '"id": "id", "name": "name"'
@@ -112,21 +112,50 @@ describe LivePaper::BaseObject do
       it 'should return the requested base object.' do
         allow(@data).to receive(:body).and_return(@data)
         expect(LivePaper::BaseObject).to receive(:parse).with(@data)
-        LivePaper::BaseObject.find('base_object')
+        LivePaper::BaseObject.get('base_object')
       end
     end
 
     context 'the requested base object does not exist or some error happened.' do
       it 'should not raise error.' do
-        expect { LivePaper::BaseObject.find('base_object_not_existent') }.to_not raise_error
+        expect { LivePaper::BaseObject.get('base_object_not_existent') }.to_not raise_error
       end
 
       it 'should return nil.' do
-        expect(LivePaper::BaseObject.find('base_object_not_existent')).to eq nil
+        expect(LivePaper::BaseObject.get('base_object_not_existent')).to eq nil
       end
     end
   end
 
+  describe '#rel' do
+    before do
+      stub_unimplemented_methods
+            @data = {
+              name: 'name',
+              date_created: 'date_created',
+              date_modified: 'date_modified',
+              link: [
+                { :rel => "self", :href => "/api/v1/payoffs/payoff_id" },
+                { :rel => "analytics", :href => "/analytics/v1/payoffs/payoff_id" }
+              ]
+            }
+            @obj = LivePaper::BaseObject.create @data
+    end
+
+    it 'should return href for rel link' do
+      expect(@obj.rel('self')).to eq '/api/v1/payoffs/payoff_id'
+    end
+
+    it 'should return nil for unknown rel link' do
+      expect(@obj.rel('invalid')).to be_nil
+    end
+
+    it 'should NOT crash when the link attribute is nil' do
+      @obj.link=nil
+      expect(@obj.rel('self')).to be_nil
+    end
+  end
+
   describe '#all_present?' do
     before do
       @all = [1, 2, {k: 'v'}, [1, 2]]

data/spec/live_paper/link_spec.rb

@@ -1,6 +1,6 @@
 require 'spec_helper'
 
-describe LivePaper::WmTrigger do
+describe LivePaper::Link do
   before do
     stub_request(:post, /.*livepaperapi.com\/auth\/token.*/).to_return(:body => lpp_auth_response_json, :status => 200)
     stub_request(:post, LivePaper::Link.api_url).to_return(:body => lpp_link_response_json, :status => 200)
@@ -10,7 +10,6 @@ describe LivePaper::WmTrigger do
     @data = {
       id: 'id',
       name: 'name',
-      analytics: 'analytics',
       trigger_id: 'trigger_id',
       payoff_id: 'payoff_id'
     }
@@ -21,10 +20,6 @@ describe LivePaper::WmTrigger do
       @link = LivePaper::Link.new @data
     end
 
-    it 'should map the analytics attribute.' do
-      expect(@link.analytics).to eq @data[:analytics]
-    end
-
     it 'should map the trigger_id attribute.' do
       expect(@link.trigger_id).to eq @data[:trigger_id]
     end
@@ -85,8 +80,13 @@ describe LivePaper::WmTrigger do
       expect(@link.name).to eq 'name'
     end
 
-    it 'should map the analytics attribute.' do
-      expect(@link.analytics).to eq 'analytics'
+    it 'should map the link array attribute.' do
+      expect(@link.link).to be_a Array
+      expect(@link.link.size).to eq 4
+      expect(@link.link).to eq [{:rel=>"self", :href=>"self_url"},
+                                {:rel=>"analytics", :href=>"analytic_url"},
+                                {:rel=>"payoff", :href=>"payoff_url"},
+                                {:rel=>"trigger", :href=>"trigger_url"}]
     end
 
     it 'should map the trigger_id attribute.' do
@@ -98,16 +98,17 @@ describe LivePaper::WmTrigger do
     end
   end
 
-  describe '.find' do
+  describe '.get' do
     context 'the requested Link exists.' do
       before do
-        @link = LivePaper::Link.find('link_id')
+        @link = LivePaper::Link.get('link_id')
       end
 
       it 'should return the requested link.' do
         expect(@link.id).to eq 'link_id'
         expect(@link.name).to eq 'name'
-        expect(@link.analytics).to eq 'analytics'
+        expect(@link.link).to be_a Array
+        expect(@link.link.size).to eq 4
         expect(@link.trigger_id).to eq 'trigger_id'
         expect(@link.payoff_id).to eq 'payoff_id'
       end
@@ -115,11 +116,11 @@ describe LivePaper::WmTrigger do
 
     context 'the requested link does not exist or some error happened.' do
       it 'should not raise error.' do
-        expect { LivePaper::Link.find('link_not_existent') }.to_not raise_error
+        expect { LivePaper::Link.get('link_not_existent') }.to_not raise_error
       end
 
       it 'should return nil.' do
-        expect(LivePaper::Link.find('link_not_existent')).to eq nil
+        expect(LivePaper::Link.get('link_not_existent')).to eq nil
       end
     end
   end
@@ -179,5 +180,4 @@ describe LivePaper::WmTrigger do
       end
     end
   end
-
 end

data/spec/live_paper/payoff_spec.rb

@@ -192,10 +192,10 @@ describe LivePaper::Payoff do
     end
   end
 
-  describe '.find' do
+  describe '.get' do
     context 'the requested payoff exists.' do
       before do
-        @payoff = LivePaper::Payoff.find('payoff_id')
+        @payoff = LivePaper::Payoff.get('payoff_id')
       end
 
       it 'should return the requested payoff.' do
@@ -208,11 +208,11 @@ describe LivePaper::Payoff do
 
     context 'the requested payoff does not exist or some error happened.' do
       it 'should not raise error.' do
-        expect { LivePaper::Payoff.find('payoff_not_existent') }.to_not raise_error
+        expect { LivePaper::Payoff.get('payoff_not_existent') }.to_not raise_error
       end
 
       it 'should return nil.' do
-        expect(LivePaper::Payoff.find('payoff_not_existent')).to eq nil
+        expect(LivePaper::Payoff.get('payoff_not_existent')).to eq nil
       end
     end
   end

data/spec/live_paper/trigger_spec.rb

@@ -96,10 +96,10 @@ describe LivePaper::WmTrigger do
     end
   end
 
-  describe '.find' do
+  describe '.get' do
     context 'the requested trigger exists.' do
       before do
-        @trigger = LivePaper::WmTrigger.find('trigger_id')
+        @trigger = LivePaper::WmTrigger.get('trigger_id')
       end
 
       it 'should return the requested trigger.' do
@@ -112,11 +112,11 @@ describe LivePaper::WmTrigger do
 
     context 'the requested trigger does not exist or some error happened.' do
       it 'should not raise error.' do
-        expect { LivePaper::WmTrigger.find('trigger_not_existent') }.to_not raise_error
+        expect { LivePaper::WmTrigger.get('trigger_not_existent') }.to_not raise_error
       end
 
       it 'should return nil.' do
-        expect(LivePaper::WmTrigger.find('trigger_not_existent')).to eq nil
+        expect(LivePaper::WmTrigger.get('trigger_not_existent')).to eq nil
       end
     end
   end

data/spec/spec_helpers/lpp_client.rb

@@ -27,14 +27,26 @@ def lpp_richpayoff_response_json
   "payoff": {
     "id": "payoff_id",
     "name": "name",
+    "dateCreated":"2014-10-07T20:57:01.083+0000",
+    "dateModified":"2014-10-07T20:57:01.083+0000",
+    "link":[
+      {
+        "rel":"self",
+        "href":"/api/v1/payoffs/payoff_id"
+      },
+      {
+        "rel":"analytics",
+        "href":"/analytics/v1/payoffs/payoff_id"
+      }
+    ],
     "richPayoff" : {
       "version": 1,
+      "public": {
+        "url": "url"
+      },
       "private": {
         "content-type": "data_type",
         "data": "#{Base64.encode64('{ "field": 1 }').gsub(/\n/,'')}"
-      },
-      "public": {
-        "url": "url"
       }
     }
   }
@@ -62,9 +74,26 @@ def lpp_link_response_json
   "link": {
     "id": "link_id",
     "name": "name",
-    "triggerId": "trigger_id",
+    "dateCreated": "2014-04-08T08:16:25.723+0000",
+    "dateModified": "2014-04-08T08:16:25.723+0000",
+    "link": [{
+      "rel": "self",
+      "href": "self_url"
+    },
+    {
+      "rel": "analytics",
+      "href": "analytic_url"
+    },
+    {
+      "rel": "payoff",
+      "href": "payoff_url"
+    },
+    {
+      "rel": "trigger",
+      "href": "trigger_url"
+    }],
     "payoffId": "payoff_id",
-    "link": [{"rel":"analytics", "href": "analytics"}]
+    "triggerId": "trigger_id"
   }
 }
   RESPONSE
@@ -72,4 +101,20 @@ end
 
 def lpp_watermark_response
   'watermark_data'
+end
+
+def lpp_delete_error_response
+  <<-RESPONSE
+{ "associatedLinks": "1",
+  "link": [{
+      "rel": "associatedLinks",
+      "href": "https://dev.livepaperapi.com/api/v1/links?trigger=uMTGQ0xoS7exSL3iujHJSA"
+  }],
+  "error": {
+      "title": "409 Conflict",
+      "message": "The trigger to be deleted has associated links. The associated links must be deleted first."
+  }
+}
+  RESPONSE
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: live_paper
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.0.10
 platform: ruby
 authors:
 - Mike Whitmarsh
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-10-29 00:00:00.000000000 Z
+date: 2014-11-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client