ISSUE-345: return 409

Author: tatevikg1

src/Validator/UniqueEmailValidator.php

@@ -5,6 +5,7 @@
 namespace PhpList\RestBundle\Validator;
 
 use PhpList\Core\Domain\Repository\Subscription\SubscriberRepository;
+use Symfony\Component\HttpKernel\Exception\ConflictHttpException;
 use Symfony\Component\Validator\Constraint;
 use Symfony\Component\Validator\ConstraintValidator;
 use Symfony\Component\Validator\Exception\UnexpectedTypeException;
@@ -36,9 +37,7 @@ public function validate($value, Constraint $constraint): void
         $existingUser = $this->subscriberRepository->findOneBy(['email' => $value]);
 
         if ($existingUser) {
-            $this->context->buildViolation($constraint->message)
-                ->setParameter('{{ value }}', $value)
-                ->addViolation();
+            throw new ConflictHttpException('Email already exists.');
         }
     }
 }

tests/Integration/Controller/AbstractTestController.php

@@ -240,13 +240,11 @@ protected function assertHttpMethodNotAllowed(): void
     protected function assertHttpConflict(): void
     {
         $this->assertHttpStatusWithJsonContentType(Response::HTTP_CONFLICT);
-
-        self::assertSame(
-            [
-                'message' => 'This resource already exists.',
-            ],
-            $this->getDecodedJsonResponseContent()
-        );
+        $data = $this->getDecodedJsonResponseContent();
+        $this->assertArrayHasKey('message', $data);
+        $this->assertIsString($data['message']);
+        $this->assertNotEmpty($data['message']);
+        $this->assertStringContainsString('already exists', $data['message']);
     }
 
     /**