Skip to content

Commit

Permalink
Merge pull request #26 from npauzenga/tasks/fix-user-factories
Browse files Browse the repository at this point in the history
replace user factories with new names
  • Loading branch information
npauzenga committed Nov 24, 2015
2 parents 06cbac9 + 9396699 commit 34589c2
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion spec/features/user_creates_todo_spec.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
require "rails_helper"

feature "User creates a todo" do
let(:user) { create(:user, email_confirmed: true) }
let(:user) { create(:confirmed_user) }

scenario "Successfully" do
visit root_path
Expand Down
2 changes: 1 addition & 1 deletion spec/features/user_deletes_todo_spec.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
require "rails_helper"

feature "User deletes todo" do
let(:user) { create(:user, email_confirmed: true) }
let(:user) { create(:confirmed_user) }

scenario "Successfully" do
visit root_path
Expand Down
2 changes: 1 addition & 1 deletion spec/features/user_signs_in_spec.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
require "rails_helper"

feature "User signs in" do
let(:user) { create(:user, email_confirmed: true) }
let(:user) { create(:confirmed_user) }

scenario "Successfully" do
visit root_path
Expand Down
2 changes: 1 addition & 1 deletion spec/features/user_visits_profile_spec.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
require "rails_helper"

feature "User visits profile" do
let(:user) { create(:user, email_confirmed: true) }
let(:user) { create(:confirmed_user) }

before do
visit root_path
Expand Down
6 changes: 3 additions & 3 deletions spec/models/user_spec.rb
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
require "rails_helper"

RSpec.describe User, type: :model do
let(:user_static_email) { create(:user, email: "test@test.com") }
let(:user_invalid_email) { build(:user, email: "xxx") }
let(:user_static_email) { create(:confirmed_user, email: "test@test.com") }
let(:user_invalid_email) { build(:unconfirmed_user, email: "xxx") }

describe "validations" do
it "rejects an invalid email address" do
Expand All @@ -11,7 +11,7 @@

it "rejects a duplicate email address" do
user_static_email
expect(build(:user, email: "test@test.com")).to_not be_valid
expect(build(:unconfirmed_user, email: "test@test.com")).to_not be_valid
end
end
end

0 comments on commit 34589c2

Please sign in to comment.