chore: update Jest

remix-run · Aug 22, 2023 · c6ad996 · c6ad996
1 parent 341e097
commit c6ad996
Show file tree

Hide file tree

Showing 26 changed files with 1,036 additions and 978 deletions.
diff --git a/jest.config.js b/jest.config.js
@@ -1,4 +1,4 @@
-/** @type {import('@jest/types').Config.InitialOptions} */
+/** @type {import('jest').Config} */
 module.exports = {
   modulePathIgnorePatterns: [
     "<rootDir>/.tmp",
@@ -23,10 +23,6 @@ module.exports = {
     "packages/remix-server-runtime",
     "packages/remix-testing",
   ],
-  watchPlugins: [
-    require.resolve("jest-watch-select-projects"),
-    require.resolve("jest-watch-typeahead/filename"),
-    require.resolve("jest-watch-typeahead/testname"),
-  ],
-  reporters: ["default"],
+  watchPlugins: ["select-projects", "typeahead/filename", "typeahead/testname"],
+  reporters: [["github-actions", { silent: false }], "summary"],
 };
diff --git a/jest/jest.config.shared.js b/jest/jest.config.shared.js
@@ -5,7 +5,8 @@ const ignorePatterns = [
   "\\/\\.tmp\\/",
   "\\/\\.cache\\/",
 ];
-/** @type {import('@jest/types').Config.InitialOptions} */
+
+/** @type {import('jest').Config} */
 module.exports = {
   testEnvironment: "node",
   modulePathIgnorePatterns: ignorePatterns,
@@ -15,9 +16,8 @@ module.exports = {
   transform: {
     "\\.[jt]sx?$": require.resolve("./transform"),
   },
-  watchPlugins: [
-    require.resolve("jest-watch-select-projects"),
-    require.resolve("jest-watch-typeahead/filename"),
-    require.resolve("jest-watch-typeahead/testname"),
+  transformIgnorePatterns: [
+    "node_modules/(?!(@remix-run/web-(blob|fetch|file|form-data|stream)|@web3-storage/multipart-parser)/)",
   ],
+  watchPlugins: ["select-projects", "typeahead/filename", "typeahead/testname"],
 };
diff --git a/package.json b/package.json
@@ -74,7 +74,7 @@
     "@types/cross-spawn": "^6.0.2",
     "@types/dedent": "^0.7.0",
     "@types/glob": "7.2.0",
-    "@types/jest": "^27.4.1",
+    "@types/jest": "^29.5.4",
     "@types/jsonfile": "^6.1.0",
     "@types/lodash": "^4.14.182",
     "@types/react": "^18.2.20",
@@ -85,7 +85,7 @@
     "@types/serialize-javascript": "^5.0.2",
     "@types/ssri": "^7.1.0",
     "@vanilla-extract/css": "^1.10.0",
-    "babel-jest": "^27.5.1",
+    "babel-jest": "^29.6.3",
     "babel-plugin-transform-remove-console": "^6.9.4",
     "chalk": "^4.1.2",
     "cheerio": "^1.0.0-rc.12",
@@ -101,9 +101,9 @@
     "front-matter": "^4.0.2",
     "glob": "8.0.3",
     "isbot": "^3.5.1",
-    "jest": "^27.5.1",
+    "jest": "^29.6.3",
     "jest-watch-select-projects": "^2.0.0",
-    "jest-watch-typeahead": "^0.6.5",
+    "jest-watch-typeahead": "^2.2.2",
     "jsonfile": "^6.0.1",
     "lodash": "^4.17.21",
     "mime": "^3.0.0",

diff --git a/packages/create-remix/__tests__/create-remix-test.ts b/packages/create-remix/__tests__/create-remix-test.ts
@@ -242,7 +242,7 @@ describe("create-remix CLI", () => {
     });
 
     expect(stderr.trim()).toMatchInlineSnapshot(
-      `"▲  Oh no! The path \\"this/path/does/not/exist\\" was not found in this GitHub repo."`
+      `"▲  Oh no! The path "this/path/does/not/exist" was not found in this GitHub repo."`
     );
     expect(status).toBe(1);
     expect(fse.existsSync(path.join(projectDir, "package.json"))).toBeFalsy();
@@ -381,7 +381,7 @@ describe("create-remix CLI", () => {
     });
 
     expect(stderr.trim()).toMatchInlineSnapshot(
-      `"▲  Oh no! The path \\"this/path/does/not/exist\\" was not found in this GitHub repo."`
+      `"▲  Oh no! The path "this/path/does/not/exist" was not found in this GitHub repo."`
     );
     expect(status).toBe(1);
     expect(fse.existsSync(path.join(projectDir, "package.json"))).toBeFalsy();
@@ -1086,12 +1086,12 @@ describe("create-remix CLI", () => {
         });
 
         expect(stderr.trim()).toMatchInlineSnapshot(`
-        "▲  Oh no! Destination directory contains files that would be overwritten
-                 and no \`--overwrite\` flag was included in a non-interactive
-                 environment. The following files would be overwritten:
-                   package.json
-                   tsconfig.json"
-      `);
+                  "▲  Oh no! Destination directory contains files that would be overwritten
+                           and no \`--overwrite\` flag was included in a non-interactive
+                           environment. The following files would be overwritten:
+                             package.json
+                             tsconfig.json"
+              `);
         expect(status).toBe(1);
         expect(
           fse.existsSync(path.join(projectDir, "app/root.tsx"))

diff --git a/packages/create-remix/jest.config.js b/packages/create-remix/jest.config.js
@@ -1,4 +1,4 @@
-/** @type {import('@jest/types').Config.InitialOptions} */
+/** @type {import('jest').Config} */
 module.exports = {
   ...require("../../jest/jest.config.shared"),
   displayName: "create-remix",

diff --git a/packages/remix-architect/__tests__/server-test.ts b/packages/remix-architect/__tests__/server-test.ts
@@ -201,9 +201,7 @@ describe("architect createRemixHeaders", () => {
   describe("creates fetch headers from architect headers", () => {
     it("handles empty headers", () => {
       let headers = createRemixHeaders({});
-      expect(Object.fromEntries(headers.entries())).toMatchInlineSnapshot(
-        `Object {}`
-      );
+      expect(Object.fromEntries(headers.entries())).toMatchInlineSnapshot(`{}`);
     });
 
     it("handles simple headers", () => {

diff --git a/packages/remix-architect/jest.config.js b/packages/remix-architect/jest.config.js
@@ -1,4 +1,4 @@
-/** @type {import('@jest/types').Config.InitialOptions} */
+/** @type {import('jest').Config} */
 module.exports = {
   ...require("../../jest/jest.config.shared"),
   displayName: "architect",

diff --git a/packages/remix-dev/__tests__/cli-test.ts b/packages/remix-dev/__tests__/cli-test.ts
@@ -128,7 +128,7 @@ describe("remix CLI", () => {
           Run your project locally in development:
 
             $ remix dev
-            $ remix dev -c \\"node ./server.js\\"
+            $ remix dev -c "node ./server.js"
 
           Start your server separately and watch for changes:
 

diff --git a/packages/remix-dev/__tests__/cssSideEffectImports-test.ts b/packages/remix-dev/__tests__/cssSideEffectImports-test.ts
@@ -10,7 +10,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"require(\\"./foo.css?__remix_sideEffect__\\");"`
+        `"require("./foo.css?__remix_sideEffect__");"`
       );
     });
 
@@ -20,7 +20,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"import \\"./foo.css?__remix_sideEffect__\\";"`
+        `"import "./foo.css?__remix_sideEffect__";"`
       );
     });
 
@@ -33,7 +33,7 @@ describe("addSuffixToCssSideEffectImports", () => {
 
       expect(addSuffixToCssSideEffectImports("jsx", code))
         .toMatchInlineSnapshot(`
-        "import \\"./foo.css?__remix_sideEffect__\\";
+        "import "./foo.css?__remix_sideEffect__";
 
         export const Foo = () => <div />;"
       `);
@@ -48,7 +48,7 @@ describe("addSuffixToCssSideEffectImports", () => {
 
       expect(addSuffixToCssSideEffectImports("ts", code))
         .toMatchInlineSnapshot(`
-        "require(\\"./foo.css?__remix_sideEffect__\\");
+        "require("./foo.css?__remix_sideEffect__");
 
         export const foo: string = ('foo' satisfies string);"
       `);
@@ -64,7 +64,7 @@ describe("addSuffixToCssSideEffectImports", () => {
 
       expect(addSuffixToCssSideEffectImports("tsx", code))
         .toMatchInlineSnapshot(`
-        "require(\\"./foo.css?__remix_sideEffect__\\");
+        "require("./foo.css?__remix_sideEffect__");
 
         export const foo: string = ('foo' satisfies string);
         export const Bar = () => <div>{foo}</div>;"
@@ -82,10 +82,10 @@ describe("addSuffixToCssSideEffectImports", () => {
 
       expect(addSuffixToCssSideEffectImports("js", code))
         .toMatchInlineSnapshot(`
-        "if (process.env.NODE_ENV === \\"production\\") {
-          require(\\"./foo.min.css?__remix_sideEffect__\\");
+        "if (process.env.NODE_ENV === "production") {
+          require("./foo.min.css?__remix_sideEffect__");
         } else {
-          require(\\"./foo.css?__remix_sideEffect__\\");
+          require("./foo.css?__remix_sideEffect__");
         }"
       `);
     });
@@ -96,7 +96,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"process.env.NODE_ENV === \\"production\\" ? require(\\"./foo.min.css?__remix_sideEffect__\\") : require(\\"./foo.css?__remix_sideEffect__\\");"`
+        `"process.env.NODE_ENV === "production" ? require("./foo.min.css?__remix_sideEffect__") : require("./foo.css?__remix_sideEffect__");"`
       );
     });
 
@@ -106,7 +106,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"process.env.NODE_ENV === \\"development\\" && require(\\"./debug.css?__remix_sideEffect__\\");"`
+        `"process.env.NODE_ENV === "development" && require("./debug.css?__remix_sideEffect__");"`
       );
     });
 
@@ -116,7 +116,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"process.env.NODE_ENV === \\"production\\" || require(\\"./debug.css?__remix_sideEffect__\\");"`
+        `"process.env.NODE_ENV === "production" || require("./debug.css?__remix_sideEffect__");"`
       );
     });
   });
@@ -128,7 +128,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"const href = require(\\"./foo.css\\");"`
+        `"const href = require("./foo.css");"`
       );
     });
 
@@ -138,7 +138,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"import href from \\"./foo.css\\";"`
+        `"import href from "./foo.css";"`
       );
     });
 
@@ -148,7 +148,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"import { foo } from \\"./foo.css\\";"`
+        `"import { foo } from "./foo.css";"`
       );
     });
 
@@ -158,7 +158,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"import * as foo from \\"./foo.css\\";"`
+        `"import * as foo from "./foo.css";"`
       );
     });
 
@@ -171,9 +171,9 @@ describe("addSuffixToCssSideEffectImports", () => {
 
       expect(addSuffixToCssSideEffectImports("js", code))
         .toMatchInlineSnapshot(`
-        "const href = process.env.NODE_ENV === \\"production\\" ?
-        require(\\"./foo.min.css\\") :
-        require(\\"./foo.css\\");"
+        "const href = process.env.NODE_ENV === "production" ?
+        require("./foo.min.css") :
+        require("./foo.css");"
       `);
     });
 
@@ -183,7 +183,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"const href = process.env.NODE_ENV === \\"production\\" && require(\\"./foo.min.css\\") || require(\\"./foo.css\\");"`
+        `"const href = process.env.NODE_ENV === "production" && require("./foo.min.css") || require("./foo.css");"`
       );
     });
 
@@ -193,7 +193,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"require(\\"./foo\\");"`
+        `"require("./foo");"`
       );
     });
 
