Fix some minor (language, not code) syntax and type errors. (#14352)

dev-itsheng · web-flow · commit 4820ea0f6e86 · 2022-03-13T19:09:38.000+01:00
Co-authored-by: gaosheng08 &lt;gaosheng08@meituan.com&gt;
diff --git a/test/jest-light-runner/src/worker-runner.js b/test/jest-light-runner/src/worker-runner.js
@@ -8,10 +8,10 @@ import { inspect } from "util";
 
 import "./global-setup.js";
 
-/** @typedef {{ failures: number, passses: number, pending: number, start: number, end: number }} Stats */
-/** @typedef {{ ancestors: string[], title: string, errors: Error[], skipped: boolean }} InternalTestResult */
+/** @typedef {{ failures: number, passes: number, pending: number, start: number, end: number }} Stats */
+/** @typedef {{ ancestors: string[], title: string, duration: number, errors: Error[], skipped: boolean }} InternalTestResult */
 
-// Node.js workers (worker_therads) don't support
+// Node.js workers (worker_threads) don't support
 // process.chdir, that we use multiple times in our tests.
 // We can "polyfill" it for process.cwd() usage, but it
 // won't affect path.* and fs.* functions.
@@ -53,7 +53,7 @@ export default async function ({
   stats.end = performance.now();
 
   snapshotState._inlineSnapshots.forEach(({ frame }) => {
-    // When using native ESM, errors have an URL location.
+    // When using native ESM, errors have a URL location.
     // Jest expects paths.
     frame.file = fileURLToPath(frame.file);
   });
@@ -140,7 +140,7 @@ async function runTest(fn, stats, results, ancestors, name) {
     errors.push(error);
   });
 
-  // Get suppressed errors from ``jest-matchers`` that weren't throw during
+  // Get suppressed errors from ``jest-matchers`` that weren't thrown during
   // test execution and add them to the test result, potentially failing
   // a passing test.
   const { suppressedErrors } = expect.getState();
diff --git a/test/runtime-integration/bundlers.cjs b/test/runtime-integration/bundlers.cjs
@@ -33,13 +33,12 @@ function test(name, command, directory, output, first) {
   });
 
   const expectedPath = path.join(__dirname, "expected-bundler.txt");
-  let expected = fs.readFileSync(expectedPath, "utf8");
+  const expected = fs.readFileSync(expectedPath, "utf8");
 
   if (expected === out) {
     console.log("OK");
   } else if (first && process.env.OVERWRITE) {
     fs.writeFileSync(expectedPath, out);
-    expected = out;
     console.log("UPDATED");
   } else {
     console.error("FAILED\n");
diff --git a/test/runtime-integration/node.cjs b/test/runtime-integration/node.cjs
@@ -18,7 +18,7 @@ if (
 
   test("ESM", "./src/main-esm.mjs", expectedEsm);
   // TODO: This never worked in any Babel version
-  // test("ESM - absoluteRuntime", "--esperimental-modules ./src/absolute/main-esm.mjs", "expected-esm-absolute.txt");
+  // test("ESM - absoluteRuntime", "--experimental-modules ./src/absolute/main-esm.mjs", "expected-esm-absolute.txt");
 }
 
 const expectedCjs =

Original file line number	Diff line number	Diff line change
`@@ -18,7 +18,7 @@ if (`
`18`	`18`
`19`	`19`	`test("ESM", "./src/main-esm.mjs", expectedEsm);`
`20`	`20`	`// TODO: This never worked in any Babel version`
`21`		`- // test("ESM - absoluteRuntime", "--esperimental-modules ./src/absolute/main-esm.mjs", "expected-esm-absolute.txt");`
	`21`	`+ // test("ESM - absoluteRuntime", "--experimental-modules ./src/absolute/main-esm.mjs", "expected-esm-absolute.txt");`
`22`	`22`	`}`
`23`	`23`
`24`	`24`	`const expectedCjs =`