Merge pull request #2969 from tarlepp/chore/test-classes-to-final

Chore(test) - Make test classes to be `final`

Author: tarlepp

compose.yaml

@@ -2,6 +2,7 @@ services:
     php:
         container_name: symfony-backend-php-fpm
         hostname: php-fpm
+        stop_signal: SIGKILL
         build:
             context: .
             dockerfile: ./Dockerfile_dev
@@ -26,6 +27,7 @@ services:
     nginx:
         container_name: symfony-backend-nginx
         hostname: nginx
+        stop_signal: SIGKILL
         build:
             context: ./docker/nginx/
             dockerfile: ./Dockerfile_dev
@@ -44,6 +46,7 @@ services:
     mariadb:
         container_name: symfony-backend-mariadb
         hostname: mariadb
+        stop_signal: SIGKILL
         build:
             context: ./docker/mariadb/
         command: --character-set-server=utf8mb4 --collation-server=utf8mb4_unicode_ci
@@ -58,6 +61,7 @@ services:
         container_name: symfony-backend-dozzle
         hostname: dozzle
         image: amir20/dozzle
+        stop_signal: SIGKILL
         ports:
             - "8100:8080"
         volumes:
@@ -67,6 +71,7 @@ services:
         container_name: symfony-backend-adminer
         hostname: adminer
         image: adminer
+        stop_signal: SIGKILL
         environment:
             ADMINER_DEFAULT_SERVER: mariadb
             ADMINER_DESIGN: pepa-linha-dark

psalm.xml

@@ -30,7 +30,11 @@
             </errorLevel>
         </ArgumentTypeCoercion>
 
-        <ClassMustBeFinal errorLevel="suppress" />
+        <ClassMustBeFinal>
+            <errorLevel type="suppress">
+                <directory name="src" />
+            </errorLevel>
+        </ClassMustBeFinal>
 
         <ImplicitToStringCast>
             <errorLevel type="suppress">

tests/DataFixtures/AppFixtures.php

@@ -16,7 +16,7 @@
  * @package App\DataFixtures
  * @author TLe, Tarmo Leppänen <tarmo.leppanen@pinja.com>
  */
-class AppFixtures extends Fixture
+final class AppFixtures extends Fixture
 {
     #[Override]
     public function load(ObjectManager $manager): void

tests/E2E/Controller/HealthzControllerTest.php

@@ -17,7 +17,7 @@
  * @package App\Tests\E2E\Controller
  * @author TLe, Tarmo Leppänen <tarmo.leppanen@pinja.com>
  */
-class HealthzControllerTest extends WebTestCase
+final class HealthzControllerTest extends WebTestCase
 {
     /**
      * @throws Throwable

tests/E2E/Controller/IndexControllerTest.php

@@ -16,7 +16,7 @@
  * @package App\Tests\E2E\Controller
  * @author TLe, Tarmo Leppänen <tarmo.leppanen@pinja.com>
  */
-class IndexControllerTest extends WebTestCase
+final class IndexControllerTest extends WebTestCase
 {
     /**
      * @throws Throwable

tests/E2E/Controller/VersionControllerTest.php

@@ -19,7 +19,7 @@
  * @package App\Tests\E2E\Controller
  * @author TLe, Tarmo Leppänen <tarmo.leppanen@pinja.com>
  */
-class VersionControllerTest extends WebTestCase
+final class VersionControllerTest extends WebTestCase
 {
     /**
      * @throws Throwable

tests/E2E/Controller/v1/ApiKey/ApiKeyControllerTest.php

@@ -19,7 +19,7 @@
  * @package App\Tests\E2E\Controller\v1\ApiKey
  * @author TLe, Tarmo Leppänen <tarmo.leppanen@pinja.com>
  */
-class ApiKeyControllerTest extends WebTestCase
+final class ApiKeyControllerTest extends WebTestCase
 {
     private string $baseUrl = '/v1/api_key';
 

tests/E2E/Controller/v1/Auth/GetTokenControllerTest.php

@@ -22,7 +22,7 @@
  * @package App\Tests\E2E\Controller\v1\Auth
  * @author TLe, Tarmo Leppänen <tarmo.leppanen@pinja.com>
  */
-class GetTokenControllerTest extends WebTestCase
+final class GetTokenControllerTest extends WebTestCase
 {
     private string $baseUrl = '/v1/auth/get_token';
 

tests/E2E/Controller/v1/Auth/LoginFailureTest.php

@@ -21,7 +21,7 @@
  * @package App\Tests\E2E\Controller\v1\Auth
  * @author TLe, Tarmo Leppänen <tarmo.leppanen@pinja.com>
  */
-class LoginFailureTest extends WebTestCase
+final class LoginFailureTest extends WebTestCase
 {
     private string $baseUrl = '/v1/auth/get_token';
 

tests/E2E/Controller/v1/Localization/LanguageControllerTest.php

@@ -19,7 +19,7 @@
  * @package App\Tests\E2E\Controller\v1\Localization
  * @author TLe, Tarmo Leppänen <tarmo.leppanen@pinja.com>
  */
-class LanguageControllerTest extends WebTestCase
+final class LanguageControllerTest extends WebTestCase
 {
     private string $baseUrl = '/v1/localization/language';