Skip to content

Commit 135af7b

Browse files
authored
Merge pull request rails#51803 from akhilgkrishnan/fix-batch-test
Fix: batch test assertion order
2 parents ad0e312 + 75bdf05 commit 135af7b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

activerecord/test/cases/batches_test.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -390,7 +390,7 @@ def test_in_batches_touch_all_affect_all_records
390390
assert_queries_count(6 + 6) do # 6 selects, 6 updates
391391
Developer.in_batches(of: 2).touch_all(time: time)
392392
end
393-
assert_equal Developer.all.pluck(:updated_at), [time] * Developer.count
393+
assert_equal [time] * Developer.count, Developer.all.pluck(:updated_at)
394394
end
395395

396396
def test_in_batches_touch_all_returns_rows_affected

0 commit comments

Comments
 (0)