Add cache invalidation tests for tag updates

Author: robzolkos

test/controllers/cards_controller_test.rb

@@ -319,6 +319,27 @@ class CardsControllerTest < ActionDispatch::IntegrationTest
     assert_equal [ tag.title ], @response.parsed_body["tags"]
   end
 
+  test "update as JSON with description and tag_ids busts the card cache key" do
+    card = cards(:logo)
+    original_cache_key = card.cache_key_with_version
+    tag = tags(:mobile)
+
+    travel 1.minute do
+      put card_path(card, format: :json), params: {
+        card: {
+          description: "Updated description",
+          tag_ids: [ tag.id ]
+        }
+      }
+    end
+
+    assert_response :success
+    assert_not_equal original_cache_key, card.reload.cache_key_with_version
+    assert_equal "Updated description", card.description.to_plain_text.strip
+    assert_equal [ tag ], card.tags
+    assert_equal [ tag.title ], @response.parsed_body["tags"]
+  end
+
   test "delete as JSON" do
     card = cards(:logo)
 

test/models/card/taggable_test.rb

@@ -25,4 +25,17 @@ class Card::TaggableTest < ActiveSupport::TestCase
 
     assert_not_equal cards(:logo).tags.last, cards(:paycheck).tags.last
   end
+
+  test "updating just tag_ids touches the card and board" do
+    board = @card.board
+    card_updated_at = @card.updated_at
+    board_updated_at = board.updated_at
+
+    travel 1.minute do
+      @card.update!(tag_ids: [ tags(:web).id, tags(:mobile).id ])
+    end
+
+    assert @card.reload.updated_at > card_updated_at
+    assert board.reload.updated_at > board_updated_at
+  end
 end