Update dependencies

TestConfiguration/Annotation/TestConfig.php → TestConfiguration/Attribute/TestConfig.php

@@ -1,16 +1,13 @@
 <?php declare(strict_types=1);
 
-namespace RichCongress\TestFramework\TestConfiguration\Annotation;
+namespace RichCongress\TestFramework\TestConfiguration\Attribute;
 
 /**
  * Class TestConfig
  *
  * @package    RichCongress\TestFramework\TestConfiguration
  * @author     Nicolas Guilloux <[email protected]>
  * @copyright  2014 - 2020 RichCongress (https://www.richcongress.com)
- *
- * @Annotation
- * @Target({"CLASS", "METHOD"})
  */
 #[\Attribute(\Attribute::IS_REPEATABLE | \Attribute::TARGET_CLASS | \Attribute::TARGET_METHOD)]
 class TestConfig

TestConfiguration/TestConfiguration.php

@@ -2,7 +2,7 @@
 
 namespace RichCongress\TestFramework\TestConfiguration;
 
-use RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig;
+use RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig;
 
 /**
  * Class TestConfiguration

TestConfiguration/TestConfigurationExtractor.php

@@ -2,8 +2,7 @@
 
 namespace RichCongress\TestFramework\TestConfiguration;
 
-use Doctrine\Common\Annotations\AnnotationReader;
-use RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig;
+use RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig;
 
 /**
  * Class TestConfigurationExtractor
@@ -14,9 +13,6 @@
  */
 final class TestConfigurationExtractor
 {
-    /** @var AnnotationReader */
-    private static $annotationReader;
-
     public static function get(string $class, string $method): ?TestConfig
     {
         $reflectionClass = new \ReflectionClass($class);
@@ -46,34 +42,8 @@ public static function getRecursively(string $class, string $method): ?TestConfi
             : null;
     }
 
-    public static function getFromReflection(\Reflector $reflection): ?TestConfig
-    {
-        if (method_exists($reflection, 'getAttributes')) {
-            $attribute = $reflection->getAttributes(TestConfig::class)[0] ?? null;
-
-            if ($attribute !== null) {
-                return $attribute->newInstance();
-            }
-        }
-
-        switch (\get_class($reflection)) {
-            case \ReflectionClass::class:
-                return self::getAnnotationReader()->getClassAnnotation($reflection, TestConfig::class);
-
-            case \ReflectionMethod::class:
-                return self::getAnnotationReader()->getMethodAnnotation($reflection, TestConfig::class);
-
-            default:
-                throw new \LogicException('Unsupported reflector.');
-        }
-    }
-
-    private static function getAnnotationReader(): AnnotationReader
+    public static function getFromReflection(\ReflectionMethod|\ReflectionClass $reflection): ?TestConfig
     {
-        if (self::$annotationReader === null) {
-            self::$annotationReader = new AnnotationReader();
-        }
-
-        return self::$annotationReader;
+        return ($reflection->getAttributes(TestConfig::class)[0] ?? null)?->newInstance();
     }
 }

TestHook/TestConfigurationHook.php

@@ -2,7 +2,7 @@
 
 namespace RichCongress\TestFramework\TestHook;
 
-use RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig;
+use RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig;
 use RichCongress\TestFramework\TestConfiguration\TestConfiguration;
 use RichCongress\TestFramework\TestConfiguration\TestConfigurationExtractor;
 

Tests/Resources/DummyTestCaseWithTestConfig.php

@@ -3,20 +3,19 @@
 namespace RichCongress\TestFramework\Tests\Resources;
 
 use PHPUnit\Framework\TestCase;
-use RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig;
+use RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig;
 
 /**
  * Class DummyTestCaseWithTestConfig
  *
  * @package    RichCongress\TestFramework\Tests\Resources
  * @author     Nicolas Guilloux <[email protected]>
  * @copyright  2014 - 2020 RichCongress (https://www.richcongress.com)
- *
- * @TestConfig({
- *     "test_case_configuration": "this is a test",
- *     "test_case_configuration_2": 10
- * })
  */
+#[TestConfig([
+    'test_case_configuration' => 'this is a test',
+    'test_case_configuration_2' => 10
+])]
 abstract class DummyTestCaseWithTestConfig extends TestCase
 {
 }

Tests/TestConfiguration/TestConfigurationTest.php

@@ -3,7 +3,7 @@
 namespace RichCongress\TestFramework\Tests\TestConfiguration;
 
 use PHPUnit\Framework\TestCase;
-use RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig;
+use RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig;
 use RichCongress\TestFramework\TestConfiguration\TestConfiguration;
 use RichCongress\TestFramework\TestConfiguration\TestConfigurationExtractor;
 
