Fix class const ignore formatting bug (#1799)

Author: wdoug

src/index.js

@@ -115,16 +115,29 @@ const printers = {
       }
     },
     hasPrettierIgnore(path) {
-      const node = path.getNode();
       const isSimpleIgnore = (comment) =>
         comment.value.includes("prettier-ignore") &&
         !comment.value.includes("prettier-ignore-start") &&
         !comment.value.includes("prettier-ignore-end");
+
+      const parentNode = path.getParentNode();
+      const node = path.getNode();
+
       return (
-        node &&
-        node.comments &&
-        node.comments.length > 0 &&
-        node.comments.some(isSimpleIgnore)
+        (node &&
+          node.kind !== "classconstant" &&
+          node.comments &&
+          node.comments.length > 0 &&
+          node.comments.some(isSimpleIgnore)) ||
+        // For proper formatting, the classconstant ignore formatting should
+        // run on the "constant" child
+        (node &&
+          node.kind === "constant" &&
+          parentNode &&
+          parentNode.kind === "classconstant" &&
+          parentNode.comments &&
+          parentNode.comments.length > 0 &&
+          parentNode.comments.some(isSimpleIgnore))
       );
     },
   },

tests/ignore/__snapshots__/jsfmt.spec.js.snap

@@ -7,6 +7,10 @@ printWidth: 80
                                                                                 | printWidth
 =====================================input======================================
 <?php
+
+// prettier-ignore
+define("FOO",    "something");
+
 // prettier-ignore
 $var1   =   1;
 
@@ -39,8 +43,24 @@ function Foo ($veryLongVeryLongVeryLongVeryLongVeryLongVeryLongVeryLong) { retur
 $info =    "prettier-ignore-start and -end is currently not supported"  ;
 // prettier-ignore-end
 
+class ExampleClass {
+    // prettier-ignore
+    const SOME_CONST    = 1;
+    const ANOTHER_CONST = 2;
+
+    // prettier-ignore
+    const CONST_ARRAY = [
+        'test'        => 0,
+        'another_key' => 1
+    ];
+}
+
 =====================================output=====================================
 <?php
+
+// prettier-ignore
+define("FOO",    "something");
+
 // prettier-ignore
 $var1   =   1;
 
@@ -72,5 +92,18 @@ function Foo ($veryLongVeryLongVeryLongVeryLongVeryLongVeryLongVeryLong) { retur
 $info = "prettier-ignore-start and -end is currently not supported";
 // prettier-ignore-end
 
+class ExampleClass
+{
+    // prettier-ignore
+    const SOME_CONST    = 1;
+    const ANOTHER_CONST = 2;
+
+    // prettier-ignore
+    const CONST_ARRAY = [
+        'test'        => 0,
+        'another_key' => 1
+    ];
+}
+
 ================================================================================
 `;

tests/ignore/ignore.php

@@ -1,4 +1,8 @@
 <?php
+
+// prettier-ignore
+define("FOO",    "something");
+
 // prettier-ignore
 $var1   =   1;
 
@@ -30,3 +34,15 @@ function Foo ($veryLongVeryLongVeryLongVeryLongVeryLongVeryLongVeryLong) { retur
 // prettier-ignore-start
 $info =    "prettier-ignore-start and -end is currently not supported"  ;
 // prettier-ignore-end
+
+class ExampleClass {
+    // prettier-ignore
+    const SOME_CONST    = 1;
+    const ANOTHER_CONST = 2;
+
+    // prettier-ignore
+    const CONST_ARRAY = [
+        'test'        => 0,
+        'another_key' => 1
+    ];
+}