kippt 1.1.0 → 2.0.0

data/spec/kippt/user_collection_spec.rb

@@ -0,0 +1,14 @@
+require "spec_helper"
+require "kippt/user_collection"
+
+describe Kippt::UserCollection do
+  let(:data) { MultiJson.load(fixture("users.json").read) }
+  let(:data_with_multiple_pages) {
+    MultiJson.load(fixture("users_with_multiple_pages.json").read)
+  }
+  let(:client) { stub }
+  subject { Kippt::UserCollection.new(data, client) }
+  let(:subject_with_multiple_pages) { Kippt::UserCollection.new(data_with_multiple_pages, client) }
+
+  it_behaves_like "collection"
+end

data/spec/kippt/user_lists_spec.rb

@@ -0,0 +1,13 @@
+require "spec_helper"
+require "kippt/user_lists"
+
+describe Kippt::UserLists do
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  subject { Kippt::User.new({:id => 10}, client).lists }
+  let(:base_uri) { "users/#{10}/lists" }
+  let(:singular_fixture) { "list" }
+  let(:collection_class) { Kippt::ListCollection }
+  let(:resource_class) { Kippt::List }
+
+  it_behaves_like "collection resource"
+end

data/spec/kippt/user_spec.rb

@@ -0,0 +1,71 @@
+require "spec_helper"
+require "kippt/user"
+
+describe Kippt::User do
+  subject { Kippt::User.new(data, client) }
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  let(:collection_resource_class) { Kippt::Users }
+
+  let(:data) { MultiJson.load(fixture("user.json").read) }
+  let(:attributes) {
+    [:username, :bio, :app_url, :avatar_url, :twitter,
+     :id, :github, :website_url, :full_name, :dribble,
+     :counts, :resource_uri]
+  }
+  let(:mapped_attributes) {
+    {}
+  }
+
+  it_behaves_like "resource"
+
+  describe "#pro?" do
+    it "gets data from is_pro" do
+      user = Kippt::User.new({:is_pro => true}, nil)
+      user.pro?.should be_true
+    end
+  end
+
+  describe "#following?" do
+    it "uses Kippt::FollowRelationship to get the information" do
+      follow_relationship = mock(:follow_relationship,
+        :following? => true)
+      Kippt::FollowRelationship.stub(:new).and_return(follow_relationship)
+      user = Kippt::User.new({:is_pro => true}, nil)
+      user.following?.should be_true
+    end
+  end
+
+  describe "#follower_count" do
+    it "returns the count from the fetched data" do
+      user = Kippt::User.new({:counts => {"follows" => 11}})
+      user.follower_count.should eq 11
+    end
+  end
+
+  describe "#following_count" do
+    it "returns the count from the fetched data" do
+      user = Kippt::User.new({:counts => {"followed_by" => 111}})
+      user.following_count.should eq 111
+    end
+  end
+
+  describe "#follow" do
+    it "sets up a Kippt::FollowRelationship and calls it" do
+      follow_relationship = mock(:follow_relationship,
+        :follow => true)
+      Kippt::FollowRelationship.stub(:new).and_return(follow_relationship)
+      user = Kippt::User.new
+      user.follow.should be_true
+    end
+  end
+
+  describe "#unfollow" do
+    it "sets up a Kippt::FollowRelationship and calls it" do
+      follow_relationship = mock(:follow_relationship,
+        :unfollow => true)
+      Kippt::FollowRelationship.stub(:new).and_return(follow_relationship)
+      user = Kippt::User.new
+      user.unfollow.should be_true
+    end
+  end
+end

data/spec/kippt/users_spec.rb

@@ -0,0 +1,59 @@
+require "spec_helper"
+require "kippt/users"
+
+describe Kippt::Users do
+  let(:client) { Kippt::Client.new(valid_user_credentials) }
+  subject { client.users }
+  let(:base_uri) { "users" }
+  let(:singular_fixture) { "user" }
+  let(:collection_class) { Kippt::UserCollection }
+  let(:resource_class) { Kippt::User }
+
+  it_behaves_like "collection resource"
+
+  describe "#search" do
+    subject { Kippt::Client.new(valid_user_credentials).users }
+
+    describe "parameters" do
+      context "when parameter is a String" do
+        it "does the search with the string being q" do
+          stub_get("/users/search?q=bunny").
+            to_return(:status => 200, :body => fixture("users_with_multiple_pages.json"))
+          subject.search("bunny")
+        end
+      end
+
+      context "when parameter is a Hash" do
+        context "with accepted keys" do
+          it "does the search" do
+            stub_get("/users/search?q=bunny").
+              to_return(:status => 200, :body => fixture("users_with_multiple_pages.json"))
+            subject.search(:q => "bunny")
+          end
+        end
+
+        context "with invalid keys" do
+          it "raises ArgumentError" do
+            lambda {
+              subject.search(:q => "bunny", :stuff => true)
+            }.should raise_error(ArgumentError, "'stuff' is not a valid search parameter")
+          end
+        end
+      end
+    end
+
+    it "returns UserCollection" do
+      stub_get("/users/search?q=bunny").
+        to_return(:status => 200, :body => fixture("users_with_multiple_pages.json"))
+      users = subject.search("bunny")
+      users.should be_a Kippt::UserCollection
+    end
+
+    it "sets UserCollection client" do
+      stub_get("/users/search?q=bunny").
+        to_return(:status => 200, :body => fixture("users_with_multiple_pages.json"))
+      Kippt::UserCollection.should_receive(:new).with(kind_of(Hash), kind_of(Kippt::Client))
+      users = subject.search("bunny")
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -11,6 +11,18 @@ def stub_get(url)
   stub_request(:get, kippt_url(url))
 end
 
