twitter 4.8.1 → 5.0.0

data/spec/twitter/place_spec.rb

@@ -3,73 +3,129 @@ require 'helper'
 describe Twitter::Place do
 
   describe "#==" do
-    it "returns true when objects IDs are the same" do
-      place = Twitter::Place.new(:id => 1, :name => "foo")
-      other = Twitter::Place.new(:id => 1, :name => "bar")
-      expect(place == other).to be_true
+    it "returns true when objects WOE IDs are the same" do
+      place = Twitter::Place.new(:woeid => 1, :name => "foo")
+      other = Twitter::Place.new(:woeid => 1, :name => "bar")
+      expect(place == other).to be true
     end
-    it "returns false when objects IDs are different" do
-      place = Twitter::Place.new(:id => 1)
-      other = Twitter::Place.new(:id => 2)
-      expect(place == other).to be_false
+    it "returns false when objects WOE IDs are different" do
+      place = Twitter::Place.new(:woeid => 1)
+      other = Twitter::Place.new(:woeid => 2)
+      expect(place == other).to be false
     end
     it "returns false when classes are different" do
-      place = Twitter::Place.new(:id => 1)
-      other = Twitter::Identity.new(:id => 1)
-      expect(place == other).to be_false
+      place = Twitter::Place.new(:woeid => 1)
+      other = Twitter::Base.new(:woeid => 1)
+      expect(place == other).to be false
     end
   end
 
   describe "#bounding_box" do
-    it "returns a Twitter::Place when set" do
-      place = Twitter::Place.new(:id => "247f43d441defc03", :bounding_box => {:type => 'Polygon', :coordinates => [[[-122.40348192, 37.77752898], [-122.387436, 37.77752898], [-122.387436, 37.79448597], [-122.40348192, 37.79448597]]]})
+    it "returns a Twitter::Geo when bounding_box is set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :bounding_box => {:type => "Polygon", :coordinates => [[[-122.40348192, 37.77752898], [-122.387436, 37.77752898], [-122.387436, 37.79448597], [-122.40348192, 37.79448597]]]})
       expect(place.bounding_box).to be_a Twitter::Geo::Polygon
     end
-    it "returns nil when not set" do
-      place = Twitter::Place.new(:id => "247f43d441defc03")
+    it "returns nil when not bounding_box is not set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03")
       expect(place.bounding_box).to be_nil
     end
   end
 
+  describe "#bounding_box?" do
+    it "returns true when bounding_box is set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :bounding_box => {:type => "Polygon", :coordinates => [[[-122.40348192, 37.77752898], [-122.387436, 37.77752898], [-122.387436, 37.79448597], [-122.40348192, 37.79448597]]]})
+      expect(place.bounding_box?).to be true
+    end
+    it "returns false when bounding_box is not set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03")
+      expect(place.bounding_box?).to be false
+    end
+  end
+
+  describe "#contained_within" do
+    it "returns a Twitter::Place when contained_within is set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :contained_within => {:woeid => "247f43d441defc04"})
+      expect(place.contained_within).to be_a Twitter::Place
+    end
+    it "returns nil when not contained_within is not set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03")
+      expect(place.contained_within).to be_nil
+    end
+  end
+
+  describe "#contained_within?" do
+    it "returns true when contained_within is set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :contained_within => {:woeid => "247f43d441defc04"})
+      expect(place.contained?).to be true
+    end
+    it "returns false when contained_within is not set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03")
+      expect(place.contained?).to be false
+    end
+  end
+
   describe "#country_code" do
     it "returns a country code when set with country_code" do
-      place = Twitter::Place.new(:id => "247f43d441defc03", :country_code => 'US')
-      expect(place.country_code).to eq 'US'
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :country_code => "US")
+      expect(place.country_code).to eq("US")
     end
     it "returns a country code when set with countryCode" do
-      place = Twitter::Place.new(:id => "247f43d441defc03", :countryCode => 'US')
-      expect(place.country_code).to eq 'US'
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :countryCode => "US")
+      expect(place.country_code).to eq("US")
     end
     it "returns nil when not set" do
