twitter 4.8.1 → 5.0.0

data/spec/fixtures/track_streaming_user.json

@@ -0,0 +1,5 @@
+{"friends":[488736931,311444249]}
+{"created_at":"Wed Apr 06 19:13:37 +0000 2011","id":55709764298092545,"id_str":"55709764298092545","text":"The problem with your code is that it's doing exactly what you told it to do.","source":"\u003ca href=\"http:\/\/twitter.com\/download\/iphone\" rel=\"nofollow\"\u003eTwitter for iPhone\u003c\/a\u003e","truncated":false,"in_reply_to_status_id":null,"in_reply_to_status_id_str":null,"in_reply_to_user_id":null,"in_reply_to_user_id_str":null,"in_reply_to_screen_name":null,"user":{"id":7505382,"id_str":"7505382","name":"Erik Michaels-Ober","screen_name":"sferik","location":"San Francisco","description":"Write code. Not too much. Mostly Ruby.","url":"https:\/\/github.com\/sferik","entities":{"url":{"urls":[{"url":"https:\/\/github.com\/sferik","expanded_url":null,"indices":[0,25]}]},"description":{"urls":[]}},"protected":false,"followers_count":2479,"friends_count":200,"listed_count":132,"created_at":"Mon Jul 16 12:59:01 +0000 2007","favourites_count":4421,"utc_offset":-28800,"time_zone":"Pacific Time (US & Canada)","geo_enabled":true,"verified":false,"statuses_count":8730,"lang":"en","contributors_enabled":false,"is_translator":false,"profile_background_color":"000000","profile_background_image_url":"http:\/\/a0.twimg.com\/profile_background_images\/677717672\/bb0b3653dcf0644e344823e0a2eb3382.png","profile_background_image_url_https":"https:\/\/si0.twimg.com\/profile_background_images\/677717672\/bb0b3653dcf0644e344823e0a2eb3382.png","profile_background_tile":false,"profile_image_url":"http:\/\/a0.twimg.com\/profile_images\/1759857427\/image1326743606_normal.png","profile_image_url_https":"https:\/\/si0.twimg.com\/profile_images\/1759857427\/image1326743606_normal.png","profile_banner_url":"https:\/\/si0.twimg.com\/profile_banners\/7505382\/1349499693","profile_link_color":"0084B4","profile_sidebar_border_color":"000000","profile_sidebar_fill_color":"DDEEF6","profile_text_color":"333333","profile_use_background_image":true,"default_profile":false,"default_profile_image":false,"following":false,"follow_request_sent":false,"notifications":false},"geo":{"type":"Point","coordinates":[37.78349999,-122.39362884]},"coordinates":{"type":"Point","coordinates":[-122.39362884,37.78349999]},"place":{"id":"5c92ab5379de3839","url":"https:\/\/api.twitter.com\/1.1\/geo\/id\/5c92ab5379de3839.json","place_type":"neighborhood","name":"South Beach","full_name":"South Beach, San Francisco","country_code":"US","country":"United States","bounding_box":{"type":"Polygon","coordinates":[[[-122.403482,37.777529],[-122.387436,37.777529],[-122.387436,37.794486],[-122.403482,37.794486]]]},"attributes":{}},"contributors":null,"retweet_count":316,"entities":{"hashtags":[],"urls":[],"user_mentions":[]},"favorited":false,"retweeted":false}
+{"created_at":"Wed Apr 06 19:13:37 +0000 2011","id":55709764298092545,"id_str":"55709764298092545","text":"The problem with your code is that it's doing exactly what you told it to do.","source":"\u003ca href=\"http:\/\/twitter.com\/download\/iphone\" rel=\"nofollow\"\u003eTwitter for iPhone\u003c\/a\u003e","truncated":false,"in_reply_to_status_id":null,"in_reply_to_status_id_str":null,"in_reply_to_user_id":null,"in_reply_to_user_id_str":null,"in_reply_to_screen_name":null,"user":{"id":7505382,"id_str":"7505382","name":"Erik Michaels-Ober","screen_name":"sferik","location":"San Francisco","description":"Write code. Not too much. Mostly Ruby.","url":"https:\/\/github.com\/sferik","entities":{"url":{"urls":[{"url":"https:\/\/github.com\/sferik","expanded_url":null,"indices":[0,25]}]},"description":{"urls":[]}},"protected":false,"followers_count":2479,"friends_count":200,"listed_count":132,"created_at":"Mon Jul 16 12:59:01 +0000 2007","favourites_count":4421,"utc_offset":-28800,"time_zone":"Pacific Time (US & Canada)","geo_enabled":true,"verified":false,"statuses_count":8730,"lang":"en","contributors_enabled":false,"is_translator":false,"profile_background_color":"000000","profile_background_image_url":"http:\/\/a0.twimg.com\/profile_background_images\/677717672\/bb0b3653dcf0644e344823e0a2eb3382.png","profile_background_image_url_https":"https:\/\/si0.twimg.com\/profile_background_images\/677717672\/bb0b3653dcf0644e344823e0a2eb3382.png","profile_background_tile":false,"profile_image_url":"http:\/\/a0.twimg.com\/profile_images\/1759857427\/image1326743606_normal.png","profile_image_url_https":"https:\/\/si0.twimg.com\/profile_images\/1759857427\/image1326743606_normal.png","profile_banner_url":"https:\/\/si0.twimg.com\/profile_banners\/7505382\/1349499693","profile_link_color":"0084B4","profile_sidebar_border_color":"000000","profile_sidebar_fill_color":"DDEEF6","profile_text_color":"333333","profile_use_background_image":true,"default_profile":false,"default_profile_image":false,"following":false,"follow_request_sent":false,"notifications":false},"geo":{"type":"Point","coordinates":[37.78349999,-122.39362884]},"coordinates":{"type":"Point","coordinates":[-122.39362884,37.78349999]},"place":{"id":"5c92ab5379de3839","url":"https:\/\/api.twitter.com\/1.1\/geo\/id\/5c92ab5379de3839.json","place_type":"neighborhood","name":"South Beach","full_name":"South Beach, San Francisco","country_code":"US","country":"United States","bounding_box":{"type":"Polygon","coordinates":[[[-122.403482,37.777529],[-122.387436,37.777529],[-122.387436,37.794486],[-122.403482,37.794486]]]},"attributes":{}},"contributors":null,"retweet_count":316,"entities":{"hashtags":[],"urls":[],"user_mentions":[]},"favorited":false,"retweeted":false}
+{"direct_message":{"id":389327745617891328,"id_str":"389327745617891328","text":"hello bot","sender":{"id":10083602,"id_str":"10083602","name":"Adam Bird","screen_name":"adambird","location":"Nottingham, UK","url":"http:\/\/about.me\/adambird", "description":"CEO @onediaryapp, ex @esendex CTO and co-founder, still cycling","protected":false,"followers_count":1286,"friends_count":850,"listed_count":51,"created_at":"Fri Nov 09 00:35:12 +0000 2007","favourites_count":93,"utc_offset":3600,"time_zone":"London","geo_enabled":true,"verified":false,"statuses_count":13480,"lang":"en","contributors_enabled":false,"is_translator":false,"profile_background_color":"C0DEED","profile_background_image_url":"http:\/\/abs.twimg.com\/images\/themes\/theme1\/bg.png","profile_background_image_url_https":"https:\/\/abs.twimg.com\/images\/themes\/theme1\/bg.png","profile_background_tile":false,"profile_image_url":"http:\/\/a0.twimg.com\/profile_images\/378800000477232297\/23d85bb78f71534eea1e1133fb771f86_normal.jpeg","profile_image_url_https":"https:\/\/si0.twimg.com\/profile_images\/378800000477232297\/23d85bb78f71534eea1e1133fb771f86_normal.jpeg","profile_link_color":"0084B4","profile_sidebar_border_color":"C0DEED","profile_sidebar_fill_color":"DDEEF6","profile_text_color":"333333","profile_use_background_image":true,"default_profile":true,"default_profile_image":false,"following":false,"follow_request_sent":false,"notifications":false},"sender_id":10083602,"sender_id_str":"10083602","sender_screen_name":"adambird","recipient":{"id":1292911088,"id_str":"1292911088","name":"One Diary Bot","screen_name":"onediarybot","location":"","url":"http:\/\/www.onediary.com","description":"I'm the One Diary bot, here to help you with your life including personal cycling weather forecasts.","protected":false,"followers_count":113,"friends_count":148,"listed_count":1,"created_at":"Sat Mar 23 23:52:18 +0000 2013","favourites_count":0,"utc_offset":"","time_zone":"","geo_enabled":false,"verified":false,"statuses_count":7919,"lang":"en","contributors_enabled":false,"is_translator":false,"profile_background_color":"C0DEED","profile_background_image_url":"http:\/\/abs.twimg.com\/images\/themes\/theme1\/bg.png","profile_background_image_url_https":"https:\/\/abs.twimg.com\/images\/themes\/theme1\/bg.png","profile_background_tile":false,"profile_image_url":"http:\/\/a0.twimg.com\/profile_images\/3651575369\/090551d8dd92080198f707769239ff43_normal.jpeg","profile_image_url_https":"https:\/\/si0.twimg.com\/profile_images\/3651575369\/090551d8dd92080198f707769239ff43_normal.jpeg","profile_link_color":"0084B4","profile_sidebar_border_color":"C0DEED","profile_sidebar_fill_color":"DDEEF6","profile_text_color":"333333","profile_use_background_image":true,"default_profile":true,"default_profile_image":false,"following":false,"follow_request_sent":false,"notifications":false},"recipient_id":1292911088,"recipient_id_str":"1292911088","recipient_screen_name":"onediarybot","created_at":"Sun Oct 13 09:52:22 +0000 2013","entities":{"hashtags":[],"symbols":[],"urls":[],"user_mentions":[]}}}
+{"event":"follow","source":{"id":10083602,"id_str":"10083602","name":"Adam Bird","screen_name":"adambird","location":"Nottingham, UK","url":"http://t.co/M1eaIKDQyz","description":"CEO @onediaryapp, ex @esendex CTO and co-founder, still cycling","protected":false,"followers_count":1295,"friends_count":850,"listed_count":53,"created_at":"Fri Nov 09 00:35:12 +0000 2007","favourites_count":93,"utc_offset":0,"time_zone":"London","geo_enabled":true,"verified":false,"statuses_count":13507,"lang":"en","contributors_enabled":false,"is_translator":false,"profile_background_color":"C0DEED","profile_background_image_url":"http://abs.twimg.com/images/themes/theme1/bg.png","profile_background_image_url_https":"https://abs.twimg.com/images/themes/theme1/bg.png","profile_background_tile":false,"profile_image_url":"http://pbs.twimg.com/profile_images/378800000477232297/23d85bb78f71534eea1e1133fb771f86_normal.jpeg","profile_image_url_https":"https://pbs.twimg.com/profile_images/378800000477232297/23d85bb78f71534eea1e1133fb771f86_normal.jpeg","profile_link_color":"0084B4","profile_sidebar_border_color":"C0DEED","profile_sidebar_fill_color":"DDEEF6","profile_text_color":"333333","profile_use_background_image":true,"default_profile":true,"default_profile_image":false,"following":false,"follow_request_sent":false,"notifications":false},"target":{"id":1292911088,"id_str":"1292911088","name":"One Diary Bot","screen_name":"onediarybot","location":null,"url":"http://t.co/etHGc0xHX4","description":"I'm the One Diary bot, here to help you with your life including personal cycling weather forecasts.","protected":false,"followers_count":123,"friends_count":157,"listed_count":1,"created_at":"Sat Mar 23 23:52:18 +0000 2013","favourites_count":0,"utc_offset":null,"time_zone":null,"geo_enabled":false,"verified":false,"statuses_count":9637,"lang":"en","contributors_enabled":false,"is_translator":false,"profile_background_color":"C0DEED","profile_background_image_url":"http://abs.twimg.com/images/themes/theme1/bg.png","profile_background_image_url_https":"https://abs.twimg.com/images/themes/theme1/bg.png","profile_background_tile":false,"profile_image_url":"http://pbs.twimg.com/profile_images/3651575369/090551d8dd92080198f707769239ff43_normal.jpeg","profile_image_url_https":"https://pbs.twimg.com/profile_images/3651575369/090551d8dd92080198f707769239ff43_normal.jpeg","profile_link_color":"0084B4","profile_sidebar_border_color":"C0DEED","profile_sidebar_fill_color":"DDEEF6","profile_text_color":"333333","profile_use_background_image":true,"default_profile":true,"default_profile_image":false,"following":false,"follow_request_sent":false,"notifications":false},"created_at":"Sun Oct 27 20:35:25 +0000 2013"}

