sorcery 0.5.1 → 0.5.2

data/spec/sinatra_modular/spec_modular/controller_brute_force_protection_spec.rb

@@ -0,0 +1,70 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe Modular do
+  before(:all) do
+    ActiveRecord::Migrator.migrate("#{APP_ROOT}/db/migrate/brute_force_protection")
+  end
+  
+  after(:all) do
+    ActiveRecord::Migrator.rollback("#{APP_ROOT}/db/migrate/brute_force_protection")
+  end
+  
+  # ----------------- SESSION TIMEOUT -----------------------
+  describe Modular, "with brute force protection features" do
+    before(:all) do
+      sorcery_reload!([:brute_force_protection])
+    end
+    
+    before(:each) do
+      create_new_user
+    end
+    
+    after(:each) do
+      Sorcery::Controller::Config.reset!
+      sorcery_controller_property_set(:user_class, User)
+      User.delete_all
+      Timecop.return
+    end
+    
+    it "should count login retries" do
+      3.times {get "/test_login", :username => 'gizmo', :password => 'blabla'}
+      User.find_by_username('gizmo').failed_logins_count.should == 3
+    end
+    
+    it "should reset the counter on a good login" do
+      sorcery_model_property_set(:consecutive_login_retries_amount_limit, 5)
+      3.times {get "/test_login", :username => 'gizmo', :password => 'blabla'}
+      get "/test_login", :username => 'gizmo', :password => 'secret'
+      User.find_by_username('gizmo').failed_logins_count.should == 0
+    end
+    
+    it "should lock user when number of retries reached the limit" do
+      User.find_by_username('gizmo').lock_expires_at.should be_nil
+      sorcery_model_property_set(:consecutive_login_retries_amount_limit, 1)
+      get "/test_login", :username => 'gizmo', :password => 'blabla'
+      User.find_by_username('gizmo').lock_expires_at.should_not be_nil
+    end
+
+    it "should unlock after lock time period passes" do
+      sorcery_model_property_set(:consecutive_login_retries_amount_limit, 2)
+      sorcery_model_property_set(:login_lock_time_period, 0.2)
+      get "/test_login", :username => 'gizmo', :password => 'blabla'
+      get "/test_login", :username => 'gizmo', :password => 'blabla'
+      User.find_by_username('gizmo').lock_expires_at.should_not be_nil
+      Timecop.travel(Time.now+0.3)
+      get "/test_login", :username => 'gizmo', :password => 'blabla'
+      User.find_by_username('gizmo').lock_expires_at.should be_nil
+    end
+
+    it "should not unlock if time period is 0 (permanent lock)" do
+      sorcery_model_property_set(:consecutive_login_retries_amount_limit, 2)
+      sorcery_model_property_set(:login_lock_time_period, 0)
+      get "/test_login", :username => 'gizmo', :password => 'blabla'
+      get "/test_login", :username => 'gizmo', :password => 'blabla'
+      unlock_date = User.find_by_username('gizmo').lock_expires_at
+      Timecop.travel(Time.now+1)
+      get "/test_login", :username => 'gizmo', :password => 'blabla'
+      User.find_by_username('gizmo').lock_expires_at.to_s.should == unlock_date.to_s
+    end
+  end
+end

data/spec/sinatra_modular/spec_modular/controller_http_basic_auth_spec.rb

@@ -0,0 +1,53 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'base64'
+
+describe Modular do
+  
+  # ----------------- HTTP BASIC AUTH -----------------------
+  describe Modular, "with http basic auth features" do
+    before(:all) do
+      sorcery_reload!([:http_basic_auth])
+      create_new_user
+    end
+    
+    after(:each) do
+      get "/test_logout"
+    end
+    
+    it "requests basic authentication when before_filter is used" do
+      session[:http_authentication_used] = nil
+      get "/test_http_basic_auth"
+      last_response.status.should == 401
+      session[:http_authentication_used].should == true
+    end
+    
+    it "authenticates from http basic if credentials are sent" do
+      session[:http_authentication_used] = true
+      get "/test_http_basic_auth", {}, {"HTTP_AUTHORIZATION" => "Basic " + Base64::encode64("#{@user.username}:secret")}
+      last_response.should be_ok
+    end
+    
+    it "fails authentication if credentials are wrong" do
+      session[:http_authentication_used] = true
+      get "/test_http_basic_auth", {}, {"HTTP_AUTHORIZATION" => "Basic " + Base64::encode64("#{@user.username}:wrong!")}
+      last_response.should redirect_to 'http://example.org/'
+    end
+    
+    it "should allow configuration option 'controller_to_realm_map'" do
+      sorcery_controller_property_set(:controller_to_realm_map, {"1" => "2"})
+      Sorcery::Controller::Config.controller_to_realm_map.should == {"1" => "2"}
+    end
+    
+    it "should display the correct realm name configured for the controller" do
+      sorcery_controller_property_set(:controller_to_realm_map, {"application" => "Salad"})
+      get "/test_http_basic_auth"
+      last_response.headers["WWW-Authenticate"].should == "Basic realm=\"Salad\""
+    end
+    
+    it "should sign in the user's session on successful login" do
+      session[:http_authentication_used] = true
+      get "/test_http_basic_auth", {}, {"HTTP_AUTHORIZATION" => "Basic " + Base64::encode64("#{@user.username}:secret")}
+      session[:user_id].should == User.find_by_username(@user.username).id
+    end
+  end
+end