-      place = Twitter::Place.new(:id => "247f43d441defc03")
+      place = Twitter::Place.new(:woeid => "247f43d441defc03")
       expect(place.country_code).to be_nil
     end
   end
 
   describe "#parent_id" do
     it "returns a parent ID when set with parentid" do
-      place = Twitter::Place.new(:id => "247f43d441defc03", :parentid => 1)
-      expect(place.parent_id).to eq 1
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :parentid => 1)
+      expect(place.parent_id).to eq(1)
     end
     it "returns nil when not set" do
-      place = Twitter::Place.new(:id => "247f43d441defc03")
+      place = Twitter::Place.new(:woeid => "247f43d441defc03")
       expect(place.parent_id).to be_nil
     end
   end
 
   describe "#place_type" do
     it "returns a place type when set with place_type" do
-      place = Twitter::Place.new(:id => "247f43d441defc03", :place_type => 'city')
-      expect(place.place_type).to eq 'city'
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :place_type => "city")
+      expect(place.place_type).to eq("city")
     end
     it "returns a place type when set with placeType[name]" do
-      place = Twitter::Place.new(:id => "247f43d441defc03", :placeType => {:name => 'Town'})
-      expect(place.place_type).to eq 'Town'
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :placeType => {:name => "Town"})
+      expect(place.place_type).to eq("Town")
     end
     it "returns nil when not set" do
-      place = Twitter::Place.new(:id => "247f43d441defc03")
+      place = Twitter::Place.new(:woeid => "247f43d441defc03")
       expect(place.place_type).to be_nil
     end
   end
 
+  describe "#uri" do
+    it "returns a URI when the url is set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :url => "https://api.twitter.com/1.1/geo/id/247f43d441defc03.json")
+      expect(place.uri).to be_a URI
+      expect(place.uri.to_s).to eq("https://api.twitter.com/1.1/geo/id/247f43d441defc03.json")
+    end
+    it "returns nil when the url is not set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03")
+      expect(place.uri).to be_nil
+    end
+  end
+
+  describe "#uri?" do
+    it "returns true when the url is set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03", :url => "https://api.twitter.com/1.1/geo/id/247f43d441defc03.json")
+      expect(place.uri?).to be true
+    end
+    it "returns false when the url is not set" do
+      place = Twitter::Place.new(:woeid => "247f43d441defc03")
+      expect(place.uri?).to be false
+    end
+  end
+
 end

data/spec/twitter/profile_banner_spec.rb

@@ -4,7 +4,7 @@ describe Twitter::ProfileBanner do
 
   describe "#sizes" do
     it "returns a hash of Sizes when sizes is set" do
-      sizes = Twitter::ProfileBanner.new(:sizes => {:small => {:h => 226, :w => 340, :resize => 'fit'}, :large => {:h => 466, :w => 700, :resize => 'fit'}, :medium => {:h => 399, :w => 600, :resize => 'fit'}, :thumb => {:h => 150, :w => 150, :resize => 'crop'}}).sizes
+      sizes = Twitter::ProfileBanner.new(:sizes => {:small => {:h => 226, :w => 340, :resize => "fit"}, :large => {:h => 466, :w => 700, :resize => "fit"}, :medium => {:h => 399, :w => 600, :resize => "fit"}, :thumb => {:h => 150, :w => 150, :resize => "crop"}}).sizes
       expect(sizes).to be_a Hash
       expect(sizes[:small]).to be_a Twitter::Size
     end

data/spec/twitter/rate_limit_spec.rb

@@ -4,9 +4,9 @@ describe Twitter::RateLimit do
 
   describe "#limit" do
     it "returns an Integer when x-rate-limit-limit header is set" do
-      rate_limit = Twitter::RateLimit.new('x-rate-limit-limit' => "150")
+      rate_limit = Twitter::RateLimit.new("x-rate-limit-limit" => "150")
       expect(rate_limit.limit).to be_an Integer
-      expect(rate_limit.limit).to eq 150
+      expect(rate_limit.limit).to eq(150)
     end
     it "returns nil when x-rate-limit-limit header is not set" do
       rate_limit = Twitter::RateLimit.new
@@ -16,9 +16,9 @@ describe Twitter::RateLimit do
 
   describe "#remaining" do
     it "returns an Integer when x-rate-limit-remaining header is set" do
