fix(vitest/no-done-callback): do not report accesssing of text contex…

…t when test runs concurrently (#313)
veritem · Dec 8, 2023 · b7eb2ce · b7eb2ce
1 parent 864205b
commit b7eb2ce
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/src/rules/no-done-callback.ts b/src/rules/no-done-callback.ts
@@ -46,6 +46,9 @@ export default createEslintRule<Options, MessageIds>({
 				if (isVitestEach && node.callee.type !== AST_NODE_TYPES.TaggedTemplateExpression)
 					return
 
+				const isVitestConcurrent = getNodeName(node.callee)?.endsWith('.concurrent') ?? false;
+				if (isVitestConcurrent) return
+
 				const callback = findCallbackArg(node, isVitestEach, context)
 				const callbackArgIndex = Number(isVitestEach)
 

diff --git a/tests/no-done-callback.test.ts b/tests/no-done-callback.test.ts
@@ -13,6 +13,8 @@ ruleTester.run(RULE_NAME, rule, {
     'it.each``("something", ({ a, b }) => {})',
     'it.each([])("something", (a, b) => { a(); b(); })',
     'it.each``("something", ({ a, b }) => { a(); b(); })',
+    'it.concurrent("something", (context) => {})',
+    'it.concurrent("something", ({ expect }) => {})',
     'test("something", async function () {})',
     'test("something", someArg)',
     'beforeEach(() => {})',