fix(typescript): handle ExperimentalSpreadOperator, optional method, and abstract interface (#1763)

src/clean-ast.js

@@ -66,6 +66,21 @@ function massageAST(ast) {
       delete newObj.params;
     }
 
+    // (TypeScript) Ignore `static` in `constructor(static p) {}`
+    // and `export` in `constructor(export p) {}`
+    if (
+      ast.type === "TSParameterProperty" &&
+      ast.accessibility === null &&
+      !ast.readonly
+    ) {
+      return {
+        type: "Identifier",
+        name: ast.parameter.name,
+        typeAnnotation: newObj.parameter.typeAnnotation,
+        decorators: newObj.decorators
+      };
+    }
+
     // We convert <div></div> to <div />
     if (ast.type === "JSXOpeningElement") {
       delete newObj.selfClosing;

src/printer.js

@@ -356,6 +356,7 @@ function genericPrintNoParens(path, options, print, args) {
     case "SpreadElementPattern":
     case "RestProperty":
     case "ExperimentalRestProperty":
+    case "ExperimentalSpreadProperty":
     case "SpreadProperty":
     case "SpreadPropertyPattern":
     case "RestElement":
@@ -779,6 +780,7 @@ function genericPrintNoParens(path, options, print, args) {
     }
     case "TSInterfaceDeclaration":
       parts.push(
+        n.abstract ? "abstract " : "",
         printTypeScriptModifiers(path, options, print),
         "interface ",
         path.call(print, "id"),
@@ -2352,6 +2354,7 @@ function genericPrintNoParens(path, options, print, args) {
     case "TSMethodSignature":
       parts.push(
         path.call(print, "name"),
+        n.questionToken ? "?" : "",
         printFunctionTypeParameters(path, options, print),
         printFunctionParams(path, print, options)
       );

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

@@ -14,3 +14,14 @@ interface AudioBufferList {
 }
 
 `;
+
+exports[`methods.ts 1`] = `
+class X {
+    optionalMethod?() {}
+}
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+class X {
+  optionalMethod?() {}
+}
+
+`;

tests/typescript_class/methods.ts

@@ -0,0 +1,3 @@
+class X {
+    optionalMethod?() {}
+}

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

@@ -0,0 +1,10 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`abstract.ts 1`] = `
+abstract interface I {
+
+}
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+abstract interface I {}
+
+`;

tests/typescript_interface/abstract.ts

@@ -0,0 +1,3 @@
+abstract interface I {
+
+}

tests/typescript_interface/jsfmt.spec.js

@@ -0,0 +1 @@
+run_spec(__dirname, { parser: "typescript" });