-      rate_limit = Twitter::RateLimit.new('x-rate-limit-remaining' => "149")
+      rate_limit = Twitter::RateLimit.new("x-rate-limit-remaining" => "149")
       expect(rate_limit.remaining).to be_an Integer
-      expect(rate_limit.remaining).to eq 149
+      expect(rate_limit.remaining).to eq(149)
     end
     it "returns nil when x-rate-limit-remaining header is not set" do
       rate_limit = Twitter::RateLimit.new
@@ -28,9 +28,9 @@ describe Twitter::RateLimit do
 
   describe "#reset_at" do
     it "returns a Time when x-rate-limit-reset header is set" do
-      rate_limit = Twitter::RateLimit.new('x-rate-limit-reset' => "1339019097")
+      rate_limit = Twitter::RateLimit.new("x-rate-limit-reset" => "1339019097")
       expect(rate_limit.reset_at).to be_a Time
-      expect(rate_limit.reset_at).to eq Time.at(1339019097)
+      expect(rate_limit.reset_at).to eq(Time.at(1339019097))
     end
     it "returns nil when x-rate-limit-reset header is not set" do
       rate_limit = Twitter::RateLimit.new
@@ -46,9 +46,9 @@ describe Twitter::RateLimit do
       Timecop.return
     end
     it "returns an Integer when x-rate-limit-reset header is set" do
-      rate_limit = Twitter::RateLimit.new('x-rate-limit-reset' => "1339019097")
+      rate_limit = Twitter::RateLimit.new("x-rate-limit-reset" => "1339019097")
       expect(rate_limit.reset_in).to be_an Integer
-      expect(rate_limit.reset_in).to eq 15777
+      expect(rate_limit.reset_in).to eq(15777)
     end
     it "returns nil when x-rate-limit-reset header is not set" do
       rate_limit = Twitter::RateLimit.new
@@ -56,21 +56,4 @@ describe Twitter::RateLimit do
     end
   end
 
-  describe "#update" do
-    before do
-      Timecop.freeze(Time.utc(2012, 6, 6, 17, 22, 0))
-    end
-    after do
-      Timecop.return
-    end
-    it "updates a rate limit" do
-      rate_limit = Twitter::RateLimit.new('x-rate-limit-reset' => "1339019097")
-      expect(rate_limit.reset_in).to be_an Integer
-      expect(rate_limit.reset_in).to eq 15777
-      rate_limit.update({'x-rate-limit-reset' => "1339019098"})
-      expect(rate_limit.reset_in).to be_an Integer
-      expect(rate_limit.reset_in).to eq 15778
-    end
-  end
-
 end

data/spec/twitter/relationship_spec.rb

@@ -4,31 +4,45 @@ describe Twitter::Relationship do
 
   describe "#source" do
     it "returns a User when source is set" do
-      source = Twitter::Relationship.new(:relationship => {:source => {:id => 7505382}}).source
-      expect(source).to be_a Twitter::SourceUser
+      relationship = Twitter::Relationship.new(:relationship => {:source => {:id => 7505382}})
+      expect(relationship.source).to be_a Twitter::SourceUser
     end
     it "returns nil when source is not set" do
-      source = Twitter::Relationship.new(:relationship => {}).source
-      expect(source).to be_nil
+      relationship = Twitter::Relationship.new(:relationship => {})
+      expect(relationship.source).to be_nil
+    end
+  end
+
+  describe "#source?" do
+    it "returns true when source is set" do
+      relationship = Twitter::Relationship.new(:relationship => {:source => {:id => 7505382}})
+      expect(relationship.source?).to be true
+    end
+    it "returns false when source is not set" do
+      relationship = Twitter::Relationship.new(:relationship => {})
+      expect(relationship.source?).to be false
     end
   end
 
   describe "#target" do
     it "returns a User when target is set" do
-      target = Twitter::Relationship.new(:relationship => {:target => {:id => 7505382}}).target
-      expect(target).to be_a Twitter::TargetUser
+      relationship = Twitter::Relationship.new(:relationship => {:target => {:id => 7505382}})
+      expect(relationship.target).to be_a Twitter::TargetUser
     end
     it "returns nil when target is not set" do
