Run cs-fixer

ksassnowski · ksassnowski · commit b30286268eff · 2021-12-22T11:39:34.000+01:00
diff --git a/.php-cs-fixer.php b/.php-cs-fixer.php
@@ -14,6 +14,14 @@
 
 $config = Config\Factory::fromRuleSet(new Config\RuleSet\Php80($header), [
     'php_unit_test_class_requires_covers' => false,
+    'class_attributes_separation' => [
+        'elements' => [
+            'const' => 'one',
+            'method' => 'one',
+            'property' => 'one',
+            'trait_import' => 'none',
+        ],
+    ],
     'error_suppression' => [
         'noise_remaining_usages' => false,
     ],
diff --git a/src/Downloader/Middleware/DownloaderMiddlewareAdapter.php b/src/Downloader/Middleware/DownloaderMiddlewareAdapter.php
@@ -23,7 +23,7 @@
 final class DownloaderMiddlewareAdapter implements DownloaderMiddlewareInterface
 {
     public function __construct(
-        private RequestMiddlewareInterface | ResponseMiddlewareInterface $middleware,
+        private RequestMiddlewareInterface|ResponseMiddlewareInterface $middleware,
     ) {
     }
 
diff --git a/src/Extensions/StatsCollectorExtension.php b/src/Extensions/StatsCollectorExtension.php
@@ -27,6 +27,7 @@
 final class StatsCollectorExtension implements ExtensionInterface
 {
     use Configurable;
+
     private ?DateTimeImmutable $startTime = null;
 
     /**
diff --git a/src/Http/Request.php b/src/Http/Request.php
@@ -30,6 +30,7 @@ final class Request implements DroppableInterface
      * @var Closure(Response): Generator<ParseResult>
      */
     private Closure $parseCallback;
+
     private GuzzleRequest $guzzleRequest;
 
     /**
diff --git a/src/Http/Response.php b/src/Http/Response.php
@@ -26,6 +26,7 @@ final class Response implements DroppableInterface
 {
     use HasMetaData;
     use Droppable;
+
     private Crawler $crawler;
 
     public function __construct(private ResponseInterface $response, private Request $request)
diff --git a/src/ItemPipeline/Processors/FakeProcessor.php b/src/ItemPipeline/Processors/FakeProcessor.php
@@ -20,6 +20,7 @@
 final class FakeProcessor implements ItemProcessorInterface
 {
     use Configurable;
+
     private array $calls = [];
 
     public function processItem(ItemInterface $item): ItemInterface
diff --git a/src/ResponseProcessing/Handlers/FakeHandler.php b/src/ResponseProcessing/Handlers/FakeHandler.php
@@ -23,7 +23,9 @@
 final class FakeHandler extends Handler implements MiddlewareInterface
 {
     private array $responseCalls = [];
+
     private array $itemCalls = [];
+
     private array $requestCalls = [];
 
     /**
diff --git a/src/ResponseProcessing/Handlers/HandlerAdapter.php b/src/ResponseProcessing/Handlers/HandlerAdapter.php
@@ -21,7 +21,7 @@
 final class HandlerAdapter implements MiddlewareInterface
 {
     public function __construct(
-        private RequestHandlerInterface | ItemHandlerInterface | ResponseHandlerInterface $handler,
+        private RequestHandlerInterface|ItemHandlerInterface|ResponseHandlerInterface $handler,
     ) {
     }
 
diff --git a/src/ResponseProcessing/ParseResult.php b/src/ResponseProcessing/ParseResult.php
@@ -22,7 +22,7 @@
 
 final class ParseResult
 {
-    private function __construct(private Request | ItemInterface $value)
+    private function __construct(private Request|ItemInterface $value)
     {
     }
 
diff --git a/src/Spider/Configuration/Configuration.php b/src/Spider/Configuration/Configuration.php
@@ -21,7 +21,7 @@
 final class Configuration
 {
     /**
-     * @param string[]                                      $startUrls
+     * @param string[] $startUrls
      * @param class-string<DownloaderMiddlewareInterface>[] $downloaderMiddleware
      * @param class-string<ItemProcessorInterface>[]        $itemProcessors
      * @param class-string<MiddlewareInterface>[]           $spiderMiddleware
diff --git a/tests/Downloader/Middleware/RequestDeduplicationMiddlewareTest.php b/tests/Downloader/Middleware/RequestDeduplicationMiddlewareTest.php
@@ -27,7 +27,9 @@
 final class RequestDeduplicationMiddlewareTest extends TestCase
 {
     use InteractsWithRequestsAndResponses;
+
     private RequestDeduplicationMiddleware $middleware;
+
     private FakeLogger $logger;
 
     protected function setUp(): void
diff --git a/tests/Extensions/ExtensionTestCase.php b/tests/Extensions/ExtensionTestCase.php
@@ -24,6 +24,7 @@
 abstract class ExtensionTestCase extends TestCase
 {
     protected ExtensionInterface $extension;
+
     private FakeDispatcher $dispatcher;
 
     protected function setUp(): void
diff --git a/tests/ItemPipeline/ItemPipelineTest.php b/tests/ItemPipeline/ItemPipelineTest.php
@@ -32,6 +32,7 @@
 final class ItemPipelineTest extends TestCase
 {
     private ItemPipeline $pipeline;
+
     private FakeDispatcher $dispatcher;
 
     protected function setUp(): void

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@`
`23`	`23`	`final class DownloaderMiddlewareAdapter implements DownloaderMiddlewareInterface`
`24`	`24`	`{`
`25`	`25`	`public function __construct(`
`26`		`- private RequestMiddlewareInterface \| ResponseMiddlewareInterface $middleware,`
	`26`	`+ private RequestMiddlewareInterface\|ResponseMiddlewareInterface $middleware,`
`27`	`27`	`) {`
`28`	`28`	`}`
`29`	`29`
Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,7 @@`
`27`	`27`	`final class StatsCollectorExtension implements ExtensionInterface`
`28`	`28`	`{`
`29`	`29`	`use Configurable;`
	`30`	`+`
`30`	`31`	`private ?DateTimeImmutable $startTime = null;`
`31`	`32`
`32`	`33`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,7 @@ final class Response implements DroppableInterface`
`26`	`26`	`{`
`27`	`27`	`use HasMetaData;`
`28`	`28`	`use Droppable;`
	`29`	`+`
`29`	`30`	`private Crawler $crawler;`
`30`	`31`
`31`	`32`	`public function __construct(private ResponseInterface $response, private Request $request)`
Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,7 @@`
`20`	`20`	`final class FakeProcessor implements ItemProcessorInterface`
`21`	`21`	`{`
`22`	`22`	`use Configurable;`
	`23`	`+`
`23`	`24`	`private array $calls = [];`
`24`	`25`
`25`	`26`	`public function processItem(ItemInterface $item): ItemInterface`
Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,9 @@`
`23`	`23`	`final class FakeHandler extends Handler implements MiddlewareInterface`
`24`	`24`	`{`
`25`	`25`	`private array $responseCalls = [];`
	`26`	`+`
`26`	`27`	`private array $itemCalls = [];`
	`28`	`+`
`27`	`29`	`private array $requestCalls = [];`
`28`	`30`
`29`	`31`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@`
`21`	`21`	`final class HandlerAdapter implements MiddlewareInterface`
`22`	`22`	`{`
`23`	`23`	`public function __construct(`
`24`		`- private RequestHandlerInterface \| ItemHandlerInterface \| ResponseHandlerInterface $handler,`
	`24`	`+ private RequestHandlerInterface\|ItemHandlerInterface\|ResponseHandlerInterface $handler,`
`25`	`25`	`) {`
`26`	`26`	`}`
`27`	`27`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@`
`22`	`22`
`23`	`23`	`final class ParseResult`
`24`	`24`	`{`
`25`		`- private function __construct(private Request \| ItemInterface $value)`
	`25`	`+ private function __construct(private Request\|ItemInterface $value)`
`26`	`26`	`{`
`27`	`27`	`}`
`28`	`28`