kippt 1.1.0 → 2.0.0

data/spec/kippt/clips_spec.rb

@@ -2,7 +2,8 @@ require "spec_helper"
 require "kippt/clips"
 
 describe Kippt::Clips do
-  subject { Kippt::Client.new(valid_user_credentials).clips }
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  subject { client.clips }
   let(:base_uri) { "clips" }
   let(:singular_fixture) { "clip" }
   let(:collection_class) { Kippt::ClipCollection }
@@ -10,6 +11,16 @@ describe Kippt::Clips do
 
   it_behaves_like "collection resource"
 
+  describe "#feed" do
+    subject { Kippt::Client.new(valid_user_credentials).clips }
+
+    it "returns ClipCollection" do
+      stub_get("/clips/feed").
+        to_return(:status => 200, :body => fixture("feed.json"))
+      subject.feed.should be_a Kippt::ClipCollection
+    end
+  end
+
   describe "#search" do
     subject { Kippt::Client.new(valid_user_credentials).clips }
 
@@ -45,7 +56,7 @@ describe Kippt::Clips do
       stub_get("/search/clips?q=bunny").
         to_return(:status => 200, :body => fixture("clips.json"))
       clips = subject.search("bunny")
-      clips.is_a? Kippt::ClipCollection
+      clips.should be_a Kippt::ClipCollection
     end
   end
 
@@ -54,7 +65,7 @@ describe Kippt::Clips do
 
     context "successful request" do
       it "returns hash with success boolean" do
-       stub_request(:post, "https://kippt.com/api/clips").
+       stub_post("/clips").
          with(:body => "{\"url\":\"http://kiskolabs.com\"}").
          to_return(:status => 200, :body => "{}", :headers => {})
 
@@ -66,7 +77,7 @@ describe Kippt::Clips do
 
     context "unsuccessful request" do
       it "returns hash with success boolean and error message" do
-       stub_request(:post, "https://kippt.com/api/clips").
+       stub_post("/clips").
          with(:body => "{\"url\":\"http://kiskolabs.com\"}").
          to_return(:status => 400, :body => "{\"message\": \"No good.\"}", :headers => {})
 
@@ -79,7 +90,7 @@ describe Kippt::Clips do
 
     context "when object doesn't have an id" do
       it "POSTs new resource to the API" do
-       stub_request(:post, "https://kippt.com/api/clips").
+       stub_post("/clips").
          with(:body => "{\"url\":\"http://kiskolabs.com\"}").
          to_return(:status => 200, :body => "{}", :headers => {})
 
@@ -90,7 +101,7 @@ describe Kippt::Clips do
     
     context "when object has an id" do
       it "PUTs new version of the resource to the API" do
-       stub_request(:put, "https://kippt.com/api/clips/22").
+       stub_put("/clips/22").
          with(:body => "{\"url\":\"http://kiskolabs.com\"}").
          to_return(:status => 200, :body => "{}", :headers => {})
 
@@ -105,7 +116,7 @@ describe Kippt::Clips do
 
     context "successful request" do
       it "returns boolean" do
-       stub_request(:delete, "https://kippt.com/api/clips/100").
+       stub_delete("/clips/100").
          to_return(:status => 200, :headers => {})
 
         clip = Kippt::Clip.new(:id => 100)

data/spec/kippt/comment_spec.rb

@@ -0,0 +1,25 @@
+require "spec_helper"
+require "kippt/comment"
+
+describe Kippt::Comment do
+  subject { Kippt::Comment.new(data, client, clip) }
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  let(:clip) { stub(:clip) }
+  let(:collection_resource_class) { Kippt::Comments }
+
+  let(:data) { MultiJson.load(fixture("comment.json").read) }
+  let(:attributes) {
+    [:body, :id, :resource_uri]
+   }
+   let(:mapped_attributes) {
+     {:created => "Time", :user => "Kippt::User"}
+   }
+
+  it_behaves_like "resource"
+
+  describe "#clip" do
+    it "returns the clip passed to it" do
+      subject.clip.should eq clip
+    end
+  end
+end

data/spec/kippt/comments_spec.rb