-      target = Twitter::Relationship.new(:relationship => {}).target
-      expect(target).to be_nil
+      relationship = Twitter::Relationship.new(:relationship => {})
+      expect(relationship.target).to be_nil
     end
   end
 
-  describe "#update" do
-    it "updates a relationship" do
+  describe "#target?" do
+    it "returns true when target is set" do
       relationship = Twitter::Relationship.new(:relationship => {:target => {:id => 7505382}})
-      relationship.update(:relationship => {:target => {:id => 14100886}})
-      expect(relationship.target.id).to eq 14100886
+      expect(relationship.target?).to be true
+    end
+    it "returns false when target is not set" do
+      relationship = Twitter::Relationship.new(:relationship => {})
+      expect(relationship.target?).to be false
     end
   end
 

data/spec/twitter/{api → rest/api}/direct_messages_spec.rb

@@ -1,9 +1,9 @@
 require 'helper'
 
-describe Twitter::API::DirectMessages do
+describe Twitter::REST::API::DirectMessages do
 
   before do
-    @client = Twitter::Client.new
+    @client = Twitter::REST::Client.new(:consumer_key => "CK", :consumer_secret => "CS", :access_token => "AT", :access_token_secret => "AS")
   end
 
   describe "#direct_messages_received" do
@@ -18,7 +18,7 @@ describe Twitter::API::DirectMessages do
       direct_messages = @client.direct_messages_received
       expect(direct_messages).to be_an Array
       expect(direct_messages.first).to be_a Twitter::DirectMessage
-      expect(direct_messages.first.sender.id).to eq 7505382
+      expect(direct_messages.first.sender.id).to eq(7505382)
     end
   end
 
@@ -34,7 +34,7 @@ describe Twitter::API::DirectMessages do
       direct_messages = @client.direct_messages_sent
       expect(direct_messages).to be_an Array
       expect(direct_messages.first).to be_a Twitter::DirectMessage
-      expect(direct_messages.first.sender.id).to eq 7505382
+      expect(direct_messages.first.sender.id).to eq(7505382)
     end
   end
 
@@ -49,7 +49,7 @@ describe Twitter::API::DirectMessages do
     it "returns the specified direct message" do
       direct_message = @client.direct_message(1825786345)
       expect(direct_message).to be_a Twitter::DirectMessage
-      expect(direct_message.sender.id).to eq 7505382
+      expect(direct_message.sender.id).to eq(7505382)
     end
   end
 
@@ -66,7 +66,7 @@ describe Twitter::API::DirectMessages do
         direct_messages = @client.direct_messages(1825786345)
         expect(direct_messages).to be_an Array
         expect(direct_messages.first).to be_a Twitter::DirectMessage
-        expect(direct_messages.first.sender.id).to eq 7505382
+        expect(direct_messages.first.sender.id).to eq(7505382)
       end
     end
     context "without ids passed" do
@@ -81,39 +81,52 @@ describe Twitter::API::DirectMessages do
         direct_messages = @client.direct_messages
         expect(direct_messages).to be_an Array
         expect(direct_messages.first).to be_a Twitter::DirectMessage
-        expect(direct_messages.first.sender.id).to eq 7505382
+        expect(direct_messages.first.sender.id).to eq(7505382)
       end
     end
   end
 
-  describe "#direct_message_destroy" do
+  describe "#destroy_direct_message" do
     before do
       stub_post("/1.1/direct_messages/destroy.json").with(:body => {:id => "1825785544"}).to_return(:body => fixture("direct_message.json"), :headers => {:content_type => "application/json; charset=utf-8"})
     end
     it "requests the correct resource" do
-      @client.direct_message_destroy(1825785544)
+      @client.destroy_direct_message(1825785544)
       expect(a_post("/1.1/direct_messages/destroy.json").with(:body => {:id => "1825785544"})).to have_been_made
     end
     it "returns an array of deleted messages" do
-      direct_messages = @client.direct_message_destroy(1825785544)
+      direct_messages = @client.destroy_direct_message(1825785544)
       expect(direct_messages).to be_an Array
       expect(direct_messages.first).to be_a Twitter::DirectMessage
-      expect(direct_messages.first.sender.id).to eq 7505382
+      expect(direct_messages.first.sender.id).to eq(7505382)
     end
   end
 
