Add tests for keyword class property (prettier#13927)

medikoo · Jan 4, 2024 · 2013a76 · 2013a76
1 parent d55e5df
commit 2013a76
Show file tree

Hide file tree

Showing 12 changed files with 607 additions and 2 deletions.
diff --git a/src/language-js/parse/postprocess/index.js b/src/language-js/parse/postprocess/index.js
@@ -153,6 +153,26 @@ function postprocess(ast, options) {
         }
         break;
       }
+      // TODO: Remove this when https://github.com/meriyah/meriyah/issues/231 get fixed
+      case "PropertyDefinition":
+        if (
+          options.parser === "meriyah" &&
+          node.static &&
+          !node.computed &&
+          !node.key
+        ) {
+          const name = "static";
+          const start = locStart(node);
+          Object.assign(node, {
+            static: false,
+            key: {
+              type: "Identifier",
+              name,
+              range: [start, start + name.length],
+            },
+          });
+        }
+        break;
     }
   });
 

diff --git a/src/language-js/print/statement.js b/src/language-js/print/statement.js
@@ -174,8 +174,6 @@ const isClassProperty = ({ type }) =>
  */
 function shouldPrintSemicolonAfterClassProperty(node, nextNode) {
   const name = node.key && node.key.name;
-  // this isn't actually possible yet with most parsers available today
-  // so isn't properly tested yet.
   if (
     (name === "static" || name === "get" || name === "set") &&
     !node.value &&