[CS][DX] Refactor

src/Sylius/Behat/Behaviour/Toggles.php

@@ -52,7 +52,7 @@ private function assertCheckboxState(NodeElement $toggleableElement, $expectedSt
             throw new \RuntimeException(sprintf(
                 "Toggleable element state is '%s' but expected '%s'.",
                 $toggleableElement->isChecked() ? 'true' : 'false',
-                $expectedState ? 'true' : 'false'
+                $expectedState ? 'true' : 'false',
             ));
         }
     }

src/Sylius/Behat/Client/ApiPlatformClient.php

@@ -38,7 +38,7 @@ public function __construct(
         SharedStorageInterface $sharedStorage,
         string $authorizationHeader,
         string $resource,
-        ?string $section = null
+        ?string $section = null,
     ) {
         $this->client = $client;
         $this->sharedStorage = $sharedStorage;
@@ -77,7 +77,7 @@ public function show(string $id): Response
             $this->resource,
             $id,
             $this->authorizationHeader,
-            $this->getToken()
+            $this->getToken(),
         ));
     }
 
@@ -98,7 +98,7 @@ public function delete(string $id): Response
             $this->resource,
             $id,
             $this->authorizationHeader,
-            $this->getToken()
+            $this->getToken(),
         ));
     }
 
@@ -167,7 +167,7 @@ public function buildUpdateRequest(string $id): void
             $this->resource,
             $id,
             $this->authorizationHeader,
-            $this->getToken()
+            $this->getToken(),
         );
         $this->request->setContent(json_decode($this->client->getResponse()->getContent(), true));
     }
@@ -179,7 +179,7 @@ public function buildCustomUpdateRequest(string $id, string $customSuffix): void
             $this->resource,
             sprintf('%s/%s', $id, $customSuffix),
             $this->authorizationHeader,
-            $this->getToken()
+            $this->getToken(),
         );
     }
 
@@ -263,7 +263,7 @@ private function request(RequestInterface $request): Response
             $request->parameters(),
             $request->files(),
             $request->headers(),
-            $request->content() ?? null
+            $request->content() ?? null,
         );
 
         return $this->getLastResponse();

src/Sylius/Behat/Client/ApiPlatformIriClient.php

@@ -29,7 +29,7 @@ final class ApiPlatformIriClient implements ApiIriClientInterface
     public function __construct(
         AbstractBrowser $client,
         SharedStorageInterface $sharedStorage,
-        string $authorizationHeader
+        string $authorizationHeader,
     ) {
         $this->client = $client;
         $this->sharedStorage = $sharedStorage;
@@ -52,7 +52,7 @@ private function request(RequestInterface $request): Response
             $request->parameters(),
             $request->files(),
             $request->headers(),
-            $request->content() ?? null
+            $request->content() ?? null,
         );
 
         return $this->client->getResponse();

src/Sylius/Behat/Client/ApiPlatformSecurityClient.php

@@ -58,7 +58,7 @@ public function call(): void
             [],
             [],
             ['CONTENT_TYPE' => 'application/json', 'HTTP_ACCEPT' => 'application/json'],
-            json_encode($this->request['body'])
+            json_encode($this->request['body']),
         );
 
         $response = $this->client->getResponse();

src/Sylius/Behat/Client/Request.php

@@ -43,22 +43,22 @@ public static function index(
         ?string $section,
         string $resource,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): RequestInterface {
         $headers = $token ? ['HTTP_' . $authorizationHeader => 'Bearer ' . $token] : [];
 
         return new self(
             sprintf('/api/v2/%s%s', self::prepareSection($section), $resource),
             HttpRequest::METHOD_GET,
-            $headers
+            $headers,
         );
     }
 
     public static function subResourceIndex(?string $section, string $resource, string $id, string $subResource): RequestInterface
     {
         return new self(
             sprintf('/api/v2/%s%s/%s/%s', self::prepareSection($section), $resource, $id, $subResource),
-            HttpRequest::METHOD_GET
+            HttpRequest::METHOD_GET,
         );
     }
 