data/spec/helper.rb

@@ -8,7 +8,6 @@ SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
 SimpleCov.start
 
 require 'twitter'
-require 'twitter/identity_map'
 require 'rspec'
 require 'stringio'
 require 'tempfile'
@@ -21,42 +20,38 @@ RSpec.configure do |config|
   config.expect_with :rspec do |c|
     c.syntax = :expect
   end
-
-  config.before(:each) do
-    stub_post("/oauth2/token").with(:body => "grant_type=client_credentials").to_return(:body => fixture("bearer_token.json"), :headers => {:content_type => "application/json; charset=utf-8"})
-  end
 end
 
 def a_delete(path)
-  a_request(:delete, Twitter::Default::ENDPOINT + path)
+  a_request(:delete, Twitter::REST::Client::ENDPOINT + path)
 end
 
 def a_get(path)
-  a_request(:get, Twitter::Default::ENDPOINT + path)
+  a_request(:get, Twitter::REST::Client::ENDPOINT + path)
 end
 
 def a_post(path)
-  a_request(:post, Twitter::Default::ENDPOINT + path)
+  a_request(:post, Twitter::REST::Client::ENDPOINT + path)
 end
 
 def a_put(path)
-  a_request(:put, Twitter::Default::ENDPOINT + path)
+  a_request(:put, Twitter::REST::Client::ENDPOINT + path)
 end
 
 def stub_delete(path)