data/spec/sinatra_modular/spec_modular/controller_oauth2_spec.rb

@@ -0,0 +1,120 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+def stub_all_oauth2_requests!
+  @client = OAuth2::Client.new("key","secret", :site => "http://myapi.com")
+  OAuth2::Client.stub!(:new).and_return(@client)
+  @acc_token = OAuth2::AccessToken.new(@client, "", "asd", nil, {})
+  @webby = @client.web_server
+  OAuth2::Strategy::WebServer.stub!(:new).and_return(@webby)
+  @webby.stub!(:get_access_token).and_return(@acc_token)
+  @acc_token.stub!(:get).and_return({"id"=>"123", "name"=>"Noam Ben Ari", "first_name"=>"Noam", "last_name"=>"Ben Ari", "link"=>"http://www.facebook.com/nbenari1", "hometown"=>{"id"=>"110619208966868", "name"=>"Haifa, Israel"}, "location"=>{"id"=>"106906559341067", "name"=>"Pardes Hanah, Hefa, Israel"}, "bio"=>"I'm a new daddy, and enjoying it!", "gender"=>"male", "email"=>"nbenari@gmail.com", "timezone"=>2, "locale"=>"en_US", "languages"=>[{"id"=>"108405449189952", "name"=>"Hebrew"}, {"id"=>"106059522759137", "name"=>"English"}, {"id"=>"112624162082677", "name"=>"Russian"}], "verified"=>true, "updated_time"=>"2011-02-16T20:59:38+0000"}.to_json)
+end
+
+describe 'MyApp' do
+  before(:all) do
+    ActiveRecord::Migrator.migrate("#{APP_ROOT}/db/migrate/external")
+    sorcery_reload!([:external])
+    sorcery_controller_property_set(:external_providers, [:facebook])
+    sorcery_controller_external_property_set(:facebook, :key, "eYVNBjBDi33aa9GkA3w")
+    sorcery_controller_external_property_set(:facebook, :secret, "XpbeSdCoaKSmQGSeokz5qcUATClRW5u08QWNfv71N8")
+    sorcery_controller_external_property_set(:facebook, :callback_url, "http://blabla.com")
+  end
+  
+  after(:all) do
+    ActiveRecord::Migrator.rollback("#{APP_ROOT}/db/migrate/external")
+  end
+  # ----------------- OAuth -----------------------
+  describe Modular, "with OAuth features" do
+  
+    before(:each) do
+      stub_all_oauth2_requests!
+    end
+      
+    after(:each) do
+      User.delete_all
+      Authentication.delete_all
+    end
+    
+    it "login_at redirects correctly" do
+      create_new_user
+      get "/login_at_test2"
+      last_response.should be_a_redirect
+      #last_response.should redirect_to("http://myapi.com/oauth/authorize?client_id=key&redirect_uri=http%3A%2F%2Fblabla.com&scope=email%2Coffline_access&type=web_server")
+      last_response.should redirect_to("http://myapi.com/oauth/authorize?client_id=key&redirect_uri=http%3A%2F%2Fblabla.com&scope=email%2Coffline_access&response_type=code")
+    end
+    
+    it "'login_from' logins if user exists" do
+      sorcery_model_property_set(:authentications_class, Authentication)
+      create_new_external_user(:facebook)
+      get "/test_login_from2"
+      last_response.body.should == "Success!"
+    end
+    
+    it "'login_from' fails if user doesn't exist" do
+      sorcery_model_property_set(:authentications_class, Authentication)
+      create_new_user
+      get "/test_login_from2"
+      last_response.body.should == "Failed!"
+    end
+  end
+  
+  describe Modular, "'create_from'" do
+    before(:each) do
+      stub_all_oauth2_requests!
+      User.delete_all
+      Authentication.delete_all
+    end
+      
+    it "should create a new user" do
+      sorcery_model_property_set(:authentications_class, Authentication)
+      sorcery_controller_external_property_set(:facebook, :user_info_mapping, {:username => "name"})
+      lambda do
+        get "/test_create_from_provider", :provider => "facebook"
+      end.should change(User, :count).by(1)
+      User.first.username.should == "Noam Ben Ari"
+    end
+    
+    it "should support nested attributes" do
+      sorcery_model_property_set(:authentications_class, Authentication)
+      sorcery_controller_external_property_set(:facebook, :user_info_mapping, {:username => "hometown/name"})
+      lambda do
+        get "/test_create_from_provider", :provider => "facebook"
+      end.should change(User, :count).by(1)
+      User.first.username.should == "Haifa, Israel"
+    end
+  end
+  
+  describe Modular, "OAuth with User Activation features" do
+    before(:all) do
+      ActiveRecord::Migrator.migrate("#{APP_ROOT}/db/migrate/activation")
+      sorcery_reload!([:user_activation,:external], :user_activation_mailer => ::SorceryMailer)
+      sorcery_controller_property_set(:external_providers, [:facebook])
+      sorcery_controller_external_property_set(:facebook, :key, "eYVNBjBDi33aa9GkA3w")
+      sorcery_controller_external_property_set(:facebook, :secret, "XpbeSdCoaKSmQGSeokz5qcUATClRW5u08QWNfv71N8")
+      sorcery_controller_external_property_set(:facebook, :callback_url, "http://blabla.com")
+    end
+    
+    after(:all) do
+      ActiveRecord::Migrator.rollback("#{APP_ROOT}/db/migrate/activation")
+    end
+    
+    after(:each) do
+      User.delete_all
+    end
+    
+    it "should not send activation email to external users" do
+      old_size = ActionMailer::Base.deliveries.size
+      create_new_external_user(:facebook)
+      ActionMailer::Base.deliveries.size.should == old_size
+    end
+    
+    it "should not send external users an activation success email" do
+      sorcery_model_property_set(:activation_success_email_method_name, nil)
+      create_new_external_user(:facebook)
+      old_size = ActionMailer::Base.deliveries.size
+      @user.activate!
+      ActionMailer::Base.deliveries.size.should == old_size
+    end
+  end
+  
+end