@@ -19,9 +19,7 @@
  */
 final class TestConfigurationTest extends TestCase
 {
-    /**
-     * @TestConfig("custom_configuration")
-     */
+    #[TestConfig('custom_configuration')]
     public function testTestConfiguration(): void
     {
         $testConfig = TestConfigurationExtractor::get(__CLASS__, __FUNCTION__);
@@ -40,15 +38,4 @@ public function testGetFromReflectionWithAttribute(): void
 
         self::assertNotNull($testConfig);
     }
-
-    public function testGetFromReflectionWithUnknownReflector(): void
-    {
-        $this->expectException(\LogicException::class);
-        $this->expectExceptionMessage('Unsupported reflector.');
-
-        $reflector = new \ReflectionObject(new \StdClass());
-        $testConfig = TestConfigurationExtractor::getFromReflection($reflector);
-
-        self::assertNotNull($testConfig);
-    }
 }

Tests/TestConfiguration/WithClassConfigTest.php

@@ -2,7 +2,7 @@
 
 namespace RichCongress\TestFramework\Tests\TestConfiguration;
 
-use RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig;
+use RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig;
 use RichCongress\TestFramework\TestConfiguration\TestConfigurationExtractor;
 use RichCongress\TestFramework\Tests\Resources\DummyTestCaseWithTestConfig;
 
@@ -13,15 +13,13 @@
  * @author     Nicolas Guilloux <[email protected]>
  * @copyright  2014 - 2020 RichCongress (https://www.richcongress.com)
  *
- * @covers \RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig
+ * @covers \RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig
  * @covers \RichCongress\TestFramework\TestConfiguration\TestConfigurationExtractor
- * @TestConfig("test_class_configuration")
  */
+#[TestConfig('test_class_configuration')]
 final class WithClassConfigTest extends DummyTestCaseWithTestConfig
 {
-    /**
-     * @TestConfig("test_method_configuration")
-     */
+    #[TestConfig('test_method_configuration')]
     public function testWithTestConfig(): void
     {
         $testConfig = TestConfigurationExtractor::getRecursively(__CLASS__, __FUNCTION__);

Tests/TestConfiguration/WithTestCaseConfigTest.php

@@ -2,7 +2,7 @@
 
 namespace RichCongress\TestFramework\Tests\TestConfiguration;
 
-use RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig;
+use RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig;
 use RichCongress\TestFramework\TestConfiguration\TestConfigurationExtractor;
 use RichCongress\TestFramework\Tests\Resources\DummyTestCaseWithTestConfig;
 
@@ -13,14 +13,12 @@
  * @author     Nicolas Guilloux <[email protected]>
  * @copyright  2014 - 2020 RichCongress (https://www.richcongress.com)
  *
- * @covers \RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig
+ * @covers \RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig
  * @covers \RichCongress\TestFramework\TestConfiguration\TestConfigurationExtractor
  */
 final class WithTestCaseConfigTest extends DummyTestCaseWithTestConfig
 {
-    /**
-     * @TestConfig("test_method_configuration")
-     */
+    #[TestConfig('test_method_configuration')]
     public function testWithTestConfig(): void
     {
         $testConfig = TestConfigurationExtractor::getRecursively(__CLASS__, __FUNCTION__);

Tests/TestConfiguration/WithoutClassConfigTest.php

@@ -3,7 +3,7 @@
 namespace RichCongress\TestFramework\Tests\TestConfiguration;
 
 use PHPUnit\Framework\TestCase;
-use RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig;
+use RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig;
 use RichCongress\TestFramework\TestConfiguration\TestConfigurationExtractor;
 
 /**
@@ -13,14 +13,12 @@
  * @author     Nicolas Guilloux <[email protected]>
  * @copyright  2014 - 2020 RichCongress (https://www.richcongress.com)
  *
- * @covers \RichCongress\TestFramework\TestConfiguration\Annotation\TestConfig
+ * @covers \RichCongress\TestFramework\TestConfiguration\Attribute\TestConfig
  * @covers \RichCongress\TestFramework\TestConfiguration\TestConfigurationExtractor
  */
 final class WithoutClassConfigTest extends TestCase
 {
-    /**
-     * @TestConfig("test_method_configuration")
-     */
+    #[TestConfig('test_method_configuration')]
     public function testWithTestConfig(): void
     {
         $testConfig = TestConfigurationExtractor::getRecursively(__CLASS__, __FUNCTION__);

composer.json

@@ -16,9 +16,8 @@
         "issues": "https://github.com/richcongress/test-framework/issues"
     },
     "require": {
-        "php": "^7.1|^8.0",
-        "doctrine/annotations": "^1.11",
-        "phpunit/phpunit": "^7|^8|^9"
+        "php": "^8.1",
+        "phpunit/phpunit": "^9.0"
     },
     "require-dev": {
         "phpunit/phpunit": "^9.5",
@@ -29,7 +28,7 @@
             "RichCongress\\TestFramework\\": ""
         },
         "files" : [
-            "TestConfiguration/Annotation/TestConfig.php"
+            "TestConfiguration/Attribute/TestConfig.php"
         ]
     },
     "autoload-dev": {