authenticate 0.2.3 → 0.3.0

data/spec/model/email_spec.rb

@@ -3,7 +3,6 @@ require 'authenticate/model/email'
 
 
 describe Authenticate::Model::Email do
-
   it 'validates email' do
     user = build(:user, :without_email)
     user.save

data/spec/model/lifetimed_spec.rb

@@ -3,31 +3,19 @@ require 'authenticate/model/lifetimed'
 
 
 describe Authenticate::Model::Lifetimed do
-
   context '#max_session_lifetime_exceeded?' do
-    before {
-      Authenticate.configure do |config|
-        config.max_session_lifetime = 10.minutes
-      end
-    }
 
     it 'passes fresh sessions' do
-      user = create(:user, current_sign_in_at: 1.minute.ago.utc)
-      expect(user).to_not be_max_session_lifetime_exceeded
+      Timecop.freeze do
+        user = create(:user, current_sign_in_at: 1.minute.ago.utc)
+        expect(user).to_not be_max_session_lifetime_exceeded
+      end
     end
 
     it 'detects timed out sessions' do
-      user = create(:user, current_sign_in_at: 5.hours.ago.utc)
-      expect(user).to be_max_session_lifetime_exceeded
-    end
-
-    describe 'max_session_lifetime param not set' do
-      it 'does not time out' do
+      Timecop.freeze do
         user = create(:user, current_sign_in_at: 5.hours.ago.utc)
-        Authenticate.configure do |config|
-          config.max_session_lifetime = nil
-        end
-        expect(user).to_not be_max_session_lifetime_exceeded
+        expect(user).to be_max_session_lifetime_exceeded
       end
     end
   end

data/spec/model/password_reset_spec.rb

@@ -3,9 +3,6 @@ require 'authenticate/model/password_reset'
 
 
 describe Authenticate::Model::PasswordReset do
-  before(:all) {
-    Authenticate.configuration.reset_password_within = 5.minutes
-  }
   context 'forgot_password!' do
     subject { create(:user) }
     before { subject.forgot_password! }
@@ -22,14 +19,13 @@ describe Authenticate::Model::PasswordReset do
 
   context '#reset_password_period_valid?' do
     subject { create(:user) }
-    before(:each) {
-      Authenticate.configuration.reset_password_within = 5.minutes
-    }
 
     it 'always true if reset_password_within config param is nil' do
+      within = Authenticate.configuration.reset_password_within
       subject.password_reset_sent_at = 10.days.ago
       Authenticate.configuration.reset_password_within = nil
       expect(subject.reset_password_period_valid?).to be_truthy
+      Authenticate.configuration.reset_password_within = within
     end
 
     it 'false if time exceeded' do
@@ -45,9 +41,6 @@ describe Authenticate::Model::PasswordReset do
 
   context '#update_password' do
     subject { create(:user) }