data/spec/sinatra_modular/spec_modular/controller_oauth_spec.rb

@@ -0,0 +1,121 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+require 'ostruct'
+
+def stub_all_oauth_requests!
+  @consumer = OAuth::Consumer.new("key","secret", :site => "http://myapi.com")
+  OAuth::Consumer.stub!(:new).and_return(@consumer)
+  @req_token = OAuth::RequestToken.new(@consumer) # OpenStruct.new() 
+  @consumer.stub!(:get_request_token).and_return(@req_token)
+  @acc_token = OAuth::AccessToken.new(@consumer)
+  @req_token.stub!(:get_access_token).and_return(@acc_token)
+  session[:request_token] = @req_token.token
+  session[:request_token_secret] = @req_token.secret
+  OAuth::RequestToken.stub!(:new).and_return(@req_token)
+  response = OpenStruct.new()
+  response.body = {"following"=>false, "listed_count"=>0, "profile_link_color"=>"0084B4", "profile_image_url"=>"http://a1.twimg.com/profile_images/536178575/noamb_normal.jpg", "description"=>"Programmer/Heavy Metal Fan/New Father", "status"=>{"text"=>"coming soon to sorcery gem: twitter and facebook authentication support.", "truncated"=>false, "favorited"=>false, "source"=>"web", "geo"=>nil, "in_reply_to_screen_name"=>nil, "in_reply_to_user_id"=>nil, "in_reply_to_status_id_str"=>nil, "created_at"=>"Sun Mar 06 23:01:12 +0000 2011", "contributors"=>nil, "place"=>nil, "retweeted"=>false, "in_reply_to_status_id"=>nil, "in_reply_to_user_id_str"=>nil, "coordinates"=>nil, "retweet_count"=>0, "id"=>44533012284706816, "id_str"=>"44533012284706816"}, "show_all_inline_media"=>false, "geo_enabled"=>true, "profile_sidebar_border_color"=>"a8c7f7", "url"=>nil, "followers_count"=>10, "screen_name"=>"nbenari", "profile_use_background_image"=>true, "location"=>"Israel", "statuses_count"=>25, "profile_background_color"=>"022330", "lang"=>"en", "verified"=>false, "notifications"=>false, "profile_background_image_url"=>"http://a3.twimg.com/profile_background_images/104087198/04042010339.jpg", "favourites_count"=>5, "created_at"=>"Fri Nov 20 21:58:19 +0000 2009", "is_translator"=>false, "contributors_enabled"=>false, "protected"=>false, "follow_request_sent"=>false, "time_zone"=>"Greenland", "profile_text_color"=>"333333", "name"=>"Noam Ben Ari", "friends_count"=>10, "profile_sidebar_fill_color"=>"C0DFEC", "id"=>123, "id_str"=>"91434812", "profile_background_tile"=>false, "utc_offset"=>-10800}.to_json
+  @acc_token.stub!(:get).and_return(response)
+end
+
+describe Modular do
+  before(:all) do
+    ActiveRecord::Migrator.migrate("#{APP_ROOT}/db/migrate/external")
+    sorcery_reload!([:external])
+    sorcery_controller_property_set(:external_providers, [:twitter])
+    sorcery_controller_external_property_set(:twitter, :key, "eYVNBjBDi33aa9GkA3w")
+    sorcery_controller_external_property_set(:twitter, :secret, "XpbeSdCoaKSmQGSeokz5qcUATClRW5u08QWNfv71N8")
+    sorcery_controller_external_property_set(:twitter, :callback_url, "http://blabla.com")
+  end
+  
+  after(:all) do
+    ActiveRecord::Migrator.rollback("#{APP_ROOT}/db/migrate/external")
+  end
+  # ----------------- OAuth -----------------------
+  describe Modular, "'login_from'" do
+  
+    before(:each) do
+      stub_all_oauth_requests!
+    end
+      
+    after(:each) do
+      User.delete_all
+      Authentication.delete_all
+    end
+    
+    it "login_at_test redirects correctly" do
+      create_new_user
+      get "/login_at_test"
+      last_response.should be_a_redirect
+      last_response.should redirect_to("http://myapi.com/oauth/authorize?oauth_callback=http%3A%2F%2Fblabla.com&oauth_token=")
+    end
+    
+    it "logins if user exists" do
+      sorcery_model_property_set(:authentications_class, Authentication)
+      create_new_external_user(:twitter)
+      get '/test_login_from', :oauth_verifier => "blablaRERASDFcxvSDFA"
+      last_response.body.should == "Success!"
+    end
+    
+    it "'login_from' fails if user doesn't exist" do
+      sorcery_model_property_set(:authentications_class, Authentication)
+      create_new_user
+      get '/test_login_from', :oauth_verifier => "blablaRERASDFcxvSDFA"
+      last_response.body.should == "Failed!"
+    end
+  end
+  
+  describe Modular, "'create_from'" do
+    before(:each) do
+      stub_all_oauth_requests!
+      User.delete_all
+      Authentication.delete_all
+    end
+      
+    it "should create a new user" do
+      sorcery_model_property_set(:authentications_class, Authentication)
+      sorcery_controller_external_property_set(:twitter, :user_info_mapping, {:username => "screen_name"})
+      lambda do
+        get :test_create_from_provider, :provider => "twitter"
+      end.should change(User, :count).by(1)
+      User.first.username.should == "nbenari"
+    end
+    
+    it "should support nested attributes" do
+      sorcery_model_property_set(:authentications_class, Authentication)
+      sorcery_controller_external_property_set(:twitter, :user_info_mapping, {:username => "status/text"})
+      lambda do
+        get :test_create_from_provider, :provider => "twitter"
+      end.should change(User, :count).by(1)
+      User.first.username.should == "coming soon to sorcery gem: twitter and facebook authentication support."
+    end
+  end
+  
+  describe Modular, "OAuth with User Activation features" do
+    before(:all) do
+      ActiveRecord::Migrator.migrate("#{APP_ROOT}/db/migrate/activation")
+      sorcery_reload!([:user_activation,:external], :user_activation_mailer => ::SorceryMailer)
+    end
+    
+    after(:all) do
+      ActiveRecord::Migrator.rollback("#{APP_ROOT}/db/migrate/activation")
+    end
+    
+    after(:each) do
+      User.delete_all
+      Authentication.delete_all
+    end
+    
+    it "should not send activation email to external users" do
+      old_size = ActionMailer::Base.deliveries.size
+      create_new_external_user(:twitter)
+      ActionMailer::Base.deliveries.size.should == old_size
+    end
+    
+    it "should not send external users an activation success email" do
+      sorcery_model_property_set(:activation_success_email_method_name, nil)
+      create_new_external_user(:twitter)
+      old_size = ActionMailer::Base.deliveries.size
+      @user.activate!
+      ActionMailer::Base.deliveries.size.should == old_size
+    end
+  end
+end