@@ -203,7 +203,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       `;
 
       expect(addSuffixToCssSideEffectImports("js", code)).toMatchInlineSnapshot(
-        `"import \\"./foo\\";"`
+        `"import "./foo";"`
       );
     });
 
@@ -217,7 +217,7 @@ describe("addSuffixToCssSideEffectImports", () => {
       expect(addSuffixToCssSideEffectImports("js", code))
         .toMatchInlineSnapshot(`
         "export const foo = async () => {
-          await import(\\"./foo.css\\");
+          await import("./foo.css");
         };"
       `);
     });
@@ -288,7 +288,7 @@ describe("addSuffixToCssSideEffectImports", () => {
 
       expect(addSuffixToCssSideEffectImports("js", code))
         .toMatchInlineSnapshot(`
-        "require(\\"./foo.css?__remix_sideEffect__\\");
+        "require("./foo.css?__remix_sideEffect__");
 
         // JS
         const topLevelAwait = await Promise.resolve('top level await');
@@ -330,7 +330,7 @@ describe("addSuffixToCssSideEffectImports", () => {
 
       expect(addSuffixToCssSideEffectImports("jsx", code))
         .toMatchInlineSnapshot(`
-        "require(\\"./foo.css?__remix_sideEffect__\\");
+        "require("./foo.css?__remix_sideEffect__");
 
         // JSX
         const ExampleComponent = () => <div>JSX element</div>;
@@ -375,7 +375,7 @@ describe("addSuffixToCssSideEffectImports", () => {
 
       expect(addSuffixToCssSideEffectImports("tsx", code))
         .toMatchInlineSnapshot(`
-        "require(\\"./foo.css?__remix_sideEffect__\\");
+        "require("./foo.css?__remix_sideEffect__");
 
         // TS
         const exampleSatisfies = ('satisfies' satisfies string);
@@ -425,7 +425,7 @@ describe("addSuffixToCssSideEffectImports", () => {
 
       expect(addSuffixToCssSideEffectImports("tsx", code))
         .toMatchInlineSnapshot(`
-        "require(\\"./foo.css?__remix_sideEffect__\\");
+        "require("./foo.css?__remix_sideEffect__");
 
         // TS
         const exampleSatisfies = ('satisfies' satisfies string);