mbleigh-twitter-auth 0.0.2 → 0.1.1

data/spec/fixtures/config/twitter_auth.yml

@@ -0,0 +1,17 @@
+development:
+  strategy: oauth
+  base_url: "https://twitter.com"
+  oauth_consumer_key: devkey
+  oauth_consumer_secret: devsecret
+  oauth_callback: "http://localhost:3000"
+test:
+  strategy: oauth
+  base_url: "https://twitter.com"
+  oauth_consumer_key: testkey
+  oauth_consumer_secret: testsecret
+production:
+  strategy: oauth
+  base_url: "https://twitter.com"
+  oauth_consumer_key: prodkey
+  oauth_consumer_secret: prodsecret
+

data/spec/fixtures/factories.rb

@@ -0,0 +1,18 @@
+require 'factory_girl'
+
+Factory.define(:twitter_oauth_user, :class => User) do |u|
+  u.login 'twitterman'
+  u.access_token 'fakeaccesstoken'
+  u.access_secret 'fakeaccesstokensecret'
+  
+  u.name 'Twitter Man'
+  u.description 'Saving the world for all Twitter kind.'
+end
+
+Factory.define(:twitter_basic_user, :class => User) do |u|
+  u.login 'twitterman'
+  u.password 'test'
+
+  u.name 'Twitter Man'
+  u.description 'Saving the world for all Twitter kind.'
+end

data/spec/fixtures/fakeweb.rb

@@ -0,0 +1,18 @@
+# This is where we fake out all of the URLs that we
+# will be calling as a part of this spec suite.
+# You must have the 'fakeweb' gem in order to run
+# the tests for TwitterAuth.
+#
+# gem install 'mbleigh-fakeweb'
+
+require 'fake_web'
+
+FakeWeb.allow_net_connect = false
+
+FakeWeb.register_uri(:post, 'https://twitter.com:443/oauth/request_token', :string => 'oauth_token=faketoken&oauth_token_secret=faketokensecret')
+
+FakeWeb.register_uri(:post, 'https://twitter.com:443/oauth/access_token', :string => 'oauth_token=fakeaccesstoken&oauth_token_secret=fakeaccesstokensecret')
+
+FakeWeb.register_uri(:get, 'https://twitter.com:443/account/verify_credentials.json', :string => "{\"profile_image_url\":\"http:\\/\\/static.twitter.com\\/images\\/default_profile_normal.png\",\"description\":\"Saving the world for all Twitter kind.\",\"utc_offset\":null,\"favourites_count\":0,\"profile_sidebar_fill_color\":\"e0ff92\",\"screen_name\":\"twitterman\",\"statuses_count\":0,\"profile_background_tile\":false,\"profile_sidebar_border_color\":\"87bc44\",\"friends_count\":2,\"url\":null,\"name\":\"Twitter Man\",\"time_zone\":null,\"protected\":false,\"profile_background_image_url\":\"http:\\/\\/static.twitter.com\\/images\\/themes\\/theme1\\/bg.gif\",\"profile_background_color\":\"9ae4e8\",\"created_at\":\"Fri Feb 06 18:10:32 +0000 2009\",\"profile_text_color\":\"000000\",\"followers_count\":2,\"location\":null,\"id\":20256865,\"profile_link_color\":\"0000ff\"}")
+
+#FakeWeb.register_uri(:get, 'https://twitter.com:443/)

data/spec/fixtures/twitter.rb

@@ -0,0 +1,5 @@
+require 'net/http'
+
+TWITTER_JSON = {
+  :verify_credentials => "{\"profile_image_url\":\"http:\\/\\/static.twitter.com\\/images\\/default_profile_normal.png\",\"description\":\"Saving the world for all Twitter kind.\",\"utc_offset\":null,\"favourites_count\":0,\"profile_sidebar_fill_color\":\"e0ff92\",\"screen_name\":\"twitterman\",\"statuses_count\":0,\"profile_background_tile\":false,\"profile_sidebar_border_color\":\"87bc44\",\"friends_count\":2,\"url\":null,\"name\":\"Twitter Man\",\"time_zone\":null,\"protected\":false,\"profile_background_image_url\":\"http:\\/\\/static.twitter.com\\/images\\/themes\\/theme1\\/bg.gif\",\"profile_background_color\":\"9ae4e8\",\"created_at\":\"Fri Feb 06 18:10:32 +0000 2009\",\"profile_text_color\":\"000000\",\"followers_count\":2,\"location\":null,\"id\":20256865,\"profile_link_color\":\"0000ff\"}"
+}