data/spec/sinatra_modular/spec_modular/controller_remember_me_spec.rb

@@ -0,0 +1,64 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe Modular do
+  
+  # ----------------- REMEMBER ME -----------------------
+  describe Modular, "with remember me features" do
+    before(:all) do
+      ActiveRecord::Migrator.migrate("#{APP_ROOT}/db/migrate/remember_me")
+      sorcery_reload!([:remember_me])
+    end
+    
+    before(:each) do
+      create_new_user
+    end
+    
+    after(:all) do
+      ActiveRecord::Migrator.rollback("#{APP_ROOT}/db/migrate/remember_me")
+    end
+    
+    after(:each) do
+      session = nil
+      clear_cookies
+      User.delete_all
+    end
+    
+    it "should set cookie on remember_me!" do
+      post "/test_login_with_remember", :username => 'gizmo', :password => 'secret'
+      cookies["remember_me_token"].should == assigns[:user].remember_me_token
+    end
+    
+    it "should clear cookie on forget_me!" do
+      cookies["remember_me_token"] == {:value => 'asd54234dsfsd43534', :expires => 3600}
+      get '/test_logout'
+      cookies["remember_me_token"].should == nil
+    end
+    
+    it "login(username,password,remember_me) should login and remember" do
+      post '/test_login_with_remember_in_login', :username => 'gizmo', :password => 'secret', :remember => "1"
+      cookies["remember_me_token"].should_not be_nil
+      cookies["remember_me_token"].should == assigns[:user].remember_me_token
+    end
+    
+    it "logout should also forget_me!" do
+      session[:user_id] = @user.id
+      get '/test_logout_with_remember'
+      cookies["remember_me_token"].should == nil
+    end
+    
+    it "should login_from_cookie" do
+      post "/test_login_with_remember", :username => 'gizmo', :password => 'secret'
+      get_sinatra_app(subject).instance_eval do
+        @current_user = nil
+      end
+      session[:user_id] = nil
+      get '/test_login_from_cookie'
+      assigns[:current_user].should == @user
+    end
+    
+    it "should not remember_me! when not asked to" do
+      post '/test_login', :username => 'gizmo', :password => 'secret'
+      cookies["remember_me_token"].should == nil
+    end
+  end
+end