+def stub_post(url)
+  stub_request(:post, kippt_url(url))
+end
+
+def stub_put(url)
+  stub_request(:put, kippt_url(url))
+end
+
+def stub_delete(url)
+  stub_request(:delete, kippt_url(url))
+end
+
 def kippt_url(url)
   "https://kippt.com/api#{url}"
 end
@@ -24,18 +36,26 @@ def fixture(file)
   File.new(fixture_path + '/' + file)
 end
 
+def json_fixture(file)
+  MultiJson.load(fixture(file).read)
+end
+
 shared_examples_for "collection resource" do
+  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("#{base_uri}.json"))
+        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("#{base_uri}.json"))
+        to_return(:status => 200, :body => fixture("#{collection_fixture}.json"))
       resources = subject.all(:limit => 10, :offset => 100)
     end
 
@@ -61,6 +81,18 @@ shared_examples_for "collection resource" do
         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
@@ -72,7 +104,7 @@ shared_examples_for "collection resource" do
   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("#{base_uri}.json"))
+        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
@@ -95,7 +127,7 @@ shared_examples_for "collection resource" do
     end
 
     it "accepts parameters" do
-      subject.object_class.should_receive(:new).with({:an => "attribute"}, subject)
+      subject.object_class.should_receive(:new).with({:an => "attribute"}, client)
       subject.build(:an => "attribute")
     end
   end
@@ -165,6 +197,8 @@ shared_examples_for "collection" do
       let(:collection_resource) { stub }
 
       it "gets the next page of results from the collection resource" do
+        client.stub(:collection_resource_for).and_return(collection_resource)
+
         results = stub
         collection_resource.should_receive(:collection_from_url).
           with(data_with_multiple_pages["meta"]["next"]).
@@ -202,6 +236,8 @@ shared_examples_for "collection" do
       let(:collection_resource) { stub }
 
       it "gets the previous page of results from the collection resource" do
+        client.stub(:collection_resource_for).and_return(collection_resource)
+
         results = stub
         collection_resource.should_receive(:collection_from_url).
           with(data_with_multiple_pages["meta"]["previous"]).
@@ -230,8 +266,19 @@ shared_examples_for "resource" do
     end
   end
 
+  describe "mapped attribute accessors" do
+    it "delegates to attributes and wraps with to a object" do
+      mapped_attributes.each do |attribute_name, attribute_class|
+        subject.send(attribute_name).class.to_s.should eq attribute_class
+      end
+    end
+  end
+
   describe "#destroy" do
+    let(:collection_resource)  { stub }
+
     it "sends delete request to the server" do
+      client.stub(:collection_resource_for).and_return(collection_resource)
       collection_resource.should_receive(:destroy_resource).with(subject).and_return(true)
       subject.destroy.should be_true
     end
@@ -239,6 +286,12 @@ shared_examples_for "resource" do
 
   describe "#save" do
     context "with valid parameters" do
+      let(:collection_resource)  { stub }
+
+      before do
+        client.stub(:collection_resource_for).and_return(collection_resource)
+      end
+
       it "sends POST request to server" do
         collection_resource.should_receive(:save_resource).with(subject).and_return({})
         subject.save
@@ -259,7 +312,10 @@ shared_examples_for "resource" do
     end
 
     context "with invalid parameters" do
+      let(:collection_resource)  { stub }
+
       before do
+        client.stub(:collection_resource_for).and_return(collection_resource)
         collection_resource.stub(:save_resource).and_return({:success => false, :error_message => "No url."})
       end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kippt
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 2.0.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,74 +9,104 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-07-19 00:00:00.000000000 Z
+date: 2013-07-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
-  requirement: &70143540468320 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.7.6
+        version: '0.8'
   type: :runtime
   prerelease: false
-  version_requirements: *70143540468320
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.8'
 - !ruby/object:Gem::Dependency
   name: faraday_middleware
-  requirement: &70143540467520 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.8.7
+        version: '0.8'
   type: :runtime
   prerelease: false
-  version_requirements: *70143540467520
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.8'
 - !ruby/object:Gem::Dependency
   name: multi_json
-  requirement: &70143540466360 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.3.4
+        version: '1.3'
   type: :runtime
   prerelease: false