data/spec/models/twitter_auth/basic_user_spec.rb

@@ -0,0 +1,122 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe TwitterAuth::BasicUser do
+  before do
+    stub_basic!
+  end
+
+  describe '#password=' do
+    before do
+      @user = Factory.build(:twitter_basic_user)
+    end
+
+    it 'should change the value of crypted_password' do
+      lambda{@user.password = 'newpass'}.should change(@user, :crypted_password)
+    end
+
+    it 'should change the value of salt' do
+      lambda{@user.password = 'newpass'}.should change(@user, :salt)
+    end
+
+    it 'should not store the plaintext password' do
+      @user.password = 'newpass'
+      @user.crypted_password.should_not == 'newpass'
+    end
+  end
+
+  describe '#password' do
+    before do
+      @user = Factory.build(:twitter_basic_user, :password => 'monkey')
+    end
+
+    it 'should return the password' do
+      @user.password.should == 'monkey'
+    end    
+
+    it 'should not be a database attribute' do
+      @user['password'].should_not == 'monkey'
+    end
+  end
+  
+  describe '.verify_credentials' do
+    before do
+      @user = Factory.create(:twitter_basic_user)
+    end
+
+    it 'should return a JSON hash of the user when successful' do
+      hash = User.verify_credentials('twitterman','test')
+      hash.should be_a(Hash)
+      hash['screen_name'].should == 'twitterman'
+      hash['name'].should == 'Twitter Man'
+    end
+
+    it 'should return false when a 401 unauthorized happens' do
+      FakeWeb.register_uri(:get, 'https://twitter.com:443/account/verify_credentials.json', :string => '401 "Unauthorized"', :status => ['401',' Unauthorized'])
+      User.verify_credentials('twitterman','wrong').should be_false
+    end
+  end
+
+  describe '.authenticate' do
+    before do
+      @user = Factory.create(:twitter_basic_user)
+    end
+
+    it 'should make a call to verify_credentials' do
+      User.should_receive(:verify_credentials).with('twitterman','test')
+      User.authenticate('twitterman','test')
+    end
+
+    it 'should return nil if verify_credentials returns false' do
+      User.stub!(:verify_credentials).and_return(false)
+      User.authenticate('twitterman','test').should be_nil
+    end
+
+    it 'should return the user if verify_credentials succeeds' do
+      User.stub!(:verify_credentials).and_return(JSON.parse("{\"profile_image_url\":\"http:\\/\\/static.twitter.com\\/images\\/default_profile_normal.png\",\"description\":\"Saving the world for all Twitter kind.\",\"utc_offset\":null,\"favourites_count\":0,\"profile_sidebar_fill_color\":\"e0ff92\",\"screen_name\":\"twitterman\",\"statuses_count\":0,\"profile_background_tile\":false,\"profile_sidebar_border_color\":\"87bc44\",\"friends_count\":2,\"url\":null,\"name\":\"Twitter Man\",\"time_zone\":null,\"protected\":false,\"profile_background_image_url\":\"http:\\/\\/static.twitter.com\\/images\\/themes\\/theme1\\/bg.gif\",\"profile_background_color\":\"9ae4e8\",\"created_at\":\"Fri Feb 06 18:10:32 +0000 2009\",\"profile_text_color\":\"000000\",\"followers_count\":2,\"location\":null,\"id\":20256865,\"profile_link_color\":\"0000ff\"}"))
+      User.authenticate('twitterman','test').should == @user
+    end
+  end
+
+  describe '.find_or_create_by_twitter_hash_and_password' do
+    before do
+      @user = Factory.create(:twitter_basic_user)
+    end
+
+    it 'should return the existing user if there is one' do
+      User.identify_or_create_from_twitter_hash_and_password({'screen_name' => 'twitterman'},'test').should == @user
+    end
+
+    it 'should update the attributes from the hash' do
+      User.identify_or_create_from_twitter_hash_and_password({'screen_name' => 'twitterman', 'name' => 'New Name'}, 'test').name.should == 'New Name'
+    end
+
+    it 'should update the password from the argument' do
+      User.identify_or_create_from_twitter_hash_and_password({'screen_name' => 'twitterman', 'name' => 'New Name'}, 'test2').password.should == 'test2'
+    end
+
+    it 'should create a user if one does not exist' do
+      lambda{User.identify_or_create_from_twitter_hash_and_password({'screen_name' => 'dude', 'name' => "Lebowski"}, 'test')}.should change(User, :count).by(1)
+    end
+
+    it 'should assign the attributes from the hash to a created user' do
+      user = User.identify_or_create_from_twitter_hash_and_password({'screen_name' => 'dude', 'name' => "Lebowski"}, 'test')
+      user.login.should == 'dude'
+      user.name.should == 'Lebowski'
+      user.password.should == 'test'
+    end
+  end
+
+  describe '#twitter' do
+    before do
+      @user = Factory.create(:twitter_basic_user)
+    end
+
+    it 'should be an instance of TwitterAuth::Dispatcher::Basic' do
+      @user.twitter.class.should == TwitterAuth::Dispatcher::Basic
+    end
+
+    it 'should have the correct user set' do
+      @user.twitter.user.should == @user
+    end
+  end
+end

