diff options
author | ankonym <ankonym@gmail.com> | 2015-09-02 16:03:48 +0200 |
---|---|---|
committer | ankonym <ankonym@gmail.com> | 2015-09-28 15:12:45 +0200 |
commit | b3762da8d8a06e164524a20d26293a8d5a9770d8 (patch) | |
tree | 08b2f85a6b087ede8998b0b390f0e0da6e5e0fc0 | |
parent | c48e921c101d49bf68fa1af489b8012517b1a105 (diff) |
Fix three unit tests by passing Factory Girl a valid invite code
The tests were failing because of a hardcoded "testcode" string so during test setup we generate a valid code and pass it to Factory Girl
-rw-r--r-- | test/unit/account_test.rb | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/test/unit/account_test.rb b/test/unit/account_test.rb index b2bfe27..8c66853 100644 --- a/test/unit/account_test.rb +++ b/test/unit/account_test.rb @@ -2,12 +2,17 @@ require 'test_helper' class AccountTest < ActiveSupport::TestCase + setup do + @testcode = InviteCode.new + @testcode.save! + end + teardown do Identity.destroy_all_disabled end test "create a new account" do - user = Account.create(FactoryGirl.attributes_for(:user)) + user = Account.create(FactoryGirl.attributes_for(:user, :invite_code => @testcode.invite_code)) assert user.valid?, "unexpected errors: #{user.errors.inspect}" assert user.persisted? assert id = user.identity @@ -20,14 +25,14 @@ class AccountTest < ActiveSupport::TestCase # We keep an identity that will block the handle from being reused. assert_difference "Identity.count" do assert_no_difference "User.count" do - user = Account.create(FactoryGirl.attributes_for(:user)) + user = Account.create(FactoryGirl.attributes_for(:user, :invite_code => @testcode.invite_code)) user.account.destroy end end end test "change username and create alias" do - user = Account.create(FactoryGirl.attributes_for(:user)) + user = Account.create(FactoryGirl.attributes_for(:user, :invite_code => @testcode.invite_code)) old_id = user.identity old_email = user.email_address user.account.update(FactoryGirl.attributes_for(:user)) |