Remove usages of assertNoCacheTag as it is deprecated

The patch no longer applies, but it looks solid.

Reroll against 8.6.x

The only remaining instances are in SearchPageCacheTagsTest, or in the legacy tests. I'm not sure if we should postpone this on that conversion, but since @alexpott didn't mention that in the original issue this looks good to go.

+++ b/core/tests/Drupal/Tests/Listeners/DeprecationListenerTrait.php
@@ -121,7 +121,6 @@ public static function getSkippedDeprecations() {
       'AssertLegacyTrait::getAllOptions() is scheduled for removal in Drupal 9.0.0. Use $element->findAll(\'xpath\', \'option\') instead.',
-      'assertNoCacheTag() is deprecated and scheduled for removal in Drupal 9.0.0. Use $this->assertSession()->responseHeaderNotContains() instead. See https://www.drupal.org/node/2864029.',
       'assertNoPattern() is deprecated and scheduled for removal in Drupal 9.0.0. Use $this->assertSession()->responseNotMatches($pattern) instead. See https://www.drupal.org/node/2864262.',

Since the patch does this, it ensures that future conversions won't use the deprecated method, so I think it's fine to get in as is.

Committed 323c56c and pushed to 8.6.x. Thanks!