data/spec/models/twitter_auth/generic_user_spec.rb

@@ -0,0 +1,48 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe TwitterAuth::GenericUser do
+  should_validate_presence_of :login
+  should_validate_format_of :login, 'some_guy', 'awesome', 'cool_man'
+  should_not_validate_format_of :login, 'with-dashes', 'with.periods', 'with spaces'
+  should_validate_length_of :login, :in => 1..15
+ 
+  it 'should validate uniqueness of login' do
+    Factory.create(:twitter_oauth_user)
+    Factory.build(:twitter_oauth_user).should have_at_least(1).errors_on(:login)
+  end
+
+  describe '.new_from_twitter_hash' do
+    it 'should raise an argument error if the hash does not have a screen_name attribute' do
+      lambda{User.new_from_twitter_hash({})}.should raise_error(ArgumentError, 'Invalid hash: must include screen_name.')
+    end
+
+    it 'should return a user' do
+      User.new_from_twitter_hash({'screen_name' => 'twitterman'}).should be_a(User)
+    end
+
+    it 'should assign login to the screen_name' do
+      User.new_from_twitter_hash({'screen_name' => 'twitterman'}).login.should == 'twitterman'
+    end
+
+    it 'should assign twitter attributes that are provided' do
+      u = User.new_from_twitter_hash({'screen_name' => 'twitterman', 'name' => 'Twitter Man', 'description' => 'Saving the world for all Tweet kind.'})
+      u.name.should == 'Twitter Man'
+      u.description.should == 'Saving the world for all Tweet kind.'
+    end
+  end
+
+  describe '#update_twitter_attributes' do
+    it 'should assign values to the user' do
+      user = Factory.create(:twitter_oauth_user, :name => "Dude", :description => "Awesome, man.")
+      user.update_twitter_attributes({'name' => 'Twitter Man', 'description' => 'Works.'})
+      user.reload
+      user.name.should == 'Twitter Man'
+      user.description.should == 'Works.'
+    end
+
+    it 'should not throw an error with extraneous info' do
+      user = Factory.create(:twitter_oauth_user, :name => "Dude", :description => "Awesome, man.")
+      lambda{user.update_twitter_attributes({'name' => 'Twitter Man', 'description' => 'Works.', 'whoopsy' => 'noworks.'})}.should_not raise_error
+    end
+  end
+end

