diff options
| author | Azul <azul@leap.se> | 2012-12-18 13:41:41 +0100 |
|---|---|---|
| committer | Azul <azul@leap.se> | 2012-12-18 13:41:41 +0100 |
| commit | 285b23f39765d8346a658a81eca8b70d70b3e9bf (patch) | |
| tree | ddb8f4d433c6f0461f7832ad08475b9a5bae99b2 /users/test/unit/email_test.rb | |
| parent | 6e8a45145722c12dee4d15b33cc28d2b09881e1a (diff) | |
refactored email_alias creation and validation
using CouchRests user.email_aliases.build so the casted_by method is set in the alias
Used this to move the validations into the alias itself. This is where they belong and allows us to render the errors inline along the email field they belong to.
Diffstat (limited to 'users/test/unit/email_test.rb')
| -rw-r--r-- | users/test/unit/email_test.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/users/test/unit/email_test.rb b/users/test/unit/email_test.rb index 1e216d6..5aa2b11 100644 --- a/users/test/unit/email_test.rb +++ b/users/test/unit/email_test.rb @@ -20,9 +20,9 @@ class EmailTest < ActiveSupport::TestCase test "email aliases need to be unique" do email_alias = "valid_alias@domain.net" - @other_user.add_email_alias email_alias + @other_user.email_aliases.build :email => email_alias @other_user.save - @user.add_email_alias email_alias + @user.email_aliases.build :email => email_alias assert @user.changed? assert !@user.save # TODO handle errors @@ -32,7 +32,7 @@ class EmailTest < ActiveSupport::TestCase email_alias = "valid_alias@domain.net" @other_user.email = email_alias @other_user.save - @user.add_email_alias email_alias + @user.email_aliases.build :email => email_alias assert @user.changed? assert !@user.save end |