#105 - array no multiline whitespace (#106)

* #105 - added array whitespace fixer

* #105 - updated tests

composer.json

@@ -36,9 +36,9 @@
   "scripts": {
     "cs": "./vendor/bin/php-cs-fixer fix --dry-run --diff --config codestyle.php",
     "csf": "./vendor/bin/php-cs-fixer fix --diff --config codestyle.php",
-    "test": "./vendor/bin/phpunit tests --colors always",
-    "unit": "./vendor/bin/phpunit tests/unit --colors always",
-    "e2e": "./vendor/bin/phpunit tests/codestyle --colors always"
+    "test": "./vendor/bin/phpunit tests --colors=always",
+    "unit": "./vendor/bin/phpunit tests/unit --colors=always",
+    "e2e": "./vendor/bin/phpunit tests/codestyle --colors=always"
   },
   "bin": [
     "bin/codestyle"

src/Configuration/Defaults/CommonRules.php

@@ -7,6 +7,7 @@
 use Blumilk\Codestyle\Fixers\DoubleQuoteFixer;
 use Blumilk\Codestyle\Fixers\NoLaravelMigrationsGeneratedCommentFixer;
 use PhpCsFixer\Fixer\ArrayNotation\ArraySyntaxFixer;
+use PhpCsFixer\Fixer\ArrayNotation\NoMultilineWhitespaceAroundDoubleArrowFixer;
 use PhpCsFixer\Fixer\ArrayNotation\NoWhitespaceBeforeCommaInArrayFixer;
 use PhpCsFixer\Fixer\ArrayNotation\TrimArraySpacesFixer;
 use PhpCsFixer\Fixer\ArrayNotation\WhitespaceAfterCommaInArrayFixer;
@@ -330,5 +331,6 @@ class CommonRules extends Rules
             "anonymous_functions_opening_brace" => "same_line",
         ],
         LowercaseKeywordsFixer::class => true,
+        NoMultilineWhitespaceAroundDoubleArrowFixer::class => true,
     ];
 }

tests/fixtures/noMultilineWhitespaceAroundDoubleArrow/actual.php

@@ -0,0 +1,24 @@
+<?php
+
+class NoMultilineWhitespaceAroundDoubleArrow
+{
+    public function getArray1(): array
+    {
+        return [
+
+        ];
+    }
+
+    public function getArray2(): array
+    {
+        return [1
+
+        => 2];
+    }
+
+    public function getClosure(): Closure
+    {
+        return fn() =>
+            [];
+    }
+}

tests/fixtures/noMultilineWhitespaceAroundDoubleArrow/expected.php

@@ -0,0 +1,21 @@
+<?php
+
+declare(strict_types=1);
+
+class NoMultilineWhitespaceAroundDoubleArrow
+{
+    public function getArray1(): array
+    {
+        return [];
+    }
+
+    public function getArray2(): array
+    {
+        return [1 => 2];
+    }
+
+    public function getClosure(): Closure
+    {
+        return fn() => [];
+    }
+}