Zend/ValidVariableName: add tests for properties with asymmetric visibility

Sniff already behaves as intended. This commit just adds some extra tests to safeguard this for the future.

Author: jrfnl

src/Standards/Zend/Tests/NamingConventions/ValidVariableNameUnitTest.inc

@@ -129,3 +129,17 @@ enum SomeEnum
         $bar_foo = 3;
     }
 }
+
+class AsymVisibility {
+    // The read scope is public, but not specified. Enforce the naming conventions anyway.
+    private(set) $asymPublicImplied  = 'hello';
+    private(set) $_asymPublicImplied = 'hello';
+
+    // The read scope is private, so these properties should be handled as private properties.
+    private private(set) $asymPrivate  = 'hello';
+    private(set) private $_asymPrivate = 'hello';
+
+    // The read scope is public/protected, so these properties should be handled as public properties.
+    protected private(set) $asymProtectedPrivate  = 'hello';
+    private(set) public $_asymPrivatePublic = 'hello';
+}

src/Standards/Zend/Tests/NamingConventions/ValidVariableNameUnitTest.php

@@ -61,6 +61,10 @@ public function getErrorList()
             121 => 1,
             126 => 1,
             129 => 1,
+            136 => 1,
+            139 => 1,
+            143 => 1,
+            144 => 1,
         ];
 
     }//end getErrorList()