feat: create new valid-mock-module-path rule

README.md

@@ -386,6 +386,7 @@ Manually fixable by
 | [valid-describe-callback](docs/rules/valid-describe-callback.md)                     | Enforce valid `describe()` callback                                       | ✅  |     |     |     |
 | [valid-expect](docs/rules/valid-expect.md)                                           | Enforce valid `expect()` usage                                            | ✅  |     | 🔧  |     |
 | [valid-expect-in-promise](docs/rules/valid-expect-in-promise.md)                     | Require promises that have expectations in their chain to be valid        | ✅  |     |     |     |
+| [valid-mocked-module-path](docs/rules/valid-mocked-module-path.md)                   | Disallow mocking of non-existing module path                              |     |     |     |     |
 | [valid-title](docs/rules/valid-title.md)                                             | Enforce valid titles                                                      | ✅  |     | 🔧  |     |
 
 ### Requires Type Checking

docs/rules/valid-mocked-module-path.md

@@ -0,0 +1,43 @@
+# Disallow mocking of non-existing module path (`valid-mocked-module-path`)
+
+<!-- end auto-generated rule header -->
+
+This rule raises an error when using `jest.mock` and `jest.doMock` and the first
+argument for mocked object (module/local file) do not exist.
+
+## Rule details
+
+This rule checks existence of the supplied path for `jest.mock` or `jest.doMock`
+in the first argument.
+
+The following patterns are considered errors:
+
+```js
+// Module(s) that cannot be found
+jest.mock('@org/some-module-not-in-package-json');
+jest.mock('some-module-not-in-package-json');
+
+// Local module (directory) that cannot be found
+jest.mock('../../this/module/does/not/exist');
+
+// Local file that cannot be found
+jest.mock('../../this/path/does/not/exist.js');
+```
+
+The following patterns are **not** considered errors:
+
+```js
+// Module(s) that can be found
+jest.mock('@org/some-module-in-package-json');
+jest.mock('some-module-in-package-json');
+
+// Local module that cannot be found
+jest.mock('../../this/module/really/does/exist');
+
+// Local file that cannot be found
+jest.mock('../../this/path/really/does/exist.js');
+```
+
+## When Not To Use It
+
+Don't use this rule on non-jest test files.

markdown_link_check_config.json

@@ -6,5 +6,11 @@
         "Accept-Encoding": "br, gzip, deflate"
       }
     }
+  ],
+  "replacementPatterns": [
+    {
+      "pattern": "^https:\\/\\/www\\.npmjs\\.com\\/package",
+      "replacement": "https://registry.npmjs.org/"
+    }
   ]
 }

package.json

@@ -112,7 +112,7 @@
     "jest": "^30.0.0",
     "jest-runner-eslint": "^2.0.0",
     "lint-staged": "^16.0.0",
-    "markdown-link-check": "^3.13.7",
+    "markdown-link-check": "^3.14.1",
     "pinst": "^3.0.0",
     "prettier": "^3.0.0",
     "rimraf": "^6.0.0",

src/__tests__/__snapshots__/rules.test.ts.snap

@@ -72,6 +72,7 @@ exports[`rules should export configs that refer to actual rules 1`] = `
       "jest/valid-describe-callback": "error",
       "jest/valid-expect": "error",
       "jest/valid-expect-in-promise": "error",
+      "jest/valid-mocked-module-path": "error",
       "jest/valid-title": "error",
     },
   },
@@ -164,6 +165,7 @@ exports[`rules should export configs that refer to actual rules 1`] = `
       "jest/valid-describe-callback": "error",
       "jest/valid-expect": "error",
       "jest/valid-expect-in-promise": "error",
+      "jest/valid-mocked-module-path": "error",
       "jest/valid-title": "error",
     },
   },

src/__tests__/rules.test.ts

@@ -2,7 +2,7 @@ import { existsSync } from 'fs';
 import { resolve } from 'path';
 import plugin from '../';
 