-    before(:each) {
-      Authenticate.configuration.reset_password_within = 5.minutes
-    }
 
     context 'within time time' do
       before(:each) {

data/spec/model/session_spec.rb

@@ -2,24 +2,23 @@ require 'spec_helper'
 
 
 describe Authenticate::Session do
-
   describe 'session token' do
     it 'finds a user from session token' do
       user = create(:user, :with_session_token)
-      request = {}
-      cookies = {authenticate_session_token: user.session_token}
+      request = mock_request
+      cookies = cookies_for user
       session = Authenticate::Session.new(request, cookies)
       expect(session.current_user).to eq user
     end
-    it 'returns nil without a session token' do
-      request = {}
-      cookies = {session_token: nil}
+    it 'nil user without a session token' do
+      request = mock_request
+      cookies = {}
       session = Authenticate::Session.new(request, cookies)
       expect(session.current_user).to be_nil
     end
     it 'returns nil with a bogus session token' do
-      request = {}
-      cookies = {session_token: 'some made up value'}
+      request = mock_request
+      cookies = {Authenticate.configuration.cookie_name.freeze.to_sym => 'some made up value'}
       session = Authenticate::Session.new(request, cookies)
       expect(session.current_user).to be_nil
     end
@@ -27,23 +26,20 @@ describe Authenticate::Session do
 
   describe '#login' do
     it 'sets current_user' do
-      user = build(:user, :with_session_token)
+      user = create(:user)
       session = Authenticate::Session.new(mock_request, {})
       session.login(user)
       expect(session.current_user).to eq user
     end
     context 'with a block' do
       it 'passes the success status to the block when login succeeds' do
-        user = build(:user, :with_session_token)
+        user = create(:user)
         session = Authenticate::Session.new(mock_request, {})
-        session.login user do |status|
+        session.login(user) do |status|
           expect(status.success?).to eq true
         end
       end
       it 'passes the failure status to the block when login fails' do
-        Authenticate.configure do |config|
-          config.max_consecutive_bad_logins_allowed = nil
-        end
         session = Authenticate::Session.new(mock_request, {})
         session.login nil do |status|
           expect(status.success?).to eq false
@@ -65,20 +61,17 @@ describe Authenticate::Session do
         expect{session.login(user)}.to change{user.sign_in_count}.by(1)
       end
       it 'fails login if a callback fails' do
-        failure_message = 'THIS IS A FORCED FAIL'
-        Authenticate.lifecycle.after_authentication do |user, session, opts|
-          throw(:failure, failure_message)
-        end
-        user = create(:user, :with_session_token, last_access_at: 10.minutes.ago)
-        cookies = {authenticate_session_token: user.session_token}
+        cookies = {}
         session = Authenticate::Session.new(mock_request, cookies)
-        session.login user do |status|
+        session.login nil do |status|
           expect(status.success?).to eq false
-          expect(status.message).to eq failure_message
+          expect(status.message).to eq I18n.t('callbacks.authenticatable.failure')
         end
       end
     end
   end
+end
 
-
+def cookies_for user
+  { Authenticate.configuration.cookie_name.freeze.to_sym => user.session_token }
 end

data/spec/model/timeoutable_spec.rb

@@ -3,18 +3,19 @@ require 'authenticate/model/timeoutable'
 
 
 describe Authenticate::Model::Timeoutable do
-  before(:all) {
-    Authenticate.configuration.timeout_in = 45.minutes
-  }
   subject { create(:user) }
 
   it 'does not timeout while last_access_at is valid' do
-    subject.last_access_at = 10.minutes.ago
-    expect(subject.timedout?).to be_falsey
+    Timecop.freeze do
+      subject.last_access_at = 10.minutes.ago
+      expect(subject.timedout?).to be_falsey
+    end
   end
 
   it 'does timeout when last_access_at is stale' do
-    subject.last_access_at = 46.minutes.ago
-    expect(subject.timedout?).to be_truthy
+    Timecop.freeze do
+      subject.last_access_at = 46.minutes.ago
+      expect(subject.timedout?).to be_truthy
+    end
   end
 end

data/spec/model/trackable_spec.rb

@@ -4,7 +4,6 @@ require 'authenticate/model/trackable'
 
 describe Authenticate::Model::Trackable do
   subject {create(:user)}
-
   context '#last_sign_in_at' do
     it 'sets to old current_sign_in_at if it is not nil' do
       old_sign_in = 2.days.ago.utc

data/spec/model/user_spec.rb

@@ -1,7 +1,6 @@
 require 'spec_helper'
 
 describe Authenticate::User do
-
   context 'session tokens' do
     it 'generates a new session token' do
       user = create(:user, :with_session_token)
@@ -19,4 +18,4 @@ describe Authenticate::User do
     end
   end
 
-end
+end

data/spec/spec_helper.rb

@@ -1,56 +1,52 @@
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
+ENV['RAILS_ENV'] ||= 'test'
 
-
-ENV["RAILS_ENV"] ||= 'test'
-
-MY_ORM = :active_record
-
-# require 'simplecov'
-# SimpleCov.root File.join(File.dirname(__FILE__), '..', 'lib')
-# SimpleCov.start
-
-
-require 'rails/all'
+require File.expand_path('../dummy/config/environment.rb',  __FILE__)
 require 'rspec/rails'
-require 'factory_girl_rails'
-# require 'timecop'
-
-require 'authenticate'
+require 'shoulda-matchers'
+require 'capybara/rails'
+require 'capybara/rspec'
+require 'database_cleaner'
+require 'factory_girl'
+require 'timecop'
 
-ENGINE_RAILS_ROOT=File.join(File.dirname(__FILE__), '../')
-Dir[File.join(ENGINE_RAILS_ROOT, "spec/factories/**/*.rb")].each {|f| require f }
+Rails.backtrace_cleaner.remove_silencers!
+DatabaseCleaner.strategy = :truncation
 
-def setup_orm; end
-def teardown_orm; end
-
-require "orm/#{MY_ORM}"
+# No longer autoloading support, individually requiring instead.
+#
+# Dir[Rails.root.join("spec/support/**/*.rb")].each { |f| require f }
 
-# require "rails_app/config/environment"
-require "dummy/config/environment"
+# Load factory girl factories.
+Dir[File.join(File.dirname(__FILE__), 'factories/**/*.rb')].each { |f| require f }
 
-# class TestMailer < ActionMailer::Base;end
+RSpec.configure do |config|
+  config.include FactoryGirl::Syntax::Methods
+  config.infer_spec_type_from_file_location!
+  config.order = :random
+  config.use_transactional_fixtures = true
 
-Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}
+  # config.mock_with :rspec
 