data/spec/models/twitter_auth/oauth_user_spec.rb

@@ -0,0 +1,85 @@
+require File.dirname(__FILE__) + '/../../spec_helper'
+
+describe TwitterAuth::OauthUser do
+  before do
+    stub_oauth!
+  end
+ 
+  describe '.identify_or_create_from_access_token' do
+    before do
+       @token = OAuth::AccessToken.new(TwitterAuth.consumer, 'faketoken', 'fakesecret')
+    end
+
+    it 'should accept an OAuth::AccessToken' do
+      lambda{ User.identify_or_create_from_access_token(@token) }.should_not raise_error(ArgumentError)
+    end
+
+    it 'should accept two strings' do
+      lambda{ User.identify_or_create_from_access_token('faketoken', 'fakesecret') }.should_not raise_error(ArgumentError)
+    end
+
+    it 'should not accept one string' do
+      lambda{ User.identify_or_create_from_access_token('faketoken') }.should raise_error(ArgumentError, 'Must authenticate with an OAuth::AccessToken or the string access token and secret.')
+    end
+
+    it 'should make a call to verify_credentials' do
+      # this is in the before, just making it explicit
+      User.identify_or_create_from_access_token(@token)
+    end
+
+    it 'should try to find the user with that login' do
+      User.should_receive(:find_by_login).once.with('twitterman')
+      User.identify_or_create_from_access_token(@token)
+    end
+
+    it 'should return the user if he/she exists' do
+      user = Factory.create(:twitter_oauth_user, :login => 'twitterman')
+      User.identify_or_create_from_access_token(@token).should == user
+    end
+
+    it 'should update the user\'s attributes based on the twitter info' do
+      user = Factory.create(:twitter_oauth_user, :login => 'twitterman', :name => 'Not Twitter Man')
+      User.identify_or_create_from_access_token(@token).name.should == 'Twitter Man'
+    end
+
+    it 'should create a user if one does not exist' do
+      lambda{User.identify_or_create_from_access_token(@token)}.should change(User, :count).by(1)
+    end
+
+    it 'should assign the oauth access token and secret' do
+      user = User.identify_or_create_from_access_token(@token)
+      user.access_token.should == @token.token
+      user.access_secret.should == @token.secret
+    end
+  end
+
+  describe '#token' do
+    before do
+      @user = Factory.create(:twitter_oauth_user, :access_token => 'token', :access_secret => 'secret')
+    end
+
+    it 'should return an AccessToken' do
+      @user.token.should be_a(OAuth::AccessToken)
+    end
+
+    it "should use the user's access_token and secret" do
+      @user.token.token.should == @user.access_token
+      @user.token.secret.should == @user.access_secret
+    end
+  end
+
+  describe '#twitter' do
+    before do
+      @user = Factory.create(:twitter_oauth_user, :access_token => 'token', :access_secret => 'secret')
+    end
+
+    it 'should return a TwitterAuth::Dispatcher::Oauth' do
+      @user.twitter.should be_a(TwitterAuth::Dispatcher::Oauth)
+    end
+
+    it 'should use my token and secret' do
+      @user.twitter.token.should == @user.access_token
+      @user.twitter.secret.should == @user.access_secret
+    end
+  end
+end

data/spec/schema.rb

@@ -0,0 +1,37 @@
+ActiveRecord::Schema.define :version => 0 do
+  create_table :twitter_auth_users, :force => true do |t|
+    t.string :login
+    
+    # OAuth fields
+    t.string :access_token
+    t.string :access_secret
+
+    # Basic fields
+    t.binary :crypted_password
+    t.string :salt
+
+    # This information is automatically kept
+    # in-sync at each login of the user. You
+    # may remove any/all of these columns.
+    t.string :name
+    t.string :location
+    t.string :description
+    t.string :profile_image_url
+    t.string :url
+    t.boolean :protected
+    t.string :profile_background_color
+    t.string :profile_sidebar_fill_color
+    t.string :profile_link_color
+    t.string :profile_sidebar_border_color
+    t.string :profile_text_color
+    t.integer :friends_count
+    t.integer :statuses_count
+    t.integer :followers_count
+    t.integer :favourites_count
+    t.integer :utc_offset
+    t.string :time_zone
+
+    t.timestamps
+  end
+end
+

