Flesh out handling of Updates

lib/kickscraper/client.rb

@@ -58,6 +58,20 @@ def load_more_projects
             end
         end
 
+        def more_updates_available?
+            !@more_updates_url.nil?
+        end
+
+        alias_method :can_load_more_updates, :more_updates_available?
+
+        def load_more_updates
+            if self::more_updates_available?
+                self::process_api_url "updates", @more_updates_url
+            else
+                []
+            end
+        end
+
         def categories
             self::process_api_call "categories", ""
         end
@@ -156,11 +170,26 @@ def coerce_api_response(expected_type, response)
 
                 return empty_response if body.comments.nil?
                 body.comments.map { |comment| Comment.coerce comment }
+
+            when "update"
+
+                Update.coerce body
 
             when "updates"
 
-                return empty_response if body.updates.nil?
-                body.updates.map { |update| Update.coerce update }
+                # Return an empty result set if the body has no updates
+                if body.updates.nil?
+
+                    @more_updates_url = nil
+                    return empty_response
+
+                # Otherwise, set the url that holds the next batch of updates (if available)
+                # and return an array of updates
+                else
+
+                    @more_updates_url = (!body.urls.nil? && !body.urls.api.nil? && !body.urls.api.more_updates.nil? && !body.urls.api.more_updates.empty?) ? body.urls.api.more_updates : nil
+                    return body.updates.map { |update| Update.coerce update }
+                end
 
             when "categories"
 

lib/kickscraper/client/update.rb

@@ -1,5 +1,36 @@
 module Kickscraper
     class Update < Api
-
+        attr_accessor :sequence, :title, :body
+
+        def to_s
+            "Update \##{sequence}: #{title}"
+        end
+
+        def inspect
+            "<Update: '#{title}'>"
+        end
+
+        def reload!
+            # The API URL must be specified here as api['update'], because api.update refers to the
+            # update method inherited from Hashie
+            @raw = Kickscraper.client.process_api_url("Update", self.urls.api['update'], false)
+            Kickscraper::Update::do_coercion(self)
+        end
+
+        def sequence
+            reload! unless @raw['sequence']
+            @raw['sequence']
+        end
+
+        def title
+            reload! unless @raw['title']
+            @raw['title']
+        end
+
+        def body
+            reload! unless @raw['body']
+            @raw['body']
+        end
+
     end
 end

spec/client_spec.rb

@@ -128,4 +128,25 @@
     its(:name) { should eq TEST_CATEGORY_NAME }
   end
 
+  context "returns an array of updates when a valid project is loaded" do
+    subject { client.find_project(TEST_PROJECT_ID).updates }
+    it_returns "a collection", Kickscraper::Update
+  end
+
+  context "loads more updates when available" do
+    before { client.find_project(TEST_PROJECT_ID) }
+    subject do
+      client.more_updates_available?.should be_true
+      client.load_more_updates
+    end
+
+    it_returns "a collection", Kickscraper::Update
+  end
+
+  it "throws an error when accessing updates without a valid project loaded" do
+    subject { client.find_project TEST_PROJECT_ID }
+    expect { subject.to be_nil }
+    expect { subject.updates }.to raise_error
+  end
+
 end

spec/test_constants.rb

@@ -15,4 +15,5 @@
 TEST_PROJECT_SLUG = 'wish-i-was-here-1'
 TEST_PROJECT_NAME = "WISH I WAS HERE"
 TEST_CATEGORY_NAME = 'Design'
-TEST_CATEGORY_ID = 7
+TEST_CATEGORY_ID = 7
+TEST_PUBLIC_UPDATE_URL = 'https://api.kickstarter.com/v1/projects/1871494789/updates/462776'

spec/update_spec.rb

@@ -0,0 +1,34 @@
+describe Kickscraper::Update do
+  let(:client) { Kickscraper.client }
+  subject(:update) { client.find_project(TEST_PROJECT_ID).updates.first }
+
+  context "loads basic update information" do
+    its(:id) { should be > 0 }
+    its(:project_id) { should be > 0 }
+    its(:sequence) { should be > 0 }
+    its(:public) { should satisfy { |s| [true,false].include?(s) } }
+    its(:visible) { should satisfy { |s| [true,false].include?(s) } }
+    its(:title) { should_not be_empty }
+    its(:published_at) { should be >= 1262304000 }
+    its(:updated_at) { should be >= 1262304000 }
+    its(:comments_count) { should be >= 0 }
+    its(:likes_count) { should be >= 0 }
+  end
+
+  context "reloads" do
+    before { update.reload! }
+
+    its(:sequence) { should be > 0 }
+    its(:title) { should_not be_empty }
+  end
+
+  context "loads additional information for a visible update" do
+    # Ensure that this test runs on a public update, so its success does not
+    # depend on the user running the tests
+    subject { client.process_api_url("update", TEST_PUBLIC_UPDATE_URL) }
+
+    its(:body) { should_not be_empty }
+    its(:images) { should have_at_least(0).items } 
+    its(:sounds) { should have_at_least(0).items } 
+  end
+end