@@ -67,22 +67,22 @@ public static function show(
         string $resource,
         string $id,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): RequestInterface {
         $headers = $token ? ['HTTP_' . $authorizationHeader => 'Bearer ' . $token] : [];
 
         return new self(
             sprintf('/api/v2/%s%s/%s', self::prepareSection($section), $resource, $id),
             HttpRequest::METHOD_GET,
-            $headers
+            $headers,
         );
     }
 
     public static function create(
         ?string $section,
         string $resource,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): RequestInterface {
         $headers = ['CONTENT_TYPE' => 'application/ld+json'];
         if ($token !== null) {
@@ -92,7 +92,7 @@ public static function create(
         return new self(
             sprintf('/api/v2/%s%s', self::prepareSection($section), $resource),
             HttpRequest::METHOD_POST,
-            $headers
+            $headers,
         );
     }
 
@@ -101,7 +101,7 @@ public static function update(
         string $resource,
         string $id,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): RequestInterface {
         $headers = ['CONTENT_TYPE' => 'application/ld+json'];
         if ($token !== null) {
@@ -111,7 +111,7 @@ public static function update(
         return new self(
             sprintf('/api/v2/%s%s/%s', self::prepareSection($section), $resource, $id),
             HttpRequest::METHOD_PUT,
-            $headers
+            $headers,
         );
     }
 
@@ -120,14 +120,14 @@ public static function delete(
         string $resource,
         string $id,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): RequestInterface {
         $headers = $token ? ['HTTP_' . $authorizationHeader => 'Bearer ' . $token] : [];
 
         return new self(
             sprintf('/api/v2/%s%s/%s', self::prepareSection($section), $resource, $id),
             HttpRequest::METHOD_DELETE,
-            $headers
+            $headers,
         );
     }
 
@@ -141,15 +141,15 @@ public static function customItemAction(?string $section, string $resource, stri
         return new self(
             sprintf('/api/v2/%s%s/%s/%s', self::prepareSection($section), $resource, $id, $action),
             $type,
-            ['CONTENT_TYPE' => 'application/merge-patch+json']
+            ['CONTENT_TYPE' => 'application/merge-patch+json'],
         );
     }
 
     public static function upload(
         ?string $section,
         string $resource,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): RequestInterface {
         $headers = ['CONTENT_TYPE' => 'multipart/form-data'];
         if ($token !== null) {
@@ -159,7 +159,7 @@ public static function upload(
         return new self(
             sprintf('/api/v2/%s%s', self::prepareSection($section), $resource),
             HttpRequest::METHOD_POST,
-            $headers
+            $headers,
         );
     }
 

src/Sylius/Behat/Client/RequestInterface.php

@@ -19,7 +19,7 @@ public static function index(
         ?string $section,
         string $resource,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): self;
 
     public static function subResourceIndex(?string $section, string $resource, string $id, string $subResource): self;
@@ -29,30 +29,30 @@ public static function show(
         string $resource,
         string $id,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): self;
 
     public static function create(
         ?string $section,
         string $resource,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): self;
 
     public static function update(
         ?string $section,
         string $resource,
         string $id,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): self;
 
     public static function delete(
         ?string $section,
         string $resource,
         string $id,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): self;
 
     public static function transition(?string $section, string $resource, string $id, string $transition): self;
@@ -63,7 +63,7 @@ public static function upload(
         ?string $section,
         string $resource,
         string $authorizationHeader,
-        ?string $token = null
+        ?string $token = null,
     ): self;
 
     public static function custom(string $url, string $method, ?string $token = null): self;

src/Sylius/Behat/Client/ResponseChecker.php

@@ -215,8 +215,8 @@ private function getResponseContentValue(Response $response, string $key)
             $content,
             SprintfResponseEscaper::provideMessageWithEscapedResponseContent(
                 'Content could not be parsed to array.',
-                $response
-            )
+                $response,
+            ),
         );
 
         Assert::keyExists($content, $key, sprintf('Expected key "%s" not found. Received response: %s', $key, $response->getContent()));