An email has been sent to <%= @user.email %>. Please click on the link in that message to verify your email address.
diff --git a/bin/rspec b/bin/rspec new file mode 100755 index 0000000..0c86b5c --- /dev/null +++ b/bin/rspec @@ -0,0 +1,16 @@ +#!/usr/bin/env ruby +# +# This file was generated by Bundler. +# +# The application 'rspec' is installed as part of a gem, and +# this file is here to facilitate running it. +# + +require 'pathname' +ENV['BUNDLE_GEMFILE'] ||= File.expand_path("../../Gemfile", + Pathname.new(__FILE__).realpath) + +require 'rubygems' +require 'bundler/setup' + +load Gem.bin_path('rspec-core', 'rspec') diff --git a/config/database.yml.sample b/config/database.yml.sample index d43355b..0cf7198 100644 --- a/config/database.yml.sample +++ b/config/database.yml.sample @@ -57,7 +57,7 @@ development: # Do not set this db to the same as development or production. test: <<: *default - database: committed_test + database: committed_development # As with config/secrets.yml, you never want to store sensitive information, # like your database password, in your source code. If your source code is diff --git a/config/environments/development.rb b/config/environments/development.rb index b55e214..ca361a6 100644 --- a/config/environments/development.rb +++ b/config/environments/development.rb @@ -38,4 +38,6 @@ # Raises error for missing translations # config.action_view.raise_on_missing_translations = true + + config.action_mailer.default_url_options = { :host => "localhost:3000" } end diff --git a/config/routes.rb b/config/routes.rb index 3f66539..ff5d781 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -1,4 +1,21 @@ Rails.application.routes.draw do + + get "signup" => "users#new", as: :signup + post "signup" => "users#create" + get "signin" => "sessions#new", as: :signin + post "signin" => "sessions#create" + delete "signout" => "sessions#destroy" + get "verify/:user_id" => "verifications#new", as: :verify + get "verify/confirm/:token" => "verifications#create", as: :confirm + get "reset_password" => "password_resets#new", as: :reset_password + post "reset_password" => "password_resets#create" + get "reset_password/:token/edit" => "password_resets#edit", as: :edit_reset_password + post "reset_password/:token/edit" => "password_resets#update" + get "dashboard" => "dashboards#new", as: :dashboard + + + + # The priority is based upon order of creation: first created -> highest priority. # See how all your routes lay out with "rake routes". diff --git a/db/migrate/20151004184620_create_users.rb b/db/migrate/20151004184620_create_users.rb new file mode 100644 index 0000000..048c182 --- /dev/null +++ b/db/migrate/20151004184620_create_users.rb @@ -0,0 +1,12 @@ +class CreateUsers < ActiveRecord::Migration + def change + create_table :users do |t| + t.string :email, null: false, index: :unique + t.string :password_digest, null: false + t.string :verification_token + t.datetime :verified_at + t.string :password_reset_token + t.timestamps null: false + end + end +end diff --git a/db/schema.rb b/db/schema.rb new file mode 100644 index 0000000..f3e9ef4 --- /dev/null +++ b/db/schema.rb @@ -0,0 +1,31 @@ +# encoding: UTF-8 +# This file is auto-generated from the current state of the database. Instead +# of editing this file, please use the migrations feature of Active Record to +# incrementally modify your database, and then regenerate this schema definition. +# +# Note that this schema.rb definition is the authoritative source for your +# database schema. If you need to create the application database on another +# system, you should be using db:schema:load, not running all the migrations +# from scratch. The latter is a flawed and unsustainable approach (the more migrations +# you'll amass, the slower it'll run and the greater likelihood for issues). +# +# It's strongly recommended that you check this file into your version control system. + +ActiveRecord::Schema.define(version: 20151004184620) do + + # These are extensions that must be enabled in order to support this database + enable_extension "plpgsql" + + create_table "users", force: :cascade do |t| + t.string "email", null: false + t.string "password_digest", null: false + t.string "verification_token" + t.datetime "verified_at" + t.string "password_reset_token" + t.datetime "created_at", null: false + t.datetime "updated_at", null: false + end + + add_index "users", ["email"], name: "index_users_on_email", using: :btree + +end diff --git a/spec/controllers/dashboards_controller_spec.rb b/spec/controllers/dashboards_controller_spec.rb new file mode 100644 index 0000000..c9e7cbc --- /dev/null +++ b/spec/controllers/dashboards_controller_spec.rb @@ -0,0 +1,5 @@ +require 'rails_helper' + +RSpec.describe DashboardsController, type: :controller do + +end diff --git a/spec/controllers/sessions_controller_spec.rb b/spec/controllers/sessions_controller_spec.rb new file mode 100644 index 0000000..9d02c1e --- /dev/null +++ b/spec/controllers/sessions_controller_spec.rb @@ -0,0 +1,17 @@ +require 'rails_helper' + +RSpec.describe SessionsController, type: :controller do + it "prevents invalid users from signing in" do + user = FactoryGirl.create(:user) + post :create, email: user.email, password: "" + assert_response(:success) + expect(session[:user_id]).not_to eq(user.id) + end + + it "signs in valid users" do + user = FactoryGirl.create(:user) + post :create, email: user.email, password: user.password + assert_response(:redirect) + expect(session[:user_id]).to eq(user.id) + end +end diff --git a/spec/controllers/users_controller_spec.rb b/spec/controllers/users_controller_spec.rb new file mode 100644 index 0000000..300064c --- /dev/null +++ b/spec/controllers/users_controller_spec.rb @@ -0,0 +1,22 @@ +require 'rails_helper' + +RSpec.describe UsersController, type: :controller do + it "creates new users" do + attributes = {user: FactoryGirl.attributes_for(:user) } + expect(User.find_by(email: attributes[:user][:email])).to be_nil + post :create, attributes + assert_response(:redirect) + expect(User.find_by(email: attributes[:user][:email])).not_to be_nil + end + + it "returns the form if there are errors" do + attributes = {user: FactoryGirl.attributes_for(:user) } + attributes[:user][:password] = "" + expect(User.find_by(email: attributes[:user][:email])).to be_nil + post :create, attributes + assert_response(:redirect) + user = assigns[:user] + expect(user.email).to eq(attributes[:user][:email]) + expect(User.find_by(email: attributes[:user][:email])).to be_nil + end +end diff --git a/spec/controllers/verifications_controller_spec.rb b/spec/controllers/verifications_controller_spec.rb new file mode 100644 index 0000000..4a4ec7b --- /dev/null +++ b/spec/controllers/verifications_controller_spec.rb @@ -0,0 +1,5 @@ +require 'rails_helper' + +RSpec.describe VerificationsController, type: :controller do + +end diff --git a/spec/factories/users.rb b/spec/factories/users.rb new file mode 100644 index 0000000..cd2d6c8 --- /dev/null +++ b/spec/factories/users.rb @@ -0,0 +1,8 @@ +FactoryGirl.define do + factory :user do + sequence(:email) { |n| "user#{n}@website.com" } + password "password" + password_confirmation "password" + verified_at Time.now + end +end diff --git a/spec/helpers/dashboards_helper_spec.rb b/spec/helpers/dashboards_helper_spec.rb new file mode 100644 index 0000000..d68481c --- /dev/null +++ b/spec/helpers/dashboards_helper_spec.rb @@ -0,0 +1,14 @@ +require 'rails_helper' + +# Specs in this file have access to a helper object that includes +# the DashboardsHelper. For example: +# +# describe DashboardsHelper do +# describe "string concat" do +# it "concats two strings with spaces" do +# expect(helper.concat_strings("this","that")).to eq("this that") +# end +# end +# end +RSpec.describe DashboardsHelper, type: :helper do +end diff --git a/spec/helpers/sessions_helper_spec.rb b/spec/helpers/sessions_helper_spec.rb new file mode 100644 index 0000000..38c13fc --- /dev/null +++ b/spec/helpers/sessions_helper_spec.rb @@ -0,0 +1,14 @@ +require 'rails_helper' + +# Specs in this file have access to a helper object that includes +# the SessionsHelper. For example: +# +# describe SessionsHelper do +# describe "string concat" do +# it "concats two strings with spaces" do +# expect(helper.concat_strings("this","that")).to eq("this that") +# end +# end +# end +RSpec.describe SessionsHelper, type: :helper do +end diff --git a/spec/helpers/verifications_helper_spec.rb b/spec/helpers/verifications_helper_spec.rb new file mode 100644 index 0000000..ec425a9 --- /dev/null +++ b/spec/helpers/verifications_helper_spec.rb @@ -0,0 +1,14 @@ +require 'rails_helper' + +# Specs in this file have access to a helper object that includes +# the VerificationsHelper. For example: +# +# describe VerificationsHelper do +# describe "string concat" do +# it "concats two strings with spaces" do +# expect(helper.concat_strings("this","that")).to eq("this that") +# end +# end +# end +RSpec.describe VerificationsHelper, type: :helper do +end diff --git a/spec/mailers/previews/user_mailer_preview.rb b/spec/mailers/previews/user_mailer_preview.rb new file mode 100644 index 0000000..957e12b --- /dev/null +++ b/spec/mailers/previews/user_mailer_preview.rb @@ -0,0 +1,4 @@ +# Preview all emails at http://localhost:3000/rails/mailers/user_mailer +class UserMailerPreview < ActionMailer::Preview + +end diff --git a/spec/mailers/user_mailer_spec.rb b/spec/mailers/user_mailer_spec.rb new file mode 100644 index 0000000..4ad85dc --- /dev/null +++ b/spec/mailers/user_mailer_spec.rb @@ -0,0 +1,4 @@ +require "rails_helper" + +RSpec.describe UserMailer, type: :mailer do +end diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb new file mode 100644 index 0000000..5ff2ca8 --- /dev/null +++ b/spec/models/user_spec.rb @@ -0,0 +1,55 @@ +require 'rails_helper' + +RSpec.describe User, type: :model do + it "makes sure the email is present" do + user = FactoryGirl.build(:user, email: nil) + expect(user).not_to be_valid + expect(user.errors[:email]).not_to be_empty + end + + it "ensures the email is well-formed" do + user = FactoryGirl.build(:user, email: "bad fake email") + expect(user).not_to be_valid + expect(user.errors[:email]).not_to be_empty + end + + it "ensures the email is unique" do + user1 = FactoryGirl.create(:user) + user2 = FactoryGirl.build(:user, email: user1.email) + expect(user1).to be_valid + expect(user2).not_to be_valid + expect(user2.errors[:email]).not_to be_empty + end + + it "ensures the password is present when user created" do + user = FactoryGirl.build(:user, password: nil) + expect(user).not_to be_valid + expect(user.errors[:password]).not_to be_empty + end + + it "ensures password confirmation is present on change" do + user = FactoryGirl.build( + :user, + password: "password", + password_confirmation: "" + ) + expect(user).not_to be_valid + expect(user.errors[:password_confirmation]).not_to be_empty + end + + it "ensures password is confirmed on change" do + user = FactoryGirl.build( + :user, + password: "password", + password_confirmation: "123" + ) + expect(user).not_to be_valid + expect(user.errors[:password_confirmation]).not_to be_empty + end + + it "allows user authentication" do + user = FactoryGirl.build(:user) + expect(user.authenticate("password")).to be_truthy + expect(user.authenticate("123")).to be_falsey + end +end diff --git a/spec/rails_helper.rb b/spec/rails_helper.rb new file mode 100644 index 0000000..88ff2d0 --- /dev/null +++ b/spec/rails_helper.rb @@ -0,0 +1,52 @@ +# This file is copied to spec/ when you run 'rails generate rspec:install' +ENV['RAILS_ENV'] ||= 'test' +require File.expand_path('../../config/environment', __FILE__) +# Prevent database truncation if the environment is production +abort("The Rails environment is running in production mode!") if Rails.env.production? +require 'spec_helper' +require 'rspec/rails' +# Add additional requires below this line. Rails is not loaded until this point! + +# Requires supporting ruby files with custom matchers and macros, etc, in +# spec/support/ and its subdirectories. Files matching `spec/**/*_spec.rb` are +# run as spec files by default. This means that files in spec/support that end +# in _spec.rb will both be required and run as specs, causing the specs to be +# run twice. It is recommended that you do not name files matching this glob to +# end with _spec.rb. You can configure this pattern with the --pattern +# option on the command line or in ~/.rspec, .rspec or `.rspec-local`. +# +# The following line is provided for convenience purposes. It has the downside +# of increasing the boot-up time by auto-requiring all files in the support +# directory. Alternatively, in the individual `*_spec.rb` files, manually +# require only the support files necessary. +# +# Dir[Rails.root.join('spec/support/**/*.rb')].each { |f| require f } + +# Checks for pending migrations before tests are run. +# If you are not using ActiveRecord, you can remove this line. +ActiveRecord::Migration.maintain_test_schema! + +RSpec.configure do |config| + # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures + config.fixture_path = "#{::Rails.root}/spec/fixtures" + + # If you're not using ActiveRecord, or you'd prefer not to run each of your + # examples within a transaction, remove the following line or assign false + # instead of true. + config.use_transactional_fixtures = true + + # RSpec Rails can automatically mix in different behaviours to your tests + # based on their file location, for example enabling you to call `get` and + # `post` in specs under `spec/controllers`. + # + # You can disable this behaviour by removing the line below, and instead + # explicitly tag your specs with their type, e.g.: + # + # RSpec.describe UsersController, :type => :controller do + # # ... + # end + # + # The different available types are documented in the features, such as in + # https://relishapp.com/rspec/rspec-rails/docs + config.infer_spec_type_from_file_location! +end diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb new file mode 100644 index 0000000..913e28a --- /dev/null +++ b/spec/spec_helper.rb @@ -0,0 +1,92 @@ +# 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://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax + # - http://www.teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/ + # - http://myronmars.to/n/dev-blog/2014/05/notable-changes-in-rspec-3#new__config_option_to_disable_rspeccore_monkey_patching + 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