data/spec/spec.opts

@@ -0,0 +1 @@
+--colour

data/spec/spec_helper.rb

@@ -5,6 +5,47 @@ rescue LoadError
   exit
 end
 
+require File.dirname(__FILE__) + '/../app/models/twitter_auth/generic_user'
+
+class TwitterAuth::GenericUser
+  def self.table_name; 'twitter_auth_users' end
+end
+
+class User < TwitterAuth::GenericUser; end
+
+require 'remarkable'
+require File.dirname(__FILE__) + '/fixtures/factories'
+require File.dirname(__FILE__) + '/fixtures/fakeweb'
+require File.dirname(__FILE__) + '/fixtures/twitter'
+
 plugin_spec_dir = File.dirname(__FILE__)
 ActiveRecord::Base.logger = Logger.new(plugin_spec_dir + "/debug.log")
 
+load(File.dirname(__FILE__) + '/schema.rb')
+
+def define_basic_user_class!
+  TwitterAuth::GenericUser.send :include, TwitterAuth::BasicUser 
+end
+
+def define_oauth_user_class!
+  TwitterAuth::GenericUser.send :include, TwitterAuth::OauthUser  
+end
+
+def stub_oauth!
+  TwitterAuth.stub!(:config).and_return({
+    'strategy' => 'oauth',
+    'oauth_consumer_key' => 'testkey',
+    'oauth_consumer_secret' => 'testsecret'
+  })
+  define_oauth_user_class!
+end
+
+def stub_basic!
+  TwitterAuth.stub!(:config).and_return({
+    'strategy' => 'basic',
+    'encryption_key' => 'secretcode'
+  })
+  define_basic_user_class!
+end
+
+define_oauth_user_class!

data/spec/twitter_auth/cryptify_spec.rb

@@ -0,0 +1,51 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe TwitterAuth::Cryptify do
+  before do
+    stub_basic!
+  end
+
+  it 'should have encrypt and decrypt methods' do
+    TwitterAuth::Cryptify.should respond_to(:encrypt)
+    TwitterAuth::Cryptify.should respond_to(:decrypt)
+  end
+
+  describe '.encrypt' do
+    it 'should return a hash with :encrypted_data and :salt keys' do
+      result = TwitterAuth::Cryptify.encrypt('some string')
+      result.should be_a(Hash)
+      result.key?(:encrypted_data).should be_true
+      result.key?(:salt).should be_true
+    end
+
+    it 'should make a call to EzCrypto::Key.encrypt_with_password' do
+      EzCrypto::Key.should_receive(:encrypt_with_password).once.and_return('gobbledygook')
+      TwitterAuth::Cryptify.encrypt('some string') 
+    end
+
+    it 'should not have the same encrypted as plaintext data' do
+      TwitterAuth::Cryptify.encrypt('some string')[:encrypted_data].should_not == 'some string'
+    end
+  end
+
+  describe '.decrypt' do
+    before do
+      @salt = TwitterAuth::Cryptify.generate_salt
+      TwitterAuth::Cryptify.stub!(:generate_salt).and_return(@salt)
+      @string = 'decrypted string'
+      @encrypted = TwitterAuth::Cryptify.encrypt(@string)
+    end
+
+    it 'should return the original string' do
+      TwitterAuth::Cryptify.decrypt(@encrypted).should == @string
+    end
+
+    it 'should raise an argument error if encrypted data is provided without a salt' do
+      lambda{TwitterAuth::Cryptify.decrypt('asodiaoie2')}.should raise_error(ArgumentError)
+    end
+
+    it 'should raise an argument error if a string or hash are not provided' do
+      lambda{TwitterAuth::Cryptify.decrypt(23)}.should raise_error(ArgumentError)
+    end
+  end
+end