-  describe "#direct_message_create" do
+  describe "#create_direct_message" do
     before do
       stub_post("/1.1/direct_messages/new.json").with(:body => {:screen_name => "pengwynn", :text => "Creating a fixture for the Twitter gem"}).to_return(:body => fixture("direct_message.json"), :headers => {:content_type => "application/json; charset=utf-8"})
     end
     it "requests the correct resource" do
-      @client.direct_message_create("pengwynn", "Creating a fixture for the Twitter gem")
+      @client.create_direct_message("pengwynn", "Creating a fixture for the Twitter gem")
       expect(a_post("/1.1/direct_messages/new.json").with(:body => {:screen_name => "pengwynn", :text => "Creating a fixture for the Twitter gem"})).to have_been_made
     end
     it "returns the sent message" do
-      direct_message = @client.direct_message_create("pengwynn", "Creating a fixture for the Twitter gem")
+      direct_message = @client.create_direct_message("pengwynn", "Creating a fixture for the Twitter gem")
       expect(direct_message).to be_a Twitter::DirectMessage
-      expect(direct_message.text).to eq "Creating a fixture for the Twitter gem"
+      expect(direct_message.text).to eq("Creating a fixture for the Twitter gem")
+    end
+    context "with a URI object passed" do
+      it "requests the correct resource" do
+        user = URI.parse("https://twitter.com/pengwynn")
+        @client.create_direct_message(user, "Creating a fixture for the Twitter gem")
+        expect(a_post("/1.1/direct_messages/new.json").with(:body => {:screen_name => "pengwynn", :text => "Creating a fixture for the Twitter gem"})).to have_been_made
+      end
+    end
+    context "with a URI string passed" do
+      it "requests the correct resource" do
+        @client.create_direct_message("https://twitter.com/pengwynn", "Creating a fixture for the Twitter gem")
+        expect(a_post("/1.1/direct_messages/new.json").with(:body => {:screen_name => "pengwynn", :text => "Creating a fixture for the Twitter gem"})).to have_been_made
+      end
     end
   end
 

data/spec/twitter/{api → rest/api}/favorites_spec.rb

@@ -1,9 +1,9 @@
 require 'helper'
 
-describe Twitter::API::Favorites do
+describe Twitter::REST::API::Favorites do
 
   before do
-    @client = Twitter::Client.new
+    @client = Twitter::REST::Client.new(:consumer_key => "CK", :consumer_secret => "CS", :access_token => "AT", :access_token_secret => "AS")
   end
 
   describe "#favorites" do
@@ -19,7 +19,20 @@ describe Twitter::API::Favorites do
         favorites = @client.favorites("sferik")
         expect(favorites).to be_an Array
         expect(favorites.first).to be_a Twitter::Tweet
-        expect(favorites.first.user.id).to eq 7505382
+        expect(favorites.first.user.id).to eq(7505382)
+      end
+      context "with a URI object passed" do
+        it "requests the correct resource" do
+          user = URI.parse("https://twitter.com/sferik")
+          @client.favorites(user)
+          expect(a_get("/1.1/favorites/list.json").with(:query => {:screen_name => "sferik"})).to have_been_made
+        end
+      end
+      context "with a URI string passed" do
+        it "requests the correct resource" do
+          @client.favorites("https://twitter.com/sferik")
+          expect(a_get("/1.1/favorites/list.json").with(:query => {:screen_name => "sferik"})).to have_been_made
+        end
       end
     end
     context "without arguments passed" do
@@ -34,7 +47,7 @@ describe Twitter::API::Favorites do
         favorites = @client.favorites
         expect(favorites).to be_an Array
         expect(favorites.first).to be_a Twitter::Tweet
-        expect(favorites.first.user.id).to eq 7505382
+        expect(favorites.first.user.id).to eq(7505382)
       end
     end
   end
@@ -51,7 +64,27 @@ describe Twitter::API::Favorites do
       tweets = @client.unfavorite(25938088801)
       expect(tweets).to be_an Array
       expect(tweets.first).to be_a Twitter::Tweet
