Permalink
Browse files

relaxes assertion

This method returns the status of the operation,
but as we generally do in the code base it does
not commit to any particular exact value. Hence,
we do not have to check for a singleton, because
if the implementation changes and returns some
other true value the test should pass.
  • Loading branch information...
1 parent f5f57e2 commit f32f150e45f3dc3ccbd0adc1c91e6144b3cf1384 @fxn fxn committed Oct 31, 2012
Showing with 1 addition and 1 deletion.
  1. +1 −1 activerecord/test/cases/persistence_test.rb
@@ -516,7 +516,7 @@ def test_update_column_changing_id
def test_update_column_should_return_correct_value
developer = Developer.find(1)
return_value = developer.update_column(:salary, 80001)
- assert_equal return_value, true
+ assert return_value
end
def test_update_attributes

0 comments on commit f32f150

Please sign in to comment.