Squiz/FunctionClosingBraceSpace: add some more tests

A significant part of this sniff (handling of nested functions) is only covered by JS tests, so let's make sure this functionality also has tests for PHP.

Author: jrfnl

src/Standards/Squiz/Tests/WhiteSpace/FunctionClosingBraceSpaceUnitTest.inc

@@ -37,3 +37,54 @@ function foo()
 
 
 }
+
+// Nested functions - okay.
+function hasNestedOkay() {
+    function nestedFunction() {
+        echo 'hello';
+    }
+
+}
+
+$cl = function() {
+    function nestedFunction() {
+        echo 'hello';
+    }
+
+};
+
+mycallback(
+    'name',
+    function() {
+        echo 'hello';
+    }
+
+);
+
+// Nested functions - not okay.
+function hasNestedNotOkay() {
+    function nestedFunction() {}
+
+}
+
+$cl = function() {
+    function nestedFunction() {
+        echo 'hello'; }
+
+};
+
+mycallback('name', function() {
+    echo 'hello';
+
+
+
+    }
+
+);
+
+mycallback('name', function() {
+    // Comment
+
+    }
+
+);

src/Standards/Squiz/Tests/WhiteSpace/FunctionClosingBraceSpaceUnitTest.inc.fixed

@@ -43,3 +43,52 @@ function foo()
     }//end for
 
 }
+
+// Nested functions - okay.
+function hasNestedOkay() {
+    function nestedFunction() {
+        echo 'hello';
+    }
+
+}
+
+$cl = function() {
+    function nestedFunction() {
+        echo 'hello';
+    }
+
+};
+
+mycallback(
+    'name',
+    function() {
+        echo 'hello';
+    }
+
+);
+
+// Nested functions - not okay.
+function hasNestedNotOkay() {
+    function nestedFunction() {
+}
+
+}
+
+$cl = function() {
+    function nestedFunction() {
+        echo 'hello'; 
+}
+
+};
+
+mycallback('name', function() {
+    echo 'hello';
+    }
+
+);
+
+mycallback('name', function() {
+    // Comment
+    }
+
+);

src/Standards/Squiz/Tests/WhiteSpace/FunctionClosingBraceSpaceUnitTest.php

@@ -41,6 +41,10 @@ public function getErrorList($testFile='')
                 29 => 1,
                 31 => 1,
                 39 => 1,
+                66 => 1,
+                72 => 1,
+                81 => 1,
+                88 => 1,
             ];
 
         case 'FunctionClosingBraceSpaceUnitTest.js':