diff options
author | jessib <jessib@riseup.net> | 2013-09-19 10:34:14 -0700 |
---|---|---|
committer | jessib <jessib@riseup.net> | 2013-09-19 10:34:14 -0700 |
commit | 8450af4c4c0ef35a0515586f83220762b54d5339 (patch) | |
tree | cff922632b3ad5983ff07d1eb77648b6f5a8b84f /users/test | |
parent | 22e6f781caec701a4d041e0ac183df4c955e89c2 (diff) | |
parent | 3ca376a4070428c862c9db48cba25c3d307955ea (diff) |
Merge pull request #85 from azul/refactor/user_account_shortcut
user.account shortcut to Account.new(user)
Diffstat (limited to 'users/test')
-rw-r--r-- | users/test/integration/browser/account_test.rb | 4 | ||||
-rw-r--r-- | users/test/unit/account_test.rb | 11 |
2 files changed, 5 insertions, 10 deletions
diff --git a/users/test/integration/browser/account_test.rb b/users/test/integration/browser/account_test.rb index a5ec2c5..06f2401 100644 --- a/users/test/integration/browser/account_test.rb +++ b/users/test/integration/browser/account_test.rb @@ -13,9 +13,7 @@ class AccountTest < BrowserIntegrationTest assert page.has_content?("Sign Up") assert_equal '/', current_path assert user = User.find_by_login(username) - assert id = user.identity - id.destroy - user.destroy + user.account.destroy end test "successful login" do diff --git a/users/test/unit/account_test.rb b/users/test/unit/account_test.rb index 39969c0..94a9980 100644 --- a/users/test/unit/account_test.rb +++ b/users/test/unit/account_test.rb @@ -9,15 +9,14 @@ class AccountTest < ActiveSupport::TestCase assert id = user.identity assert_equal user.email_address, id.address assert_equal user.email_address, id.destination - id.destroy - user.destroy + user.account.destroy end test "create and remove a user account" do assert_no_difference "Identity.count" do assert_no_difference "User.count" do user = Account.create(FactoryGirl.attributes_for(:user)) - Account.new(user).destroy + user.account.destroy end end end @@ -26,7 +25,7 @@ class AccountTest < ActiveSupport::TestCase user = Account.create(FactoryGirl.attributes_for(:user)) old_id = user.identity old_email = user.email_address - Account.new(user).update(FactoryGirl.attributes_for(:user)) + user.account.update(FactoryGirl.attributes_for(:user)) user.reload old_id.reload assert user.valid? @@ -37,9 +36,7 @@ class AccountTest < ActiveSupport::TestCase assert_equal user.email_address, id.destination assert_equal user.email_address, old_id.destination assert_equal old_email, old_id.address - old_id.destroy - id.destroy - user.destroy + user.account.destroy end end |