diff options
author | Kasper Timm Hansen <kaspth@gmail.com> | 2017-01-02 21:10:32 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-01-02 21:10:32 +0100 |
commit | 33e60514aed85b3076f2636d5f1ccfb513aace1c (patch) | |
tree | 56953381ef0764357f22d0ce497400ee43a5598f /activerecord/test/cases | |
parent | 4bf04471f525571146f238e200467724b25689d8 (diff) | |
parent | 059a476c570bc3451df855b99724e74bc2cba604 (diff) | |
download | rails-33e60514aed85b3076f2636d5f1ccfb513aace1c.tar.gz rails-33e60514aed85b3076f2636d5f1ccfb513aace1c.tar.bz2 rails-33e60514aed85b3076f2636d5f1ccfb513aace1c.zip |
Merge pull request #27543 from kamipo/fix_update_counters_of_multiple_records_with_touch
Fix update counters of multiple records with touch: true
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r-- | activerecord/test/cases/counter_cache_test.rb | 10 | ||||
-rw-r--r-- | activerecord/test/cases/timestamp_test.rb | 15 |
2 files changed, 10 insertions, 15 deletions
diff --git a/activerecord/test/cases/counter_cache_test.rb b/activerecord/test/cases/counter_cache_test.rb index c735e13715..c7d0ba32b4 100644 --- a/activerecord/test/cases/counter_cache_test.rb +++ b/activerecord/test/cases/counter_cache_test.rb @@ -227,6 +227,16 @@ class CounterCacheTest < ActiveRecord::TestCase end end + test "update counters of multiple records with touch: true" do + t1, t2 = topics(:first, :second) + + assert_touching t1, :updated_at do + assert_difference ["t1.reload.replies_count", "t2.reload.replies_count"], 2 do + Topic.update_counters([t1.id, t2.id], replies_count: 2, touch: true) + end + end + end + test "update multiple counters with touch: true" do assert_touching @topic, :updated_at do Topic.update_counters(@topic.id, replies_count: 2, unique_replies_count: 2, touch: true) diff --git a/activerecord/test/cases/timestamp_test.rb b/activerecord/test/cases/timestamp_test.rb index 7327926706..7766a74612 100644 --- a/activerecord/test/cases/timestamp_test.rb +++ b/activerecord/test/cases/timestamp_test.rb @@ -430,21 +430,6 @@ class TimestampTest < ActiveRecord::TestCase assert_not_equal person.born_at, nil end - def test_timestamp_attributes_for_create - toy = Toy.first - assert_equal ["created_at", "created_on"], toy.send(:timestamp_attributes_for_create) - end - - def test_timestamp_attributes_for_update - toy = Toy.first - assert_equal ["updated_at", "updated_on"], toy.send(:timestamp_attributes_for_update) - end - - def test_all_timestamp_attributes - toy = Toy.first - assert_equal ["created_at", "created_on", "updated_at", "updated_on"], toy.send(:all_timestamp_attributes) - end - def test_timestamp_attributes_for_create_in_model toy = Toy.first assert_equal ["created_at"], toy.send(:timestamp_attributes_for_create_in_model) |