pr-fix

SirDomin · SirDomin · commit 762cf3b5d68b · 2021-06-23T13:55:25.000+02:00
diff --git a/src/Sylius/Behat/Context/Api/Shop/CheckoutContext.php b/src/Sylius/Behat/Context/Api/Shop/CheckoutContext.php
@@ -424,9 +424,11 @@ public function iTryToSelectPaymentMethod(string $paymentMethodCode): void
      */
     public function iShouldBeNotifiedThatTheOrderShouldBeAddressedFirst()
     {
-        Assert::same($this->ordersClient->getLastResponse()->getStatusCode(), Response::HTTP_UNPROCESSABLE_ENTITY);
+        $response = $this->ordersClient->getLastResponse();
+
+        Assert::same($response->getStatusCode(), Response::HTTP_UNPROCESSABLE_ENTITY);
 
-        $content = json_decode($this->ordersClient->getLastResponse()->getContent(), true);
+        $content = json_decode($response->getContent(), true);
 
         Assert::same($content['message'], 'Order should be addressed first.');
     }
diff --git a/src/Sylius/Bundle/ApiBundle/Exception/OrderCannotBeShippedWithoutAddressing.php b/src/Sylius/Bundle/ApiBundle/Exception/OrderCannotBeShippedWithoutAddressing.php
@@ -13,7 +13,7 @@
 
 namespace Sylius\Bundle\ApiBundle\Exception;
 
-class OrderCannotBeShippedWithoutAddressing extends \RuntimeException
+final class OrderCannotBeShippedWithoutAddressing extends \RuntimeException
 {
     public function __construct()
     {

Original file line number	Diff line number	Diff line change
`@@ -424,9 +424,11 @@ public function iTryToSelectPaymentMethod(string $paymentMethodCode): void`
`424`	`424`	`*/`
`425`	`425`	`public function iShouldBeNotifiedThatTheOrderShouldBeAddressedFirst()`
`426`	`426`	`{`
`427`		`- Assert::same($this->ordersClient->getLastResponse()->getStatusCode(), Response::HTTP_UNPROCESSABLE_ENTITY);`
	`427`	`+ $response = $this->ordersClient->getLastResponse();`
	`428`	`+`
	`429`	`+ Assert::same($response->getStatusCode(), Response::HTTP_UNPROCESSABLE_ENTITY);`
`428`	`430`
`429`		`- $content = json_decode($this->ordersClient->getLastResponse()->getContent(), true);`
	`431`	`+ $content = json_decode($response->getContent(), true);`
`430`	`432`
`431`	`433`	`Assert::same($content['message'], 'Order should be addressed first.');`
`432`	`434`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@`
`13`	`13`
`14`	`14`	`namespace Sylius\Bundle\ApiBundle\Exception;`
`15`	`15`
`16`		`-class OrderCannotBeShippedWithoutAddressing extends \RuntimeException`
	`16`	`+final class OrderCannotBeShippedWithoutAddressing extends \RuntimeException`
`17`	`17`	`{`
`18`	`18`	`public function __construct()`
`19`	`19`	`{`