-const numberOfRules = 63;
+const numberOfRules = 64;
 const ruleNames = Object.keys(plugin.rules);
 const deprecatedRules = Object.entries(plugin.rules)
   .filter(([, rule]) => rule.meta.deprecated)

src/rules/__tests__/fixtures/module/foo.js

@@ -0,0 +1 @@
+export const foo = 'foo_js';

src/rules/__tests__/fixtures/module/foo.ts

@@ -0,0 +1 @@
+export const foo = 'foo_ts';

src/rules/__tests__/fixtures/module/index.ts

@@ -0,0 +1 @@
+export * from './foo';

src/rules/__tests__/valid-mocked-module-path.test.ts

@@ -0,0 +1,74 @@
+import dedent from 'dedent';
+import rule from '../valid-mocked-module-path';
+import { FlatCompatRuleTester as RuleTester, espreeParser } from './test-utils';
+
+const ruleTester = new RuleTester({
+  parser: espreeParser,
+  parserOptions: {
+    ecmaVersion: 2015,
+  },
+});
+
+ruleTester.run('valid-mocked-module-path', rule, {
+  valid: [
+    { filename: __filename, code: 'jest.mock("./fixtures/module")' },
+    { filename: __filename, code: 'jest.mock("./fixtures/module", () => {})' },
+    { filename: __filename, code: 'jest.mock()' },
+    {
+      filename: __filename,
+      code: 'jest.doMock("./fixtures/module", () => {})',
+    },
+    {
+      filename: __filename,
+      code: dedent`
+        describe("foo", () => {});
+      `,
+    },
+    { filename: __filename, code: 'jest.doMock("./fixtures/module")' },
+    { filename: __filename, code: 'jest.mock("./fixtures/module/foo.ts")' },
+    { filename: __filename, code: 'jest.doMock("./fixtures/module/foo.ts")' },
+    { filename: __filename, code: 'jest.mock("./fixtures/module/foo.js")' },
+    { filename: __filename, code: 'jest.doMock("./fixtures/module/foo.js")' },
+    'jest.mock("eslint")',
+    'jest.doMock("eslint")',
+    'jest.mock("child_process")',
+  ],
+  invalid: [
+    {
+      filename: __filename,
+      code: "jest.mock('../module/does/not/exist')",
+      errors: [
+        {
+          messageId: 'invalidMockModulePath',
+          data: { moduleName: "'../module/does/not/exist'" },
+          column: 1,
+          line: 1,
+        },
+      ],
+    },
+    {
+      filename: __filename,
+      code: 'jest.mock("../file/does/not/exist.ts")',
+      errors: [
+        {
+          messageId: 'invalidMockModulePath',
+          data: { moduleName: '"../file/does/not/exist.ts"' },
+          column: 1,
+          line: 1,
+        },
+      ],
+    },
+    {
+      filename: __filename,
+      code: 'jest.mock("@doesnotexist/module")',
+      errors: [
+        {
+          messageId: 'invalidMockModulePath',
+          data: { moduleName: '"@doesnotexist/module"' },
+          column: 1,
+          line: 1,
+        },
+      ],
+    },
+  ],
+});

src/rules/no-untyped-mock-factory.ts

@@ -1,22 +1,13 @@
 import { AST_NODE_TYPES, type TSESTree } from '@typescript-eslint/utils';
 import {
   createRule,
+  findModuleName,
   getAccessorValue,
   isFunction,
   isSupportedAccessor,
   isTypeOfJestFnCall,
 } from './utils';
 