-RSpec.configure do |config|
-  config.fixture_path = "#{::Rails.root}/spec/factories"
+  config.expect_with :rspec do |expectations|
+    expectations.syntax = :expect
+  end
 
-  config.include RSpec::Rails::ControllerExampleGroup, :file_path => /controller(.)*_spec.rb$/
-  config.mock_with :rspec
-  config.include FactoryGirl::Syntax::Methods
+  config.mock_with :rspec do |mocks|
+    mocks.syntax = :expect
+  end
 
-  config.use_transactional_fixtures = true
 
-  config.before(:suite) { setup_orm }
-  config.after(:suite) { teardown_orm }
-  config.before(:each) { ActionMailer::Base.deliveries.clear }
+  config.after(:each, :type => :feature) do
+    DatabaseCleaner.clean       # Truncate the database
+    Capybara.reset_sessions!    # Forget the (simulated) browser state
+    Capybara.use_default_driver # Revert Capybara.current_driver to Capybara.default_driver
+  end
 end
 
 
 def restore_default_configuration
-  Authenticate.configuration = nil
-  Authenticate.configure {}
+  puts 'restore_default_configuration called!!!!!!!!!!!!!!!!!!!!!'
 end
 
 def mock_request(params = {})
@@ -59,97 +55,3 @@ def mock_request(params = {})
   allow(req).to receive(:remote_ip).and_return('111.111.111.111')
   req
 end
