Merge pull request #962 from PHPCSStandards/feature/squiz-membervarspacing-add-extra-tests

Squiz/MemberVarSpacing: add extra tests

Author: jrfnl

src/Standards/Squiz/Tests/WhiteSpace/MemberVarSpacingUnitTest.1.inc

@@ -428,3 +428,19 @@ class PHP84FinalProperties {
 
     final private bool $finalPrivate;
 }
+
+class MultilineCommentShouldNotBeSplitUp {
+    // This is the first line
+    // of a multi-line comment
+    // which should be recognized as belonging
+    // with the property.
+    public $prop;
+
+
+    /* This is the first line
+       of a multi-line comment
+       which should be recognized as belonging
+       with the property.
+     */
+    public $prop;
+}

src/Standards/Squiz/Tests/WhiteSpace/MemberVarSpacingUnitTest.1.inc.fixed

@@ -415,3 +415,19 @@ class PHP84FinalProperties {
 
     final private bool $finalPrivate;
 }
+
+class MultilineCommentShouldNotBeSplitUp {
+
+    // This is the first line
+    // of a multi-line comment
+    // which should be recognized as belonging
+    // with the property.
+    public $prop;
+
+    /* This is the first line
+       of a multi-line comment
+       which should be recognized as belonging
+       with the property.
+     */
+    public $prop;
+}

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

@@ -85,6 +85,8 @@ public function getErrorList($testFile='')
                 416 => 1,
                 420 => 1,
                 427 => 1,
+                437 => 1,
+                445 => 1,
             ];
 
         default: