devise 3.2.4 → 4.7.1

data/test/integration/rememberable_test.rb

@@ -1,167 +0,0 @@
-require 'test_helper'
-
-class RememberMeTest < ActionDispatch::IntegrationTest
-  def create_user_and_remember(add_to_token='')
-    user = create_user
-    user.remember_me!
-    raw_cookie = User.serialize_into_cookie(user).tap { |a| a.last << add_to_token }
-    cookies['remember_user_token'] = generate_signed_cookie(raw_cookie)
-    user
-  end
-
-  def generate_signed_cookie(raw_cookie)
-    request = ActionDispatch::TestRequest.new
-    request.cookie_jar.signed['raw_cookie'] = raw_cookie
-    request.cookie_jar['raw_cookie']
-  end
-
-  def signed_cookie(key)
-    controller.send(:cookies).signed[key]
-  end
-
-  def cookie_expires(key)
-    cookie  = response.headers["Set-Cookie"].split("\n").grep(/^#{key}/).first
-    expires = cookie.split(";").map(&:strip).grep(/^expires=/).first
-    Time.parse(expires).utc
-  end
-
-  test 'do not remember the user if they have not checked remember me option' do
-    sign_in_as_user
-    assert_nil request.cookies["remember_user_cookie"]
-  end
-
-  test 'handle unverified requests gets rid of caches' do
-    swap ApplicationController, allow_forgery_protection: true do
-      post exhibit_user_url(1)
-      assert_not warden.authenticated?(:user)
-
-      create_user_and_remember
-      post exhibit_user_url(1)
-      assert_equal "User is not authenticated", response.body
-      assert_not warden.authenticated?(:user)
-    end
-  end
-
-  test 'handle unverified requests does not create cookies on sign in' do
-    swap ApplicationController, allow_forgery_protection: true do
-      get new_user_session_path
-      assert request.session[:_csrf_token]
-
-      post user_session_path, authenticity_token: "oops", user:
-           { email: "jose.valim@gmail.com", password: "123456", remember_me: "1" }
-      assert_not warden.authenticated?(:user)
-      assert_not request.cookies['remember_user_token']
-    end
-  end
-
-  test 'generate remember token after sign in' do
-    sign_in_as_user remember_me: true
-    assert request.cookies['remember_user_token']
-  end
-
-  test 'generate remember token after sign in setting cookie options' do
-    # We test this by asserting the cookie is not sent after the redirect
-    # since we changed the domain. This is the only difference with the
-    # previous test.
-    swap Devise, rememberable_options: { domain: "omg.somewhere.com" } do
-      sign_in_as_user remember_me: true
-      assert_nil request.cookies["remember_user_token"]
-    end
-  end
-
-  test 'generate remember token with a custom key' do
-    swap Devise, rememberable_options: { key: "v1lat_token" } do
-      sign_in_as_user remember_me: true
-      assert request.cookies["v1lat_token"]
-    end
-  end
-
-  test 'generate remember token after sign in setting session options' do
-    begin
-      Rails.configuration.session_options[:domain] = "omg.somewhere.com"
-      sign_in_as_user remember_me: true
-      assert_nil request.cookies["remember_user_token"]
-    ensure
-      Rails.configuration.session_options.delete(:domain)
-    end
-  end
-
-  test 'remember the user before sign in' do
-    user = create_user_and_remember
-    get users_path
-    assert_response :success
-    assert warden.authenticated?(:user)
-    assert warden.user(:user) == user
-    assert_match /remember_user_token[^\n]*HttpOnly/, response.headers["Set-Cookie"], "Expected Set-Cookie header in response to set HttpOnly flag on remember_user_token cookie."
-  end
-
-  test 'remember the user before sign up and redirect them to their home' do
-    create_user_and_remember
-    get new_user_registration_path
-    assert warden.authenticated?(:user)
-    assert_redirected_to root_path
-  end
-
-  test 'does not extend remember period through sign in' do
-    swap Devise, extend_remember_period: true, remember_for: 1.year do
-      user = create_user
-      user.remember_me!
-
-      user.remember_created_at = old = 10.days.ago
-      user.save
-
-      sign_in_as_user remember_me: true
-      user.reload
-
-      assert warden.user(:user) == user
-      assert_equal old.to_i, user.remember_created_at.to_i
-    end
-  end
-
-  test 'do not remember other scopes' do
-    create_user_and_remember
-    get root_path
-    assert_response :success
-    assert warden.authenticated?(:user)
-    assert_not warden.authenticated?(:admin)
-  end
-
-  test 'do not remember with invalid token' do
-    create_user_and_remember('add')
-    get users_path
-    assert_not warden.authenticated?(:user)
-    assert_redirected_to new_user_session_path
-  end
-
-  test 'do not remember with expired token' do
-    create_user_and_remember
-    swap Devise, remember_for: 0 do
-      get users_path
-      assert_not warden.authenticated?(:user)
-      assert_redirected_to new_user_session_path
-    end
-  end
-
-  test 'do not remember the user anymore after forget' do
-    create_user_and_remember
-    get users_path
-    assert warden.authenticated?(:user)
-
-    get destroy_user_session_path
-    assert_not warden.authenticated?(:user)
-    assert_nil warden.cookies['remember_user_token']
-
-    get users_path
-    assert_not warden.authenticated?(:user)
-  end
-
-  test 'changing user password expires remember me token' do
-    user = create_user_and_remember
-    user.password = "another_password"
-    user.password_confirmation = "another_password"
-    user.save!
-
-    get users_path
-    assert_not warden.authenticated?(:user)
-  end
-end

data/test/integration/timeoutable_test.rb

@@ -1,183 +0,0 @@
-require 'test_helper'
-
-class SessionTimeoutTest < ActionDispatch::IntegrationTest
-
-  def last_request_at
-    @controller.user_session['last_request_at']
-  end
-
-  test 'set last request at in user session after each request' do
-    sign_in_as_user
-    old_last_request = last_request_at
-    assert_not_nil last_request_at
-
-    get users_path
-    assert_not_nil last_request_at
-    assert_not_equal old_last_request, last_request_at
-  end
-
-  test 'set last request at in user session after each request is skipped if tracking is disabled' do
-    sign_in_as_user
-    old_last_request = last_request_at
-    assert_not_nil last_request_at
-
-    get users_path, {}, 'devise.skip_trackable' => true
-    assert_equal old_last_request, last_request_at
-  end
-
-  test 'does not time out user session before default limit time' do
-    sign_in_as_user
-    assert_response :success
-    assert warden.authenticated?(:user)
-
-    get users_path
-    assert_response :success
-    assert warden.authenticated?(:user)
-  end
-
-  test 'time out user session after default limit time when sign_out_all_scopes is false' do
-    swap Devise, sign_out_all_scopes: false do
-      sign_in_as_admin
-
-      user = sign_in_as_user
-      get expire_user_path(user)
-      assert_not_nil last_request_at
-
-      get users_path
-      assert_redirected_to users_path
-      assert_not warden.authenticated?(:user)
-      assert warden.authenticated?(:admin)
-    end
-  end
-
-  test 'time out all sessions after default limit time when sign_out_all_scopes is true' do
-    swap Devise, sign_out_all_scopes: true do
-      sign_in_as_admin
-
-      user = sign_in_as_user
-      get expire_user_path(user)
-      assert_not_nil last_request_at
-
-      get root_path
-      assert_not warden.authenticated?(:user)
-      assert_not warden.authenticated?(:admin)
-    end
-  end
-
-  test 'time out user session after deault limit time and redirect to latest get request' do
-    user = sign_in_as_user
-    visit edit_form_user_path(user)
-
-    click_button 'Update'
-    sign_in_as_user
-
-    assert_equal edit_form_user_url(user), current_url
-  end
-
-  test 'time out is not triggered on sign out' do
-    user = sign_in_as_user
-    get expire_user_path(user)
-
-    get destroy_user_session_path
-
-    assert_response :redirect
-    assert_redirected_to root_path
-    follow_redirect!
-    assert_contain 'Signed out successfully'
-  end
-
-  test 'expired session is not extended by sign in page' do
-    user = sign_in_as_user
-    get expire_user_path(user)
-    assert warden.authenticated?(:user)
-
-    get "/users/sign_in"
-    assert_redirected_to "/users/sign_in"
-    follow_redirect!
-
-    assert_response :success
-    assert_contain 'Sign in'
-    assert_not warden.authenticated?(:user)
-  end
-
-  test 'time out is not triggered on sign in' do
-    user = sign_in_as_user
-    get expire_user_path(user)
-
-    post "/users/sign_in", email: user.email, password: "123456"
-
-    assert_response :redirect
-    follow_redirect!
-    assert_contain 'You are signed in'
-  end
-
-  test 'admin does not explode on time out' do
-    admin = sign_in_as_admin
-    get expire_admin_path(admin)
-
-    Admin.send :define_method, :reset_authentication_token! do
-      nil
-    end
-
-    begin
-      get admins_path
-      assert_redirected_to admins_path
-      assert_not warden.authenticated?(:admin)
-    ensure
-      Admin.send(:remove_method, :reset_authentication_token!)
-    end
-  end
-
-  test 'user configured timeout limit' do
-    swap Devise, timeout_in: 8.minutes do
-      user = sign_in_as_user
-
-      get users_path
-      assert_not_nil last_request_at
-      assert_response :success
-      assert warden.authenticated?(:user)
-
-      get expire_user_path(user)
-      get users_path
-      assert_redirected_to users_path
-      assert_not warden.authenticated?(:user)
-    end
-  end
-
-  test 'error message with i18n' do
-    store_translations :en, devise: {
-      failure: { user: { timeout: 'Session expired!' } }
-    } do
-      user = sign_in_as_user
-
-      get expire_user_path(user)
-      get root_path
-      follow_redirect!
-      assert_contain 'Session expired!'
-    end
-  end
-
-  test 'error message with i18n with double redirect' do
-    store_translations :en, devise: {
-      failure: { user: { timeout: 'Session expired!' } }
-    } do
-      user = sign_in_as_user
-
-      get expire_user_path(user)
-      get users_path
-      follow_redirect!
-      follow_redirect!
-      assert_contain 'Session expired!'
-    end
-  end
-
-  test 'time out not triggered if remembered' do
-    user = sign_in_as_user remember_me: true
-    get expire_user_path(user)
-    assert_not_nil last_request_at
-
-    get users_path
-    assert_response :success
-    assert warden.authenticated?(:user)
-  end
-end

data/test/integration/trackable_test.rb

@@ -1,92 +0,0 @@
-require 'test_helper'
-
-class TrackableHooksTest < ActionDispatch::IntegrationTest
-
-  test "current and last sign in timestamps are updated on each sign in" do
-    user = create_user
-    assert_nil user.current_sign_in_at
-    assert_nil user.last_sign_in_at
-
-    sign_in_as_user
-    user.reload
-
-    assert_kind_of Time, user.current_sign_in_at
-    assert_kind_of Time, user.last_sign_in_at
-
-    assert_equal user.current_sign_in_at, user.last_sign_in_at
-    assert user.current_sign_in_at >= user.created_at
-
-    visit destroy_user_session_path
-    new_time = 2.seconds.from_now
-    Time.stubs(:now).returns(new_time)
-
-    sign_in_as_user
-    user.reload
-    assert user.current_sign_in_at > user.last_sign_in_at
-  end
-
-  test "current and last sign in remote ip are updated on each sign in" do
-    user = create_user
-    assert_nil user.current_sign_in_ip
-    assert_nil user.last_sign_in_ip
-
-    sign_in_as_user
-    user.reload
-
-    assert_equal "127.0.0.1", user.current_sign_in_ip
-    assert_equal "127.0.0.1", user.last_sign_in_ip
-  end
-
-  test "current remote ip returns original ip behind a non transparent proxy" do
-    user = create_user
-
-    arbitrary_ip = '200.121.1.69'
-    sign_in_as_user do
-      header 'HTTP_X_FORWARDED_FOR', arbitrary_ip
-    end
-    user.reload
-    assert_equal arbitrary_ip, user.current_sign_in_ip
-  end
-
-  test "increase sign in count" do
-    user = create_user
-    assert_equal 0, user.sign_in_count
-
-    sign_in_as_user
-    user.reload
-    assert_equal 1, user.sign_in_count
-
-    visit destroy_user_session_path
-    sign_in_as_user
-    user.reload
-    assert_equal 2, user.sign_in_count
-  end
-
-  test "does not update anything if user has signed out along the way" do
-    swap Devise, allow_unconfirmed_access_for: 0.days do
-      user = create_user(confirm: false)
-      sign_in_as_user
-
-      user.reload
-      assert_nil user.current_sign_in_at
-      assert_nil user.last_sign_in_at
-    end
-  end
-
-  test "do not track if devise.skip_trackable is set" do
-    user = create_user
-    sign_in_as_user do
-      header 'devise.skip_trackable', '1'
-    end
-    user.reload
-    assert_equal 0, user.sign_in_count
-    visit destroy_user_session_path
-
-    sign_in_as_user do
-      header 'devise.skip_trackable', false
-    end
-    user.reload
-    assert_equal 1, user.sign_in_count
-  end
-
-end

data/test/mailers/confirmation_instructions_test.rb

@@ -1,115 +0,0 @@
-require 'test_helper'
-
-class ConfirmationInstructionsTest < ActionMailer::TestCase
-
-  def setup
-    setup_mailer
-    Devise.mailer = 'Devise::Mailer'
-    Devise.mailer_sender = 'test@example.com'
-  end
-
-  def teardown
-    Devise.mailer = 'Devise::Mailer'
-    Devise.mailer_sender = 'please-change-me@config-initializers-devise.com'
-  end
-
-  def user
-    @user ||= create_user
-  end
-
-  def mail
-    @mail ||= begin
-      user
-      ActionMailer::Base.deliveries.first
-    end
-  end
-
-  test 'email sent after creating the user' do
-    assert_not_nil mail
-  end
-
-  test 'content type should be set to html' do
-    assert mail.content_type.include?('text/html')
-  end
-
-  test 'send confirmation instructions to the user email' do
-    mail
-    assert_equal [user.email], mail.to
-  end
-
-  test 'setup sender from configuration' do
-    assert_equal ['test@example.com'], mail.from
-  end
-
-  test 'setup sender from custom mailer defaults' do
-    Devise.mailer = 'Users::Mailer'
-    assert_equal ['custom@example.com'], mail.from
-  end
-
-  test 'setup sender from custom mailer defaults with proc' do
-    Devise.mailer = 'Users::FromProcMailer'
-    assert_equal ['custom@example.com'], mail.from
-  end
-
-  test 'custom mailer renders parent mailer template' do
-    Devise.mailer = 'Users::Mailer'
-    assert_not_blank mail.body.encoded
-  end
-
-  test 'setup reply to as copy from sender' do
-    assert_equal ['test@example.com'], mail.reply_to
-  end
-
-  test 'setup reply to as different if set in defaults' do
-    Devise.mailer = 'Users::ReplyToMailer'
-    assert_equal ['custom@example.com'], mail.from
-    assert_equal ['custom_reply_to@example.com'], mail.reply_to
-  end
-
-  test 'setup subject from I18n' do
-    store_translations :en, devise: { mailer: { confirmation_instructions: { subject: 'Account Confirmation' } } } do
-      assert_equal 'Account Confirmation', mail.subject
-    end
-  end
-
-  test 'subject namespaced by model' do
-    store_translations :en, devise: { mailer: { confirmation_instructions: { user_subject: 'User Account Confirmation' } } } do
-      assert_equal 'User Account Confirmation', mail.subject
-    end
-  end
-
-  test 'body should have user info' do
-    assert_match user.email, mail.body.encoded
-  end
-
-  test 'body should have link to confirm the account' do
-    host = ActionMailer::Base.default_url_options[:host]
-
-    if mail.body.encoded =~ %r{<a href=\"http://#{host}/users/confirmation\?confirmation_token=([^"]+)">}
-      assert_equal Devise.token_generator.digest(user.class, :confirmation_token, $1), user.confirmation_token
-    else
-      flunk "expected confirmation url regex to match"
-    end
-  end
-
-  test 'renders a scoped if scoped_views is set to true' do
-    swap Devise, scoped_views: true do
-      assert_equal user.email, mail.body.decoded
-    end
-  end
-
-  test 'renders a scoped if scoped_views is set in the mailer class' do
-    begin
-      Devise::Mailer.scoped_views = true
-      assert_equal user.email, mail.body.decoded
-    ensure
-      Devise::Mailer.send :remove_instance_variable, :@scoped_views
-    end
-  end
-
-  test 'mailer sender accepts a proc' do
-    swap Devise, mailer_sender: proc { "another@example.com" } do
-      assert_equal ['another@example.com'], mail.from
-    end
-  end
-end