data/spec/sinatra_modular/spec_modular/controller_session_timeout_spec.rb

@@ -0,0 +1,57 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe Modular do
+  
+  # ----------------- SESSION TIMEOUT -----------------------
+  describe Modular, "with session timeout features" do
+    before(:all) do
+      sorcery_reload!([:session_timeout])
+      sorcery_controller_property_set(:session_timeout,0.5)
+      create_new_user
+    end
+    
+    after(:each) do
+      Timecop.return
+    end
+    
+    it "should not reset session before session timeout" do
+      session[:user_id] = User.first.id
+      get "/test_should_be_logged_in"
+      last_response.should be_ok
+    end
+    
+    it "should reset session after session timeout" do
+      get "/test_login", :username => 'gizmo', :password => 'secret'
+      session[:user_id].should_not be_nil
+      Timecop.travel(Time.now+0.6)
+      get "/test_should_be_logged_in"
+      last_response.should be_a_redirect
+    end
+    
+    context "with 'session_timeout_from_last_action'" do
+      it "should not logout if there was activity" do
+        session[:user_id] = nil
+        sorcery_controller_property_set(:session_timeout,2)
+        sorcery_controller_property_set(:session_timeout_from_last_action, true)
+        get "/test_login", :username => 'gizmo', :password => 'secret'
+        Timecop.travel(Time.now+1)
+        get "/test_should_be_logged_in"
+        session[:user_id].should_not be_nil
+        Timecop.travel(Time.now+1)
+        get "/test_should_be_logged_in"
+        session[:user_id].should_not be_nil
+        last_response.should be_ok
+      end
+    
+      it "should logout if there was no activity" do
+        sorcery_controller_property_set(:session_timeout,0.5)
+        sorcery_controller_property_set(:session_timeout_from_last_action, true)
+        get "/test_login", :username => 'gizmo', :password => 'secret'
+        Timecop.travel(Time.now+0.6)
+        get "/test_should_be_logged_in"
+        session[:user_id].should be_nil
+        last_response.should be_a_redirect
+      end
+    end
+  end
+end