[Address] Handle adress exists validation in case when address does n…

…ot belong to any customer
Sylius · Mar 31, 2021 · c91a315 · c91a315
1 parent e6bd790
commit c91a315
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 1 deletion.
diff --git a/spec/Validator/Address/AddressExistsValidatorSpec.php b/spec/Validator/Address/AddressExistsValidatorSpec.php
@@ -79,4 +79,26 @@ function it_adds_constraint_if_current_user_is_not_address_owner(
 
         $this->validate('ADDRESS_ID', new AddressExists());
     }
+
+    function it_adds_constraint_if_the_address_is_not_owned_by_anyone(
+        AddressInterface $address,
+        AddressRepositoryInterface $addressRepository,
+        LoggedInShopUserProviderInterface $currentUserProvider,
+        ShopUserInterface $shopUser,
+        CustomerInterface $customerOwner,
+        ExecutionContextInterface $executionContext
+    ): void {
+        $addressRepository->findOneBy(['id' => 'ADDRESS_ID'])->willReturn($address);
+
+        $customerOwner->getEmail()->willReturn('oliver@queen.com');
+        $address->getCustomer()->willReturn(null);
+
+        $currentUserProvider->provide()->willReturn($shopUser);
+
+        $shopUser->getEmail()->willReturn('shop@example.com');
+
+        $executionContext->addViolation('sylius.shop_api.address.not_exists')->shouldBeCalled();
+
+        $this->validate('ADDRESS_ID', new AddressExists());
+    }
 }
diff --git a/src/Validator/Address/AddressExistsValidator.php b/src/Validator/Address/AddressExistsValidator.php
@@ -42,8 +42,9 @@ public function validate($id, Constraint $constraint): void
         }
 
         $user = $this->loggedInUserProvider->provide();
+        $customer = $address->getCustomer();
 
-        if ($address->getCustomer()->getEmail() !== $user->getEmail()) {
+        if (null === $customer || $customer->getEmail() !== $user->getEmail()) {
             $this->context->addViolation($constraint->message);
 
             return;