-
-
-# # This file was generated by the `rails generate rspec:install` command. Conventionally, all
-# # specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
-# # The generated `.rspec` file contains `--require spec_helper` which will cause
-# # this file to always be loaded, without a need to explicitly require it in any
-# # files.
-# #
-# # Given that it is always loaded, you are encouraged to keep this file as
-# # light-weight as possible. Requiring heavyweight dependencies from this file
-# # will add to the boot time of your test suite on EVERY test run, even for an
-# # individual file that may not need all of that loaded. Instead, consider making
-# # a separate helper file that requires the additional dependencies and performs
-# # the additional setup, and require it from the spec files that actually need
-# # it.
-# #
-# # The `.rspec` file also contains a few flags that are not defaults but that
-# # users commonly want.
-# #
-# # See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
-# RSpec.configure do |config|
-#   # rspec-expectations config goes here. You can use an alternate
-#   # assertion/expectation library such as wrong or the stdlib/minitest
-#   # assertions if you prefer.
-#   config.expect_with :rspec do |expectations|
-#     # This option will default to `true` in RSpec 4. It makes the `description`
-#     # and `failure_message` of custom matchers include text for helper methods
-#     # defined using `chain`, e.g.:
-#     #     be_bigger_than(2).and_smaller_than(4).description
-#     #     # => "be bigger than 2 and smaller than 4"
-#     # ...rather than:
-#     #     # => "be bigger than 2"
-#     expectations.include_chain_clauses_in_custom_matcher_descriptions = true
-#   end
-#
-#   # rspec-mocks config goes here. You can use an alternate test double
-#   # library (such as bogus or mocha) by changing the `mock_with` option here.
-#   config.mock_with :rspec do |mocks|
-#     # Prevents you from mocking or stubbing a method that does not exist on
-#     # a real object. This is generally recommended, and will default to
-#     # `true` in RSpec 4.
-#     mocks.verify_partial_doubles = true
-#   end
-#
-# # The settings below are suggested to provide a good initial experience
-# # with RSpec, but feel free to customize to your heart's content.
-# =begin
-#   # These two settings work together to allow you to limit a spec run
-#   # to individual examples or groups you care about by tagging them with
-#   # `:focus` metadata. When nothing is tagged with `:focus`, all examples
-#   # get run.
-#   config.filter_run :focus
-#   config.run_all_when_everything_filtered = true
-#
-#   # Allows RSpec to persist some state between runs in order to support
-#   # the `--only-failures` and `--next-failure` CLI options. We recommend
-#   # you configure your source control system to ignore this file.
-#   config.example_status_persistence_file_path = "spec/examples.txt"
-#
-#   # Limits the available syntax to the non-monkey patched syntax that is
-#   # recommended. For more details, see:
-#   #   - http://rspec.info/blog/2012/06/rspecs-new-expectation-syntax/
-#   #   - http://www.teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/
-#   #   - http://rspec.info/blog/2014/05/notable-changes-in-rspec-3/#zero-monkey-patching-mode
-#   config.disable_monkey_patching!
-#
-#   # Many RSpec users commonly either run the entire suite or an individual
-#   # file, and it's useful to allow more verbose output when running an
-#   # individual spec file.
-#   if config.files_to_run.one?
-#     # Use the documentation formatter for detailed output,
-#     # unless a formatter has already been configured
-#     # (e.g. via a command-line flag).
-#     config.default_formatter = 'doc'
-#   end
-#
-#   # Print the 10 slowest examples and example groups at the
-#   # end of the spec run, to help surface which specs are running
-#   # particularly slow.
-#   config.profile_examples = 10
-#
-#   # Run specs in random order to surface order dependencies. If you find an
-#   # order dependency and want to debug it, you can fix the order by providing
-#   # the seed, which is printed after each run.
-#   #     --seed 1234
-#   config.order = :random
-#
-#   # Seed global randomization in this process using the `--seed` CLI option.
-#   # Setting this allows you to use `--seed` to deterministically reproduce
-#   # test failures related to randomization by passing the same `--seed` value
-#   # as the one that triggered the failure.
-#   Kernel.srand config.seed
-# =end
-# end

data/spec/support/controllers/controller_helpers.rb

@@ -0,0 +1,24 @@
+require 'authenticate/controller'
+
+module Controllers
+  module ControllerHelpers
+
+    def sign_in
+      user = create(:user)
+      sign_in_as user
+    end
+
+    def sign_in_as(user)
+      controller.login user
+    end
+
+    def sign_out
+      controller.logout
+    end
+
+  end
+end
+
+RSpec.configure do |config|
+  config.include Controllers::ControllerHelpers, type: :controller
+end

data/spec/support/features/feature_helpers.rb

@@ -0,0 +1,36 @@
+module Features
+  module FeatureHelpers
+
+
+    def sign_in_with(email, password)
+      visit sign_in_path
+      fill_in 'session_email', with: email
+      fill_in 'session_password', with: password
+      click_button 'Sign in'
+    end
+
+    def sign_out
+      within '#header' do
+        click_link I18n.t("layouts.application.sign_out")
+      end
+    end
+
+    def expect_user_to_be_signed_in
+      visit root_path
+      expect(page).to have_link 'Sign out'
+    end
+
+    def expect_page_to_display_sign_in_error
+      expect(page).to have_content 'Invalid id or password'
+    end
+
+    def expect_user_to_be_signed_out
+      expect(page).to have_content 'Sign in'
+    end
+
+  end
+end
+
+RSpec.configure do |config|
+  config.include Features::FeatureHelpers, type: :feature
+end