Skip to content

Commit

Permalink
Make a basic User model including secure passwords page 267
Browse files Browse the repository at this point in the history
  • Loading branch information
wdk122 committed Jan 30, 2013
1 parent 06bd3ed commit e42afc7
Show file tree
Hide file tree
Showing 8 changed files with 246 additions and 0 deletions.
4 changes: 4 additions & 0 deletions Gemfile
Expand Up @@ -8,6 +8,10 @@ gem 'will_paginate', '3.0.3'
gem 'bootstrap-will_paginate', '0.0.6'
gem 'jquery-rails', '2.0.2'

group :development do
gem 'annotate', '2.5.0'
end

group :development, :test do
gem 'sqlite3', '1.3.5'
gem 'rspec-rails', '2.11.0'
Expand Down
3 changes: 3 additions & 0 deletions Gemfile.lock
Expand Up @@ -28,6 +28,8 @@ GEM
activesupport (3.2.11)
i18n (~> 0.6)
multi_json (~> 1.0)
annotate (2.5.0)
rake
arel (3.0.2)
bcrypt-ruby (3.0.1)
bootstrap-sass (2.1.0.0)
Expand Down Expand Up @@ -188,6 +190,7 @@ PLATFORMS
ruby

DEPENDENCIES
annotate (= 2.5.0)
bcrypt-ruby (= 3.0.1)
bootstrap-sass (= 2.1)
bootstrap-will_paginate (= 0.0.6)
Expand Down
54 changes: 54 additions & 0 deletions app/models/user.rb
@@ -0,0 +1,54 @@
# == Schema Information
#
# Table name: users
#
# id :integer not null, primary key
# name :string(255)
# email :string(255)
# created_at :datetime not null
# updated_at :datetime not null
#

class User < ActiveRecord::Base
attr_accessible :name, :email, :password, :password_confirmation
has_secure_password

before_save { |user| user.email = email.downcase }

validates :name, presence: true, length: { maximum: 50 }
VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
validates :email, presence: true,
format: { with: VALID_EMAIL_REGEX },
uniqueness: { case_sensitive: false }
validates :password, presence: true, length: { minimum: 6 }
validates :password_confirmation, presence: true
end





























10 changes: 10 additions & 0 deletions db/migrate/20130129060634_create_users.rb
@@ -0,0 +1,10 @@
class CreateUsers < ActiveRecord::Migration
def change
create_table :users do |t|
t.string :name
t.string :email

t.timestamps
end
end
end
5 changes: 5 additions & 0 deletions db/migrate/20130129182633_add_index_to_users_email.rb
@@ -0,0 +1,5 @@
class AddIndexToUsersEmail < ActiveRecord::Migration
def change
add_index :users, :email, unique: true
end
end
5 changes: 5 additions & 0 deletions db/migrate/20130129212917_add_password_digest_to_users.rb
@@ -0,0 +1,5 @@
class AddPasswordDigestToUsers < ActiveRecord::Migration
def change
add_column :users, :password_digest, :string
end
end
26 changes: 26 additions & 0 deletions db/schema.rb
@@ -0,0 +1,26 @@
# 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 to check this file into your version control system.

ActiveRecord::Schema.define(:version => 20130129212917) do

create_table "users", :force => true do |t|
t.string "name"
t.string "email"
t.datetime "created_at", :null => false
t.datetime "updated_at", :null => false
t.string "password_digest"
end

add_index "users", ["email"], :name => "index_users_on_email", :unique => true

end
139 changes: 139 additions & 0 deletions spec/models/user_spec.rb
@@ -0,0 +1,139 @@
# == Schema Information
#
# Table name: users
#
# id :integer not null, primary key
# name :string(255)
# email :string(255)
# created_at :datetime not null
# updated_at :datetime not null
#

require 'spec_helper'

describe User do

before do
@user = User.new(name: "Example User", email: "user@example.com", password: "foobar",
password_confirmation: "foobar")
end

subject { @user }

it { should respond_to(:name) }
it { should respond_to(:email) }
it { should respond_to(:password_digest) }
it { should respond_to(:password) }
it { should respond_to(:password_confirmation) }
it { should respond_to(:authenticate) }

it { should be_valid }

describe "when name is not present" do
before { @user.name = " " }
it { should_not be_valid }
end

describe "when email is not present" do
before { @user.email = " " }
it { should_not be_valid }
end

describe "when name is too long" do
before { @user.name = "a" * 51 }
it { should_not be_valid }
end

describe "when email format is invalid" do
it "should be invalid" do
addresses = %w[user@foo,com user_at_foo.org example.user@foo.
foo@bar_baz.com foo@bar+baz.com]
addresses.each do |invalid_address|
@user.email = invalid_address
@user.should_not be_valid
end
end
end

describe "when email format is valid" do
it "should be valid" do
addresses = %w[user@foo.COM A_US-er@f.b.org frst.lst@foo.jp a+b@baz.cn]
addresses.each do |valid_address|
@user.email = valid_address
@user.should be_valid
end
end
end

describe "when email address is already taken" do
before do
user_with_same_email = @user.dup
user_with_same_email.email = @user.email.upcase
user_with_same_email.save
end

it { should_not be_valid }
end

describe "when password is not present" do
before { @user.password = @user.password_confirmation = " " }
it { should_not be_valid }
end

describe "when password doesn't match confirmation" do
before { @user.password_confirmation = "mismatch" }
it { should_not be_valid }
end

describe "when password confirmation is nil" do
before { @user.password_confirmation = nil }
it { should_not be_valid }
end

describe "with a password that's too short" do
before { @user.password = @user.password_confirmation = "a" * 5 }
it { should be_invalid }
end

describe "return value of authenticate method" do
before { @user.save }
let(:found_user) { User.find_by_email(@user.email) }

describe "with valid password" do
it { should == found_user.authenticate(@user.password) }
end

describe "with invalid password" do
let(:user_for_invalid_password) { found_user.authenticate("invalid") }

it { should_not == user_for_invalid_password }
specify { user_for_invalid_password.should be_false }
end
end
end


























0 comments on commit e42afc7

Please sign in to comment.