@nx/jest 22.7.0 → 22.7.2

package/migrations.json

@@ -42,6 +42,14 @@
       },
       "description": "Replace removed matcher aliases in Jest v30 with their corresponding matcher",
       "implementation": "./src/migrations/update-21-3-0/replace-removed-matcher-aliases"
+    },
+    "update-snapshot-guide-link": {
+      "version": "23.0.0-beta.6",
+      "requires": {
+        "jest": ">=30.0.0"
+      },
+      "description": "Update the Jest snapshot guide link in `.snap` files from the legacy `https://goo.gl/fbAQLP` URL to `https://jestjs.io/docs/snapshot-testing`, which Jest v30 now requires.",
+      "implementation": "./src/migrations/update-23-0-0/update-snapshot-guide-link"
     }
   },
   "packageJsonUpdates": {
@@ -171,6 +179,26 @@
           "alwaysAddToPackageJson": false
         }
       }
+    },
+    "23.0.0-pin-jest-30-3-for-rn-compat": {
+      "version": "23.0.0-beta.9",
+      "requires": {
+        "jest": ">=30.0.0"
+      },
+      "packages": {
+        "jest": {
+          "version": "~30.3.0",
+          "alwaysAddToPackageJson": false
+        },
+        "babel-jest": {
+          "version": "~30.3.0",
+          "alwaysAddToPackageJson": false
+        },
+        "@types/jest": {
+          "version": "~30.0.0",
+          "alwaysAddToPackageJson": false
+        }
+      }
     }
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nx/jest",
-  "version": "22.7.0",
+  "version": "22.7.2",
   "private": false,
   "description": "The Nx Plugin for Jest contains executors and generators allowing your workspace to use the powerful Jest testing capabilities.",
   "repository": {
@@ -38,14 +38,14 @@
   "dependencies": {
     "@jest/reporters": "^30.0.2",
     "@jest/test-result": "^30.0.2",
-    "@nx/devkit": "22.7.0",
-    "@nx/js": "22.7.0",
-    "@phenomnomnominal/tsquery": "~6.1.4",
+    "@nx/devkit": "22.7.2",
+    "@nx/js": "22.7.2",
+    "@phenomnomnominal/tsquery": "~6.2.0",
     "identity-obj-proxy": "3.0.0",
     "jest-config": "^30.0.2",
     "jest-resolve": "^30.0.2",
     "jest-util": "^30.0.2",
-    "minimatch": "10.2.4",
+    "minimatch": "10.2.5",
     "picocolors": "^1.1.0",
     "resolve.exports": "2.0.3",
     "semver": "^7.6.3",
@@ -53,7 +53,7 @@
     "yargs-parser": "21.1.1"
   },
   "devDependencies": {
-    "nx": "22.7.0"
+    "nx": "22.7.2"
   },
   "publishConfig": {
     "access": "public"

package/preset/jest-preset.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"jest-preset.d.ts","sourceRoot":"","sources":["../../../../packages/jest/preset/jest-preset.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,EAAc,MAAM,MAAM,CAAC;AAK1C,eAAO,MAAM,QAAQ,EAAE,MAsCtB,CAAC"}
+{"version":3,"file":"jest-preset.d.ts","sourceRoot":"","sources":["../../../../packages/jest/preset/jest-preset.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,EAAc,MAAM,MAAM,CAAC;AAK1C,eAAO,MAAM,QAAQ,EAAE,MAuCtB,CAAC"}

package/preset/jest-preset.js

@@ -25,6 +25,7 @@ exports.nxPreset = {
         ],
     },
     testEnvironment: 'jsdom',
+    modulePathIgnorePatterns: ['<rootDir>/dist/', '<rootDir>/out-tsc/'],
     /**
      * manually set the exports names to load in common js, to mimic the behaviors of jest 27
      * before jest didn't fully support package exports and would load in common js code (typically via main field). now jest 28+ will load in the browser esm code, but jest esm support is not fully supported.

package/src/migrations/update-22-2-0/convert-jest-config-to-cjs.d.ts

@@ -5,8 +5,6 @@ import { Tree } from '@nx/devkit';
  * in newer versions (22+, 24+) can cause issues with ESM syntax in .ts files
  * when the project is configured for CommonJS.
  *
- * This migration only runs if @nx/jest/plugin is registered in nx.json.
- *
  * Conversions:
  * - `export default { ... }` -> `module.exports = { ... }`
  * - `import { x } from 'y'` -> `const { x } = require('y')`
@@ -16,8 +14,8 @@ import { Tree } from '@nx/devkit';
  * - `import.meta`
  * - top-level `await`
  *
- * Projects with `type: module` in package.json will be warned as they are
- * incompatible with @nx/jest/plugin which forces CommonJS resolution.
+ * Projects with `type: module` in package.json are skipped and warned about,
+ * as they use ESM semantics and don't need the CJS conversion.
  */
 export default function convertJestConfigToCjs(tree: Tree): Promise<() => void>;
 //# sourceMappingURL=convert-jest-config-to-cjs.d.ts.map

package/src/migrations/update-22-2-0/convert-jest-config-to-cjs.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"convert-jest-config-to-cjs.d.ts","sourceRoot":"","sources":["../../../../../../packages/jest/src/migrations/update-22-2-0/convert-jest-config-to-cjs.ts"],"names":[],"mappings":"AAAA,OAAO,EAOL,IAAI,EACL,MAAM,YAAY,CAAC;AAIpB;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAA8B,sBAAsB,CAAC,IAAI,EAAE,IAAI,uBA+F9D"}
+{"version":3,"file":"convert-jest-config-to-cjs.d.ts","sourceRoot":"","sources":["../../../../../../packages/jest/src/migrations/update-22-2-0/convert-jest-config-to-cjs.ts"],"names":[],"mappings":"AAAA,OAAO,EAML,IAAI,EACL,MAAM,YAAY,CAAC;AAGpB;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAA8B,sBAAsB,CAAC,IAAI,EAAE,IAAI,uBAmF9D"}

package/src/migrations/update-22-2-0/convert-jest-config-to-cjs.js

@@ -2,7 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.default = convertJestConfigToCjs;
 const devkit_1 = require("@nx/devkit");
-const find_plugin_for_config_file_1 = require("@nx/devkit/src/utils/find-plugin-for-config-file");
 const path_1 = require("path");
 /**
  * Migration to convert jest.config.ts files from ESM to CJS syntax for projects
@@ -10,8 +9,6 @@ const path_1 = require("path");
  * in newer versions (22+, 24+) can cause issues with ESM syntax in .ts files
  * when the project is configured for CommonJS.
  *
- * This migration only runs if @nx/jest/plugin is registered in nx.json.
- *
  * Conversions:
  * - `export default { ... }` -> `module.exports = { ... }`
  * - `import { x } from 'y'` -> `const { x } = require('y')`
@@ -21,24 +18,16 @@ const path_1 = require("path");
  * - `import.meta`
  * - top-level `await`
  *
- * Projects with `type: module` in package.json will be warned as they are
- * incompatible with @nx/jest/plugin which forces CommonJS resolution.
+ * Projects with `type: module` in package.json are skipped and warned about,
+ * as they use ESM semantics and don't need the CJS conversion.
  */
 async function convertJestConfigToCjs(tree) {
-    // If @nx/jest/plugin not used, then there will not be any problems with graph construction, which
-    // is what we're trying to address.
-    if (!isJestPluginRegistered(tree))
-        return;
     const { ast: parseAst, query } = require('@phenomnomnominal/tsquery');
     const jestConfigPaths = await (0, devkit_1.globAsync)(tree, ['**/jest.config.ts']);
     const projectsWithEsmOnlyFeatures = [];
     const projectsWithTypeModule = [];
     const modifiedFiles = [];
     for (const configPath of jestConfigPaths) {
-        // Skip config files that are excluded from the plugin via include/exclude patterns
-        const pluginRegistration = await (0, find_plugin_for_config_file_1.findPluginForConfigFile)(tree, '@nx/jest/plugin', configPath);
-        if (!pluginRegistration)
-            continue;
         const projectRoot = (0, path_1.dirname)(configPath);
         const packageJsonPath = (0, devkit_1.joinPathFragments)(projectRoot, 'package.json');
         const rootPackageJsonPath = 'package.json';
@@ -52,8 +41,8 @@ async function convertJestConfigToCjs(tree) {
             rootPackageJson = (0, devkit_1.readJson)(tree, rootPackageJsonPath);
         }
         const effectiveType = projectPackageJson?.type ?? rootPackageJson?.type ?? 'commonjs'; // CJS is default if missing
-        // If type is "module", warn user - this is incompatible with @nx/jest/plugin
-        // Should not be possible, but it's possible that there's a way to get this working that we're unaware of
+        // If type is "module", skip conversion: the file already runs under ESM
+        // semantics and does not need the CJS rewrite.
         if (effectiveType === 'module') {
             projectsWithTypeModule.push(configPath);
             continue;
@@ -79,9 +68,9 @@ async function convertJestConfigToCjs(tree) {
     if (hasWarnings) {
         return () => {
             if (projectsWithTypeModule.length > 0) {
-                devkit_1.logger.warn(`The following projects have "type": "module" in their package.json which is incompatible ` +
-                    `with @nx/jest/plugin. Consider removing "type": "module" ` +
-                    `or using a different Jest configuration approach:\n` +
+                devkit_1.logger.warn(`The following jest.config.ts files belong to projects with "type": "module" in their package.json ` +
+                    `and were left as-is. If you use @nx/jest/plugin, it forces CommonJS resolution, so consider ` +
+                    `removing "type": "module" or using a different Jest configuration approach:\n` +
                     projectsWithTypeModule.map((p) => `  - ${p}`).join('\n'));
             }
             if (projectsWithEsmOnlyFeatures.length > 0) {
@@ -134,26 +123,36 @@ function convertImportsToRequire(content, parseAst, query) {
             content = replaceNode(content, importDecl, requireStatement);
             continue;
         }
+        // `import type ...` is erased by Node's type-stripping at runtime, so it
+        // can remain in the file untouched — this preserves editor/tsc type safety.
+        if (importClause.isTypeOnly) {
+            continue;
+        }
         const parts = [];
+        const typeOnlySpecifiers = [];
         // Default import: import x from 'module'
         if (importClause.name) {
             const defaultName = importClause.name.getText();
             parts.push(`const ${defaultName} = require('${moduleSpecifier}').default ?? require('${moduleSpecifier}')`);
         }
-        // Named imports: import { a, b } from 'module'
         if (importClause.namedBindings) {
             if (ts.isNamedImports(importClause.namedBindings)) {
-                const namedImports = importClause.namedBindings.elements
-                    .map((element) => {
+                const valueSpecifiers = [];
+                for (const element of importClause.namedBindings.elements) {
                     const name = element.name.getText();
                     const propertyName = element.propertyName?.getText();
-                    if (propertyName) {
-                        return `${propertyName}: ${name}`;
+                    if (element.isTypeOnly) {
+                        // Inline `type` modifier: `import { type Foo, bar } from 'x'`.
+                        // Preserve the type-only portion so type references still resolve.
+                        typeOnlySpecifiers.push(propertyName ? `${propertyName} as ${name}` : name);
+                    }
+                    else {
+                        valueSpecifiers.push(propertyName ? `${propertyName}: ${name}` : name);
                     }
-                    return name;
-                })
-                    .join(', ');
-                parts.push(`const { ${namedImports} } = require('${moduleSpecifier}')`);
+                }
+                if (valueSpecifiers.length) {
+                    parts.push(`const { ${valueSpecifiers.join(', ')} } = require('${moduleSpecifier}')`);
+                }
             }
             else if (ts.isNamespaceImport(importClause.namedBindings)) {
                 // Namespace import: import * as x from 'module'
@@ -161,8 +160,16 @@ function convertImportsToRequire(content, parseAst, query) {
                 parts.push(`const ${namespaceName} = require('${moduleSpecifier}')`);
             }
         }
-        const requireStatement = parts.join(';\n');
-        content = replaceNode(content, importDecl, requireStatement);
+        const replacementParts = [];
+        if (typeOnlySpecifiers.length) {
+            replacementParts.push(`import type { ${typeOnlySpecifiers.join(', ')} } from '${moduleSpecifier}'`);
+        }
+        replacementParts.push(...parts);
+        if (replacementParts.length === 0) {
+            continue;
+        }
+        const replacement = replacementParts.join(';\n');
+        content = replaceNode(content, importDecl, replacement);
     }
     return content;
 }
@@ -191,14 +198,3 @@ function replaceNode(content, node, replacement) {
     }
     return content.slice(0, start) + replacement + ';' + content.slice(endPos);
 }
-function isJestPluginRegistered(tree) {
-    if (!tree.exists('nx.json')) {
-        return false;
-    }
-    const nxJson = (0, devkit_1.readJson)(tree, 'nx.json');
-    const plugins = nxJson.plugins ?? [];
-    return plugins.some((plugin) => {
-        const pluginName = typeof plugin === 'string' ? plugin : plugin.plugin;
-        return pluginName === '@nx/jest/plugin' || pluginName === '@nx/jest';
-    });
-}

package/src/migrations/update-23-0-0/update-snapshot-guide-link.d.ts

@@ -0,0 +1,3 @@
+import { type Tree } from '@nx/devkit';
+export default function updateSnapshotGuideLink(tree: Tree): Promise<void>;
+//# sourceMappingURL=update-snapshot-guide-link.d.ts.map

package/src/migrations/update-23-0-0/update-snapshot-guide-link.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"update-snapshot-guide-link.d.ts","sourceRoot":"","sources":["../../../../../../packages/jest/src/migrations/update-23-0-0/update-snapshot-guide-link.ts"],"names":[],"mappings":"AAAA,OAAO,EAA0B,KAAK,IAAI,EAAE,MAAM,YAAY,CAAC;AAM/D,wBAA8B,uBAAuB,CAAC,IAAI,EAAE,IAAI,iBA2B/D"}

package/src/migrations/update-23-0-0/update-snapshot-guide-link.js

@@ -0,0 +1,28 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.default = updateSnapshotGuideLink;
+const devkit_1 = require("@nx/devkit");
+const OLD_HEADER = '// Jest Snapshot v1, https://goo.gl/fbAQLP';
+const NEW_HEADER = '// Jest Snapshot v1, https://jestjs.io/docs/snapshot-testing';
+async function updateSnapshotGuideLink(tree) {
+    const snapshotFiles = await (0, devkit_1.globAsync)(tree, ['**/__snapshots__/*.snap']);
+    for (const snapshotFile of snapshotFiles) {
+        const content = tree.read(snapshotFile, 'utf-8');
+        if (!content) {
+            continue;
+        }
+        const newlineMatch = content.match(/\r?\n/);
+        if (!newlineMatch) {
+            continue;
+        }
+        const newline = newlineMatch[0];
+        const firstNewlineIndex = content.indexOf(newline);
+        const firstLine = content.slice(0, firstNewlineIndex);
+        if (firstLine !== OLD_HEADER) {
+            continue;
+        }
+        const updated = NEW_HEADER + content.slice(firstNewlineIndex);
+        tree.write(snapshotFile, updated);
+    }
+    await (0, devkit_1.formatFiles)(tree);
+}

package/src/utils/versions.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"versions.d.ts","sourceRoot":"","sources":["../../../../../packages/jest/src/utils/versions.ts"],"names":[],"mappings":"AAAA,OAAO,EAAuC,KAAK,IAAI,EAAE,MAAM,YAAY,CAAC;AAK5E,eAAO,MAAM,cAAc;;;;;;;;;;CAU1B,CAAC;AAEF,QAAA,MAAM,sBAAsB,mBAAoB,CAAC;AAIjD,KAAK,iBAAiB,GAAG,CAAC,OAAO,sBAAsB,CAAC,CAAC,MAAM,CAAC,CAAC;AACjE,KAAK,mBAAmB,GAAG,MAAM,OAAO,cAAc,CAAC;AACvD,MAAM,MAAM,UAAU,GAAG;KACtB,GAAG,IAAI,iBAAiB,GAAG,MAAM,CAAC,mBAAmB,EAAE,MAAM,CAAC;CAChE,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,UAaxB,CAAC;AAEF,wBAAgB,QAAQ,CAAC,IAAI,EAAE,IAAI,OAiBlC;AAED,wBAAgB,uBAAuB,CAAC,IAAI,CAAC,EAAE,IAAI,GAAG,MAAM,GAAG,IAAI,CAclE;AAED,wBAAgB,2BAA2B,CAAC,IAAI,CAAC,EAAE,IAAI,GAAG;IACxD,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;CACtB,CAMA;AAED,wBAAgB,4BAA4B,CAAC,IAAI,CAAC,EAAE,IAAI,GAAG,MAAM,GAAG,IAAI,CAIvE;AAED,wBAAgB,4BAA4B,CAAC,IAAI,CAAC,EAAE,IAAI,GAAG,IAAI,CAc9D"}
+{"version":3,"file":"versions.d.ts","sourceRoot":"","sources":["../../../../../packages/jest/src/utils/versions.ts"],"names":[],"mappings":"AAAA,OAAO,EAAuC,KAAK,IAAI,EAAE,MAAM,YAAY,CAAC;AAU5E,eAAO,MAAM,cAAc;;;;;;;;;;CAU1B,CAAC;AAEF,QAAA,MAAM,sBAAsB,mBAAoB,CAAC;AAIjD,KAAK,iBAAiB,GAAG,CAAC,OAAO,sBAAsB,CAAC,CAAC,MAAM,CAAC,CAAC;AACjE,KAAK,mBAAmB,GAAG,MAAM,OAAO,cAAc,CAAC;AACvD,MAAM,MAAM,UAAU,GAAG;KACtB,GAAG,IAAI,iBAAiB,GAAG,MAAM,CAAC,mBAAmB,EAAE,MAAM,CAAC;CAChE,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,UAaxB,CAAC;AAEF,wBAAgB,QAAQ,CAAC,IAAI,EAAE,IAAI,OAiBlC;AAED,wBAAgB,uBAAuB,CAAC,IAAI,CAAC,EAAE,IAAI,GAAG,MAAM,GAAG,IAAI,CAclE;AAED,wBAAgB,2BAA2B,CAAC,IAAI,CAAC,EAAE,IAAI,GAAG;IACxD,OAAO,EAAE,MAAM,GAAG,IAAI,CAAC;IACvB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;CACtB,CAMA;AAED,wBAAgB,4BAA4B,CAAC,IAAI,CAAC,EAAE,IAAI,GAAG,MAAM,GAAG,IAAI,CAIvE;AAED,wBAAgB,4BAA4B,CAAC,IAAI,CAAC,EAAE,IAAI,GAAG,IAAI,CAc9D"}

package/src/utils/versions.js

@@ -9,11 +9,16 @@ exports.validateInstalledJestVersion = validateInstalledJestVersion;
 const devkit_1 = require("@nx/devkit");
 const semver_1 = require("semver");
 const nxVersion = require('../../package.json').version;
+// Jest is pinned to 30.3.x because jest-runtime@30.4.0 added a call to
+// `_moduleMocker.clearMocksOnScope()`, which doesn't exist on the
+// jest-mock@29 ModuleMocker that React Native's preset still feeds in
+// (via `@react-native/jest-preset`'s pinned `jest-environment-node@^29.7.0`).
+// Lift this once Meta ships a Jest-30-aware preset on react-native.
 exports.latestVersions = {
     nxVersion,
-    jestVersion: '^30.0.2',
-    babelJestVersion: '^30.0.2',
-    jestTypesVersion: '^30.0.0',
+    jestVersion: '~30.3.0',
+    babelJestVersion: '~30.3.0',
+    jestTypesVersion: '~30.0.0',
     tsJestVersion: '^29.4.0',
     tslibVersion: '^2.3.0',
     swcJestVersion: '~0.2.38',