@vitest/eslint-plugin 1.5.4 → 1.6.3

package/README.md

@@ -251,12 +251,12 @@ export default defineConfig({
 | [prefer-to-be-object](docs/rules/prefer-to-be-object.md)                                                                 | enforce using toBeObject()                                                                      |     | 🌐    |     | 🔧  |     |     |     |
 | [prefer-to-be-truthy](docs/rules/prefer-to-be-truthy.md)                                                                 | enforce using `toBeTruthy`                                                                      |     |       | 🌐  | 🔧  |     |     |     |
 | [prefer-to-contain](docs/rules/prefer-to-contain.md)                                                                     | enforce using toContain()                                                                       |     | 🌐    |     | 🔧  |     |     |     |
+| [prefer-to-have-been-called-times](docs/rules/prefer-to-have-been-called-times.md)                                       | Suggest using `toHaveBeenCalledTimes()`                                                         |     | 🌐    |     | 🔧  |     |     |     |
 | [prefer-to-have-length](docs/rules/prefer-to-have-length.md)                                                             | enforce using toHaveLength()                                                                    |     | 🌐    |     | 🔧  |     |     |     |
 | [prefer-todo](docs/rules/prefer-todo.md)                                                                                 | enforce using `test.todo`                                                                       |     | 🌐    |     | 🔧  |     |     |     |
 | [prefer-vi-mocked](docs/rules/prefer-vi-mocked.md)                                                                       | require `vi.mocked()` over `fn as Mock`                                                         |     | 🌐    |     | 🔧  |     | 💭  |     |
 | [require-awaited-expect-poll](docs/rules/require-awaited-expect-poll.md)                                                 | ensure that every `expect.poll` call is awaited                                                 |     | 🌐    |     |     |     |     |     |
 | [require-hook](docs/rules/require-hook.md)                                                                               | require setup and teardown to be within a hook                                                  |     | 🌐    |     |     |     |     |     |
-| [require-import-vi-mock](docs/rules/require-import-vi-mock.md)                                                           | require usage of import in vi.mock()                                                            |     | 🌐    |     | 🔧  |     |     |     |
 | [require-local-test-context-for-concurrent-snapshots](docs/rules/require-local-test-context-for-concurrent-snapshots.md) | require local Test Context for concurrent snapshot tests                                        | ✅  | 🌐    |     |     |     |     |     |
 | [require-mock-type-parameters](docs/rules/require-mock-type-parameters.md)                                               | enforce using type parameters with vitest mock functions                                        |     | 🌐    |     | 🔧  |     |     |     |
 | [require-to-throw-message](docs/rules/require-to-throw-message.md)                                                       | require toThrow() to be called with an error message                                            |     | 🌐    |     |     |     |     |     |

package/dist/index.cjs

@@ -31,7 +31,7 @@ let __typescript_eslint_scope_manager = require("@typescript-eslint/scope-manage
 __typescript_eslint_scope_manager = __toESM(__typescript_eslint_scope_manager);
 
 //#region package.json
-var version = "1.5.4";
+var version = "1.6.3";
 
 //#endregion
 //#region src/utils/index.ts
@@ -1738,7 +1738,16 @@ const resolveScope = (scope, identifier) => {
 				if ((property?.key.type === __typescript_eslint_utils.AST_NODE_TYPES.Identifier ? property.key : void 0)?.name === identifier) return "testContext";
 			}
 			/** if detect test function is created with `.extend()` */
-			if (def.node.type === __typescript_eslint_utils.AST_NODE_TYPES.VariableDeclarator && def.node.id.type === __typescript_eslint_utils.AST_NODE_TYPES.Identifier && Object.prototype.hasOwnProperty.call(TestCaseName, def.node.id.name) && def.node.init?.type === __typescript_eslint_utils.AST_NODE_TYPES.CallExpression && def.node.init.callee.type === __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression && isIdentifier(def.node.init.callee.property, "extend")) return "testContext";
+			if (def.node.type === __typescript_eslint_utils.AST_NODE_TYPES.VariableDeclarator && def.node.id.type === __typescript_eslint_utils.AST_NODE_TYPES.Identifier && def.node.init?.type === __typescript_eslint_utils.AST_NODE_TYPES.CallExpression && def.node.init.callee.type === __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression && isIdentifier(def.node.init.callee.property, "extend")) {
+				const rootName = getNodeName(def.node.init.callee.object)?.split(".")[0];
+				if (rootName && rootName !== identifier) {
+					const resolved = resolveScope(currentScope, rootName);
+					if (resolved && typeof resolved === "object" && Object.hasOwn(TestCaseName, resolved.imported)) return {
+						...resolved,
+						local: identifier
+					};
+				}
+			}
 			const namedParam = isFunction(def.node) ? def.node.params.find((params) => params.type === __typescript_eslint_utils.AST_NODE_TYPES.Identifier) : void 0;
 			if (namedParam && isAncestorTestCaseCall(namedParam.parent)) return "testContext";
 			const importDetails = describePossibleImportDef(def);
@@ -4719,21 +4728,30 @@ var prefer_import_in_mock_default = createEslintRule({
 		type: "suggestion",
 		docs: { description: "prefer dynamic import in mock" },
 		messages: { preferImport: "Replace '{{path}}' with import('{{path}}')" },
-		schema: []
+		schema: [{
+			type: "object",
+			properties: { fixable: {
+				type: "boolean",
+				default: true
+			} },
+			additionalProperties: false
+		}]
 	},
-	defaultOptions: [],
-	create(context) {
+	defaultOptions: [{ fixable: true }],
+	create(context, options) {
+		const fixable = options[0].fixable;
 		return { CallExpression(node) {
 			if (node.callee.type !== __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression) return;
-			const { object, property } = node.callee;
-			if (object.type !== __typescript_eslint_utils.AST_NODE_TYPES.Identifier || object.name !== "vi" || property.type !== __typescript_eslint_utils.AST_NODE_TYPES.Identifier) return;
-			const apiName = property.name;
+			if (parseVitestFnCall(node, context)?.type !== "vi") return false;
+			const { property } = node.callee;
+			if (property.type != __typescript_eslint_utils.AST_NODE_TYPES.Identifier || property.name != "mock") return;
 			const pathArg = node.arguments[0];
-			if (apiName === "mock" && pathArg && pathArg.type === __typescript_eslint_utils.AST_NODE_TYPES.Literal) context.report({
+			if (pathArg && pathArg.type === __typescript_eslint_utils.AST_NODE_TYPES.Literal) context.report({
 				messageId: "preferImport",
 				data: { path: pathArg.value },
 				node,
 				fix(fixer) {
+					if (!fixable) return null;
 					return fixer.replaceText(pathArg, `import('${pathArg.value}')`);
 				}
 			});
@@ -5657,47 +5675,11 @@ var require_hook_default = createEslintRule({
 	}
 });
 
-//#endregion
-//#region src/rules/require-import-vi-mock.ts
-const RULE_NAME$10 = "require-import-vi-mock";
-var require_import_vi_mock_default = createEslintRule({
-	name: RULE_NAME$10,
-	meta: {
-		fixable: "code",
-		type: "suggestion",
-		docs: {
-			description: "require usage of import in vi.mock()",
-			requiresTypeChecking: false,
-			recommended: false
-		},
-		messages: { requireImport: "Replace '{{path}}' with import('{{path}}')" },
-		schema: []
-	},
-	defaultOptions: [],
-	create(context) {
-		return { CallExpression(node) {
-			if (node.callee.type !== __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression) return;
-			if (parseVitestFnCall(node, context)?.type !== "vi") return false;
-			const { property } = node.callee;
-			if (property.type !== __typescript_eslint_utils.AST_NODE_TYPES.Identifier || property.name !== "mock") return;
-			const pathArg = node.arguments[0];
-			if (pathArg && pathArg.type === __typescript_eslint_utils.AST_NODE_TYPES.Literal) context.report({
-				messageId: "requireImport",
-				data: { path: pathArg.value },
-				node: pathArg,
-				fix(fixer) {
-					return fixer.replaceText(pathArg, `import('${pathArg.value}')`);
-				}
-			});
-		} };
-	}
-});
-
 //#endregion
 //#region src/rules/require-local-test-context-for-concurrent-snapshots.ts
-const RULE_NAME$9 = "require-local-test-context-for-concurrent-snapshots";
+const RULE_NAME$10 = "require-local-test-context-for-concurrent-snapshots";
 var require_local_test_context_for_concurrent_snapshots_default = createEslintRule({
-	name: RULE_NAME$9,
+	name: RULE_NAME$10,
 	meta: {
 		docs: {
 			description: "require local Test Context for concurrent snapshot tests",
@@ -5738,9 +5720,9 @@ var require_local_test_context_for_concurrent_snapshots_default = createEslintRu
 
 //#endregion
 //#region src/rules/require-mock-type-parameters.ts
-const RULE_NAME$8 = "require-mock-type-parameters";
+const RULE_NAME$9 = "require-mock-type-parameters";
 var require_mock_type_parameters_default = createEslintRule({
-	name: RULE_NAME$8,
+	name: RULE_NAME$9,
 	meta: {
 		type: "suggestion",
 		docs: {
@@ -5777,9 +5759,9 @@ var require_mock_type_parameters_default = createEslintRule({
 
 //#endregion
 //#region src/rules/require-to-throw-message.ts
-const RULE_NAME$7 = "require-to-throw-message";
+const RULE_NAME$8 = "require-to-throw-message";
 var require_to_throw_message_default = createEslintRule({
-	name: RULE_NAME$7,
+	name: RULE_NAME$8,
 	meta: {
 		type: "suggestion",
 		docs: {
@@ -5807,9 +5789,9 @@ var require_to_throw_message_default = createEslintRule({
 
 //#endregion
 //#region src/rules/require-top-level-describe.ts
-const RULE_NAME$6 = "require-top-level-describe";
+const RULE_NAME$7 = "require-top-level-describe";
 var require_top_level_describe_default = createEslintRule({
-	name: RULE_NAME$6,
+	name: RULE_NAME$7,
 	meta: {
 		docs: {
 			description: "enforce that all tests are in a top-level describe",
@@ -5878,7 +5860,7 @@ var require_top_level_describe_default = createEslintRule({
 
 //#endregion
 //#region src/rules/valid-describe-callback.ts
-const RULE_NAME$5 = "valid-describe-callback";
+const RULE_NAME$6 = "valid-describe-callback";
 const paramsLocation = (params) => {
 	const [first] = params;
 	const last = params[params.length - 1];
@@ -5900,7 +5882,7 @@ const reportUnexpectedReturnInDescribe = (blockStatement, context) => {
 	});
 };
 var valid_describe_callback_default = createEslintRule({
-	name: RULE_NAME$5,
+	name: RULE_NAME$6,
 	meta: {
 		type: "problem",
 		docs: {
@@ -5967,7 +5949,7 @@ var valid_describe_callback_default = createEslintRule({
 
 //#endregion
 //#region src/rules/valid-expect-in-promise.ts
-const RULE_NAME$4 = "valid-expect-in-promise";
+const RULE_NAME$5 = "valid-expect-in-promise";
 const defaultAsyncMatchers$1 = ["toRejectWith", "toResolveWith"];
 const isPromiseChainCall = (node) => {
 	if (node.type === __typescript_eslint_utils.AST_NODE_TYPES.CallExpression && node.callee.type === __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression && isSupportedAccessor(node.callee.property)) {
@@ -6099,7 +6081,7 @@ const isVariableAwaitedOrReturned = (variable, context) => {
 	return isValueAwaitedOrReturned(variable.id, body, context);
 };
 var valid_expect_in_promise_default = createEslintRule({
-	name: RULE_NAME$4,
+	name: RULE_NAME$5,
 	meta: {
 		docs: { description: "require promises that have expectations in their chain to be valid" },
 		messages: { expectInFloatingPromise: "This promise should either be returned or awaited to ensure the expects in its chain are called" },
@@ -6159,7 +6141,7 @@ var valid_expect_in_promise_default = createEslintRule({
 
 //#endregion
 //#region src/rules/valid-expect.ts
-const RULE_NAME$3 = "valid-expect";
+const RULE_NAME$4 = "valid-expect";
 const defaultAsyncMatchers = ["toReject", "toResolve"];
 /**
 * Async assertions might be called in Promise
@@ -6194,7 +6176,7 @@ const isAcceptableReturnNode = (node, allowReturn) => {
 	return [__typescript_eslint_utils.AST_NODE_TYPES.ArrowFunctionExpression, __typescript_eslint_utils.AST_NODE_TYPES.AwaitExpression].includes(node.type);
 };
 var valid_expect_default = createEslintRule({
-	name: RULE_NAME$3,
+	name: RULE_NAME$4,
 	meta: {
 		docs: {
 			description: "enforce valid `expect()` usage",
@@ -6387,7 +6369,7 @@ var valid_expect_default = createEslintRule({
 
 //#endregion
 //#region src/rules/valid-title.ts
-const RULE_NAME$2 = "valid-title";
+const RULE_NAME$3 = "valid-title";
 const trimFXPrefix = (word) => ["f", "x"].includes(word.charAt(0)) ? word.substring(1) : word;
 const quoteStringValue = (node) => node.type === __typescript_eslint_utils.AST_NODE_TYPES.TemplateLiteral ? `\`${node.quasis[0].value.raw}\`` : node.raw;
 const MatcherAndMessageSchema = {
@@ -6426,7 +6408,7 @@ const doesBinaryExpressionContainStringNode = (binaryExp) => {
 	return isStringNode(binaryExp.left);
 };
 var valid_title_default = createEslintRule({
-	name: RULE_NAME$2,
+	name: RULE_NAME$3,
 	meta: {
 		docs: {
 			description: "enforce valid titles",
@@ -6579,9 +6561,9 @@ var valid_title_default = createEslintRule({
 
 //#endregion
 //#region src/rules/warn-todo.ts
-const RULE_NAME$1 = "warn-todo";
+const RULE_NAME$2 = "warn-todo";
 var warn_todo_default = createEslintRule({
-	name: RULE_NAME$1,
+	name: RULE_NAME$2,
 	meta: {
 		docs: {
 			description: "disallow `.todo` usage",
@@ -6608,7 +6590,7 @@ var warn_todo_default = createEslintRule({
 
 //#endregion
 //#region src/rules/no-unneeded-async-expect-function.ts
-const RULE_NAME = "no-unneeded-async-expect-function";
+const RULE_NAME$1 = "no-unneeded-async-expect-function";
 const getAwaitedCallExpression = (expression) => {
 	if (!expression.async) return null;
 	if (expression.type === __typescript_eslint_utils.AST_NODE_TYPES.ArrowFunctionExpression && expression.body.type === __typescript_eslint_utils.AST_NODE_TYPES.AwaitExpression && expression.body.argument.type === __typescript_eslint_utils.AST_NODE_TYPES.CallExpression) return expression.body.argument;
@@ -6618,7 +6600,7 @@ const getAwaitedCallExpression = (expression) => {
 	return null;
 };
 var no_unneeded_async_expect_function_default = createEslintRule({
-	name: RULE_NAME,
+	name: RULE_NAME$1,
 	meta: {
 		docs: { description: "Disallow unnecessary async function wrapper for expected promises" },
 		fixable: "code",
@@ -6649,6 +6631,42 @@ var no_unneeded_async_expect_function_default = createEslintRule({
 	}
 });
 
+//#endregion
+//#region src/rules/prefer-to-have-been-called-times.ts
+const RULE_NAME = "prefer-to-have-been-called-times";
+var prefer_to_have_been_called_times_default = createEslintRule({
+	name: RULE_NAME,
+	meta: {
+		fixable: "code",
+		docs: { description: "Suggest using `toHaveBeenCalledTimes()`" },
+		messages: { preferMatcher: "Prefer `toHaveBeenCalledTimes`" },
+		type: "suggestion",
+		schema: []
+	},
+	defaultOptions: [],
+	create(context) {
+		return { CallExpression(node) {
+			const vitestFnCall = parseVitestFnCall(node, context);
+			if (vitestFnCall?.type !== "expect") return;
+			const { parent: expect } = vitestFnCall.head.node;
+			if (expect?.type !== __typescript_eslint_utils.AST_NODE_TYPES.CallExpression) return;
+			const { matcher } = vitestFnCall;
+			if (!isSupportedAccessor(matcher, "toHaveLength")) return;
+			const [argument] = expect.arguments;
+			if (argument?.type !== __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || !isSupportedAccessor(argument.property, "calls")) return;
+			const { object } = argument;
+			if (object.type !== __typescript_eslint_utils.AST_NODE_TYPES.MemberExpression || !isSupportedAccessor(object.property, "mock")) return;
+			context.report({
+				messageId: "preferMatcher",
+				node: matcher,
+				fix(fixer) {
+					return [fixer.removeRange([object.property.range[0] - 1, argument.range[1]]), fixer.replaceTextRange([matcher.parent.object.range[1], matcher.parent.range[1]], ".toHaveBeenCalledTimes")];
+				}
+			});
+		} };
+	}
+});
+
 //#endregion
 //#region src/rules/index.ts
 const rules = {
@@ -6716,12 +6734,12 @@ const rules = {
 	"prefer-to-be-truthy": prefer_to_be_truthy_default,
 	"prefer-to-be": prefer_to_be_default,
 	"prefer-to-contain": prefer_to_contain_default,
+	"prefer-to-have-been-called-times": prefer_to_have_been_called_times_default,
 	"prefer-to-have-length": prefer_to_have_length_default,
 	"prefer-todo": prefer_todo_default,
 	"prefer-vi-mocked": prefer_vi_mocked_default,
 	"require-awaited-expect-poll": require_awaited_expect_poll_default,
 	"require-hook": require_hook_default,
-	"require-import-vi-mock": require_import_vi_mock_default,
 	"require-local-test-context-for-concurrent-snapshots": require_local_test_context_for_concurrent_snapshots_default,
 	"require-mock-type-parameters": require_mock_type_parameters_default,
 	"require-to-throw-message": require_to_throw_message_default,
@@ -6814,6 +6832,7 @@ const allRules = {
 	"prefer-to-be-truthy": "off",
 	"prefer-to-be": "warn",
 	"prefer-to-contain": "warn",
+	"prefer-to-have-been-called-times": "warn",
 	"prefer-to-have-length": "warn",
 	"prefer-todo": "warn",
 	"prefer-vi-mocked": "warn",
@@ -6826,8 +6845,7 @@ const allRules = {
 	"valid-expect-in-promise": "warn",
 	"valid-expect": "warn",
 	"valid-title": "warn",
-	"require-awaited-expect-poll": "warn",
-	"require-import-vi-mock": "warn"
+	"require-awaited-expect-poll": "warn"
 };
 const recommendedRules = {
 	"expect-expect": "error",

package/dist/index.d.cts

@@ -133,6 +133,7 @@ declare const plugin: {
         readonly "vitest/prefer-to-be-truthy": "off";
         readonly "vitest/prefer-to-be": "warn";
         readonly "vitest/prefer-to-contain": "warn";
+        readonly "vitest/prefer-to-have-been-called-times": "warn";
         readonly "vitest/prefer-to-have-length": "warn";
         readonly "vitest/prefer-todo": "warn";
         readonly "vitest/prefer-vi-mocked": "warn";
@@ -146,7 +147,6 @@ declare const plugin: {
         readonly "vitest/valid-expect": "warn";
         readonly "vitest/valid-title": "warn";
         readonly "vitest/require-awaited-expect-poll": "warn";
-        readonly "vitest/require-import-vi-mock": "warn";
       };
     };
     readonly env: {

package/dist/index.d.ts

@@ -133,6 +133,7 @@ declare const plugin: {
         readonly "vitest/prefer-to-be-truthy": "off";
         readonly "vitest/prefer-to-be": "warn";
         readonly "vitest/prefer-to-contain": "warn";
+        readonly "vitest/prefer-to-have-been-called-times": "warn";
         readonly "vitest/prefer-to-have-length": "warn";
         readonly "vitest/prefer-todo": "warn";
         readonly "vitest/prefer-vi-mocked": "warn";
@@ -146,7 +147,6 @@ declare const plugin: {
         readonly "vitest/valid-expect": "warn";
         readonly "vitest/valid-title": "warn";
         readonly "vitest/require-awaited-expect-poll": "warn";
-        readonly "vitest/require-import-vi-mock": "warn";
       };
     };
     readonly env: {

package/dist/index.js

@@ -4,7 +4,7 @@ import { isAbsolute, posix } from "node:path";
 import { DefinitionType } from "@typescript-eslint/scope-manager";
 
 //#region package.json
-var version = "1.5.4";
+var version = "1.6.3";
 
 //#endregion
 //#region src/utils/index.ts
@@ -1711,7 +1711,16 @@ const resolveScope = (scope, identifier) => {
 				if ((property?.key.type === AST_NODE_TYPES.Identifier ? property.key : void 0)?.name === identifier) return "testContext";
 			}
 			/** if detect test function is created with `.extend()` */
-			if (def.node.type === AST_NODE_TYPES.VariableDeclarator && def.node.id.type === AST_NODE_TYPES.Identifier && Object.prototype.hasOwnProperty.call(TestCaseName, def.node.id.name) && def.node.init?.type === AST_NODE_TYPES.CallExpression && def.node.init.callee.type === AST_NODE_TYPES.MemberExpression && isIdentifier(def.node.init.callee.property, "extend")) return "testContext";
+			if (def.node.type === AST_NODE_TYPES.VariableDeclarator && def.node.id.type === AST_NODE_TYPES.Identifier && def.node.init?.type === AST_NODE_TYPES.CallExpression && def.node.init.callee.type === AST_NODE_TYPES.MemberExpression && isIdentifier(def.node.init.callee.property, "extend")) {
+				const rootName = getNodeName(def.node.init.callee.object)?.split(".")[0];
+				if (rootName && rootName !== identifier) {
+					const resolved = resolveScope(currentScope, rootName);
+					if (resolved && typeof resolved === "object" && Object.hasOwn(TestCaseName, resolved.imported)) return {
+						...resolved,
+						local: identifier
+					};
+				}
+			}
 			const namedParam = isFunction(def.node) ? def.node.params.find((params) => params.type === AST_NODE_TYPES.Identifier) : void 0;
 			if (namedParam && isAncestorTestCaseCall(namedParam.parent)) return "testContext";
 			const importDetails = describePossibleImportDef(def);
@@ -4692,21 +4701,30 @@ var prefer_import_in_mock_default = createEslintRule({
 		type: "suggestion",
 		docs: { description: "prefer dynamic import in mock" },
 		messages: { preferImport: "Replace '{{path}}' with import('{{path}}')" },
-		schema: []
+		schema: [{
+			type: "object",
+			properties: { fixable: {
+				type: "boolean",
+				default: true
+			} },
+			additionalProperties: false
+		}]
 	},
-	defaultOptions: [],
-	create(context) {
+	defaultOptions: [{ fixable: true }],
+	create(context, options) {
+		const fixable = options[0].fixable;
 		return { CallExpression(node) {
 			if (node.callee.type !== AST_NODE_TYPES.MemberExpression) return;
-			const { object, property } = node.callee;
-			if (object.type !== AST_NODE_TYPES.Identifier || object.name !== "vi" || property.type !== AST_NODE_TYPES.Identifier) return;
-			const apiName = property.name;
+			if (parseVitestFnCall(node, context)?.type !== "vi") return false;
+			const { property } = node.callee;
+			if (property.type != AST_NODE_TYPES.Identifier || property.name != "mock") return;
 			const pathArg = node.arguments[0];
-			if (apiName === "mock" && pathArg && pathArg.type === AST_NODE_TYPES.Literal) context.report({
+			if (pathArg && pathArg.type === AST_NODE_TYPES.Literal) context.report({
 				messageId: "preferImport",
 				data: { path: pathArg.value },
 				node,
 				fix(fixer) {
+					if (!fixable) return null;
 					return fixer.replaceText(pathArg, `import('${pathArg.value}')`);
 				}
 			});
@@ -5630,47 +5648,11 @@ var require_hook_default = createEslintRule({
 	}
 });
 
-//#endregion
-//#region src/rules/require-import-vi-mock.ts
-const RULE_NAME$10 = "require-import-vi-mock";
-var require_import_vi_mock_default = createEslintRule({
-	name: RULE_NAME$10,
-	meta: {
-		fixable: "code",
-		type: "suggestion",
-		docs: {
-			description: "require usage of import in vi.mock()",
-			requiresTypeChecking: false,
-			recommended: false
-		},
-		messages: { requireImport: "Replace '{{path}}' with import('{{path}}')" },
-		schema: []
-	},
-	defaultOptions: [],
-	create(context) {
-		return { CallExpression(node) {
-			if (node.callee.type !== AST_NODE_TYPES.MemberExpression) return;
-			if (parseVitestFnCall(node, context)?.type !== "vi") return false;
-			const { property } = node.callee;
-			if (property.type !== AST_NODE_TYPES.Identifier || property.name !== "mock") return;
-			const pathArg = node.arguments[0];
-			if (pathArg && pathArg.type === AST_NODE_TYPES.Literal) context.report({
-				messageId: "requireImport",
-				data: { path: pathArg.value },
-				node: pathArg,
-				fix(fixer) {
-					return fixer.replaceText(pathArg, `import('${pathArg.value}')`);
-				}
-			});
-		} };
-	}
-});
-
 //#endregion
 //#region src/rules/require-local-test-context-for-concurrent-snapshots.ts
-const RULE_NAME$9 = "require-local-test-context-for-concurrent-snapshots";
+const RULE_NAME$10 = "require-local-test-context-for-concurrent-snapshots";
 var require_local_test_context_for_concurrent_snapshots_default = createEslintRule({
-	name: RULE_NAME$9,
+	name: RULE_NAME$10,
 	meta: {
 		docs: {
 			description: "require local Test Context for concurrent snapshot tests",
@@ -5711,9 +5693,9 @@ var require_local_test_context_for_concurrent_snapshots_default = createEslintRu
 
 //#endregion
 //#region src/rules/require-mock-type-parameters.ts
-const RULE_NAME$8 = "require-mock-type-parameters";
+const RULE_NAME$9 = "require-mock-type-parameters";
 var require_mock_type_parameters_default = createEslintRule({
-	name: RULE_NAME$8,
+	name: RULE_NAME$9,
 	meta: {
 		type: "suggestion",
 		docs: {
@@ -5750,9 +5732,9 @@ var require_mock_type_parameters_default = createEslintRule({
 
 //#endregion
 //#region src/rules/require-to-throw-message.ts
-const RULE_NAME$7 = "require-to-throw-message";
+const RULE_NAME$8 = "require-to-throw-message";
 var require_to_throw_message_default = createEslintRule({
-	name: RULE_NAME$7,
+	name: RULE_NAME$8,
 	meta: {
 		type: "suggestion",
 		docs: {
@@ -5780,9 +5762,9 @@ var require_to_throw_message_default = createEslintRule({
 
 //#endregion
 //#region src/rules/require-top-level-describe.ts
-const RULE_NAME$6 = "require-top-level-describe";
+const RULE_NAME$7 = "require-top-level-describe";
 var require_top_level_describe_default = createEslintRule({
-	name: RULE_NAME$6,
+	name: RULE_NAME$7,
 	meta: {
 		docs: {
 			description: "enforce that all tests are in a top-level describe",
@@ -5851,7 +5833,7 @@ var require_top_level_describe_default = createEslintRule({
 
 //#endregion
 //#region src/rules/valid-describe-callback.ts
-const RULE_NAME$5 = "valid-describe-callback";
+const RULE_NAME$6 = "valid-describe-callback";
 const paramsLocation = (params) => {
 	const [first] = params;
 	const last = params[params.length - 1];
@@ -5873,7 +5855,7 @@ const reportUnexpectedReturnInDescribe = (blockStatement, context) => {
 	});
 };
 var valid_describe_callback_default = createEslintRule({
-	name: RULE_NAME$5,
+	name: RULE_NAME$6,
 	meta: {
 		type: "problem",
 		docs: {
@@ -5940,7 +5922,7 @@ var valid_describe_callback_default = createEslintRule({
 
 //#endregion
 //#region src/rules/valid-expect-in-promise.ts
-const RULE_NAME$4 = "valid-expect-in-promise";
+const RULE_NAME$5 = "valid-expect-in-promise";
 const defaultAsyncMatchers$1 = ["toRejectWith", "toResolveWith"];
 const isPromiseChainCall = (node) => {
 	if (node.type === AST_NODE_TYPES.CallExpression && node.callee.type === AST_NODE_TYPES.MemberExpression && isSupportedAccessor(node.callee.property)) {
@@ -6072,7 +6054,7 @@ const isVariableAwaitedOrReturned = (variable, context) => {
 	return isValueAwaitedOrReturned(variable.id, body, context);
 };
 var valid_expect_in_promise_default = createEslintRule({
-	name: RULE_NAME$4,
+	name: RULE_NAME$5,
 	meta: {
 		docs: { description: "require promises that have expectations in their chain to be valid" },
 		messages: { expectInFloatingPromise: "This promise should either be returned or awaited to ensure the expects in its chain are called" },
@@ -6132,7 +6114,7 @@ var valid_expect_in_promise_default = createEslintRule({
 
 //#endregion
 //#region src/rules/valid-expect.ts
-const RULE_NAME$3 = "valid-expect";
+const RULE_NAME$4 = "valid-expect";
 const defaultAsyncMatchers = ["toReject", "toResolve"];
 /**
 * Async assertions might be called in Promise
@@ -6167,7 +6149,7 @@ const isAcceptableReturnNode = (node, allowReturn) => {
 	return [AST_NODE_TYPES.ArrowFunctionExpression, AST_NODE_TYPES.AwaitExpression].includes(node.type);
 };
 var valid_expect_default = createEslintRule({
-	name: RULE_NAME$3,
+	name: RULE_NAME$4,
 	meta: {
 		docs: {
 			description: "enforce valid `expect()` usage",
@@ -6360,7 +6342,7 @@ var valid_expect_default = createEslintRule({
 
 //#endregion
 //#region src/rules/valid-title.ts
-const RULE_NAME$2 = "valid-title";
+const RULE_NAME$3 = "valid-title";
 const trimFXPrefix = (word) => ["f", "x"].includes(word.charAt(0)) ? word.substring(1) : word;
 const quoteStringValue = (node) => node.type === AST_NODE_TYPES.TemplateLiteral ? `\`${node.quasis[0].value.raw}\`` : node.raw;
 const MatcherAndMessageSchema = {
@@ -6399,7 +6381,7 @@ const doesBinaryExpressionContainStringNode = (binaryExp) => {
 	return isStringNode(binaryExp.left);
 };
 var valid_title_default = createEslintRule({
-	name: RULE_NAME$2,
+	name: RULE_NAME$3,
 	meta: {
 		docs: {
 			description: "enforce valid titles",
@@ -6552,9 +6534,9 @@ var valid_title_default = createEslintRule({
 
 //#endregion
 //#region src/rules/warn-todo.ts
-const RULE_NAME$1 = "warn-todo";
+const RULE_NAME$2 = "warn-todo";
 var warn_todo_default = createEslintRule({
-	name: RULE_NAME$1,
+	name: RULE_NAME$2,
 	meta: {
 		docs: {
 			description: "disallow `.todo` usage",
@@ -6581,7 +6563,7 @@ var warn_todo_default = createEslintRule({
 
 //#endregion
 //#region src/rules/no-unneeded-async-expect-function.ts
-const RULE_NAME = "no-unneeded-async-expect-function";
+const RULE_NAME$1 = "no-unneeded-async-expect-function";
 const getAwaitedCallExpression = (expression) => {
 	if (!expression.async) return null;
 	if (expression.type === AST_NODE_TYPES.ArrowFunctionExpression && expression.body.type === AST_NODE_TYPES.AwaitExpression && expression.body.argument.type === AST_NODE_TYPES.CallExpression) return expression.body.argument;
@@ -6591,7 +6573,7 @@ const getAwaitedCallExpression = (expression) => {
 	return null;
 };
 var no_unneeded_async_expect_function_default = createEslintRule({
-	name: RULE_NAME,
+	name: RULE_NAME$1,
 	meta: {
 		docs: { description: "Disallow unnecessary async function wrapper for expected promises" },
 		fixable: "code",
@@ -6622,6 +6604,42 @@ var no_unneeded_async_expect_function_default = createEslintRule({
 	}
 });
 
+//#endregion
+//#region src/rules/prefer-to-have-been-called-times.ts
+const RULE_NAME = "prefer-to-have-been-called-times";
+var prefer_to_have_been_called_times_default = createEslintRule({
+	name: RULE_NAME,
+	meta: {
+		fixable: "code",
+		docs: { description: "Suggest using `toHaveBeenCalledTimes()`" },
+		messages: { preferMatcher: "Prefer `toHaveBeenCalledTimes`" },
+		type: "suggestion",
+		schema: []
+	},
+	defaultOptions: [],
+	create(context) {
+		return { CallExpression(node) {
+			const vitestFnCall = parseVitestFnCall(node, context);
+			if (vitestFnCall?.type !== "expect") return;
+			const { parent: expect } = vitestFnCall.head.node;
+			if (expect?.type !== AST_NODE_TYPES.CallExpression) return;
+			const { matcher } = vitestFnCall;
+			if (!isSupportedAccessor(matcher, "toHaveLength")) return;
+			const [argument] = expect.arguments;
+			if (argument?.type !== AST_NODE_TYPES.MemberExpression || !isSupportedAccessor(argument.property, "calls")) return;
+			const { object } = argument;
+			if (object.type !== AST_NODE_TYPES.MemberExpression || !isSupportedAccessor(object.property, "mock")) return;
+			context.report({
+				messageId: "preferMatcher",
+				node: matcher,
+				fix(fixer) {
+					return [fixer.removeRange([object.property.range[0] - 1, argument.range[1]]), fixer.replaceTextRange([matcher.parent.object.range[1], matcher.parent.range[1]], ".toHaveBeenCalledTimes")];
+				}
+			});
+		} };
+	}
+});
+
 //#endregion
 //#region src/rules/index.ts
 const rules = {
@@ -6689,12 +6707,12 @@ const rules = {
 	"prefer-to-be-truthy": prefer_to_be_truthy_default,
 	"prefer-to-be": prefer_to_be_default,
 	"prefer-to-contain": prefer_to_contain_default,
+	"prefer-to-have-been-called-times": prefer_to_have_been_called_times_default,
 	"prefer-to-have-length": prefer_to_have_length_default,
 	"prefer-todo": prefer_todo_default,
 	"prefer-vi-mocked": prefer_vi_mocked_default,
 	"require-awaited-expect-poll": require_awaited_expect_poll_default,
 	"require-hook": require_hook_default,
-	"require-import-vi-mock": require_import_vi_mock_default,
 	"require-local-test-context-for-concurrent-snapshots": require_local_test_context_for_concurrent_snapshots_default,
 	"require-mock-type-parameters": require_mock_type_parameters_default,
 	"require-to-throw-message": require_to_throw_message_default,
@@ -6787,6 +6805,7 @@ const allRules = {
 	"prefer-to-be-truthy": "off",
 	"prefer-to-be": "warn",
 	"prefer-to-contain": "warn",
+	"prefer-to-have-been-called-times": "warn",
 	"prefer-to-have-length": "warn",
 	"prefer-todo": "warn",
 	"prefer-vi-mocked": "warn",
@@ -6799,8 +6818,7 @@ const allRules = {
 	"valid-expect-in-promise": "warn",
 	"valid-expect": "warn",
 	"valid-title": "warn",
-	"require-awaited-expect-poll": "warn",
-	"require-import-vi-mock": "warn"
+	"require-awaited-expect-poll": "warn"
 };
 const recommendedRules = {
 	"expect-expect": "error",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vitest/eslint-plugin",
-  "version": "1.5.4",
+  "version": "1.6.3",
   "license": "MIT",
   "description": "ESLint plugin for Vitest",
   "repository": "vitest-dev/eslint-plugin-vitest",
@@ -42,7 +42,7 @@
     "eslint": "^9.37.0",
     "eslint-config-flat-gitignore": "^2.1.0",
     "eslint-config-prettier": "^10.1.8",
-    "eslint-doc-generator": "^2.3.0",
+    "eslint-doc-generator": "^2.4.0",
     "eslint-plugin-eslint-plugin": "^7.0.0",
     "eslint-remote-tester": "^4.0.3",
     "eslint-remote-tester-repositories": "^2.0.2",