-  stub_request(:delete, Twitter::Default::ENDPOINT + path)
+  stub_request(:delete, Twitter::REST::Client::ENDPOINT + path)
 end
 
 def stub_get(path)
-  stub_request(:get, Twitter::Default::ENDPOINT + path)
+  stub_request(:get, Twitter::REST::Client::ENDPOINT + path)
 end
 
 def stub_post(path)
-  stub_request(:post, Twitter::Default::ENDPOINT + path)
+  stub_request(:post, Twitter::REST::Client::ENDPOINT + path)
 end
 
 def stub_put(path)
-  stub_request(:put, Twitter::Default::ENDPOINT + path)
+  stub_request(:put, Twitter::REST::Client::ENDPOINT + path)
 end
 
 def fixture_path

data/spec/twitter/base_spec.rb

@@ -1,116 +1,26 @@
 require 'helper'
 
 describe Twitter::Base do
-
-  context "identity map enabled" do
-    before do
-      Twitter.identity_map = Twitter::IdentityMap
-      object = Twitter::Base.new(:id => 1)
-      @base = Twitter::Base.store(object)
-    end
-
-    after do
-      Twitter.identity_map = false
-    end
-
-    describe ".identity_map" do
-      it "returns an instance of the identity map" do
-        expect(Twitter::Base.identity_map).to be_a Twitter::IdentityMap
-      end
-    end
-
-    describe ".fetch" do
-      it "returns existing objects" do
-        expect(Twitter::Base.fetch(:id => 1)).to be
-      end
-
-      it "raises an error on objects that don't exist" do
-        expect{Twitter::Base.fetch(:id => 6)}.to raise_error Twitter::Error::IdentityMapKeyError
-      end
-    end
-
-    describe ".store" do
-      it "stores Twitter::Base objects" do
-        object = Twitter::Base.new(:id => 4)
-        expect(Twitter::Base.store(object)).to be_a Twitter::Base
-      end
-    end
-
-    describe ".fetch_or_new" do
-      it "returns existing objects" do
-        expect(Twitter::Base.fetch_or_new(:id => 1)).to be
-      end
-      it "creates new objects and stores them" do
-        expect(Twitter::Base.fetch_or_new(:id => 2)).to be
-        expect(Twitter::Base.fetch(:id => 2)).to be
-      end
-    end
-
-    describe "#[]" do
-      it "calls methods using [] with symbol" do
-        expect(@base[:object_id]).to be_an Integer
-      end
-      it "calls methods using [] with string" do
-        expect(@base['object_id']).to be_an Integer
-      end
-      it "returns nil for missing method" do
-        expect(@base[:foo]).to be_nil
-        expect(@base['foo']).to be_nil
-      end
-    end
-
-    describe "#to_hash" do
-      it "returns a hash" do
-        expect(@base.to_hash).to be_a Hash
-        expect(@base.to_hash[:id]).to eq 1
-      end
-    end
-
-    describe "identical objects" do
-      it "have the same object_id" do
-        expect(@base.object_id).to eq Twitter::Base.fetch(:id => 1).object_id
-      end
-    end
-
+  before do
+    @base = Twitter::Base.new(:id => 1)
   end
 