@@ -0,0 +1,103 @@
+require "spec_helper"
+require "kippt/comments"
+
+describe Kippt::Comments do
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  let(:clip) { stub :clip, :id => 100, :all_comments_embedded? => false }
+  subject { Kippt::Comments.new(client, clip) }
+  let(:base_uri) { "clips/#{clip.id}/comments" }
+  let(:singular_fixture) { "comment" }
+  let(:collection_class) { Kippt::CommentCollection }
+  let(:resource_class) { Kippt::Comment }
+
+  def collection_fixture
+    base_uri.split("/").last
+  end
+
+  describe "#all" do
+    it "returns collection class" do
+      stub_get("/#{base_uri}").
+        to_return(:status => 200, :body => fixture("#{collection_fixture}.json"))
+      all_resources = subject.all
+      all_resources.is_a? collection_class
+    end
+
+    it "accepts limit and offset options" do
+      stub_get("/#{base_uri}?limit=10&offset=100").
+        to_return(:status => 200, :body => fixture("#{collection_fixture}.json"))
+      resources = subject.all(:limit => 10, :offset => 100)
+    end
+
+    context "when passed unrecognized arguments" do
+      it "raises error" do
+        lambda {
+          subject.all(:foobar => true)
+        }.should raise_error(
+          ArgumentError, "Unrecognized argument: foobar")
+      end
+    end
+  end
+
+  describe "#[]" do
+    it "fetches single resource" do
+      stub_get("/#{base_uri}/10").
+        to_return(:status => 200, :body => fixture("#{singular_fixture}.json"))
+      subject[10].id.should eq 10
+    end
+
+    it "returns resource" do
+      stub_get("/#{base_uri}/10").
+        to_return(:status => 200, :body => fixture("#{singular_fixture}.json"))
+      subject[10].should be_a(resource_class)
+    end
+
+    context "when resource is not found" do
+      it "raises exception" do
+        stub_get("/#{base_uri}/10").
+          to_return(:status => 404, :body => {"message" => "Resource not found."})
+        lambda {
+          subject[10]
+          subject.all(:foobar => true)
+        }.should raise_error(
+          Kippt::APIError, "Resource could not be loaded: Resource not found.")
+      end
+    end
+  end
+
+  describe "#find" do
+    it "exists" do
+      subject.respond_to?(:find).should be_true
+    end
+  end
+
+  describe "#collection_from_url" do
+    it "returns a new collection" do
+      stub_get("/#{base_uri}/?limit=20&offset=20").
+        to_return(:status => 200, :body => fixture("#{collection_fixture}.json"))
+      collection = subject.collection_from_url("/api/#{base_uri}/?limit=20&offset=20")
+      collection.should be_a(collection_class)
+    end
+
+    context "when passed URL is blank" do
+      it "raises ArgumentError" do
+        lambda {
+          subject.collection_from_url("")
+        }.should raise_error(ArgumentError, "The parameter URL can't be blank")
+        lambda {
+          subject.collection_from_url(nil)
+        }.should raise_error(ArgumentError, "The parameter URL can't be blank")
+      end
+    end
+  end
+
+  describe "#build" do
+    it "returns new resource" do
+      subject.build.should be_a(resource_class)
+    end
+
+    it "accepts parameters" do
+      subject.object_class.should_receive(:new).with({:an => "attribute"}, client, clip)
+      subject.build(:an => "attribute")
+    end
+  end
+end

data/spec/kippt/follow_relationship_spec.rb

@@ -0,0 +1,34 @@
+require "spec_helper"
+require "kippt/follow_relationship"
+
+describe Kippt::FollowRelationship do
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  let(:user) { stub :user, :id => 10 }
+  subject { Kippt::FollowRelationship.new(client, user) }
+
+  describe "#following?" do
+    it "returns boolean" do
+      stub_get("/users/10/relationship/").
+        to_return(:status => 200, :body => '{"following": true}')
+      subject.following?.should be_true
+    end
+  end
+
+  describe "#follow" do
+    it "updates the value on the server" do
+      stub_post("/users/10/relationship/").
+         with(:body => "{\"action\":\"follow\"}").
+         to_return(:status => 200, :body => "", :headers => {})
+      subject.follow.should be_true
+    end
+  end
+
+  describe "#unfollow" do
+    it "updates the value on the server" do
+      stub_post("/users/10/relationship/").
+         with(:body => "{\"action\":\"unfollow\"}").
+         to_return(:status => 200, :body => "", :headers => {})
+      subject.unfollow.should be_true
+    end
+  end
+end

