Merge pull request #6 from KaririCode-Framework/develop

fix and enhance trait tests

Author: walmir-silva

.gitattributes

@@ -14,4 +14,5 @@
 /phpstan.neon   export-ignore
 /phpunit.xml    export-ignore
 /psalm.xml      export-ignore
-/Makefile       export-ignore
+/Makefile       export-ignore
+/composer.lock

composer.lock

@@ -6,11 +6,7 @@
 
 interface SanitizationResult
 {
-    public function addError(string $property, string $errorKey, string $message): void;
-
-    public function setSanitizedData(string $property, mixed $value): void;
-
-    public function hasErrors(): bool;
+    public function isValid(): bool;
 
     public function getErrors(): array;
 

src/Contract/SanitizationResult.php

@@ -0,0 +1,36 @@
+<?php
+
+declare(strict_types=1);
+
+namespace KaririCode\Sanitizer\Result;
+
+use KaririCode\ProcessorPipeline\Result\ProcessingResultCollection;
+use KaririCode\Sanitizer\Contract\SanitizationResult as SanitizationResultcontract;
+
+final class SanitizationResult implements SanitizationResultcontract
+{
+    public function __construct(
+        private readonly ProcessingResultCollection $results
+    ) {
+    }
+
+    public function isValid(): bool
+    {
+        return !$this->results->hasErrors();
+    }
+
+    public function getErrors(): array
+    {
+        return $this->results->getErrors();
+    }
+
+    public function getSanitizedData(): array
+    {
+        return $this->results->getProcessedData();
+    }
+
+    public function toArray(): array
+    {
+        return $this->results->toArray();
+    }
+}

src/Contract/SanitizationResultProcessor.php

@@ -6,38 +6,43 @@
 
 use KaririCode\Contract\Processor\ProcessorRegistry;
 use KaririCode\Contract\Sanitizer\Sanitizer as SanitizerContract;
+use KaririCode\ProcessorPipeline\Handler\ProcessorAttributeHandler;
 use KaririCode\ProcessorPipeline\ProcessorBuilder;
 use KaririCode\PropertyInspector\AttributeAnalyzer;
-use KaririCode\PropertyInspector\AttributeHandler;
 use KaririCode\PropertyInspector\Utility\PropertyInspector;
 use KaririCode\Sanitizer\Attribute\Sanitize;
-use KaririCode\Sanitizer\Contract\SanitizationResult;
-use KaririCode\Sanitizer\Processor\DefaultSanitizationResultProcessor;
+use KaririCode\Sanitizer\Contract\SanitizationResult as SanitizationResultContract;
+use KaririCode\Sanitizer\Result\SanitizationResult;
 
-class Sanitizer implements SanitizerContract
+final class Sanitizer implements SanitizerContract
 {
     private const IDENTIFIER = 'sanitizer';
 
-    private ProcessorBuilder $builder;
+    private readonly ProcessorBuilder $builder;
 
     public function __construct(
         private readonly ProcessorRegistry $registry
     ) {
         $this->builder = new ProcessorBuilder($this->registry);
     }
 
-    public function sanitize(mixed $object): SanitizationResult
+    public function sanitize(mixed $object): SanitizationResultContract
     {
-        $attributeHandler = new AttributeHandler(self::IDENTIFIER, $this->builder);
+        $attributeHandler = new ProcessorAttributeHandler(
+            self::IDENTIFIER,
+            $this->builder
+        );
+
         $propertyInspector = new PropertyInspector(
             new AttributeAnalyzer(Sanitize::class)
         );
 
-        $propertyInspector->inspect($object, $attributeHandler);
+        /** @var PropertyAttributeHandler */
+        $handler = $propertyInspector->inspect($object, $attributeHandler);
         $attributeHandler->applyChanges($object);
 
-        $resultProcessor = new DefaultSanitizationResultProcessor();
-
-        return $resultProcessor->process($attributeHandler);
+        return new SanitizationResult(
+            $handler->getProcessingResults()
+        );
     }
 }