-  context "identity map disabled" do
-    before(:all) do
-      Twitter.identity_map = false
+  describe "#[]" do
+    it "calls methods using [] with symbol" do
+      expect(@base[:object_id]).to be_an Integer
     end
-    after(:all) do
-      Twitter.identity_map = Twitter::IdentityMap
+    it "calls methods using [] with string" do
+      expect(@base["object_id"]).to be_an Integer
     end
-
-    describe ".identity_map" do
-      it "returns nil" do
-        expect(Twitter::Base.identity_map).to be_nil
-      end
-    end
-
-    describe ".fetch" do
-      it "returns nil" do
-        expect(Twitter::Base.fetch(:id => 1)).to be_nil
-      end
-    end
-
-    describe ".store" do
-      it "returns an instance of the object" do
-        expect(Twitter::Base.store(Twitter::Base.new(:id => 1))).to be_a Twitter::Base
-      end
-    end
-
-    describe ".fetch_or_new" do
-      it "creates new objects" do
-        expect(Twitter::Base.fetch_or_new(:id => 2)).to be
-        expect(Twitter.identity_map).to be_false
-      end
+    it "returns nil for missing method" do
+      expect(@base[:foo]).to be_nil
+      expect(@base["foo"]).to be_nil
     end
   end
 
-  describe '#attrs' do
-    it 'returns a hash of attributes' do
-      expect(Twitter::Base.new(:id => 1).attrs).to eq({:id => 1})
+  describe "#attrs" do
+    it "returns a hash of attributes" do
+      expect(@base.attrs).to eq({:id => 1})
     end
   end
 

