Merge pull request #77 from Yoast/feature/qa-use-final-classes

QA: make most classes `final`
Yoast · Jun 15, 2022 · a6e9d5b · a6e9d5b
2 parents 998c1f4 + a7b4cfc
commit a6e9d5b
Show file tree

Hide file tree

Showing 33 changed files with 33 additions and 33 deletions.
diff --git a/phpunitpolyfills-autoload.php b/phpunitpolyfills-autoload.php
@@ -10,7 +10,7 @@
 	/**
 	 * Custom autoloader.
 	 */
-	class Autoload {
+	final class Autoload {
 
 		/**
 		 * Version number.

diff --git a/tests/Helpers/AssertAttributesHelperTest.php b/tests/Helpers/AssertAttributesHelperTest.php
@@ -10,7 +10,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Helpers\AssertAttributeHelper
  */
-class AssertAttributesHelperTest extends TestCase {
+final class AssertAttributesHelperTest extends TestCase {
 
 	/**
 	 * Instance of the ClassWithProperties class.

diff --git a/tests/Polyfills/AssertClosedResourceBzip2Test.php b/tests/Polyfills/AssertClosedResourceBzip2Test.php
@@ -15,7 +15,7 @@
  *
  * @requires extension bz2
  */
-class AssertClosedResourceBzip2Test extends AssertClosedResourceTestCase {
+final class AssertClosedResourceBzip2Test extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceCurlTest.php b/tests/Polyfills/AssertClosedResourceCurlTest.php
@@ -19,7 +19,7 @@
  * @requires extension curl
  * @requires PHP < 8.0
  */
-class AssertClosedResourceCurlTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceCurlTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceDirTest.php b/tests/Polyfills/AssertClosedResourceDirTest.php
@@ -13,7 +13,7 @@
  * @covers \Yoast\PHPUnitPolyfills\Helpers\ResourceHelper
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertClosedResource
  */
-class AssertClosedResourceDirTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceDirTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceEnchantTest.php b/tests/Polyfills/AssertClosedResourceEnchantTest.php
@@ -22,7 +22,7 @@
  * @phpcs:disable Generic.PHP.DeprecatedFunctions.Deprecated
  * @phpcs:disable PHPCompatibility.FunctionUse.RemovedFunctions.enchant_broker_freeDeprecated
  */
-class AssertClosedResourceEnchantTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceEnchantTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceFileTest.php b/tests/Polyfills/AssertClosedResourceFileTest.php
@@ -13,7 +13,7 @@
  * @covers \Yoast\PHPUnitPolyfills\Helpers\ResourceHelper
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertClosedResource
  */
-class AssertClosedResourceFileTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceFileTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceFinfoTest.php b/tests/Polyfills/AssertClosedResourceFinfoTest.php
@@ -19,7 +19,7 @@
  * @requires extension finfo
  * @requires PHP < 8.1
  */
-class AssertClosedResourceFinfoTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceFinfoTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceGdTest.php b/tests/Polyfills/AssertClosedResourceGdTest.php
@@ -19,7 +19,7 @@
  * @requires extension gd
  * @requires PHP < 8.0
  */
-class AssertClosedResourceGdTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceGdTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceNotResourceTest.php b/tests/Polyfills/AssertClosedResourceNotResourceTest.php
@@ -14,7 +14,7 @@
  * @covers \Yoast\PHPUnitPolyfills\Helpers\ResourceHelper
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertClosedResource
  */
-class AssertClosedResourceNotResourceTest extends TestCase {
+final class AssertClosedResourceNotResourceTest extends TestCase {
 
 	use AssertClosedResource;
 	use ExpectException; // Needed for PHPUnit < 5.2.0 support.

diff --git a/tests/Polyfills/AssertClosedResourceProcessTest.php b/tests/Polyfills/AssertClosedResourceProcessTest.php
@@ -13,7 +13,7 @@
  * @covers \Yoast\PHPUnitPolyfills\Helpers\ResourceHelper
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertClosedResource
  */
-class AssertClosedResourceProcessTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceProcessTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceShmopTest.php b/tests/Polyfills/AssertClosedResourceShmopTest.php
@@ -24,7 +24,7 @@
  * @phpcs:disable Generic.PHP.DeprecatedFunctions.Deprecated
  * @phpcs:disable PHPCompatibility.FunctionUse.RemovedFunctions.shmop_closeDeprecated
  */
-class AssertClosedResourceShmopTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceShmopTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceWddxTest.php b/tests/Polyfills/AssertClosedResourceWddxTest.php
@@ -22,7 +22,7 @@
  * @phpcs:disable PHPCompatibility.FunctionUse.RemovedFunctions.wddx_packet_startRemoved
  * @phpcs:disable PHPCompatibility.FunctionUse.RemovedFunctions.wddx_packet_endRemoved
  */
-class AssertClosedResourceWddxTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceWddxTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceXmlParserTest.php b/tests/Polyfills/AssertClosedResourceXmlParserTest.php
@@ -20,7 +20,7 @@
  * @requires extension libxml
  * @requires PHP < 8.0
  */
-class AssertClosedResourceXmlParserTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceXmlParserTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceZipTest.php b/tests/Polyfills/AssertClosedResourceZipTest.php
@@ -23,7 +23,7 @@
  * @phpcs:disable PHPCompatibility.FunctionUse.RemovedFunctions.zip_closeDeprecated
  * @phpcs:disable WordPress.PHP.NoSilencedErrors.Discouraged
  */
-class AssertClosedResourceZipTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceZipTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertClosedResourceZlibTest.php b/tests/Polyfills/AssertClosedResourceZlibTest.php
@@ -15,7 +15,7 @@
  *
  * @requires extension zlib
  */
-class AssertClosedResourceZlibTest extends AssertClosedResourceTestCase {
+final class AssertClosedResourceZlibTest extends AssertClosedResourceTestCase {
 
 	use AssertClosedResource;
 

diff --git a/tests/Polyfills/AssertEqualsSpecializationsTest.php b/tests/Polyfills/AssertEqualsSpecializationsTest.php
@@ -10,7 +10,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertEqualsSpecializations
  */
-class AssertEqualsSpecializationsTest extends TestCase {
+final class AssertEqualsSpecializationsTest extends TestCase {
 
 	use AssertEqualsSpecializations;
 

diff --git a/tests/Polyfills/AssertFileDirectoryTest.php b/tests/Polyfills/AssertFileDirectoryTest.php
@@ -15,7 +15,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertFileDirectory
  */
-class AssertFileDirectoryTest extends TestCase {
+final class AssertFileDirectoryTest extends TestCase {
 
 	use AssertFileDirectory;
 	use AssertionRenames;

diff --git a/tests/Polyfills/AssertFileEqualsSpecializationsTest.php b/tests/Polyfills/AssertFileEqualsSpecializationsTest.php
@@ -10,7 +10,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertFileEqualsSpecializations
  */
-class AssertFileEqualsSpecializationsTest extends TestCase {
+final class AssertFileEqualsSpecializationsTest extends TestCase {
 
 	use AssertFileEqualsSpecializations;
 

diff --git a/tests/Polyfills/AssertIsTypeTest.php b/tests/Polyfills/AssertIsTypeTest.php
@@ -11,7 +11,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertIsType
  */
-class AssertIsTypeTest extends TestCase {
+final class AssertIsTypeTest extends TestCase {
 
 	use AssertIsType;
 

diff --git a/tests/Polyfills/AssertNumericTypeTest.php b/tests/Polyfills/AssertNumericTypeTest.php
@@ -10,7 +10,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertNumericType
  */
-class AssertNumericTypeTest extends TestCase {
+final class AssertNumericTypeTest extends TestCase {
 
 	use AssertNumericType;
 

diff --git a/tests/Polyfills/AssertObjectEqualsPHPUnitLt940Test.php b/tests/Polyfills/AssertObjectEqualsPHPUnitLt940Test.php
@@ -25,7 +25,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertObjectEquals
  */
-class AssertObjectEqualsPHPUnitLt940Test extends TestCase {
+final class AssertObjectEqualsPHPUnitLt940Test extends TestCase {
 
 	use AssertObjectEquals;
 	use ExpectException; // Needed for PHPUnit < 5.2.0 support.

diff --git a/tests/Polyfills/AssertObjectEqualsTest.php b/tests/Polyfills/AssertObjectEqualsTest.php
@@ -25,7 +25,7 @@
  *
  * @requires PHP 7.0
  */
-class AssertObjectEqualsTest extends TestCase {
+final class AssertObjectEqualsTest extends TestCase {
 
 	use AssertObjectEquals;
 	use ExpectException; // Needed for PHPUnit < 5.2.0 support.

diff --git a/tests/Polyfills/AssertStringContainsTest.php b/tests/Polyfills/AssertStringContainsTest.php
@@ -11,7 +11,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertStringContains
  */
-class AssertStringContainsTest extends TestCase {
+final class AssertStringContainsTest extends TestCase {
 
 	use AssertStringContains;
 	use ExpectException; // Needed for PHPUnit < 5.2.0 support.

diff --git a/tests/Polyfills/AssertionRenamesTest.php b/tests/Polyfills/AssertionRenamesTest.php
@@ -12,7 +12,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\AssertionRenames
  */
-class AssertionRenamesTest extends TestCase {
+final class AssertionRenamesTest extends TestCase {
 
 	use AssertFileDirectory; // Needed for PHPUnit < 5.6.0 support.
 	use AssertionRenames;

diff --git a/tests/Polyfills/EqualToSpecializationsTest.php b/tests/Polyfills/EqualToSpecializationsTest.php
@@ -10,7 +10,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\EqualToSpecializations
  */
-class EqualToSpecializationsTest extends TestCase {
+final class EqualToSpecializationsTest extends TestCase {
 
 	use EqualToSpecializations;
 

diff --git a/tests/Polyfills/ExpectExceptionMessageMatchesTest.php b/tests/Polyfills/ExpectExceptionMessageMatchesTest.php
@@ -12,7 +12,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\ExpectExceptionMessageMatches
  */
-class ExpectExceptionMessageMatchesTest extends TestCase {
+final class ExpectExceptionMessageMatchesTest extends TestCase {
 
 	use ExpectException; // Needed for PHPUnit < 5.2.0 support.
 	use ExpectExceptionMessageMatches;

diff --git a/tests/Polyfills/ExpectExceptionObjectTest.php b/tests/Polyfills/ExpectExceptionObjectTest.php
@@ -12,7 +12,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\ExpectExceptionObject
  */
-class ExpectExceptionObjectTest extends TestCase {
+final class ExpectExceptionObjectTest extends TestCase {
 
 	use ExpectException; // Needed for PHPUnit < 5.2.0 support.
 	use ExpectExceptionObject;

diff --git a/tests/Polyfills/ExpectExceptionTest.php b/tests/Polyfills/ExpectExceptionTest.php
@@ -19,7 +19,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\ExpectException
  */
-class ExpectExceptionTest extends TestCase {
+final class ExpectExceptionTest extends TestCase {
 
 	use AssertionRenames;
 	use ExpectException;

diff --git a/tests/Polyfills/ExpectPHPExceptionTest.php b/tests/Polyfills/ExpectPHPExceptionTest.php
@@ -11,7 +11,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\Polyfills\ExpectPHPException
  */
-class ExpectPHPExceptionTest extends TestCase {
+final class ExpectPHPExceptionTest extends TestCase {
 
 	use ExpectException; // Needed for PHPUnit < 5.2.0 support.
 	use ExpectPHPException;

diff --git a/tests/TestCases/TestCaseTest.php b/tests/TestCases/TestCaseTest.php
@@ -9,7 +9,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\TestCases\TestCase
  */
-class TestCaseTest extends TestCase {
+final class TestCaseTest extends TestCase {
 
 	use TestCaseTestTrait;
 

diff --git a/tests/TestCases/XTestCaseTest.php b/tests/TestCases/XTestCaseTest.php
@@ -9,7 +9,7 @@
  *
  * @covers \Yoast\PHPUnitPolyfills\TestCases\XTestCase
  */
-class XTestCaseTest extends XTestCase {
+final class XTestCaseTest extends XTestCase {
 
 	use TestCaseTestTrait;
 

diff --git a/tests/TestListeners/TestListenerTest.php b/tests/TestListeners/TestListenerTest.php
@@ -14,7 +14,7 @@
  * @covers \Yoast\PHPUnitPolyfills\TestListeners\TestListenerDefaultImplementation
  * @covers \Yoast\PHPUnitPolyfills\TestListeners\TestListenerSnakeCaseMethods
  */
-class TestListenerTest extends TestCase {
+final class TestListenerTest extends TestCase {
 
 	/**
 	 * The current test result instance.