Fix printing of Flow internal slot functions (#14962)

liuxingbaoyu · web-flow · commit 624c78d99e8f · 2022-09-22T18:24:40.000+02:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -272,7 +272,7 @@ jobs:
             !**/node_modules/**
 
   test-babel-8-breaking:
-    name: Test Babel 8 breaking changes on Node.js
+    name: Test Babel 8 breaking changes on
     needs: build-babel8
     strategy:
       matrix:
diff --git a/packages/babel-generator/src/generators/expressions.ts b/packages/babel-generator/src/generators/expressions.ts
@@ -190,12 +190,14 @@ export function OptionalCallExpression(
 ) {
   this.print(node.callee, node);
 
-  this.print(node.typeArguments, node); // Flow
   this.print(node.typeParameters, node); // TS
 
   if (node.optional) {
     this.token("?.");
   }
+
+  this.print(node.typeArguments, node); // Flow
+
   this.token("(");
   this.printList(node.arguments, node);
   this.token(")");
diff --git a/packages/babel-generator/src/generators/flow.ts b/packages/babel-generator/src/generators/flow.ts
@@ -332,6 +332,7 @@ export function FunctionTypeAnnotation(
   if (
     parent &&
     (parent.type === "ObjectTypeCallProperty" ||
+      parent.type === "ObjectTypeInternalSlot" ||
       parent.type === "DeclareFunction" ||
       (parent.type === "ObjectTypeProperty" && parent.method))
   ) {
diff --git a/packages/babel-generator/test/fixtures/decoratorsBeforeExport/false-to-true/options.json b/packages/babel-generator/test/fixtures/decoratorsBeforeExport/false-to-true/options.json
@@ -1,5 +1,6 @@
 {
   "BABEL_8_BREAKING": false,
   "plugins": [["decorators", { "decoratorsBeforeExport": false }]],
-  "decoratorsBeforeExport": true
+  "decoratorsBeforeExport": true,
+  "expectedReParseError": true
 }
diff --git a/packages/babel-generator/test/fixtures/decoratorsBeforeExport/true-to-false/options.json b/packages/babel-generator/test/fixtures/decoratorsBeforeExport/true-to-false/options.json
@@ -1,5 +1,6 @@
 {
   "BABEL_8_BREAKING": false,
   "plugins": [["decorators", { "decoratorsBeforeExport": true }]],
-  "decoratorsBeforeExport": false
+  "decoratorsBeforeExport": false,
+  "expectedReParseError": true
 }
diff --git a/packages/babel-generator/test/fixtures/flow/internal-slot/output.js b/packages/babel-generator/test/fixtures/flow/internal-slot/output.js
@@ -8,13 +8,13 @@ interface I1 {
   [[foo]]: X
 }
 interface I2 {
-  [[foo]]() => X
+  [[foo]](): X
 }
 type T1 = {
   [[foo]]: X
 };
 type T2 = {
-  [[foo]]() => X
+  [[foo]](): X
 };
 type T3 = {
   [[foo]]?: X
diff --git a/packages/babel-generator/test/fixtures/flow/typeapp-call/output.js b/packages/babel-generator/test/fixtures/flow/typeapp-call/output.js
@@ -16,6 +16,6 @@ new C<T>();
 o.m<T>();
 f < T > .0;
 o?.m<T>(e);
-o.m<T>?.(e);
+o.m?.<T>(e);
 async<T>();
-f<T>?.(e);
+f?.<T>(e);
diff --git a/packages/babel-generator/test/fixtures/types/Decorator/options.json b/packages/babel-generator/test/fixtures/types/Decorator/options.json
@@ -1 +1,5 @@
-{ "plugins": ["decorators-legacy"] }
+{
+  "plugins": ["decorators-legacy"],
+  "decoratorsBeforeExport": true,
+  "BABEL_8_BREAKING": false
+}
diff --git a/packages/babel-generator/test/fixtures/types/Decorator/output.js b/packages/babel-generator/test/fixtures/types/Decorator/output.js
@@ -32,15 +32,15 @@ class Foo {
 
 }
 
-export default @foo
-class Foo2 {
+@foo
+export default class Foo2 {
   bar() {
     class Baz {}
   }
 
 }
-export @foo
-class Foo3 {
+@foo
+export class Foo3 {
   bar() {
     class Baz {}
   }
diff --git a/packages/babel-generator/test/index.js b/packages/babel-generator/test/index.js
@@ -811,14 +811,15 @@ suites.forEach(function (testSuite) {
           const actualCode = actual.code;
 
           if (actualCode) {
-            const actualAst = parse(actualCode, {
+            const parserOpts = {
               filename: actual.loc,
               plugins: task.options.plugins || [],
               strictMode: task.options.strictMode === false ? false : true,
               sourceType: "module",
               sourceMaps: !!task.sourceMap,
               ...task.options.parserOpts,
-            });
+            };
+            const actualAst = parse(actualCode, parserOpts);
             const options = {
               sourceFileName: path.relative(
                 path.dirname(fileURLToPath(import.meta.url)),
@@ -832,7 +833,7 @@ suites.forEach(function (testSuite) {
               return generate(actualAst, options, actualCode);
             };
 
-            const throwMsg = task.options.throws;
+            const throwMsg = options.throws;
             if (throwMsg) {
               expect(() => run()).toThrow(
                 throwMsg === true ? undefined : throwMsg,
@@ -864,6 +865,11 @@ suites.forEach(function (testSuite) {
               } else {
                 try {
                   expect(result.code).toBe(expected.code);
+                  if (!options.expectedReParseError) {
+                    expect(() => {
+                      parse(result.code, parserOpts);
+                    }).not.toThrow();
+                  }
                 } catch (e) {
                   if (!process.env.OVERWRITE) throw e;
                   console.log(`Updated test file: ${expected.loc}`);

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"BABEL_8_BREAKING": false,`
`3`	`3`	`"plugins": [["decorators", { "decoratorsBeforeExport": false }]],`
`4`		`- "decoratorsBeforeExport": true`
	`4`	`+ "decoratorsBeforeExport": true,`
	`5`	`+ "expectedReParseError": true`
`5`	`6`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,15 +32,15 @@ class Foo {`
`32`	`32`
`33`	`33`	`}`
`34`	`34`
`35`		`-export default @foo`
`36`		`-class Foo2 {`
	`35`	`+@foo`
	`36`	`+export default class Foo2 {`
`37`	`37`	`bar() {`
`38`	`38`	`class Baz {}`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`}`
`42`		`-export @foo`
`43`		`-class Foo3 {`
	`42`	`+@foo`
	`43`	`+export class Foo3 {`
`44`	`44`	`bar() {`
`45`	`45`	`class Baz {}`
`46`	`46`	`}`