[ECS] Apply ecs fixes

Sylius · Jan 17, 2024 · b1d58fa · b1d58fa
1 parent 0fc94be
commit b1d58fa
Show file tree

Hide file tree

Showing 5 changed files with 24 additions and 18 deletions.
diff --git a/src/Sylius/Bundle/AdminBundle/Command/ChangeAdminUserPasswordCommand.php b/src/Sylius/Bundle/AdminBundle/Command/ChangeAdminUserPasswordCommand.php
@@ -25,7 +25,7 @@
 
 #[AsCommand(
     name: 'sylius:admin-user:change-password',
-    description: 'Change password of admin user'
+    description: 'Change password of admin user',
 )]
 final class ChangeAdminUserPasswordCommand extends Command
 {

diff --git a/src/Sylius/Bundle/AdminBundle/Command/Factory/QuestionFactoryInterface.php b/src/Sylius/Bundle/AdminBundle/Command/Factory/QuestionFactoryInterface.php
@@ -1,5 +1,16 @@
 <?php
 
+/*
+ * This file is part of the Sylius package.
+ *
+ * (c) Sylius Sp. z o.o.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
 namespace Sylius\Bundle\AdminBundle\Command\Factory;
 
 use Symfony\Component\Console\Question\Question;

diff --git a/src/Sylius/Bundle/AdminBundle/Tests/Command/ChangeAdminUserPasswordCommandTest.php b/src/Sylius/Bundle/AdminBundle/Tests/Command/ChangeAdminUserPasswordCommandTest.php
@@ -77,7 +77,7 @@ public function it_does_not_change_password_when_admin_user_is_not_found(): void
         $this
             ->command
             ->setInputs([
-                'email' => self::EMAIL
+                'email' => self::EMAIL,
             ]);
 
         $this->command->execute([]);
@@ -128,7 +128,7 @@ public function it_changes_password_for_existing_admin_user(): void
             ->command
             ->setInputs([
                 'email' => self::EMAIL,
-                'password' => self::PASSWORD
+                'password' => self::PASSWORD,
             ]);
 
         $this->command->execute([]);

diff --git a/tests/Api/Admin/ProductAssociationsTest.php b/tests/Api/Admin/ProductAssociationsTest.php
@@ -36,8 +36,7 @@ public function it_gets_product_association(): void
                 ->headerBuilder()
                 ->withJsonLdAccept()
                 ->withAdminUserAuthorization('api@example.com')
-                ->build()
-            ,
+                ->build(),
         );
 
         $this->assertResponse(
@@ -62,8 +61,7 @@ public function it_returns_nothing_if_association_not_found(): void
                 ->headerBuilder()
                 ->withJsonLdAccept()
                 ->withAdminUserAuthorization('api@example.com')
-                ->build()
-            ,
+                ->build(),
         );
 
         $response = $this->client->getResponse();
@@ -76,7 +74,7 @@ public function it_returns_product_association_collection(): void
     {
         $this->loadFixturesFromFiles([
             'product/product_with_many_locales.yaml',
-            'authentication/api_administrator.yaml'
+            'authentication/api_administrator.yaml',
         ]);
 
         $this->client->request(
@@ -86,8 +84,7 @@ public function it_returns_product_association_collection(): void
                 ->headerBuilder()
                 ->withJsonLdAccept()
                 ->withAdminUserAuthorization('api@example.com')
-                ->build()
-            ,
+                ->build(),
         );
 
         $this->assertResponse(
@@ -113,8 +110,7 @@ public function it_creates_product_association(): void
                 ->withJsonLdContentType()
                 ->withJsonLdAccept()
                 ->withAdminUserAuthorization('api@example.com')
-                ->build()
-            ,
+                ->build(),
             content: json_encode([
                 'type' => '/api/v2/admin/product-association-types/similar_products',
                 'owner' => '/api/v2/admin/products/CUP',
@@ -149,8 +145,7 @@ public function it_updates_product_association(): void
                 ->withJsonLdContentType()
                 ->withJsonLdAccept()
                 ->withAdminUserAuthorization('api@example.com')
-                ->build()
-            ,
+                ->build(),
             content: json_encode([
                 'associatedProducts' => [
                     '/api/v2/admin/products/TANKARD',
@@ -217,8 +212,7 @@ public function it_does_not_create_product_association_without_required_data():
                 ->withJsonLdContentType()
                 ->withJsonLdAccept()
                 ->withAdminUserAuthorization('api@example.com')
-                ->build()
-            ,
+                ->build(),
             content: '{}',
         );
 
@@ -248,8 +242,7 @@ public function it_does_not_create_duplicated_product_association(): void
                 ->withJsonLdContentType()
                 ->withJsonLdAccept()
                 ->withAdminUserAuthorization('api@example.com')
-                ->build()
-            ,
+                ->build(),
             content: json_encode([
                 'type' => sprintf('/api/v2/admin/product-association-types/%s', $association->getType()->getCode()),
                 'owner' => sprintf('/api/v2/admin/products/%s', $association->getOwner()->getCode()),

diff --git a/tests/Api/Shop/ProductsTest.php b/tests/Api/Shop/ProductsTest.php
@@ -57,6 +57,7 @@ public function it_returns_product_with_translations_in_default_locale(): void
 
     /**
      * @test
+     *
      * @dataProvider getGermanLocales
      */
     public function it_returns_product_with_translations_in_locale_from_header(string $germanLocale): void
@@ -138,6 +139,7 @@ public function it_returns_product_item_with_only_available_associations(): void
 
     /**
      * @test
+     *
      * @dataProvider getPolishLocales
      */
     public function it_returns_product_attributes_collection_with_translations_in_locale_from_header(