data/spec/kippt/followers_spec.rb

@@ -0,0 +1,17 @@
+require "spec_helper"
+require "kippt/followers"
+
+describe Kippt::Followers do
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  subject { Kippt::User.new({:id => 10}, client).followers }
+
+  it "is a collection resource" do
+    subject.should be_a Kippt::CollectionResource
+  end
+
+  it "uses the correct base uri" do
+    stub_get("/users/10/followers").
+      to_return(:status => 200, :body => fixture("users.json"))
+    subject.all
+  end
+end

data/spec/kippt/following_spec.rb

@@ -0,0 +1,17 @@
+require "spec_helper"
+require "kippt/following"
+
+describe Kippt::Following do
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  subject { Kippt::User.new({:id => 10}, client).following }
+
+  it "is a collection resource" do
+    subject.should be_a Kippt::CollectionResource
+  end
+
+  it "uses the correct base uri" do
+    stub_get("/users/10/following").
+      to_return(:status => 200, :body => fixture("users.json"))
+    subject.all
+  end
+end

data/spec/kippt/list_collection_spec.rb

@@ -5,9 +5,9 @@ describe Kippt::ListCollection do
   let(:data_with_multiple_pages) {
     MultiJson.load(fixture("lists_with_multiple_pages.json").read)
   }
-  subject { Kippt::ListCollection.new(data) }
-  let(:subject_with_multiple_pages) { Kippt::ListCollection.new(data_with_multiple_pages, collection_resource) }
-  let(:collection_resource) { nil }
+  let(:client) { stub }
+  subject { Kippt::ListCollection.new(data, client) }
+  let(:subject_with_multiple_pages) { Kippt::ListCollection.new(data_with_multiple_pages, client) }
 
   it_behaves_like "collection"
 end

data/spec/kippt/list_spec.rb

@@ -2,14 +2,100 @@ require "spec_helper"
 require "kippt/list"
 
 describe Kippt::List do
-  subject { Kippt::List.new(data, collection_resource) }
-  let(:collection_resource) { Kippt::Client.new(valid_user_credentials).lists }
+  subject { Kippt::List.new(data, client) }
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  let(:collection_resource_class) { Kippt::Lists }
 
