Merge pull request #79 from WendellAdriel/feat/drop-unmapped-data

Clearing data property after DTO validation

.gitignore

@@ -3,5 +3,6 @@
 /.idea
 /.vagrant
 /.phpstorm.*
+/.phpunit.cache
 composer.lock
-.phpunit.result.cache
+.phpunit.result.cache

src/Attributes/Cast.php

@@ -18,6 +18,5 @@ public function __construct(
          * @var class-string
          */
         public ?string $param = null,
-    ) {
-    }
+    ) {}
 }

src/Attributes/DefaultValue.php

@@ -11,6 +11,5 @@ final class DefaultValue
 {
     public function __construct(
         public mixed $value,
-    ) {
-    }
+    ) {}
 }

src/Attributes/Map.php

@@ -12,6 +12,5 @@ final class Map
     public function __construct(
         public ?string $data = null,
         public ?string $transform = null,
-    ) {
-    }
+    ) {}
 }

src/Attributes/Rules.php

@@ -18,6 +18,5 @@ public function __construct(
          * @var array<string, string>
          */
         public array $messages = [],
-    ) {
-    }
+    ) {}
 }

src/Casting/ArrayCast.php

@@ -6,9 +6,7 @@
 
 final class ArrayCast implements Castable
 {
-    public function __construct(private ?Castable $type = null)
-    {
-    }
+    public function __construct(private ?Castable $type = null) {}
 
     public function cast(string $property, mixed $value): array
     {

src/Casting/CarbonCast.php

@@ -13,8 +13,7 @@ final class CarbonCast implements Castable
     public function __construct(
         private ?string $timezone = null,
         private ?string $format = null
-    ) {
-    }
+    ) {}
 
     /**
      * @throws CastException

src/Casting/CarbonImmutableCast.php

@@ -13,8 +13,7 @@ final class CarbonImmutableCast implements Castable
     public function __construct(
         private ?string $timezone = null,
         private ?string $format = null
-    ) {
-    }
+    ) {}
 
     /**
      * @throws CastException

src/Casting/CollectionCast.php

@@ -8,9 +8,7 @@
 
 final class CollectionCast implements Castable
 {
-    public function __construct(private ?Castable $type = null)
-    {
-    }
+    public function __construct(private ?Castable $type = null) {}
 
     public function cast(string $property, mixed $value): Collection
     {

src/Casting/DTOCast.php

@@ -12,9 +12,7 @@
 
 final class DTOCast implements Castable
 {
-    public function __construct(private string $dtoClass)
-    {
-    }
+    public function __construct(private string $dtoClass) {}
 
     /**
      * @throws CastException|CastTargetException|ValidationException

src/Casting/EnumCast.php

@@ -14,9 +14,7 @@ final class EnumCast implements Castable
     /**
      * @param  class-string<UnitEnum|BackedEnum>  $enum
      */
-    public function __construct(protected string $enum)
-    {
-    }
+    public function __construct(protected string $enum) {}
 
     /**
      * @throws CastException|CastTargetException

src/Casting/ModelCast.php

@@ -11,9 +11,7 @@
 
 final class ModelCast implements Castable
 {
-    public function __construct(private string $modelClass)
-    {
-    }
+    public function __construct(private string $modelClass) {}
 
     /**
      * @throws CastException|CastTargetException

src/SimpleDTO.php

@@ -191,6 +191,8 @@ protected function passedValidation(): void
                 $this->validatedData[$key] = $formatted;
             }
         }
+
+        $this->data = [];
     }
 
     protected function failedValidation(): void

src/Support/ResourceCollection.php

@@ -22,8 +22,7 @@ public function __construct(
         private string $dtoClass,
         private int $status = 200,
         private array $headers = []
-    ) {
-    }
+    ) {}
 
     /**
      * @param  Request  $request

tests/Unit/SimpleDTOTest.php

@@ -108,9 +108,7 @@
             = 'test:command
             {name : The name of the user}';
 
-        public function __invoke()
-        {
-        }
+        public function __invoke() {}
     };
 
     Application::starting(function ($artisan) use ($command) {
@@ -132,9 +130,7 @@ public function __invoke()
             = 'test:command
             {--name= : The name of the user}';
 
-        public function __invoke()
-        {
-        }
+        public function __invoke() {}
     };
 
     Application::starting(function ($artisan) use ($command) {
@@ -157,9 +153,7 @@ public function __invoke()
             {name : The name of the user}
             {--age= : The age of the user}';
 
-        public function __invoke()
-        {
-        }
+        public function __invoke() {}
     };
 
     Application::starting(function ($artisan) use ($command) {

tests/Unit/ValidatedDTOTest.php

@@ -140,9 +140,7 @@
             = 'test:command
             {name : The name of the user}';
 
-        public function __invoke()
-        {
-        }
+        public function __invoke() {}
     };
 
     Application::starting(function ($artisan) use ($command) {
@@ -166,9 +164,7 @@ public function __invoke()
             = 'test:command
             {--name= : The name of the user}';
 
-        public function __invoke()
-        {
-        }
+        public function __invoke() {}
     };
 
     Application::starting(function ($artisan) use ($command) {
@@ -193,9 +189,7 @@ public function __invoke()
             {name : The name of the user}
             {--age= : The age of the user}';
 
-        public function __invoke()
-        {
-        }
+        public function __invoke() {}
     };
 
     Application::starting(function ($artisan) use ($command) {