-const findModuleName = (
-  node: TSESTree.Literal | TSESTree.Node,
-): TSESTree.StringLiteral | null => {
-  if (node.type === AST_NODE_TYPES.Literal && typeof node.value === 'string') {
-    return node;
-  }
-
-  return null;
-};
-
 export default createRule({
   name: __filename,
   meta: {

src/rules/utils/misc.ts

@@ -225,3 +225,13 @@ export const getFirstMatcherArg = (
 
   return followTypeAssertionChain(firstArg);
 };
+
+export const findModuleName = (
+  node: TSESTree.Literal | TSESTree.Node,
+): TSESTree.StringLiteral | null => {
+  if (node.type === AST_NODE_TYPES.Literal && typeof node.value === 'string') {
+    return node;
+  }
+
+  return null;
+};

src/rules/valid-mocked-module-path.ts

@@ -0,0 +1,99 @@
+import { statSync } from 'fs';
+import path from 'path';
+import { AST_NODE_TYPES, type TSESTree } from '@typescript-eslint/utils';
+import {
+  createRule,
+  findModuleName,
+  getAccessorValue,
+  isSupportedAccessor,
+  isTypeOfJestFnCall,
+} from './utils';
+
+export default createRule({
+  name: __filename,
+  meta: {
+    type: 'problem',
+    docs: {
+      description: 'Disallow mocking of non-existing module path',
+    },
+    messages: {
+      invalidMockModulePath: 'Mocked module path {{moduleName}} does not exist',
+    },
+    schema: [],
+  },
+  defaultOptions: [],
+  create(context) {
+    return {
+      CallExpression(node: TSESTree.CallExpression): void {
+        const { callee } = node;
+
+        if (callee.type !== AST_NODE_TYPES.MemberExpression) {
+          return;
+        }
+
+        const { property } = callee;
+
+        if (
+          !node.arguments.length ||
+          !isTypeOfJestFnCall(node, context, ['jest']) ||
+          !(
+            isSupportedAccessor(property) &&
+            ['mock', 'doMock'].includes(getAccessorValue(property))
+          )
+        ) {
+          return;
+        }
+
+        const [nameNode] = node.arguments;
+        const moduleName = findModuleName(nameNode);
+
+        /* istanbul ignore if */
+        if (!moduleName) {
+          throw new Error(
+            'Cannot parse mocked module name from `jest.mock` -  - please file a github issue at https://github.com/jest-community/eslint-plugin-jest`',
+          );
+        }
+
+        try {
+          if (moduleName.value.startsWith('.')) {
+            const resolvedModulePath = path.resolve(
+              path.dirname(context.filename),
+              moduleName.value,
+            );
+
+            const hasPossiblyModulePaths = ['', '.js', '.ts']
+              .map(ext => `${resolvedModulePath}${ext}`)
+              .some(modPath => {
+                try {
+                  statSync(modPath);
+
+                  return true;
+                } catch {
+                  return false;
+                }
+              });
+
+            if (!hasPossiblyModulePaths) {
+              throw { code: 'MODULE_NOT_FOUND' };
+            }
+          } else {
+            require.resolve(moduleName.value);
+          }
+        } catch (err: any) {
+          // Skip over any unexpected issues when attempt to verify mocked module path.
+          // The list of possible errors is non-exhaustive.
+          /* istanbul ignore if */
+          if (!['MODULE_NOT_FOUND', 'ENOENT'].includes(err.code)) {
+            return;
+          }
+
+          context.report({
+            messageId: 'invalidMockModulePath',
+            data: { moduleName: moduleName.raw },
+            node,
+          });
+        }
+      },
+    };
+  },
+});

yarn.lock

@@ -5463,7 +5463,7 @@ __metadata:
     jest: "npm:^30.0.0"
     jest-runner-eslint: "npm:^2.0.0"
     lint-staged: "npm:^16.0.0"
-    markdown-link-check: "npm:^3.13.7"
+    markdown-link-check: "npm:^3.14.1"
     pinst: "npm:^3.0.0"
     prettier: "npm:^3.0.0"
     rimraf: "npm:^6.0.0"
@@ -8442,7 +8442,7 @@ __metadata:
   languageName: node
   linkType: hard
 
-"markdown-link-check@npm:^3.13.7":
+"markdown-link-check@npm:^3.14.1":
   version: 3.14.1
   resolution: "markdown-link-check@npm:3.14.1"
   dependencies: