Changelog + tests for PR #869

Author: gsherwood

CodeSniffer/Standards/Generic/Tests/NamingConventions/CamelCapsFunctionNameUnitTest.inc

@@ -114,4 +114,8 @@ class Foo extends \SoapClient
         // body
     }
 }
-?>
+
+function __debugInfo() {}
+class Foo {
+    function __debugInfo() {}
+}

CodeSniffer/Standards/Generic/Tests/NamingConventions/CamelCapsFunctionNameUnitTest.php

@@ -41,38 +41,39 @@ class Generic_Tests_NamingConventions_CamelCapsFunctionNameUnitTest extends Abst
     public function getErrorList()
     {
         $errors = array(
-                   10 => 1,
-                   11 => 1,
-                   12 => 1,
-                   13 => 1,
-                   16 => 1,
-                   17 => 1,
-                   20 => 1,
-                   21 => 1,
-                   24 => 1,
-                   25 => 1,
-                   30 => 1,
-                   31 => 1,
-                   50 => 1,
-                   52 => 1,
-                   53 => 1,
-                   57 => 1,
-                   58 => 1,
-                   59 => 1,
-                   60 => 1,
-                   61 => 1,
-                   62 => 1,
-                   63 => 1,
-                   64 => 1,
-                   65 => 1,
-                   66 => 1,
-                   67 => 1,
-                   68 => 1,
-                   69 => 1,
-                   71 => 1,
-                   72 => 1,
-                   73 => 1,
-                   74 => 1,
+                   10  => 1,
+                   11  => 1,
+                   12  => 1,
+                   13  => 1,
+                   16  => 1,
+                   17  => 1,
+                   20  => 1,
+                   21  => 1,
+                   24  => 1,
+                   25  => 1,
+                   30  => 1,
+                   31  => 1,
+                   50  => 1,
+                   52  => 1,
+                   53  => 1,
+                   57  => 1,
+                   58  => 1,
+                   59  => 1,
+                   60  => 1,
+                   61  => 1,
+                   62  => 1,
+                   63  => 1,
+                   64  => 1,
+                   65  => 1,
+                   66  => 1,
+                   67  => 1,
+                   68  => 1,
+                   69  => 1,
+                   71  => 1,
+                   72  => 1,
+                   73  => 1,
+                   74  => 1,
+                   118 => 1,
                   );
 
         return $errors;

CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidFunctionNameUnitTest.inc

@@ -203,3 +203,8 @@ class Magic_Case_Test {
 }
 function __autoLoad() {}
 function _() {}
+
+function __debugInfo() {}
+class Foo {
+    function __debugInfo() {}
+}

CodeSniffer/Standards/PEAR/Tests/NamingConventions/ValidFunctionNameUnitTest.php

@@ -151,6 +151,7 @@ public function getErrorList()
                    173 => 1,
                    174 => 1,
                    175 => 1,
+                   207 => 1,
                );
 
         return $errors;

package.xml

@@ -37,6 +37,8 @@ http://pear.php.net/dtd/package-2.0.xsd">
   -- Added error Squiz.Commenting.FunctionComment.ScalarTypeHintMissing for PHP7 only (request #858)
     -- These errors were previously reported as Squiz.Commenting.FunctionComment.TypeHintMissing on PHP7
     -- Disable this error message in a ruleset.xml file if your code needs to run on both PHP5 and PHP7
+  - The PHP 5.6 __debugInfo magic method no longer produces naming convention errors
+    -- Thanks to Michael Nowack for the patch
   - PEAR and Squiz FunctionComment sniffs now support variadic functions (request #841)
   - Fixed bug #622 : Wrong detection of Squiz.CSS.DuplicateStyleDefinition with media queries
   - Fixed bug #752 : The missing exception error is reported in first found DocBlock