-  version_requirements: *70143540466360
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.3'
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70143540465780 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 2.9.0
+        version: '2.9'
   type: :development
   prerelease: false
-  version_requirements: *70143540465780
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.9'
 - !ruby/object:Gem::Dependency
   name: webmock
-  requirement: &70143540465120 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 1.8.6
+        version: '1.11'
   type: :development
   prerelease: false
-  version_requirements: *70143540465120
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.11'
 - !ruby/object:Gem::Dependency
   name: simplecov
-  requirement: &70143540463960 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.6.4
+        version: '0.7'
   type: :development
   prerelease: false
-  version_requirements: *70143540463960
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.7'
 description: Client library for using Kippt.com API
 email:
 - vesa@vesavanska.com
@@ -92,36 +122,66 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- TODO.md
 - kippt.gemspec
 - lib/core_ext/open_struct.rb
 - lib/kippt.rb
-- lib/kippt/account.rb
 - lib/kippt/client.rb
 - lib/kippt/clip.rb
 - lib/kippt/clip_collection.rb
 - lib/kippt/clips.rb
 - lib/kippt/collection.rb
 - lib/kippt/collection_resource.rb
+- lib/kippt/comment.rb
+- lib/kippt/comment_collection.rb
+- lib/kippt/comments.rb
 - lib/kippt/connection.rb
+- lib/kippt/follow_relationship.rb
+- lib/kippt/followers.rb
+- lib/kippt/following.rb
+- lib/kippt/like.rb
+- lib/kippt/likes.rb
 - lib/kippt/list.rb
 - lib/kippt/list_collection.rb
 - lib/kippt/lists.rb
 - lib/kippt/resource.rb
+- lib/kippt/saves.rb
+- lib/kippt/user.rb
+- lib/kippt/user_clips.rb
+- lib/kippt/user_collection.rb
+- lib/kippt/user_lists.rb
+- lib/kippt/users.rb
 - lib/kippt/version.rb
 - spec/fixtures/clip.json
 - spec/fixtures/clips.json
 - spec/fixtures/clips_with_multiple_pages.json
+- spec/fixtures/comment.json
+- spec/fixtures/comments.json
+- spec/fixtures/feed.json
 - spec/fixtures/list.json
 - spec/fixtures/lists.json
 - spec/fixtures/lists_with_multiple_pages.json
-- spec/kippt/account_spec.rb
+- spec/fixtures/user.json
+- spec/fixtures/users.json
+- spec/fixtures/users_with_multiple_pages.json
 - spec/kippt/client_spec.rb
 - spec/kippt/clip_collection_spec.rb
 - spec/kippt/clip_spec.rb
 - spec/kippt/clips_spec.rb
+- spec/kippt/comment_spec.rb
+- spec/kippt/comments_spec.rb
+- spec/kippt/follow_relationship_spec.rb
+- spec/kippt/followers_spec.rb
+- spec/kippt/following_spec.rb
 - spec/kippt/list_collection_spec.rb
 - spec/kippt/list_spec.rb
 - spec/kippt/lists_spec.rb
+- spec/kippt/saves_spec.rb
+- spec/kippt/user_clips_spec.rb
+- spec/kippt/user_collection_spec.rb
+- spec/kippt/user_lists_spec.rb
+- spec/kippt/user_spec.rb
+- spec/kippt/users_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/vesan/kippt
 licenses: []
@@ -135,15 +195,21 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: -2217435892223786900
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: -2217435892223786900
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.10
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Client library for using Kippt.com API
@@ -151,15 +217,31 @@ test_files:
 - spec/fixtures/clip.json
 - spec/fixtures/clips.json
 - spec/fixtures/clips_with_multiple_pages.json
+- spec/fixtures/comment.json
+- spec/fixtures/comments.json
+- spec/fixtures/feed.json
 - spec/fixtures/list.json
 - spec/fixtures/lists.json
 - spec/fixtures/lists_with_multiple_pages.json
-- spec/kippt/account_spec.rb
+- spec/fixtures/user.json
+- spec/fixtures/users.json
+- spec/fixtures/users_with_multiple_pages.json
 - spec/kippt/client_spec.rb
 - spec/kippt/clip_collection_spec.rb
 - spec/kippt/clip_spec.rb
 - spec/kippt/clips_spec.rb
+- spec/kippt/comment_spec.rb
+- spec/kippt/comments_spec.rb
+- spec/kippt/follow_relationship_spec.rb
+- spec/kippt/followers_spec.rb
+- spec/kippt/following_spec.rb
 - spec/kippt/list_collection_spec.rb
 - spec/kippt/list_spec.rb
 - spec/kippt/lists_spec.rb
+- spec/kippt/saves_spec.rb
+- spec/kippt/user_clips_spec.rb
+- spec/kippt/user_collection_spec.rb
+- spec/kippt/user_lists_spec.rb
+- spec/kippt/user_spec.rb
+- spec/kippt/users_spec.rb
 - spec/spec_helper.rb