data/spec/twitter/basic_user_spec.rb

@@ -6,17 +6,17 @@ describe Twitter::BasicUser do
     it "returns true when objects IDs are the same" do
       saved_search = Twitter::BasicUser.new(:id => 1, :name => "foo")
       other = Twitter::BasicUser.new(:id => 1, :name => "bar")
-      expect(saved_search == other).to be_true
+      expect(saved_search == other).to be true
     end
     it "returns false when objects IDs are different" do
       saved_search = Twitter::BasicUser.new(:id => 1)
       other = Twitter::BasicUser.new(:id => 2)
-      expect(saved_search == other).to be_false
+      expect(saved_search == other).to be false
     end
     it "returns false when classes are different" do
       saved_search = Twitter::BasicUser.new(:id => 1)
       other = Twitter::Identity.new(:id => 1)
-      expect(saved_search == other).to be_false
+      expect(saved_search == other).to be false
     end
   end
 

data/spec/twitter/configuration_spec.rb

@@ -4,7 +4,7 @@ describe Twitter::Configuration do
 
   describe "#photo_sizes" do
     it "returns a hash of sizes when photo_sizes is set" do
-      photo_sizes = Twitter::Configuration.new(:photo_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'}}).photo_sizes
+      photo_sizes = Twitter::Configuration.new(:photo_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"}}).photo_sizes
       expect(photo_sizes).to be_a Hash
       expect(photo_sizes[:small]).to be_a Twitter::Size
     end

data/spec/twitter/cursor_spec.rb

@@ -2,34 +2,9 @@ require 'helper'
 
 describe Twitter::Cursor do
 
-  describe "#collection" do
-    it "returns a collection" do
-      collection = Twitter::Cursor.new({:ids => [1, 2, 3, 4, 5]}, :ids, nil, Twitter::Client.new, :follower_ids, {}).collection
-      expect(collection).to be_an Array
-      expect(collection.first).to be_a Fixnum
-    end
-  end
-
-  describe "#all" do
-    before do
-      @client = Twitter::Client.new
-      stub_get("/1.1/followers/ids.json").with(:query => {:cursor => "-1", :screen_name => "sferik"}).to_return(:body => fixture("ids_list.json"), :headers => {:content_type => "application/json; charset=utf-8"})
-      stub_get("/1.1/followers/ids.json").with(:query => {:cursor => "1305102810874389703", :screen_name => "sferik"}).to_return(:body => fixture("ids_list2.json"), :headers => {:content_type => "application/json; charset=utf-8"})
-    end
-    it "requests the correct resources" do
-      @client.follower_ids("sferik").all
-      expect(a_get("/1.1/followers/ids.json").with(:query => {:cursor => "-1", :screen_name => "sferik"})).to have_been_made
-      expect(a_get("/1.1/followers/ids.json").with(:query => {:cursor => "1305102810874389703", :screen_name => "sferik"})).to have_been_made
-    end
-    it "fetches all" do
-      follower_ids = @client.follower_ids("sferik").all
-      expect(follower_ids.size).to eq 6
-    end
-  end
-
   describe "#each" 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")
       stub_get("/1.1/followers/ids.json").with(:query => {:cursor => "-1", :screen_name => "sferik"}).to_return(:body => fixture("ids_list.json"), :headers => {:content_type => "application/json; charset=utf-8"})
       stub_get("/1.1/followers/ids.json").with(:query => {:cursor => "1305102810874389703", :screen_name => "sferik"}).to_return(:body => fixture("ids_list2.json"), :headers => {:content_type => "application/json; charset=utf-8"})
     end
@@ -41,25 +16,32 @@ describe Twitter::Cursor do
     it "iterates" do
       count = 0
       @client.follower_ids("sferik").each{count += 1}
-      expect(count).to eq 6
+      expect(count).to eq(6)
+    end
+    context "with start" do
+      it "iterates" do
+        count = 0
+        @client.follower_ids("sferik").each(5){count += 1}
+        expect(count).to eq(1)
+      end
     end
   end
 
   describe "#first?" do
     context "when previous cursor equals zero" do
       before do
-        @cursor = Twitter::Cursor.new({:previous_cursor => 0}, :ids, nil, Twitter::Client.new, :follower_ids, {})
+        @cursor = Twitter::Cursor.new({:previous_cursor => 0}, :ids, nil, Twitter::REST::Client.new, :get, "/1.1/followers/ids.json", {})
       end
       it "returns true" do
-        expect(@cursor.first?).to be_true
+        expect(@cursor.first?).to be true
       end
     end
     context "when previous cursor does not equal zero" do
       before do
-        @cursor = Twitter::Cursor.new({:previous_cursor => 1}, :ids, nil, Twitter::Client.new, :follower_ids, {})
+        @cursor = Twitter::Cursor.new({:previous_cursor => 1}, :ids, nil, Twitter::REST::Client.new, :get, "/1.1/followers/ids.json", {})
       end
       it "returns true" do
-        expect(@cursor.first?).to be_false
+        expect(@cursor.first?).to be false
       end
     end
   end
@@ -67,18 +49,18 @@ describe Twitter::Cursor do
   describe "#last?" do
     context "when next cursor equals zero" do
       before do
-        @cursor = Twitter::Cursor.new({:next_cursor => 0}, :ids, nil, Twitter::Client.new, :follower_ids, {})
+        @cursor = Twitter::Cursor.new({:next_cursor => 0}, :ids, nil, Twitter::REST::Client.new, :get, "/1.1/followers/ids.json", {})
       end
       it "returns true" do
-        expect(@cursor.last?).to be_true
+        expect(@cursor.last?).to be true
       end
     end
     context "when next cursor does not equal zero" do
       before do
-        @cursor = Twitter::Cursor.new({:next_cursor => 1}, :ids, nil, Twitter::Client.new, :follower_ids, {})
+        @cursor = Twitter::Cursor.new({:next_cursor => 1}, :ids, nil, Twitter::REST::Client.new, :get, "/1.1/followers/ids.json", {})
       end
       it "returns false" do
-        expect(@cursor.last?).to be_false
+        expect(@cursor.last?).to be false
       end
     end
   end

data/spec/twitter/direct_message_spec.rb

@@ -6,17 +6,17 @@ describe Twitter::DirectMessage do
     it "returns true when objects IDs are the same" do
       direct_message = Twitter::DirectMessage.new(:id => 1, :text => "foo")
       other = Twitter::DirectMessage.new(:id => 1, :text => "bar")
-      expect(direct_message == other).to be_true
+      expect(direct_message == other).to be true
     end
     it "returns false when objects IDs are different" do
       direct_message = Twitter::DirectMessage.new(:id => 1)
       other = Twitter::DirectMessage.new(:id => 2)
-      expect(direct_message == other).to be_false
+      expect(direct_message == other).to be false
     end
     it "returns false when classes are different" do
       direct_message = Twitter::DirectMessage.new(:id => 1)
       other = Twitter::Identity.new(:id => 1)
-      expect(direct_message == other).to be_false
+      expect(direct_message == other).to be false
     end
   end
 
@@ -31,25 +31,58 @@ describe Twitter::DirectMessage do
     end
   end
 
+  describe "#created?" do
+    it "returns true when created_at is set" do
+      direct_message = Twitter::DirectMessage.new(:id => 1825786345, :created_at => "Mon Jul 16 12:59:01 +0000 2007")
+      expect(direct_message.created?).to be true
+    end
+    it "returns false when created_at is not set" do
+      direct_message = Twitter::DirectMessage.new(:id => 1825786345)
+      expect(direct_message.created?).to be false
+    end
+  end
+
   describe "#recipient" do
     it "returns a User when recipient is set" do
-      recipient = Twitter::DirectMessage.new(:id => 1825786345, :recipient => {:id => 7505382}).recipient
-      expect(recipient).to be_a Twitter::User
+      direct_message = Twitter::DirectMessage.new(:id => 1825786345, :recipient => {:id => 7505382})
+      expect(direct_message.recipient).to be_a Twitter::User
     end
     it "returns nil when recipient is not set" do
-      recipient = Twitter::DirectMessage.new(:id => 1825786345).recipient
-      expect(recipient).to be_nil
+      direct_message = Twitter::DirectMessage.new(:id => 1825786345)
+      expect(direct_message.recipient).to be_nil
+    end
+  end
+
+  describe "#recipient?" do
+    it "returns true when recipient is set" do
+      direct_message = Twitter::DirectMessage.new(:id => 1825786345, :recipient => {:id => 7505382})
+      expect(direct_message.recipient?).to be true
+    end
+    it "returns false when recipient is not set" do
+      direct_message = Twitter::DirectMessage.new(:id => 1825786345)
+      expect(direct_message.recipient?).to be false
     end
   end
 
   describe "#sender" do
     it "returns a User when sender is set" do
-      sender = Twitter::DirectMessage.new(:id => 1825786345, :sender => {:id => 7505382}).sender
-      expect(sender).to be_a Twitter::User
+      direct_message = Twitter::DirectMessage.new(:id => 1825786345, :sender => {:id => 7505382})
+      expect(direct_message.sender).to be_a Twitter::User
     end
     it "returns nil when sender is not set" do
-      sender = Twitter::DirectMessage.new(:id => 1825786345).sender
-      expect(sender).to be_nil
+      direct_message = Twitter::DirectMessage.new(:id => 1825786345)
+      expect(direct_message.sender).to be_nil
+    end
+  end
+
+  describe "#sender?" do
+    it "returns true when sender is set" do
+      direct_message = Twitter::DirectMessage.new(:id => 1825786345, :sender => {:id => 7505382})
+      expect(direct_message.sender?).to be true
+    end
+    it "returns false when sender is not set" do
+      direct_message = Twitter::DirectMessage.new(:id => 1825786345)
+      expect(direct_message.sender?).to be false
     end
   end
 

data/spec/twitter/entity/uri_spec.rb

@@ -0,0 +1,75 @@
+# coding: utf-8
+require 'helper'
+
+describe Twitter::Entity::URI do
+
+  describe "#display_uri" do
+    it "returns a String when the display_url is set" do
+      uri = Twitter::Entity::URI.new(:display_url => "example.com/expanded…")
+      expect(uri.display_uri).to be_a String
+      expect(uri.display_uri).to eq("example.com/expanded…")
+    end
+    it "returns nil when the display_url is not set" do
+      uri = Twitter::Entity::URI.new
+      expect(uri.display_uri).to be_nil
+    end
+  end
+
+  describe "#display_uri?" do
+    it "returns true when the display_url is set" do
+      uri = Twitter::Entity::URI.new(:display_url => "example.com/expanded…")
+      expect(uri.display_uri?).to be true
+    end
+    it "returns false when the display_url is not set" do
+      uri = Twitter::Entity::URI.new
+      expect(uri.display_uri?).to be false
+    end
+  end
+
+  describe "#expanded_uri" do
+    it "returns a URI when the expanded_url is set" do
+      uri = Twitter::Entity::URI.new(:expanded_url => "https://github.com/sferik")
+      expect(uri.expanded_uri).to be_a URI
+      expect(uri.expanded_uri.to_s).to eq("https://github.com/sferik")
+    end
+    it "returns nil when the expanded_url is not set" do
+      uri = Twitter::Entity::URI.new
+      expect(uri.expanded_uri).to be_nil
+    end
+  end
+
+  describe "#expanded_uri?" do
+    it "returns true when the expanded_url is set" do
+      uri = Twitter::Entity::URI.new(:expanded_url => "https://github.com/sferik")
+      expect(uri.expanded_uri?).to be true
+    end
+    it "returns false when the expanded_url is not set" do
+      uri = Twitter::Entity::URI.new
+      expect(uri.expanded_uri?).to be false
+    end
+  end
+
+  describe "#uri" do
+    it "returns a URI when the url is set" do
+      uri = Twitter::Entity::URI.new(:url => "https://github.com/sferik")
+      expect(uri.uri).to be_a URI
+      expect(uri.uri.to_s).to eq("https://github.com/sferik")
+    end
+    it "returns nil when the url is not set" do
+      uri = Twitter::Entity::URI.new
+      expect(uri.uri).to be_nil
+    end
+  end
+
+  describe "#uri?" do
+    it "returns true when the url is set" do
+      uri = Twitter::Entity::URI.new(:url => "https://github.com/sferik")
+      expect(uri.uri?).to be true
+    end
+    it "returns false when the url is not set" do
+      uri = Twitter::Entity::URI.new
+      expect(uri.uri?).to be false
+    end
+  end
+
+end

data/spec/twitter/error_spec.rb

@@ -2,19 +2,67 @@ require 'helper'
 
 describe Twitter::Error do
 
-  describe "#initialize" do
-    it "wraps another error class" do
-      begin
-        raise Faraday::Error::ClientError.new("Oops")
-      rescue Faraday::Error::ClientError
-        begin
-          raise Twitter::Error
-        rescue Twitter::Error => error
-          expect(error.message).to eq "Oops"
-          expect(error.wrapped_exception.class).to eq Faraday::Error::ClientError
+  before do
+    @client = Twitter::REST::Client.new(:consumer_key => "CK", :consumer_secret => "CS", :access_token => "AT", :access_token_secret => "AS")
+  end
+
+  describe "#message" do
+    it "returns the message of the wrapped exception" do
+      error = Twitter::Error.new(Faraday::Error::ClientError.new("Oops"))
+      expect(error.message).to eq("Oops")
+    end
+  end
+
+  describe "#rate_limit" do
+    it "returns the wrapped exception" do
+      error = Twitter::Error.new(Faraday::Error::ClientError.new("Oops"))
+      expect(error.rate_limit).to be_a Twitter::RateLimit
+    end
+  end
+
+  describe "#wrapped_exception" do
+    it "returns the wrapped exception" do
+      error = Twitter::Error.new(Faraday::Error::ClientError.new("Oops"))
+      expect(error.wrapped_exception.class).to eq(Faraday::Error::ClientError)
+    end
+  end
+
+  Twitter::Error.errors.each do |status, exception|
+
+    [nil, "error", "errors"].each do |body|
+      context "when HTTP status is #{status} and body is #{body.inspect}" do
+        before do
+          body_message = '{"' + body + '":"Client Error"}' unless body.nil?
+          stub_get("/1.1/statuses/user_timeline.json").with(:query => {:screen_name => "sferik"}).to_return(:status => status, :body => body_message)
+        end
+        it "raises #{exception.name}" do
+          expect{@client.user_timeline("sferik")}.to raise_error exception
         end
       end
     end
-  end
 
+    context "when HTTP status is #{status} and body is errors" do
+      context "when errors is an array of hashes" do
+        context "when error code is nil" do
+          before do
+            body_message = '{"errors":[{"message":"Client Error"}]}'
+            stub_get("/1.1/statuses/user_timeline.json").with(:query => {:screen_name => "sferik"}).to_return(:status => status, :body => body_message)
+          end
+          it "raises #{exception.name}" do
+            expect{@client.user_timeline("sferik")}.to raise_error{|error| expect(error.code).to be_nil}
+          end
+          context "when error code is 187" do
+            before do
+              body_message = '{"errors":[{"message":"Client Error","code":187}]}'
+              stub_get("/1.1/statuses/user_timeline.json").with(:query => {:screen_name => "sferik"}).to_return(:status => status, :body => body_message)
+            end
+            it "raises #{exception.name}" do
+              expect{@client.user_timeline("sferik")}.to raise_error{|error| expect(error.code).to eq(187)}
+            end
+          end
+        end
+      end
+    end
+
+  end
 end

data/spec/twitter/geo/point_spec.rb

@@ -7,22 +7,22 @@ describe Twitter::Geo::Point do
   end
 
   describe "#==" do
-    it "returns false for empty objects" do
+    it "returns true for empty objects" do
       point = Twitter::Geo::Point.new
       other = Twitter::Geo::Point.new
-      expect(point == other).to be_false
+      expect(point == other).to be true
     end
     it "returns true when objects coordinates are the same" do
       other = Twitter::Geo::Point.new(:coordinates => [-122.399983, 37.788299])
-      expect(@point == other).to be_true
+      expect(@point == other).to be true
     end
     it "returns false when objects coordinates are different" do
       other = Twitter::Geo::Point.new(:coordinates => [37.788299, -122.399983])
-      expect(@point == other).to be_false
+      expect(@point == other).to be false
     end
     it "returns false when classes are different" do
       other = Twitter::Geo.new(:coordinates => [-122.399983, 37.788299])
-      expect(@point == other).to be_false
+      expect(@point == other).to be false
     end
   end
 
@@ -34,7 +34,7 @@ describe Twitter::Geo::Point do
 
   describe "#longitude" do
     it "returns the longitude" do
-      expect(@point.longitude).to eq 37.788299
+      expect(@point.longitude).to eq(37.788299)
     end
   end