Update babel 8 preset env fixtures (#17356)

JLHwung · web-flow · commit 7ba1afa90371 · 2025-06-03T08:36:18.000-04:00
* test: revert preset-env debug syntax strip

The debug test cases have been duplicated anyway

* support stdout in generate-babel-7-tests

* update test fixtures

* add support for stderr
diff --git a/packages/babel-helper-transform-fixture-test-runner/src/index.ts b/packages/babel-helper-transform-fixture-test-runner/src/index.ts
@@ -385,15 +385,7 @@ async function run(task: Test) {
       validateFile(
         normalizeOutput(actualLogs.stdout, normalizationOpts),
         stdout.loc,
-        process.env.BABEL_8_BREAKING
-          ? // In Babel 8, preset-env does not enable all the unnecessary syntax
-            // plugins. For simplicity, just strip them fro the expected output
-            // so that we do not need to separate tests for every fixture.
-            stdout.code.replace(
-              /\n\s*syntax-(?!import-attributes|import-assertions).*/g,
-              "",
-            )
-          : stdout.code,
+        stdout.code,
       );
       validateFile(
         normalizeOutput(actualLogs.stderr, normalizationOpts),
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100-babel-7/input.js b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100-babel-7/input.js
@@ -0,0 +1,4 @@
+class A {
+  #x;
+  [class {}]
+}
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100-babel-7/options.json b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100-babel-7/options.json
@@ -0,0 +1,15 @@
+{
+  "validateLogs": true,
+  "targets": {
+    "chrome": 100
+  },
+  "presets": [
+    [
+      "env",
+      {
+        "debug": true
+      }
+    ]
+  ],
+  "BABEL_8_BREAKING": false
+}
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100-babel-7/output.js b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100-babel-7/output.js
@@ -0,0 +1,4 @@
+class A {
+  #x;
+  [class {}];
+}
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100-babel-7/stdout.txt b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100-babel-7/stdout.txt
@@ -0,0 +1,30 @@
+@babel/preset-env: `DEBUG` option
+
+Using targets:
+{
+  "chrome": "100"
+}
+
+Using modules transform: auto
+
+Using plugins:
+  transform-duplicate-named-capturing-groups-regex { chrome < 126 }
+  transform-regexp-modifiers { chrome < 125 }
+  transform-unicode-sets-regex { chrome < 112 }
+  syntax-class-static-block
+  syntax-private-property-in-object
+  syntax-class-properties
+  syntax-numeric-separator
+  syntax-nullish-coalescing-operator
+  syntax-optional-chaining
+  syntax-json-strings
+  syntax-optional-catch-binding
+  syntax-async-generators
+  syntax-object-rest-spread
+  transform-export-namespace-from { }
+  transform-modules-commonjs
+  transform-dynamic-import
+  syntax-top-level-await
+  syntax-import-meta
+
+Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100/options.json b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100/options.json
@@ -1,5 +1,15 @@
 {
   "validateLogs": true,
-  "targets": { "chrome": 100 },
-  "presets": [["env", { "debug": true }]]
+  "targets": {
+    "chrome": 100
+  },
+  "presets": [
+    [
+      "env",
+      {
+        "debug": true
+      }
+    ]
+  ],
+  "BABEL_8_BREAKING": true
 }
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100/stdout.txt b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-chrome-100/stdout.txt
@@ -11,20 +11,8 @@ Using plugins:
   transform-duplicate-named-capturing-groups-regex { chrome < 126 }
   transform-regexp-modifiers { chrome < 125 }
   transform-unicode-sets-regex { chrome < 112 }
-  syntax-class-static-block
-  syntax-private-property-in-object
-  syntax-class-properties
-  syntax-numeric-separator
-  syntax-nullish-coalescing-operator
-  syntax-optional-chaining
-  syntax-json-strings
-  syntax-optional-catch-binding
-  syntax-async-generators
-  syntax-object-rest-spread
   transform-export-namespace-from { }
   transform-modules-commonjs
   transform-dynamic-import
-  syntax-top-level-await
-  syntax-import-meta
 
 Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50-babel-7/input.js b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50-babel-7/input.js
@@ -0,0 +1,4 @@
+class A {
+  #x;
+  [class {}]
+}
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50-babel-7/options.json b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50-babel-7/options.json
@@ -0,0 +1,15 @@
+{
+  "validateLogs": true,
+  "targets": {
+    "firefox": 50
+  },
+  "presets": [
+    [
+      "env",
+      {
+        "debug": true
+      }
+    ]
+  ],
+  "BABEL_8_BREAKING": false
+}
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50-babel-7/output.js b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50-babel-7/output.js
@@ -0,0 +1,9 @@
+var _ref;
+var _x = /*#__PURE__*/new WeakMap();
+_ref = class {};
+class A {
+  constructor() {
+    babelHelpers.classPrivateFieldInitSpec(this, _x, void 0);
+    babelHelpers.defineProperty(this, _ref, void 0);
+  }
+}
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50-babel-7/stdout.txt b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50-babel-7/stdout.txt
@@ -0,0 +1,45 @@
+@babel/preset-env: `DEBUG` option
+
+Using targets:
+{
+  "firefox": "50"
+}
+
+Using modules transform: auto
+
+Using plugins:
+  transform-duplicate-named-capturing-groups-regex { firefox < 129 }
+  transform-regexp-modifiers { firefox < 132 }
+  transform-unicode-sets-regex { firefox < 116 }
+  transform-class-static-block { firefox < 93 }
+  transform-private-property-in-object { firefox < 90 }
+  transform-class-properties { firefox < 90 }
+  transform-private-methods { firefox < 90 }
+  transform-numeric-separator { firefox < 70 }
+  transform-logical-assignment-operators { firefox < 79 }
+  transform-nullish-coalescing-operator { firefox < 72 }
+  transform-optional-chaining { firefox < 74 }
+  transform-json-strings { firefox < 62 }
+  transform-optional-catch-binding { firefox < 58 }
+  transform-parameters { firefox < 52 }
+  transform-async-generator-functions { firefox < 57 }
+  transform-object-rest-spread { firefox < 55 }
+  transform-dotall-regex { firefox < 78 }
+  transform-unicode-property-regex { firefox < 78 }
+  transform-named-capturing-groups-regex { firefox < 78 }
+  transform-async-to-generator { firefox < 52 }
+  transform-exponentiation-operator { firefox < 52 }
+  transform-literals { firefox < 53 }
+  transform-function-name { firefox < 53 }
+  transform-for-of { firefox < 53 }
+  transform-unicode-escapes { firefox < 53 }
+  transform-destructuring { firefox < 53 }
+  transform-block-scoping { firefox < 53 }
+  transform-regenerator { firefox < 53 }
+  transform-export-namespace-from { firefox < 80 }
+  transform-modules-commonjs
+  transform-dynamic-import
+  syntax-top-level-await
+  syntax-import-meta
+
+Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50/options.json b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50/options.json
@@ -1,5 +1,15 @@
 {
   "validateLogs": true,
-  "targets": { "firefox": 50 },
-  "presets": [["env", { "debug": true }]]
+  "targets": {
+    "firefox": 50
+  },
+  "presets": [
+    [
+      "env",
+      {
+        "debug": true
+      }
+    ]
+  ],
+  "BABEL_8_BREAKING": true
 }
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50/stdout.txt b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-50/stdout.txt
@@ -39,7 +39,5 @@ Using plugins:
   transform-export-namespace-from { firefox < 80 }
   transform-modules-commonjs
   transform-dynamic-import
-  syntax-top-level-await
-  syntax-import-meta
 
 Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90-babel-7/input.js b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90-babel-7/input.js
@@ -0,0 +1,4 @@
+class A {
+  #x;
+  [class {}]
+}
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90-babel-7/options.json b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90-babel-7/options.json
@@ -0,0 +1,15 @@
+{
+  "validateLogs": true,
+  "targets": {
+    "firefox": 90
+  },
+  "presets": [
+    [
+      "env",
+      {
+        "debug": true
+      }
+    ]
+  ],
+  "BABEL_8_BREAKING": false
+}
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90-babel-7/output.js b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90-babel-7/output.js
@@ -0,0 +1,4 @@
+class A {
+  #x;
+  [(() => class {})()];
+}
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90-babel-7/stdout.txt b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90-babel-7/stdout.txt
@@ -0,0 +1,31 @@
+@babel/preset-env: `DEBUG` option
+
+Using targets:
+{
+  "firefox": "90"
+}
+
+Using modules transform: auto
+
+Using plugins:
+  transform-duplicate-named-capturing-groups-regex { firefox < 129 }
+  transform-regexp-modifiers { firefox < 132 }
+  transform-unicode-sets-regex { firefox < 116 }
+  bugfix/transform-firefox-class-in-computed-class-key { firefox < 126 }
+  transform-class-static-block { firefox < 93 }
+  syntax-private-property-in-object
+  syntax-class-properties
+  syntax-numeric-separator
+  syntax-nullish-coalescing-operator
+  syntax-optional-chaining
+  syntax-json-strings
+  syntax-optional-catch-binding
+  syntax-async-generators
+  syntax-object-rest-spread
+  transform-export-namespace-from { }
+  transform-modules-commonjs
+  transform-dynamic-import
+  syntax-top-level-await
+  syntax-import-meta
+
+Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90/options.json b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90/options.json
@@ -1,5 +1,15 @@
 {
   "validateLogs": true,
-  "targets": { "firefox": 90 },
-  "presets": [["env", { "debug": true }]]
+  "targets": {
+    "firefox": 90
+  },
+  "presets": [
+    [
+      "env",
+      {
+        "debug": true
+      }
+    ]
+  ],
+  "BABEL_8_BREAKING": true
 }
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90/stdout.txt b/packages/babel-preset-env/test/fixtures/bugfixes-always-enabled/class-in-computed-field-firefox-90/stdout.txt
@@ -13,19 +13,8 @@ Using plugins:
   transform-unicode-sets-regex { firefox < 116 }
   bugfix/transform-firefox-class-in-computed-class-key { firefox < 126 }
   transform-class-static-block { firefox < 93 }
-  syntax-private-property-in-object
-  syntax-class-properties
-  syntax-numeric-separator
-  syntax-nullish-coalescing-operator
-  syntax-optional-chaining
-  syntax-json-strings
-  syntax-optional-catch-binding
-  syntax-async-generators
-  syntax-object-rest-spread
   transform-export-namespace-from { }
   transform-modules-commonjs
   transform-dynamic-import
-  syntax-top-level-await
-  syntax-import-meta
 
 Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-babel-8/edge-default-params-chrome-70/stdout.txt b/packages/babel-preset-env/test/fixtures/bugfixes-babel-8/edge-default-params-chrome-70/stdout.txt
@@ -19,14 +19,8 @@ Using plugins:
   transform-logical-assignment-operators { chrome < 85 }
   transform-nullish-coalescing-operator { chrome < 80 }
   transform-optional-chaining { chrome < 80 }
-  syntax-json-strings
-  syntax-optional-catch-binding
-  syntax-async-generators
-  syntax-object-rest-spread
   transform-export-namespace-from { chrome < 72 }
   transform-modules-commonjs
   transform-dynamic-import
-  syntax-top-level-await
-  syntax-import-meta
 
 Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-babel-8/safari-block-scoping-safari-11/stdout.txt b/packages/babel-preset-env/test/fixtures/bugfixes-babel-8/safari-block-scoping-safari-11/stdout.txt
@@ -32,7 +32,5 @@ Using plugins:
   bugfix/transform-tagged-template-caching { safari < 13 }
   transform-modules-commonjs
   transform-dynamic-import
-  syntax-top-level-await
-  syntax-import-meta
 
 Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-babel-8/safari-class-fields-safari-17/stdout.txt b/packages/babel-preset-env/test/fixtures/bugfixes-babel-8/safari-class-fields-safari-17/stdout.txt
@@ -10,19 +10,5 @@ Using modules transform: false
 Using plugins:
   transform-duplicate-named-capturing-groups-regex { safari < 17.4 }
   transform-regexp-modifiers { safari }
-  syntax-class-static-block
-  syntax-private-property-in-object
-  syntax-class-properties
-  syntax-numeric-separator
-  syntax-nullish-coalescing-operator
-  syntax-optional-chaining
-  syntax-json-strings
-  syntax-optional-catch-binding
-  syntax-async-generators
-  syntax-object-rest-spread
-  syntax-export-namespace-from
-  syntax-dynamic-import
-  syntax-top-level-await
-  syntax-import-meta
 
 Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
diff --git a/packages/babel-preset-env/test/fixtures/bugfixes-babel-8/v8-spread-parameters-in-optional-chaining-chrome-89/stdout.txt b/packages/babel-preset-env/test/fixtures/bugfixes-babel-8/v8-spread-parameters-in-optional-chaining-chrome-89/stdout.txt
@@ -14,19 +14,9 @@ Using plugins:
   bugfix/transform-v8-static-class-fields-redefine-readonly { chrome < 98 }
   transform-class-static-block { chrome < 94 }
   transform-private-property-in-object { chrome < 91 }
-  syntax-class-properties
-  syntax-numeric-separator
-  syntax-nullish-coalescing-operator
-  syntax-optional-chaining
-  syntax-json-strings
-  syntax-optional-catch-binding
-  syntax-async-generators
-  syntax-object-rest-spread
   transform-export-namespace-from { }
   bugfix/transform-v8-spread-parameters-in-optional-chaining { chrome < 91 }
   transform-modules-commonjs
   transform-dynamic-import
-  syntax-top-level-await
-  syntax-import-meta
 
 Using polyfills: No polyfills were added, since the `useBuiltIns` option was not set.
diff --git a/scripts/generate-babel-7-tests.mjs b/scripts/generate-babel-7-tests.mjs
@@ -16,7 +16,7 @@ function getUnstagedModifiedOutputJSON() {
   try {
     // Get the status and filter for unstaged files
     const output = execSync(
-      "git status --porcelain=v1 -z -- './packages/*/test/fixtures/**/output.*'",
+      "git status --porcelain=v1 -z -- './packages/*/test/fixtures/**/output.*' './packages/*/test/fixtures/**/stdout.txt' './packages/*/test/fixtures/**/stderr.txt'",
       { encoding: "utf-8" }
     );
 

-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +class A {
 +  #x;
 +  [class {}]
 +}