Fix cs

Author: veewee

.php_cs.cache

@@ -1 +1 @@
-{"php":"8.0.7","version":"2.19.0","indent":"    ","lineEnding":"\n","rules":{"blank_line_after_namespace":true,"braces":true,"class_definition":true,"constant_case":true,"elseif":true,"function_declaration":true,"indentation_type":true,"line_ending":true,"lowercase_keywords":true,"method_argument_space":{"on_multiline":"ensure_fully_multiline"},"no_break_comment":true,"no_closing_tag":true,"no_spaces_after_function_name":true,"no_spaces_inside_parenthesis":true,"no_trailing_whitespace":true,"no_trailing_whitespace_in_comment":true,"single_blank_line_at_eof":true,"single_class_element_per_statement":{"elements":["property"]},"single_import_per_statement":true,"single_line_after_imports":true,"switch_case_semicolon_to_colon":true,"switch_case_space":true,"visibility_required":true,"encoding":true,"full_opening_tag":true,"align_multiline_comment":true,"array_indentation":true,"declare_strict_types":true,"final_class":true,"global_namespace_import":{"import_classes":true,"import_constants":true,"import_functions":true},"list_syntax":{"syntax":"short"},"lowercase_constants":true,"multiline_comment_opening_closing":true,"native_function_casing":true,"no_empty_phpdoc":true,"no_leading_import_slash":true,"no_superfluous_phpdoc_tags":{"allow_mixed":true},"no_unused_imports":true,"no_useless_else":true,"no_useless_return":true,"ordered_imports":{"imports_order":["class","function","const"]},"ordered_interfaces":true,"php_unit_test_annotation":true,"php_unit_test_case_static_method_calls":{"call_type":"static"},"php_unit_method_casing":{"case":"snake_case"},"single_trait_insert_per_statement":true,"static_lambda":true,"strict_comparison":true,"strict_param":true},"hashes":{"src\/Engine.php":3551558931,"src\/Encoder.php":3739827989,"src\/SimpleEngine.php":4290238216,"src\/HttpBinding\/SoapResponse.php":61122178,"src\/HttpBinding\/SoapRequest.php":1864463774,"src\/Decoder.php":2387333807,"src\/LazyEngine.php":3266250299,"src\/Driver.php":2063335675,"src\/Transport.php":4030453571,"src\/Exception\/RuntimeException.php":1324169754,"src\/Exception\/MetadataException.php":1209161575,"src\/Metadata\/Collection\/XsdTypeCollection.php":4292607689,"src\/Metadata\/Collection\/PropertyCollection.php":4194651118,"src\/Metadata\/Collection\/TypeCollection.php":2172162984,"src\/Metadata\/Collection\/ParameterCollection.php":1402962356,"src\/Metadata\/Collection\/MethodCollection.php":3303625563,"src\/Metadata\/LazyInMemoryMetadata.php":3644769467,"src\/Metadata\/Metadata.php":2999606874,"src\/Metadata\/Model\/Type.php":4034737462,"src\/Metadata\/Model\/XsdType.php":184744432,"src\/Metadata\/Model\/Method.php":2848582160,"src\/Metadata\/Model\/Property.php":3557835267,"src\/Metadata\/Model\/Parameter.php":1678467590,"src\/Metadata\/MetadataProvider.php":1541487960,"tests\/Unit\/HttpBinding\/SoapResponseTest.php":481419200,"tests\/Unit\/HttpBinding\/SoapRequestTest.php":785126059,"tests\/Unit\/SimpleEngineTest.php":367020725,"tests\/Unit\/LazyEngineTest.php":3809437634,"tests\/Unit\/Metadata\/Collection\/PropertyCollectionTest.php":881532723,"tests\/Unit\/Metadata\/Collection\/ParameterCollectionTest.php":3791764129,"tests\/Unit\/Metadata\/Collection\/MethodCollectionTest.php":1077807463,"tests\/Unit\/Metadata\/Collection\/TypeCollectionTest.php":2615673185,"tests\/Unit\/Metadata\/Collection\/XsdTypeCollectionTest.php":2917714206,"tests\/Unit\/Metadata\/LazyInMemoryMetadataTest.php":2020186336,"tests\/Unit\/Metadata\/Model\/MethodTest.php":1711170727,"tests\/Unit\/Metadata\/Model\/ParameterTest.php":2603296264,"tests\/Unit\/Metadata\/Model\/PropertyTest.php":1378006238,"tests\/Unit\/Metadata\/Model\/TypeTest.php":3185340972,"tests\/Unit\/Metadata\/Model\/XsdTypeTest.php":4263506530,"tests\/Fixtures\/DummyTransport.php":2371990076,"tests\/Fixtures\/PassThroughDriver.php":2891185017,"tests\/Fixtures\/InmemoryMetadata.php":2701470598}}
+{"php":"8.0.7","version":"2.19.0","indent":"    ","lineEnding":"\n","rules":{"blank_line_after_namespace":true,"braces":true,"class_definition":true,"constant_case":true,"elseif":true,"function_declaration":true,"indentation_type":true,"line_ending":true,"lowercase_keywords":true,"method_argument_space":{"on_multiline":"ensure_fully_multiline"},"no_break_comment":true,"no_closing_tag":true,"no_spaces_after_function_name":true,"no_spaces_inside_parenthesis":true,"no_trailing_whitespace":true,"no_trailing_whitespace_in_comment":true,"single_blank_line_at_eof":true,"single_class_element_per_statement":{"elements":["property"]},"single_import_per_statement":true,"single_line_after_imports":true,"switch_case_semicolon_to_colon":true,"switch_case_space":true,"visibility_required":true,"encoding":true,"full_opening_tag":true,"align_multiline_comment":true,"array_indentation":true,"declare_strict_types":true,"final_class":true,"global_namespace_import":{"import_classes":true,"import_constants":true,"import_functions":true},"list_syntax":{"syntax":"short"},"lowercase_constants":true,"multiline_comment_opening_closing":true,"native_function_casing":true,"no_empty_phpdoc":true,"no_leading_import_slash":true,"no_superfluous_phpdoc_tags":{"allow_mixed":true},"no_unused_imports":true,"no_useless_else":true,"no_useless_return":true,"ordered_imports":{"imports_order":["class","function","const"]},"ordered_interfaces":true,"php_unit_test_annotation":true,"php_unit_test_case_static_method_calls":{"call_type":"static"},"php_unit_method_casing":{"case":"snake_case"},"single_trait_insert_per_statement":true,"static_lambda":true,"strict_comparison":true,"strict_param":true},"hashes":{"src\/Engine.php":3551558931,"src\/Encoder.php":3739827989,"src\/SimpleEngine.php":4290238216,"src\/HttpBinding\/SoapResponse.php":61122178,"src\/HttpBinding\/SoapRequest.php":1864463774,"src\/Decoder.php":2387333807,"src\/LazyEngine.php":3266250299,"src\/Driver.php":2063335675,"src\/Transport.php":4030453571,"src\/Exception\/RuntimeException.php":1324169754,"src\/Exception\/MetadataException.php":1209161575,"src\/Metadata\/Collection\/XsdTypeCollection.php":1310232606,"src\/Metadata\/Collection\/PropertyCollection.php":2735700407,"src\/Metadata\/Collection\/TypeCollection.php":3153869548,"src\/Metadata\/Collection\/ParameterCollection.php":3283787126,"src\/Metadata\/Collection\/MethodCollection.php":2932134934,"src\/Metadata\/LazyInMemoryMetadata.php":3644769467,"src\/Metadata\/Metadata.php":2999606874,"src\/Metadata\/Model\/Type.php":4034737462,"src\/Metadata\/Model\/XsdType.php":184744432,"src\/Metadata\/Model\/Method.php":2848582160,"src\/Metadata\/Model\/Property.php":3557835267,"src\/Metadata\/Model\/Parameter.php":1678467590,"src\/Metadata\/MetadataProvider.php":1541487960,"tests\/Unit\/HttpBinding\/SoapResponseTest.php":481419200,"tests\/Unit\/HttpBinding\/SoapRequestTest.php":785126059,"tests\/Unit\/SimpleEngineTest.php":367020725,"tests\/Unit\/LazyEngineTest.php":3809437634,"tests\/Unit\/Metadata\/Collection\/PropertyCollectionTest.php":881532723,"tests\/Unit\/Metadata\/Collection\/ParameterCollectionTest.php":3791764129,"tests\/Unit\/Metadata\/Collection\/MethodCollectionTest.php":1077807463,"tests\/Unit\/Metadata\/Collection\/TypeCollectionTest.php":2615673185,"tests\/Unit\/Metadata\/Collection\/XsdTypeCollectionTest.php":2917714206,"tests\/Unit\/Metadata\/LazyInMemoryMetadataTest.php":2020186336,"tests\/Unit\/Metadata\/Model\/MethodTest.php":1711170727,"tests\/Unit\/Metadata\/Model\/ParameterTest.php":2603296264,"tests\/Unit\/Metadata\/Model\/PropertyTest.php":1378006238,"tests\/Unit\/Metadata\/Model\/TypeTest.php":3185340972,"tests\/Unit\/Metadata\/Model\/XsdTypeTest.php":4263506530,"tests\/Fixtures\/DummyTransport.php":2371990076,"tests\/Fixtures\/PassThroughDriver.php":2891185017,"tests\/Fixtures\/InmemoryMetadata.php":2701470598}}

