ItemCacheTagsTest and ShortcutCacheTagsTest needlessly test the render cache.

Rerolled patch from #14

ItemCacheTagsTest was removed with aggregator so title doesn't need it

Left a comment on MR.

Trying to get feedback.

Think my one comment still stands but will leave in review for others. Certainly not a hill I’ll die on lol

Haha, I’m with you on being efficient and avoiding unnecessary duplication of resources.

We could refactor EntityCacheTagsTestBase so that the cache bins are stored in a class member variable within the setUp() method, which is a common practice. However, I always try to be cautious about not straying too far from the original scope.

You have a lot of experience, so I’m always a bit worried about overlooking something.

Let’s see if others can share their thoughts, and we’ll decide from there. :)

Thanks so much for all the effort you put into reviewing so many (often complex) issue queues!