data/test/mailers/reset_password_instructions_test.rb

@@ -1,96 +0,0 @@
-require 'test_helper'
-
-class ResetPasswordInstructionsTest < ActionMailer::TestCase
-  def setup
-    setup_mailer
-    Devise.mailer = 'Devise::Mailer'
-    Devise.mailer_sender = 'test@example.com'
-  end
-
-  def teardown
-    Devise.mailer = 'Devise::Mailer'
-    Devise.mailer_sender = 'please-change-me@config-initializers-devise.com'
-  end
-
-  def user
-    @user ||= begin
-      user = create_user
-      user.send_reset_password_instructions
-      user
-    end
-  end
-
-  def mail
-    @mail ||= begin
-      user
-      ActionMailer::Base.deliveries.last
-    end
-  end
-
-  test 'email sent after reseting the user password' do
-    assert_not_nil mail
-  end
-
-  test 'content type should be set to html' do
-    assert mail.content_type.include?('text/html')
-  end
-
-  test 'send confirmation instructions to the user email' do
-    assert_equal [user.email], mail.to
-  end
-
-  test 'setup sender from configuration' do
-    assert_equal ['test@example.com'], mail.from
-  end
-
-  test 'setup sender from custom mailer defaults' do
-    Devise.mailer = 'Users::Mailer'
-    assert_equal ['custom@example.com'], mail.from
-  end
-
-  test 'setup sender from custom mailer defaults with proc' do
-    Devise.mailer = 'Users::FromProcMailer'
-    assert_equal ['custom@example.com'], mail.from
-  end
-
-  test 'custom mailer renders parent mailer template' do
-    Devise.mailer = 'Users::Mailer'
-    assert_not_blank mail.body.encoded
-  end
-
-  test 'setup reply to as copy from sender' do
-    assert_equal ['test@example.com'], mail.reply_to
-  end
-
-  test 'setup subject from I18n' do
-    store_translations :en, devise: { mailer: { reset_password_instructions: { subject: 'Reset instructions' } } } do
-      assert_equal 'Reset instructions', mail.subject
-    end
-  end
-
-  test 'subject namespaced by model' do
-    store_translations :en, devise: { mailer: { reset_password_instructions: { user_subject: 'User Reset Instructions' } } } do
-      assert_equal 'User Reset Instructions', mail.subject
-    end
-  end
-
-  test 'body should have user info' do
-    assert_match user.email, mail.body.encoded
-  end
-
-  test 'body should have link to confirm the account' do
-    host = ActionMailer::Base.default_url_options[:host]
-
-    if mail.body.encoded =~ %r{<a href=\"http://#{host}/users/password/edit\?reset_password_token=([^"]+)">}
-      assert_equal Devise.token_generator.digest(user.class, :reset_password_token, $1), user.reset_password_token
-    else
-      flunk "expected reset password url regex to match"
-    end
-  end
-
-  test 'mailer sender accepts a proc' do
-    swap Devise, mailer_sender: proc { "another@example.com" } do
-      assert_equal ['another@example.com'], mail.from
-    end
-  end
-end