diff options
author | Yves Senn <yves.senn@gmail.com> | 2015-08-13 09:20:39 +0200 |
---|---|---|
committer | Yves Senn <yves.senn@gmail.com> | 2015-08-13 09:20:39 +0200 |
commit | 48a183ecb9992723a0b7dc8eee1c4bd41cf8d921 (patch) | |
tree | 9cf5019046a88622899a4f0cae396f0476f4f98b /activerecord/test/cases/associations | |
parent | 3b0e1e49e092d49db5f8ca590ae1e4342d16ba5f (diff) | |
download | rails-48a183ecb9992723a0b7dc8eee1c4bd41cf8d921.tar.gz rails-48a183ecb9992723a0b7dc8eee1c4bd41cf8d921.tar.bz2 rails-48a183ecb9992723a0b7dc8eee1c4bd41cf8d921.zip |
use `assert_not` instead of `refute` as mentioned in our guides.
As described in the "Follow Coding Conventions" section in our
contribution guide (http://edgeguides.rubyonrails.org/contributing_to_ruby_on_rails.html#follow-the-coding-conventions)
we favor `assert_not` over `refute`.
While we don't usually make stylistic changes on it's own I opted to do
it in this case. The reason being that test cases are usually copied as
a starting point for new tests. This results in a spread of `refute` in
files that have been using it already.
Diffstat (limited to 'activerecord/test/cases/associations')
-rw-r--r-- | activerecord/test/cases/associations/belongs_to_associations_test.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/activerecord/test/cases/associations/belongs_to_associations_test.rb b/activerecord/test/cases/associations/belongs_to_associations_test.rb index 73cde05504..e7a269c695 100644 --- a/activerecord/test/cases/associations/belongs_to_associations_test.rb +++ b/activerecord/test/cases/associations/belongs_to_associations_test.rb @@ -91,7 +91,7 @@ class BelongsToAssociationsTest < ActiveRecord::TestCase end account = model.new - refute account.valid? + assert_not account.valid? assert_equal [{error: :blank}], account.errors.details[:company] ensure ActiveRecord::Base.belongs_to_required_by_default = original_value @@ -108,7 +108,7 @@ class BelongsToAssociationsTest < ActiveRecord::TestCase end account = model.new - refute account.valid? + assert_not account.valid? assert_equal [{error: :blank}], account.errors.details[:company] ensure ActiveRecord::Base.belongs_to_required_by_default = original_value |