CS fixes

Author: emodric

tests/lib/Parameters/ParameterType/ChannelTypeTest.php

@@ -5,7 +5,6 @@
 namespace Netgen\Layouts\Sylius\Tests\Parameters\ParameterType;
 
 use Netgen\Layouts\Parameters\ParameterDefinition;
-use Netgen\Layouts\Parameters\ParameterTypeInterface;
 use Netgen\Layouts\Sylius\Parameters\ParameterType\ChannelType;
 use Netgen\Layouts\Sylius\Tests\Stubs\Channel as ChannelStub;
 use Netgen\Layouts\Sylius\Tests\Validator\RepositoryValidatorFactory;
@@ -23,9 +22,6 @@ final class ChannelTypeTest extends TestCase
 {
     use ParameterTypeTestTrait;
 
-    /** @var \Netgen\Layouts\Sylius\Parameters\ParameterType\ChannelType */
-    private ParameterTypeInterface $type;
-
     private MockObject&ChannelRepositoryInterface $repositoryMock;
 
     protected function setUp(): void
@@ -110,30 +106,6 @@ public static function invalidOptionsDataProvider(): iterable
         ];
     }
 
-    public function testValueObjectNull(): void
-    {
-        $this->repositoryMock
-            ->expects(self::once())
-            ->method('find')
-            ->with(self::identicalTo(null))
-            ->willReturn(null);
-
-        self::assertNull($this->type->getValueObject(null));
-    }
-
-    public function testValueObjectIsValidObject(): void
-    {
-        $stub = new ChannelStub(1, 'test', 'test');
-
-        $this->repositoryMock
-            ->expects(self::once())
-            ->method('find')
-            ->with(self::identicalTo(1))
-            ->willReturn($stub);
-
-        self::assertSame($stub, $this->type->getValueObject(1));
-    }
-
     public function testValidationValid(): void
     {
         $this->repositoryMock
@@ -196,4 +168,20 @@ public static function emptyDataProvider(): iterable
             [new ChannelStub(42, 'WEBSHOP', 'Webshop'), false],
         ];
     }
+
+    public function testGetValueObject(): void
+    {
+        $stub = new ChannelStub(1, 'test', 'test');
+
+        $this->repositoryMock
+            ->expects(self::once())
+            ->method('find')
+            ->with(self::identicalTo(1))
+            ->willReturn($stub);
+
+        /** @var \Netgen\Layouts\Sylius\Parameters\ParameterType\ChannelType $type */
+        $type = $this->type;
+
+        self::assertSame($stub, $type->getValueObject(1));
+    }
 }

tests/lib/Parameters/ParameterType/ProductTypeTest.php

@@ -5,7 +5,6 @@
 namespace Netgen\Layouts\Sylius\Tests\Parameters\ParameterType;
 
 use Netgen\Layouts\Parameters\ParameterDefinition;
-use Netgen\Layouts\Parameters\ParameterTypeInterface;
 use Netgen\Layouts\Sylius\Parameters\ParameterType\ProductType;
 use Netgen\Layouts\Sylius\Tests\Stubs\Product as ProductStub;
 use Netgen\Layouts\Sylius\Tests\Validator\RepositoryValidatorFactory;
@@ -23,9 +22,6 @@ final class ProductTypeTest extends TestCase
 {
     use ParameterTypeTestTrait;
 
-    /** @var \Netgen\Layouts\Sylius\Parameters\ParameterType\ProductType */
-    private ParameterTypeInterface $type;
-
     private MockObject&ProductRepositoryInterface $repositoryMock;
 
     protected function setUp(): void
@@ -89,30 +85,6 @@ public static function invalidOptionsDataProvider(): iterable
         ];
     }
 
-    public function testValueObjectNull(): void
-    {
-        $this->repositoryMock
-            ->expects(self::once())
-            ->method('find')
-            ->with(self::identicalTo(null))
-            ->willReturn(null);
-
-        self::assertNull($this->type->getValueObject(null));
-    }
-
-    public function testValueObjectIsValidObject(): void
-    {
-        $stub = new ProductStub(1);
-
-        $this->repositoryMock
-            ->expects(self::once())
-            ->method('find')
-            ->with(self::identicalTo(1))
-            ->willReturn($stub);
-
-        self::assertSame($stub, $this->type->getValueObject(1));
-    }
-
     public function testValidationValid(): void
     {
         $this->repositoryMock
@@ -175,4 +147,20 @@ public static function emptyDataProvider(): iterable
             [new ProductStub(42), false],
         ];
     }
+
+    public function testGetValueObject(): void
+    {
+        $stub = new ProductStub(1);
+
+        $this->repositoryMock
+            ->expects(self::once())
+            ->method('find')
+            ->with(self::identicalTo(1))
+            ->willReturn($stub);
+
+        /** @var \Netgen\Layouts\Sylius\Parameters\ParameterType\ProductType $type */
+        $type = $this->type;
+
+        self::assertSame($stub, $type->getValueObject(1));
+    }
 }

tests/lib/Parameters/ParameterType/TaxonTypeTest.php

@@ -5,7 +5,6 @@
 namespace Netgen\Layouts\Sylius\Tests\Parameters\ParameterType;
 
 use Netgen\Layouts\Parameters\ParameterDefinition;
-use Netgen\Layouts\Parameters\ParameterTypeInterface;
 use Netgen\Layouts\Sylius\Parameters\ParameterType\TaxonType;
 use Netgen\Layouts\Sylius\Tests\Stubs\Taxon as TaxonStub;
 use Netgen\Layouts\Sylius\Tests\Validator\RepositoryValidatorFactory;
@@ -23,9 +22,6 @@ final class TaxonTypeTest extends TestCase
 {
     use ParameterTypeTestTrait;
 
-    /** @var \Netgen\Layouts\Sylius\Parameters\ParameterType\TaxonType */
-    private ParameterTypeInterface $type;
-
     private MockObject&TaxonRepositoryInterface $repositoryMock;
 
     protected function setUp(): void
@@ -89,30 +85,6 @@ public static function invalidOptionsDataProvider(): iterable
         ];
     }
 
-    public function testValueObjectNull(): void
-    {
-        $this->repositoryMock
-            ->expects(self::once())
-            ->method('find')
-            ->with(self::identicalTo(null))
-            ->willReturn(null);
-
-        self::assertNull($this->type->getValueObject(null));
-    }
-
-    public function testValueObjectIsValidObject(): void
-    {
-        $stub = new TaxonStub(1);
-
-        $this->repositoryMock
-            ->expects(self::once())
-            ->method('find')
-            ->with(self::identicalTo(1))
-            ->willReturn($stub);
-
-        self::assertSame($stub, $this->type->getValueObject(1));
-    }
-
     public function testValidationValid(): void
     {
         $this->repositoryMock
@@ -175,4 +147,20 @@ public static function emptyDataProvider(): iterable
             [new TaxonStub(42), false],
         ];
     }
+
+    public function testGetValueObject(): void
+    {
+        $stub = new TaxonStub(1);
+
+        $this->repositoryMock
+            ->expects(self::once())
+            ->method('find')
+            ->with(self::identicalTo(1))
+            ->willReturn($stub);
+
+        /** @var \Netgen\Layouts\Sylius\Parameters\ParameterType\TaxonType $type */
+        $type = $this->type;
+
+        self::assertSame($stub, $type->getValueObject(1));
+    }
 }