-      expect(tweets.first.text).to eq "The problem with your code is that it's doing exactly what you told it to do."
+      expect(tweets.first.text).to eq("The problem with your code is that it's doing exactly what you told it to do.")
+    end
+    context "with a URI object passed" do
+      it "requests the correct resource" do
+        tweet = URI.parse("https://twitter.com/sferik/status/25938088801")
+        @client.unfavorite(tweet)
+        expect(a_post("/1.1/favorites/destroy.json").with(:body => {:id => "25938088801"})).to have_been_made
+      end
+    end
+    context "with a URI string passed" do
+      it "requests the correct resource" do
+        @client.unfavorite("https://twitter.com/sferik/status/25938088801")
+        expect(a_post("/1.1/favorites/destroy.json").with(:body => {:id => "25938088801"})).to have_been_made
+      end
+    end
+    context "with a Tweet passed" do
+      it "requests the correct resource" do
+        tweet = Twitter::Tweet.new(:id => 25938088801)
+        @client.unfavorite(tweet)
+        expect(a_post("/1.1/favorites/destroy.json").with(:body => {:id => "25938088801"})).to have_been_made
+      end
     end
   end
 
@@ -67,7 +100,7 @@ describe Twitter::API::Favorites do
       tweets = @client.favorite(25938088801)
       expect(tweets).to be_an Array
       expect(tweets.first).to be_a Twitter::Tweet
-      expect(tweets.first.text).to eq "The problem with your code is that it's doing exactly what you told it to do."
+      expect(tweets.first.text).to eq("The problem with your code is that it's doing exactly what you told it to do.")
     end
     context "already favorited" do
       before do
@@ -77,6 +110,26 @@ describe Twitter::API::Favorites do
         expect{@client.favorite(25938088801)}.not_to raise_error
       end
     end
+    context "with a URI object passed" do
+      it "requests the correct resource" do
+        tweet = URI.parse("https://twitter.com/sferik/status/25938088801")
+        @client.favorite(tweet)
+        expect(a_post("/1.1/favorites/create.json").with(:body => {:id => "25938088801"})).to have_been_made
+      end
+    end
+    context "with a URI string passed" do
+      it "requests the correct resource" do
+        @client.favorite("https://twitter.com/sferik/status/25938088801")
+        expect(a_post("/1.1/favorites/create.json").with(:body => {:id => "25938088801"})).to have_been_made
+      end
+    end
+    context "with a Tweet passed" do
+      it "requests the correct resource" do
+        tweet = Twitter::Tweet.new(:id => 25938088801)
+        @client.favorite(tweet)
+        expect(a_post("/1.1/favorites/create.json").with(:body => {:id => "25938088801"})).to have_been_made
+      end
+    end
   end
 
   describe "#favorite!" do
@@ -91,7 +144,7 @@ describe Twitter::API::Favorites do
       tweets = @client.favorite!(25938088801)
       expect(tweets).to be_an Array
       expect(tweets.first).to be_a Twitter::Tweet
-      expect(tweets.first.text).to eq "The problem with your code is that it's doing exactly what you told it to do."
+      expect(tweets.first.text).to eq("The problem with your code is that it's doing exactly what you told it to do.")
     end
     context "forbidden" do
       before do
@@ -109,6 +162,26 @@ describe Twitter::API::Favorites do
         expect{@client.favorite!(25938088801)}.to raise_error Twitter::Error::AlreadyFavorited
       end
     end
+    context "with a URI object passed" do
+      it "requests the correct resource" do
+        tweet = URI.parse("https://twitter.com/sferik/status/25938088801")
+        @client.favorite!(tweet)
+        expect(a_post("/1.1/favorites/create.json").with(:body => {:id => "25938088801"})).to have_been_made
+      end
+    end
+    context "with a URI string passed" do
+      it "requests the correct resource" do
+        @client.favorite!("https://twitter.com/sferik/status/25938088801")
+        expect(a_post("/1.1/favorites/create.json").with(:body => {:id => "25938088801"})).to have_been_made
+      end
+    end
+    context "with a Tweet passed" do
+      it "requests the correct resource" do
+        tweet = Twitter::Tweet.new(:id => 25938088801)
+        @client.favorite!(tweet)
+        expect(a_post("/1.1/favorites/create.json").with(:body => {:id => "25938088801"})).to have_been_made
+      end
+    end
   end
 
 end