-
Notifications
You must be signed in to change notification settings - Fork 3.1k
TESTS_FILE edges are never created for JVM (Kotlin/JUnit) tests #1556
Copy link
Copy link
Open
Labels
bugSomething isn't workingSomething isn't workingparsing/qualityGraph extraction bugs, false positives, missing edgesGraph extraction bugs, false positives, missing edgespriority/highNeeds near-term maintainer attention; high-impact bug, regression, safety issue, or release blocker.Needs near-term maintainer attention; high-impact bug, regression, safety issue, or release blocker.windowsWindows-specific issuesWindows-specific issues
Description
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingparsing/qualityGraph extraction bugs, false positives, missing edgesGraph extraction bugs, false positives, missing edgespriority/highNeeds near-term maintainer attention; high-impact bug, regression, safety issue, or release blocker.Needs near-term maintainer attention; high-impact bug, regression, safety issue, or release blocker.windowsWindows-specific issuesWindows-specific issues
Environment: v0.10.2, Windows 11,
--mode full. Kotlin (JUnit5/Testcontainers) + TypeScript (vitest) monorepo.Symptom
TESTS_FILEedges are created only for the TypeScript side. Measured on a full index:.*(Test|IT)\.ktor.*\.test\.tsx?: 283TESTS_FILEedges: 92 — all of them.test.tsx(66) /.test.ts(26)AgendarUseCaseTest.kt→AgendarUseCase.kt(same package, siblingsrc/test/kotlinsource set) andJooqCompromissoRepositoryIT.kt→JooqCompromissoRepository.kt.Impact
Any "which tests cover this class?" query over the graph silently answers "none" for the whole JVM half of a polyglot repo. We now instruct agents to fall back to naming convention +
OVERRIDEedges of test fakes for Kotlin, but the edge type advertises coverage it doesn't have.Suggestion
Apply the same filename/convention heuristic used for
*.test.ts(x)to JVM conventions (XTest/XIT,src/test/{kotlin,java}mirroringsrc/main/...), or documentTESTS_FILEas TS-only.