src/Decoder.php

@@ -1,4 +1,4 @@
-<?php
+<?php declare(strict_types=1);
 
 namespace Soap\Engine;
 

src/Driver.php

@@ -1,11 +1,11 @@
 <?php
 
-declare( strict_types=1 );
+declare(strict_types=1);
 
 namespace Soap\Engine;
 
 use Soap\Engine\Metadata\MetadataProvider;
 
-interface Driver extends Encoder, Decoder, MetadataProvider
+interface Driver extends Decoder, Encoder, MetadataProvider
 {
 }

src/Encoder.php

@@ -1,4 +1,4 @@
-<?php
+<?php declare(strict_types=1);
 
 namespace Soap\Engine;
 

src/Exception/MetadataException.php

@@ -4,7 +4,7 @@
 
 namespace Soap\Engine\Exception;
 
-class MetadataException extends RuntimeException
+final class MetadataException extends RuntimeException
 {
     public static function typeNotFound(string $name): self
     {

src/HttpBinding/SoapRequest.php

@@ -1,8 +1,8 @@
-<?php
+<?php declare(strict_types=1);
 
 namespace Soap\Engine\HttpBinding;
 
-class SoapRequest
+final class SoapRequest
 {
     public const SOAP_1_1 = 1;
     public const SOAP_1_2 = 2;

src/HttpBinding/SoapResponse.php

@@ -1,8 +1,8 @@
-<?php
+<?php declare(strict_types=1);
 
 namespace Soap\Engine\HttpBinding;
 
-class SoapResponse
+final class SoapResponse
 {
     private string $payload;
 

src/Metadata/Collection/MethodCollection.php

@@ -13,13 +13,16 @@
 /**
  * @implements IteratorAggregate<Method>
  */
-final class MethodCollection implements IteratorAggregate, Countable
+final class MethodCollection implements Countable, IteratorAggregate
 {
     /**
-     * @var Method[]
+     * @var list<Method>
      */
     private array $methods;
 
+    /**
+     * @no-named-arguments
+     */
     public function __construct(Method ...$methods)
     {
         $this->methods = $methods;

src/Metadata/Collection/ParameterCollection.php

@@ -12,13 +12,16 @@
 /**
  * @implements IteratorAggregate<Parameter>
  */
-final class ParameterCollection implements IteratorAggregate, Countable
+final class ParameterCollection implements Countable, IteratorAggregate
 {
     /**
-     * @var Parameter[]
+     * @var list<Parameter>
      */
     private array $parameters;
 
+    /**
+     * @no-named-arguments
+     */
     public function __construct(Parameter ...$parameters)
     {
         $this->parameters = $parameters;

src/Metadata/Collection/PropertyCollection.php

@@ -12,13 +12,16 @@
 /**
  * @implements IteratorAggregate<Property>
  */
-final class PropertyCollection implements IteratorAggregate, Countable
+final class PropertyCollection implements Countable, IteratorAggregate
 {
     /**
-     * @var Property[]
+     * @var list<Property>
      */
     private array $properties;
 
+    /**
+     * @no-named-arguments
+     */
     public function __construct(Property ...$properties)
     {
         $this->properties = $properties;

src/Metadata/Collection/TypeCollection.php

@@ -13,13 +13,16 @@
 /**
  * @implements IteratorAggregate<Type>
  */
-final class TypeCollection implements IteratorAggregate, Countable
+final class TypeCollection implements Countable, IteratorAggregate
 {
     /**
      * @var list<Type>
      */
     private array $types;
 
+    /**
+     * @no-named-arguments
+     */
     public function __construct(Type ...$types)
     {
         $this->types = $types;

src/Metadata/Collection/XsdTypeCollection.php

@@ -12,13 +12,16 @@
 /**
  * @implements IteratorAggregate<XsdType>
  */
-final class XsdTypeCollection implements IteratorAggregate, Countable
+final class XsdTypeCollection implements Countable, IteratorAggregate
 {
     /**
-     * @var XsdType[]
+     * @var list<XsdType>
      */
     private array $types;
 
+    /**
+     * @no-named-arguments
+     */
     public function __construct(XsdType ...$types)
     {
         $this->types = $types;

src/Metadata/MetadataProvider.php

@@ -1,6 +1,6 @@
 <?php
 
-declare( strict_types=1 );
+declare(strict_types=1);
 
 namespace Soap\Engine\Metadata;
 

src/Transport.php

@@ -1,9 +1,8 @@
-<?php
+<?php declare(strict_types=1);
 
 
 namespace Soap\Engine;
 
-
 use Soap\Engine\HttpBinding\SoapRequest;
 use Soap\Engine\HttpBinding\SoapResponse;
 

tests/Fixtures/DummyTransport.php

@@ -8,7 +8,7 @@
 use Soap\Engine\HttpBinding\SoapResponse;
 use Soap\Engine\Transport;
 
-class DummyTransport implements Transport
+final class DummyTransport implements Transport
 {
     private string $response;
 

tests/Fixtures/InmemoryMetadata.php

@@ -8,7 +8,7 @@
 use Soap\Engine\Metadata\Collection\TypeCollection;
 use Soap\Engine\Metadata\Metadata;
 
-class InmemoryMetadata implements Metadata
+final class InmemoryMetadata implements Metadata
 {
     private TypeCollection $types;
     private MethodCollection $methods;

tests/Fixtures/PassThroughDriver.php

@@ -1,4 +1,4 @@
-<?php
+<?php declare(strict_types=1);
 
 namespace SoapTest\Engine\Fixtures;
 
@@ -7,7 +7,7 @@
 use Soap\Engine\HttpBinding\SoapResponse;
 use Soap\Engine\Metadata\Metadata;
 
-class PassThroughDriver implements Driver
+final class PassThroughDriver implements Driver
 {
     private string $request;
     private array $response;

tests/Unit/HttpBinding/SoapRequestTest.php

@@ -7,16 +7,16 @@
 use PHPUnit\Framework\TestCase;
 use Soap\Engine\HttpBinding\SoapRequest;
 
-class SoapRequestTest extends TestCase
+final class SoapRequestTest extends TestCase
 {
-    public function testRequest()
+    public function test_request()
     {
         $request = new SoapRequest('requestbody', 'location', 'action', SoapRequest::SOAP_1_1, 0);
 
-        self::assertSame('requestbody', $request->getRequest());
-        self::assertSame('location', $request->getLocation());
-        self::assertSame('action', $request->getAction());
-        self::assertSame(SoapRequest::SOAP_1_1, $request->getVersion());
-        self::assertSame(0, $request->getOneWay());
+        static::assertSame('requestbody', $request->getRequest());
+        static::assertSame('location', $request->getLocation());
+        static::assertSame('action', $request->getAction());
+        static::assertSame(SoapRequest::SOAP_1_1, $request->getVersion());
+        static::assertSame(0, $request->getOneWay());
     }
 }

tests/Unit/HttpBinding/SoapResponseTest.php

@@ -7,12 +7,12 @@
 use PHPUnit\Framework\TestCase;
 use Soap\Engine\HttpBinding\SoapResponse;
 
-class SoapResponseTest extends TestCase
+final class SoapResponseTest extends TestCase
 {
-    public function testResponse()
+    public function test_response()
     {
         $request = new SoapResponse('response');
 
-        self::assertSame('response', $request->getPayload());
+        static::assertSame('response', $request->getPayload());
     }
 }

tests/Unit/LazyEngineTest.php

@@ -5,6 +5,7 @@
 namespace SoapTest\Engine;
 
 use PHPUnit\Framework\TestCase;
+use RuntimeException;
 use Soap\Engine\LazyEngine;
 use Soap\Engine\Metadata\Metadata;
 use Soap\Engine\SimpleEngine;
@@ -13,7 +14,7 @@
 use SoapTest\Engine\Fixtures\InmemoryMetadata;
 use SoapTest\Engine\Fixtures\PassThroughDriver;
 
-class LazyEngineTest extends TestCase
+final class LazyEngineTest extends TestCase
 {
     private LazyEngine $engine;
     private Metadata $metadata;
@@ -36,23 +37,23 @@ protected function setUp(): void
         );
     }
 
-    public function testItCanRequestSoap()
+    public function test_it_can_request_soap()
     {
         $response = $this->engine->request('hello', ['world']);
-        self::assertSame(['object'], $response);
+        static::assertSame(['object'], $response);
     }
 
-    public function testItCanLoadMetadata()
+    public function test_it_can_load_metadata()
     {
-        self::assertSame($this->metadata, $this->engine->getMetadata());
+        static::assertSame($this->metadata, $this->engine->getMetadata());
     }
 
-    public function testIt_does_not_load_until_needed(): void
+    public function test_it_does_not_load_until_needed(): void
     {
         $this->expectNotToPerformAssertions();
 
-        new LazyEngine(function () {
-            throw new \RuntimeException('You shall not pass!');
+        new LazyEngine(static function () {
+            throw new RuntimeException('You shall not pass!');
         });
     }
 }

tests/Unit/Metadata/Collection/MethodCollectionTest.php

@@ -11,7 +11,7 @@
 use Soap\Engine\Metadata\Model\Method;
 use Soap\Engine\Metadata\Model\XsdType;
 
-class MethodCollectionTest extends TestCase
+final class MethodCollectionTest extends TestCase
 {
     private MethodCollection $collection;
 
@@ -22,22 +22,22 @@ protected function setUp(): void
         );
     }
 
-    /** @test */
-    public function it_can_iterate_over_methods(): void
+    
+    public function test_it_can_iterate_over_methods(): void
     {
-        self::assertCount(1, $this->collection);
-        self::assertSame([...$this->collection], $this->collection->map(static fn ($item) => $item));
+        static::assertCount(1, $this->collection);
+        static::assertSame([...$this->collection], $this->collection->map(static fn ($item) => $item));
     }
 
-    /** @test */
-    public function it_can_fetch_by_name(): void
+    
+    public function test_it_can_fetch_by_name(): void
     {
         $method = $this->collection->fetchByName('hello');
-        self::assertSame('hello', $method->getName());
+        static::assertSame('hello', $method->getName());
     }
 
-    /** @test */
-    public function it_can_fail_fetching_by_name(): void
+    
+    public function test_it_can_fail_fetching_by_name(): void
     {
         $this->expectException(MetadataException::class);
         $this->collection->fetchByName('nope');

tests/Unit/Metadata/Collection/ParameterCollectionTest.php

@@ -9,7 +9,7 @@
 use Soap\Engine\Metadata\Model\Parameter;
 use Soap\Engine\Metadata\Model\XsdType;
 
-class ParameterCollectionTest extends TestCase
+final class ParameterCollectionTest extends TestCase
 {
     private ParameterCollection $collection;
 
@@ -20,10 +20,10 @@ protected function setUp(): void
         );
     }
 
-    /** @test */
-    public function it_can_iterate_over_parameters(): void
+    
+    public function test_it_can_iterate_over_parameters(): void
     {
-        self::assertCount(1, $this->collection);
-        self::assertSame([...$this->collection], $this->collection->map(static fn ($item) => $item));
+        static::assertCount(1, $this->collection);
+        static::assertSame([...$this->collection], $this->collection->map(static fn ($item) => $item));
     }
 }

tests/Unit/Metadata/Collection/PropertyCollectionTest.php

@@ -9,7 +9,7 @@
 use Soap\Engine\Metadata\Model\Property;
 use Soap\Engine\Metadata\Model\XsdType;
 
-class PropertyCollectionTest extends TestCase
+final class PropertyCollectionTest extends TestCase
 {
     private PropertyCollection $collection;
 
@@ -20,10 +20,10 @@ protected function setUp(): void
         );
     }
 
-    /** @test */
-    public function it_can_iterate_over_properties(): void
+    
+    public function test_it_can_iterate_over_properties(): void
     {
-        self::assertCount(1, $this->collection);
-        self::assertSame([...$this->collection], $this->collection->map(static fn ($item) => $item));
+        static::assertCount(1, $this->collection);
+        static::assertSame([...$this->collection], $this->collection->map(static fn ($item) => $item));
     }
 }