[fix/test] attributes

1. all attribute classes changed from abstract to final with private constructor (PHP 8.5 compatibility)
2. add reflection-based instantiation tests for attribute classes

Author: ReallifeKip

src/Attributes/ArrayOf.php

@@ -16,7 +16,7 @@
 final class ArrayOf
 {
     /** @param class-string $class */
-    public function __construct(
+    private function __construct(
         public string $class
     ) {}
 }

src/Attributes/KeepOnNull.php

@@ -16,6 +16,8 @@
  * Has no effect on its own.
  */
 #[\Attribute(\Attribute::TARGET_CLASS | \Attribute::TARGET_PROPERTY)]
-abstract class KeepOnNull
+final class KeepOnNull
 {
+    private function __construct()
+    {}
 }

src/Attributes/Lax.php

@@ -12,6 +12,8 @@
  * Takes precedence over both ImmutableBase::strict(true) and #[Strict].
  */
 #[\Attribute(\Attribute::TARGET_CLASS)]
-abstract class Lax
+final class Lax
 {
+    private function __construct()
+    {}
 }

src/Attributes/SkipOnNull.php

@@ -12,6 +12,8 @@
  * Can be overridden per-property with #[KeepOnNull].
  */
 #[\Attribute(\Attribute::TARGET_CLASS | \Attribute::TARGET_PROPERTY)]
-abstract class SkipOnNull
+final class SkipOnNull
 {
+    private function __construct()
+    {}
 }

src/Attributes/Spec.php

@@ -13,6 +13,8 @@
  * @example #[Spec('Email must contain exactly one @ symbol')]
  */
 #[\Attribute(\Attribute::TARGET_CLASS)]
-abstract class Spec
+final class Spec
 {
+    private function __construct()
+    {}
 }

src/Attributes/Strict.php

@@ -13,6 +13,8 @@
  * toggle achieves the same effect without requiring an attribute.
  */
 #[\Attribute(\Attribute::TARGET_CLASS)]
-abstract class Strict
+final class Strict
 {
+    private function __construct()
+    {}
 }

src/Attributes/ValidateFromSelf.php

@@ -13,6 +13,8 @@
  * be checked before parent validators execute.
  */
 #[\Attribute(\Attribute::TARGET_CLASS)]
-abstract class ValidateFromSelf
+final class ValidateFromSelf
 {
+    private function __construct()
+    {}
 }

tests/DefaultTest.php

@@ -6,6 +6,12 @@
 use org\bovigo\vfs\vfsStreamDirectory;
 use PHPUnit\Framework\TestCase;
 use ReallifeKip\ImmutableBase\Attributes\ArrayOf;
+use ReallifeKip\ImmutableBase\Attributes\KeepOnNull;
+use ReallifeKip\ImmutableBase\Attributes\Lax;
+use ReallifeKip\ImmutableBase\Attributes\SkipOnNull;
+use ReallifeKip\ImmutableBase\Attributes\Spec;
+use ReallifeKip\ImmutableBase\Attributes\Strict;
+use ReallifeKip\ImmutableBase\Attributes\ValidateFromSelf;
 use ReallifeKip\ImmutableBase\CLI\Cacher;
 use ReallifeKip\ImmutableBase\Exceptions\DefinitionExceptions\DebugLogDirectoryInvalidException;
 use ReallifeKip\ImmutableBase\Exceptions\DefinitionExceptions\InvalidArrayOfTargetException;
@@ -20,6 +26,7 @@
 use ReallifeKip\ImmutableBase\Exceptions\ValidationExceptions\StrictViolationException;
 use ReallifeKip\ImmutableBase\ImmutableBase;
 use ReallifeKip\ImmutableBase\StaticStatus;
+use ReflectionClass;
 use Tests\DataTransferObjects\DTO;
 use Tests\DataTransferObjects\EmptyArrayOfClassDTO;
 use Tests\DataTransferObjects\ExtraDTO;
@@ -206,7 +213,6 @@ public function testBasic()
         ImmutableBase::strict(true);
         LaxDTO::fromArray([]);
         ImmutableBase::strict(false);
-        new ArrayOf('');
         $nestedDTO = NestedVO::fromArray([
             'nested2' => [
                 'value' => 'svo',
@@ -216,6 +222,24 @@ public function testBasic()
         UnionWithImmutableBaseTypeDTO::fromArray([
             'mixed' => DTO::fromArray($this->array),
         ]);
+
+        foreach (
+            [
+                ArrayOf::class,
+                KeepOnNull::class,
+                Lax::class,
+                SkipOnNull::class,
+                Spec::class,
+                Strict::class,
+                ValidateFromSelf::class,
+            ] as $class
+        ) {
+            $ref         = new ReflectionClass($class);
+            $constructor = $ref->getConstructor();
+            $constructor->setAccessible(true);
+            $instance = $ref->newInstanceWithoutConstructor();
+            $constructor->invoke($instance, '');
+        }
     }
     public function testBasicWithCache()
     {