Squiz/FunctionDeclarationArgumentSpacing: bug fix / unclear SpaceBeforeComma message

When there would be a new line between the end of the previous parameter and the comma, the error message was not always descriptive enough.

For example, for the test added in this commit, the messages would be:
```
 182 | ERROR | [x] Expected 0 spaces between argument "$paramA" and comma; 0 found
     |       |     (Squiz.Functions.FunctionDeclarationArgumentSpacing.SpaceBeforeComma)
 185 | ERROR | [x] Expected 0 spaces between argument "$paramB" and comma; 4 found
     |       |     (Squiz.Functions.FunctionDeclarationArgumentSpacing.SpaceBeforeComma)
```

In particular, take not of the "Expected 0 spaces... 0 found".

This commit improves the messaging to take new lines into account.

Includes tests.

Author: jrfnl

src/Standards/Squiz/Sniffs/Functions/FunctionDeclarationArgumentSpacingSniff.php

@@ -312,11 +312,20 @@ public function processBracket($phpcsFile, $openBracket)
             }
 
             if ($commaToken !== false) {
-                if ($tokens[($commaToken - 1)]['code'] === T_WHITESPACE) {
+                $endOfPreviousParam = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($commaToken - 1), null, true);
+
+                $spaceBeforeComma = 0;
+                if ($tokens[$endOfPreviousParam]['line'] !== $tokens[$commaToken]['line']) {
+                    $spaceBeforeComma = 'newline';
+                } else if ($tokens[($commaToken - 1)]['code'] === T_WHITESPACE) {
+                    $spaceBeforeComma = $tokens[($commaToken - 1)]['length'];
+                }
+
+                if ($spaceBeforeComma !== 0) {
                     $error = 'Expected 0 spaces between argument "%s" and comma; %s found';
                     $data  = [
                         $params[($paramNumber - 1)]['name'],
-                        $tokens[($commaToken - 1)]['length'],
+                        $spaceBeforeComma,
                     ];
 
                     $fix = $phpcsFile->addFixableError($error, $commaToken, 'SpaceBeforeComma', $data);

src/Standards/Squiz/Tests/Functions/FunctionDeclarationArgumentSpacingUnitTest.inc

@@ -176,3 +176,12 @@ function newlineBeforeAndAfterEqualsSignShouldBeFixedForSpacing1(
     true
 ) {}
 // phpcs:set Squiz.Functions.FunctionDeclarationArgumentSpacing equalsSpacing 0
+
+function newlineBeforeCommaShouldBeFixedInOneGo(
+    $paramA
+,
+    $paramB
+
+    ,
+    $paramC
+) {}

src/Standards/Squiz/Tests/Functions/FunctionDeclarationArgumentSpacingUnitTest.inc.fixed

@@ -157,3 +157,9 @@ function newlineBeforeAndAfterEqualsSignShouldBeFixedForSpacing1(
     $param = true
 ) {}
 // phpcs:set Squiz.Functions.FunctionDeclarationArgumentSpacing equalsSpacing 0
+
+function newlineBeforeCommaShouldBeFixedInOneGo(
+    $paramA,
+    $paramB,
+    $paramC
+) {}

src/Standards/Squiz/Tests/Functions/FunctionDeclarationArgumentSpacingUnitTest.php

@@ -78,6 +78,8 @@ public function getErrorList()
             155 => 2,
             163 => 2,
             174 => 2,
+            182 => 1,
+            185 => 1,
         ];
 
     }//end getErrorList()