phpunit: Remove superfluous Title::clearCaches() calls

Why:

- Various tests have been calling Title::clearCaches() on an ad-hoc
  basis at the end of test methods, or in tearDown(), to work around
  stale Title instances persisting across tests in Title's internal
  caches.
- MediaWikiIntegrationTestCase takes care of this since
  Ie6cac7e1282f794277dfc8ff3673f12e9969818c, making these calls
  unnecessary.

What:

- Remove now-redundant Title::clearCaches() calls from tearDown()
  overrides and the end of test methods.
- Preserve in-test Title::clearCaches() calls that are likely
  intended to force internal caches to reflect changes happening
  within the test itself.

Bug: T395214
Change-Id: Ib5d42389cef20f3839c7f7a80041d002f0f09449
(cherry picked from commit 6483181044496c339ad497cc885319b20de7f6e7)

tests/phpunit/includes/ResourceLoader/WikiModuleTest.php

@@ -448,11 +448,6 @@ class WikiModuleTest extends ResourceLoaderTestCase {
 			'Redirect resolved by getContent'
 		);
 	}
-
-	protected function tearDown(): void {
-		Title::clearCaches();
-		parent::tearDown();
-	}
 }
 
 class TestResourceLoaderWikiModule extends WikiModule {

tests/phpunit/includes/api/ApiComparePagesTest.php

@@ -138,8 +138,6 @@ class ApiComparePagesTest extends ApiTestCase {
 			],
 			'comment' => 'Test for ApiComparePages',
 		] );
-
-		Title::clearCaches(); // Otherwise it has the wrong latest revision for some reason
 	}
 
 	protected function doReplacements( &$value ) {

tests/phpunit/includes/linker/LinkRendererTest.php

@@ -304,9 +304,4 @@ class LinkRendererTest extends MediaWikiLangTestCase {
 			$linkRenderer->getLinkClasses( $redirectTitle )
 		);
 	}
-
-	protected function tearDown(): void {
-		Title::clearCaches();
-		parent::tearDown();
-	}
 }

tests/phpunit/includes/parser/ExtraParserTest.php

@@ -274,7 +274,7 @@ class ExtraParserTest extends MediaWikiIntegrationTestCase {
 			MainConfigNames::InterwikiCache,
 			ClassicInterwikiLookup::buildCdbHash( $testInterwikis )
 		);
-		Title::clearCaches();
+
 		$this->parser->startExternalParse(
 			Title::newFromText( __FUNCTION__ ),
 			$this->options,

tests/phpunit/includes/search/TitleMatcherTest.php

@@ -168,9 +168,4 @@ class TitleMatcherTest extends MediaWikiIntegrationTestCase {
 		$result = $matcher->getNearMatchResultSet( 'Test Link Wrong' );
 		$this->assertSame( 0, $result->numRows() );
 	}
-
-	protected function tearDown(): void {
-		Title::clearCaches();
-		parent::tearDown();
-	}
 }

tests/phpunit/includes/title/TitleTest.php

@@ -82,7 +82,6 @@ class TitleTest extends MediaWikiIntegrationTestCase {
 	}
 
 	protected function tearDown(): void {
-		Title::clearCaches();
 		parent::tearDown();
 		// delete dummy pages
 		$this->getNonexistingTestPage( 'UTest1' );

tests/phpunit/includes/title/TitleUrlTest.php

@@ -52,11 +52,6 @@ class TitleUrlTest extends MediaWikiLangTestCase {
 		] );
 	}
 
-	protected function tearDown(): void {
-		Title::clearCaches();
-		parent::tearDown();
-	}
-
 	public function testUrlsForSimpleTitle() {
 		$title = Title::makeTitle( NS_USER, 'Göatee' );
 		$name = $title->getPrefixedURL();