feat(require-test-timeout): support ancestor traversal (#890)

Author: hamirmahal

src/rules/require-test-timeout.ts

@@ -1,3 +1,4 @@
+import { Scope } from '@typescript-eslint/scope-manager'
 import { createEslintRule, getAccessorValue } from '../utils'
 import { parseVitestFnCall } from '../utils/parse-vitest-fn-call'
 import { getScope } from '../utils/scope'
@@ -74,8 +75,19 @@ export default createEslintRule<Options, MESSAGE_ID>({
       }
 
       if (node.type === AST_NODE_TYPES.Identifier) {
-        const scope = getScope(context, node)
-        const variable = scope.set.get(node.name)
+        let currentScope: Scope | null = getScope(context, node)
+        let variable
+
+        // Walk parent scopes until we find a definition for this identifier
+        while (currentScope !== null) {
+          const ref = currentScope.set.get(node.name)
+          if (ref && ref.defs && ref.defs.length > 0) {
+            variable = ref
+            break
+          }
+          currentScope = currentScope.upper
+        }
+
         if (!variable || !variable.defs || variable.defs.length === 0)
           return undefined
 

tests/require-test-timeout.test.ts

@@ -33,6 +33,9 @@ ruleTester.run(RULE_NAME, rule, {
     // both object and numeric timeout present
     'test("a", { timeout: 500 }, 1000, () => {})',
     'test("a", () => {}, 1000, { extra: true })',
+    // in-source unit tests
+    'if (import.meta.vitest) { const opts = { timeout: 500 }; describe("outer", () => { it("repro: same-file opts object", opts, () => {}); }); }',
+    'if (import.meta.vitest) { const T = 500; describe("outer", () => { describe("inner", () => { it("repro: same-file const timeout", () => {}, T); }); }); }',
   ],
   invalid: [
     {