-  let(:data) { MultiJson.load(fixture("list.json").read) }
-  let(:attributes) { 
-    [:id, :rss_url, :updated, :title,
-     :created, :slug, :resource_uri]
+  let(:data) { json_fixture("list.json") }
+  let(:attributes) {
+    [:id, :rss_url, :title,
+     :slug, :resource_uri]
   }
+   let(:mapped_attributes) {
+     {:updated => "Time", :created => "Time",
+      :user => "Kippt::User"}
+   }
 
   it_behaves_like "resource"
+
+  describe "#private?" do
+    it "gets data from is_private" do
+      list = Kippt::List.new({"is_private" => true}, nil)
+      list.private?.should be_true
+    end
+  end
+
+  describe "#collaborators" do
+    it "returns the users generated from the data" do
+      list = Kippt::List.new({"collaborators" => {"count" => 1, "data" => [json_fixture("user.json")]}}, nil)
+      list.collaborators.size.should eq 1
+      list.collaborators.first.should be_a Kippt::User
+    end
+  end
+
+  describe "#follow" do
+    context "when request is successful" do
+      let(:client) { Kippt::Client.new(valid_user_credentials) }
+
+      it "makes a request" do
+        response = stub(:success? => true)
+        client.should_receive(:post).with("/api/lists/10/relationship", :data => {:action => "follow"}).and_return(response)
+        list = Kippt::List.new(json_fixture("list.json"), client)
+        list.follow
+      end
+
+      it "returns true" do
+        stub_request(:post, "https://kippt.com/api/lists/10/relationship").
+           with(:body => "{\"action\":\"follow\"}").
+           to_return(:status => 200, :body => "{\"folloring\":true}")
+        list = Kippt::List.new(json_fixture("list.json"), client)
+        list.follow.should eq true
+      end
+    end
+
+    context "when request is unsuccessful" do
+      it "raises an exception" do
+        response = stub(:success? => false, :body => {"message" => "Weird issue going on."})
+        client.should_receive(:post).with("/api/lists/10/relationship", :data => {:action => "follow"}).and_return(response)
+        list = Kippt::List.new(json_fixture("list.json"), client)
+
+        expect {
+          list.follow
+        }.to raise_error(Kippt::APIError, "There was an error with the request: Weird issue going on.")
+      end
+    end
+  end
+
+  describe "#unfollow" do
+    context "when request is successful" do
+      let(:client) { Kippt::Client.new(valid_user_credentials) }
+
+      it "makes a request" do
+        response = stub(:success? => true)
+        client.should_receive(:post).with("/api/lists/10/relationship", :data => {:action => "unfollow"}).and_return(response)
+        list = Kippt::List.new(json_fixture("list.json"), client)
+        list.unfollow
+      end
+
+      it "returns true" do
+        stub_request(:post, "https://kippt.com/api/lists/10/relationship").
+           with(:body => "{\"action\":\"unfollow\"}").
+           to_return(:status => 200, :body => "{\"folloring\":false}")
+        list = Kippt::List.new(json_fixture("list.json"), client)
+        list.unfollow.should eq true
+      end
+    end
+
+    context "when request is unsuccessful" do
+      it "raises an exception" do
+        response = stub(:success? => false, :body => {"message" => "Weird issue going on."})
+        client.should_receive(:post).with("/api/lists/10/relationship", :data => {:action => "unfollow"}).and_return(response)
+        list = Kippt::List.new(json_fixture("list.json"), client)
+
+        expect {
+          list.unfollow
+        }.to raise_error(Kippt::APIError, "There was an error with the request: Weird issue going on.")
+      end
+    end
+  end
 end

data/spec/kippt/lists_spec.rb

@@ -2,7 +2,8 @@ require "spec_helper"
 require "kippt/lists"
 
 describe Kippt::Lists do
-  subject { Kippt::Client.new(valid_user_credentials).lists }
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  subject { client.lists }
   let(:base_uri) { "lists" }
   let(:singular_fixture) { "list" }
   let(:collection_class) { Kippt::ListCollection }

data/spec/kippt/saves_spec.rb

@@ -0,0 +1,20 @@
+require "spec_helper"
+require "kippt/saves"
+
+describe Kippt::Saves do
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  let(:clip) { stub :clip, :saves_data => fixture("users.json") }
+  subject { Kippt::Saves.new(client, clip) }
+
+  describe "#all" do
+    it "returns collection class" do
+      all_resources = subject.all
+      all_resources.is_a? Kippt::UserCollection
+    end
+
+    it "uses the clip saves data" do
+      Kippt::UserCollection.should_receive(:new).with({"objects" => clip.saves_data}, client)
+      subject.all
+    end
+  end
+end

data/spec/kippt/user_clips_spec.rb

@@ -0,0 +1,29 @@
+require "spec_helper"
+require "kippt/user_clips"
+
+describe Kippt::UserClips do
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  subject { Kippt::User.new({:id => 10}, client).clips }
+  let(:base_uri) { "users/#{10}/clips" }
+  let(:singular_fixture) { "clip" }
+  let(:collection_class) { Kippt::ClipCollection }
+  let(:resource_class) { Kippt::Clip }
+
+  it_behaves_like "collection resource"
+
+  describe "#favorites" do
+    it "returns ClipCollection" do
+      stub_get("/#{base_uri}/favorites").
+        to_return(:status => 200, :body => fixture("clips.json"))
+      subject.favorites.should be_a Kippt::ClipCollection
+    end
+  end
+
+  describe "#likes" do
+    it "returns ClipCollection" do
+      stub_get("/#{base_uri}/likes").
+        to_return(:status => 200, :body => fixture("clips.json"))
+      subject.likes.should be_a Kippt::ClipCollection
+    end
+  end
+end