From 347a9e6defb5ed9a2aa95e766d795f8c5f9c7598 Mon Sep 17 00:00:00 2001
From: jrfnl <jrfnl@users.noreply.github.com>
Date: Sun, 24 Sep 2023 04:40:46 +0200
Subject: [PATCH] Tests: make all classes either abstract or final

---
 tests/Unit/Assets_Test.php                            | 2 +-
 tests/Unit/Configuration/Configuration_Test.php       | 2 +-
 tests/Unit/Configuration/String_Store_Test.php        | 2 +-
 tests/Unit/Dependencies/Dependency_ACF_Test.php       | 2 +-
 tests/Unit/Dependencies/Dependency_Yoast_SEO_Test.php | 2 +-
 tests/Unit/Doubles/Failing_Dependency_Mock.php        | 2 +-
 tests/Unit/Doubles/Passing_Dependency_Mock.php        | 2 +-
 tests/Unit/Main_Test.php                              | 2 +-
 tests/Unit/Registry_Test.php                          | 2 +-
 tests/Unit/Requirements_Test.php                      | 2 +-
 10 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/tests/Unit/Assets_Test.php b/tests/Unit/Assets_Test.php
index 672730e..a4b15f7 100644
--- a/tests/Unit/Assets_Test.php
+++ b/tests/Unit/Assets_Test.php
@@ -14,7 +14,7 @@
  * @preserveGlobalState disabled
  * @runTestsInSeparateProcesses
  */
-class Assets_Test extends TestCase {
+final class Assets_Test extends TestCase {
 
 	/**
 	 * Test the init hook and determines whether the proper assets are loaded.
diff --git a/tests/Unit/Configuration/Configuration_Test.php b/tests/Unit/Configuration/Configuration_Test.php
index 71841b3..4fcca48 100644
--- a/tests/Unit/Configuration/Configuration_Test.php
+++ b/tests/Unit/Configuration/Configuration_Test.php
@@ -14,7 +14,7 @@
  *
  * @covers Yoast_ACF_Analysis_Configuration
  */
-class Configuration_Test extends TestCase {
+final class Configuration_Test extends TestCase {
 
 	/**
 	 * Tests empty configurations.
diff --git a/tests/Unit/Configuration/String_Store_Test.php b/tests/Unit/Configuration/String_Store_Test.php
index b01228c..40467de 100644
--- a/tests/Unit/Configuration/String_Store_Test.php
+++ b/tests/Unit/Configuration/String_Store_Test.php
@@ -10,7 +10,7 @@
  *
  * @covers \Yoast_ACF_Analysis_String_Store
  */
-class String_Store_Test extends TestCase {
+final class String_Store_Test extends TestCase {
 
 	/**
 	 * Gets the blacklist string store.
diff --git a/tests/Unit/Dependencies/Dependency_ACF_Test.php b/tests/Unit/Dependencies/Dependency_ACF_Test.php
index 727d5d4..8a797a5 100644
--- a/tests/Unit/Dependencies/Dependency_ACF_Test.php
+++ b/tests/Unit/Dependencies/Dependency_ACF_Test.php
@@ -10,7 +10,7 @@
  *
  * @covers Yoast_ACF_Analysis_Dependency_ACF
  */
-class Dependency_ACF_Test extends TestCase {
+final class Dependency_ACF_Test extends TestCase {
 
 	/**
 	 * Tests the situation where no ACF class exists.
diff --git a/tests/Unit/Dependencies/Dependency_Yoast_SEO_Test.php b/tests/Unit/Dependencies/Dependency_Yoast_SEO_Test.php
index 08c1cc9..aa9b609 100644
--- a/tests/Unit/Dependencies/Dependency_Yoast_SEO_Test.php
+++ b/tests/Unit/Dependencies/Dependency_Yoast_SEO_Test.php
@@ -12,7 +12,7 @@
  *
  * @runTestsInSeparateProcesses
  */
-class Dependency_Yoast_SEO_Test extends TestCase {
+final class Dependency_Yoast_SEO_Test extends TestCase {
 
 	/**
 	 * Tests that requirements are not met when Yoast SEO can't be found.
diff --git a/tests/Unit/Doubles/Failing_Dependency_Mock.php b/tests/Unit/Doubles/Failing_Dependency_Mock.php
index a71d282..394fc05 100644
--- a/tests/Unit/Doubles/Failing_Dependency_Mock.php
+++ b/tests/Unit/Doubles/Failing_Dependency_Mock.php
@@ -7,7 +7,7 @@
 /**
  * Class Failing_Dependency_Mock.
  */
-class Failing_Dependency_Mock implements Yoast_ACF_Analysis_Dependency {
+final class Failing_Dependency_Mock implements Yoast_ACF_Analysis_Dependency {
 
 	/**
 	 * Checks if this dependency is met.
diff --git a/tests/Unit/Doubles/Passing_Dependency_Mock.php b/tests/Unit/Doubles/Passing_Dependency_Mock.php
index ab064cd..45c3445 100644
--- a/tests/Unit/Doubles/Passing_Dependency_Mock.php
+++ b/tests/Unit/Doubles/Passing_Dependency_Mock.php
@@ -7,7 +7,7 @@
 /**
  * Class Passing_Dependency_Mock.
  */
-class Passing_Dependency_Mock implements Yoast_ACF_Analysis_Dependency {
+final class Passing_Dependency_Mock implements Yoast_ACF_Analysis_Dependency {
 
 	/**
 	 * Checks if this dependency is met.
diff --git a/tests/Unit/Main_Test.php b/tests/Unit/Main_Test.php
index ab73ffd..f2445ae 100644
--- a/tests/Unit/Main_Test.php
+++ b/tests/Unit/Main_Test.php
@@ -10,7 +10,7 @@
 /**
  * Class Main_Test.
  */
-class Main_Test extends TestCase {
+final class Main_Test extends TestCase {
 
 	/**
 	 * Tests invalid configurations.
diff --git a/tests/Unit/Registry_Test.php b/tests/Unit/Registry_Test.php
index 1f3bcc5..1aac3ba 100644
--- a/tests/Unit/Registry_Test.php
+++ b/tests/Unit/Registry_Test.php
@@ -13,7 +13,7 @@
  *
  * @covers Yoast_ACF_Analysis_Registry
  */
-class Registry_Test extends TestCase {
+final class Registry_Test extends TestCase {
 
 	/**
 	 * Tests that the singleton instance is properly set and that every call to the registry, is the same instance.
diff --git a/tests/Unit/Requirements_Test.php b/tests/Unit/Requirements_Test.php
index 54e5859..cac4a75 100644
--- a/tests/Unit/Requirements_Test.php
+++ b/tests/Unit/Requirements_Test.php
@@ -11,7 +11,7 @@
 /**
  * Class Requirements_Test.
  */
-class Requirements_Test extends TestCase {
+final class Requirements_Test extends TestCase {
 
 	/**
 	 * Sets up test fixtures.