From 4e0fc9810eeb467690be83c5427f55ad9819db33 Mon Sep 17 00:00:00 2001
From: Pierre PLAZANET <plazanet.pierre@gmail.com>
Date: Thu, 27 May 2021 16:32:28 +0200
Subject: [PATCH] fix: white space before semi-colon on interface functions
 (#141)

* test: try to reproduce regression

* fix: white space before semi-colon on interface functions
---
 .../LineBreakBetweenMethodArgumentsFixer.php  |  5 +-
 .../Regression/Case6.php                      | 58 +++++++++++++++++++
 2 files changed, 62 insertions(+), 1 deletion(-)
 create mode 100644 tests/UseCase/LineBreakBetweenMethods/Regression/Case6.php

diff --git a/src/PedroTroller/CS/Fixer/CodingStyle/LineBreakBetweenMethodArgumentsFixer.php b/src/PedroTroller/CS/Fixer/CodingStyle/LineBreakBetweenMethodArgumentsFixer.php
index 4a76f8c..208ef3c 100644
--- a/src/PedroTroller/CS/Fixer/CodingStyle/LineBreakBetweenMethodArgumentsFixer.php
+++ b/src/PedroTroller/CS/Fixer/CodingStyle/LineBreakBetweenMethodArgumentsFixer.php
@@ -168,7 +168,10 @@ private function splitArgs(Tokens $tokens, $index): void
             $end = $tokens->getNextTokenOfKind($closeBraceIndex, [';', '{']);
 
             $tokens->removeLeadingWhitespace($end);
-            $tokens->ensureWhitespaceAtIndex($end, 0, ' ');
+
+            if (';' !== $tokens[$end]->getContent()) {
+                $tokens->ensureWhitespaceAtIndex($end, 0, ' ');
+            }
         }
 
         $linebreaks = [$openBraceIndex, $closeBraceIndex - 1];
diff --git a/tests/UseCase/LineBreakBetweenMethods/Regression/Case6.php b/tests/UseCase/LineBreakBetweenMethods/Regression/Case6.php
new file mode 100644
index 0000000..f2bf77b
--- /dev/null
+++ b/tests/UseCase/LineBreakBetweenMethods/Regression/Case6.php
@@ -0,0 +1,58 @@
+<?php
+
+declare(strict_types=1);
+
+namespace tests\UseCase\LineBreakBetweenMethods\Regression;
+
+use PedroTroller\CS\Fixer\CodingStyle\LineBreakBetweenMethodArgumentsFixer;
+use tests\UseCase;
+
+/**
+ * https://github.com/PedroTroller/PhpCSFixer-Custom-Fixers/issues/139.
+ */
+final class Case6 implements UseCase
+{
+    public function getFixers(): iterable
+    {
+        $fixer = new LineBreakBetweenMethodArgumentsFixer();
+
+        $fixer->configure([
+            'max-args'                 => false,
+            'max-length'               => 1,
+            'automatic-argument-merge' => false,
+        ]);
+
+        yield $fixer;
+    }
+
+    public function getRawScript(): string
+    {
+        return <<<'PHP'
+            <?php
+
+            interface Foo {
+                public function bar(
+                    string $path
+                ): string;
+            }
+            PHP;
+    }
+
+    public function getExpectation(): string
+    {
+        return <<<'PHP'
+            <?php
+
+            interface Foo {
+                public function bar(
+                    string $path
+                ): string;
+            }
+            PHP;
+    }
+
+    public function getMinSupportedPhpVersion(): int
+    {
+        return 0;
+    }
+}