Fix cs

Author: odan

src/Validation/ValidationException.php

@@ -23,8 +23,12 @@ class ValidationException extends DomainException
      * @param int $code [optional] The Exception code
      * @param Throwable|null $previous [optional] The previous throwable used for the exception chaining
      */
-    public function __construct(ValidationResult $validationResult, string $message = '', int $code = 0, Throwable $previous = null)
-    {
+    public function __construct(
+        ValidationResult $validationResult,
+        string $message = '',
+        int $code = 0,
+        Throwable $previous = null
+    ) {
         parent::__construct($message, $code, $previous);
 
         $this->validationResult = $validationResult;

tests/ValidationExceptionTest.php

@@ -4,6 +4,7 @@
 
 use Odan\Validation\ValidationException;
 use PHPUnit\Framework\TestCase;
+use RuntimeException;
 
 /**
  * Tests.
@@ -50,7 +51,13 @@ public function testValidationFailedAction()
      */
     protected function withJson($data): string
     {
-        return json_encode($data);
+        $result = json_encode($data);
+
+        if ($result === false) {
+            throw new RuntimeException('Json encoding failed');
+        }
+
+        return $result;
     }
 
     /**

tests/ValidationResultTest.php

@@ -90,8 +90,9 @@ public function testErrorsWithMessage()
         $result = $val->isFailed();
         $this->assertTrue($result);
         $expected = ['field' => 'email', 'message' => 'required',  'code' => '5000'];
+        $firstError = $val->getFirstError();
 
-        $this->assertEquals($expected, $val->getFirstError()->toArray());
+        $this->assertEquals($expected, $firstError ? $firstError->toArray() : null);
     }
 
     /**
@@ -112,7 +113,9 @@ public function testAddErrorMessage()
 
         $expected = ['field' => 'email', 'message' => 'required',  'code' => '5000'];
 
-        $this->assertEquals($expected, $val->getFirstError()->toArray());
+        $firstError = $val->getFirstError();
+
+        $this->assertEquals($expected, $firstError ? $firstError->toArray() : null);
     }
 
     /**