mirror of
https://github.com/abhigyanpatwari/GitNexus.git
synced 2026-09-07 08:26:11 +00:00
6388113e10
* Initial plan * fix: prevent stack overflow and memory issues on large repo analysis - Convert c3Linearize from recursive to iterative (explicit work stack) to handle deep class hierarchies without stack overflow - Replace push(...arr) spread patterns with safe loops in parse-worker.ts and lbug-adapter.ts to prevent stack overflow on large arrays - Stream relationship CSV lines directly to per-pair temp files in lbug-adapter.ts instead of accumulating millions of lines in memory - Add test for deep 500-level inheritance chain Agent-Logs-Url: https://github.com/abhigyanpatwari/GitNexus/sessions/9cb2eed2-adc7-4fa4-9216-e7ac3facb9b5 Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com> * fix: add stack size flag and enhanced error messages for large repos - Auto-set --stack-size=4096 alongside --max-old-space-size in analyze command to prevent stack overflow on deep class hierarchies - Add helpful error guidance for known large-repo failure modes (stack overflow, heap OOM, Map size limits) Agent-Logs-Url: https://github.com/abhigyanpatwari/GitNexus/sessions/9cb2eed2-adc7-4fa4-9216-e7ac3facb9b5 Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com> * fix: address code review feedback - Add error handling for write stream close in lbug-adapter.ts - Handle backpressure when writing relationship CSV lines to disk - Clarify ENTER/MERGE phase transition comment in resolve.ts - Fix inconsistent stack size in error message (4096 not 8192) Agent-Logs-Url: https://github.com/abhigyanpatwari/GitNexus/sessions/9cb2eed2-adc7-4fa4-9216-e7ac3facb9b5 Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com> * fix: address PR review — resource leak, redundant guard, Set, test depth - Fix write-stream resource leak on readline error by destroying all open WriteStreams before rejecting (lbug-adapter.ts) - Switch failedPairCsvPaths from array to Set for O(1) lookup - Remove redundant MERGE-phase empty-parents guard in resolve.ts (unreachable — ENTER phase already handles that case) - Increase deep inheritance test DEPTH from 500 to 2000 for reliable regression coverage across platforms Agent-Logs-Url: https://github.com/abhigyanpatwari/GitNexus/sessions/cf1f3e22-3864-454a-a3a5-2bded9ebfdba Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com> * style: fix prettier formatting in lbug-adapter.ts Agent-Logs-Url: https://github.com/abhigyanpatwari/GitNexus/sessions/b5ca33c4-bb03-402f-a206-21ea7e1e310e Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com> * chore: revert unintended package.json/lock changes Agent-Logs-Url: https://github.com/abhigyanpatwari/GitNexus/sessions/b5ca33c4-bb03-402f-a206-21ea7e1e310e Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com> * fix: strip NODE_OPTIONS in skip-git-cli test child processes Agent-Logs-Url: https://github.com/abhigyanpatwari/GitNexus/sessions/fc59cd11-348b-4e22-b9ea-98787300de48 Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com> * fix: don't put --stack-size in NODE_OPTIONS (rejected by Node 24) Agent-Logs-Url: https://github.com/abhigyanpatwari/GitNexus/sessions/fc59cd11-348b-4e22-b9ea-98787300de48 Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com> * fix: pass --stack-size as CLI arg only, not in NODE_OPTIONS (Node 24 compat) Agent-Logs-Url: https://github.com/abhigyanpatwari/GitNexus/sessions/fc59cd11-348b-4e22-b9ea-98787300de48 Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: magyargergo <11230420+magyargergo@users.noreply.github.com>
1743 lines
66 KiB
TypeScript
1743 lines
66 KiB
TypeScript
import { describe, it, expect } from 'vitest';
|
||
import { computeMRO } from '../../src/core/ingestion/mro-processor.js';
|
||
import { createKnowledgeGraph } from '../../src/core/graph/graph.js';
|
||
import type { KnowledgeGraph } from '../../src/core/graph/types.js';
|
||
import { generateId } from '../../src/lib/utils.js';
|
||
|
||
// ---------------------------------------------------------------------------
|
||
// Helpers
|
||
// ---------------------------------------------------------------------------
|
||
|
||
function addClass(
|
||
graph: KnowledgeGraph,
|
||
name: string,
|
||
language: string,
|
||
label: 'Class' | 'Interface' | 'Struct' | 'Trait' = 'Class',
|
||
) {
|
||
const id = generateId(label, name);
|
||
graph.addNode({
|
||
id,
|
||
label,
|
||
properties: { name, filePath: `src/${name}.ts`, language },
|
||
});
|
||
return id;
|
||
}
|
||
|
||
function addMethod(
|
||
graph: KnowledgeGraph,
|
||
className: string,
|
||
methodName: string,
|
||
classLabel: 'Class' | 'Interface' | 'Struct' | 'Trait' = 'Class',
|
||
parameterTypes?: string[],
|
||
opts?: { isAbstract?: boolean; parameterCount?: number },
|
||
) {
|
||
// Derive arity for the ID suffix: explicit parameterCount > parameterTypes.length > 0
|
||
const arity = opts?.parameterCount ?? parameterTypes?.length ?? 0;
|
||
const classId = generateId(classLabel, className);
|
||
const methodId = generateId('Method', `${className}.${methodName}#${arity}`);
|
||
graph.addNode({
|
||
id: methodId,
|
||
label: 'Method',
|
||
properties: {
|
||
name: methodName,
|
||
filePath: `src/${className}.ts`,
|
||
parameterCount: arity,
|
||
...(parameterTypes ? { parameterTypes } : {}),
|
||
...(opts?.isAbstract !== undefined ? { isAbstract: opts.isAbstract } : {}),
|
||
},
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${classId}->${methodId}`),
|
||
sourceId: classId,
|
||
targetId: methodId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
return methodId;
|
||
}
|
||
|
||
function addExtends(
|
||
graph: KnowledgeGraph,
|
||
childName: string,
|
||
parentName: string,
|
||
childLabel: 'Class' | 'Struct' = 'Class',
|
||
parentLabel: 'Class' | 'Interface' | 'Trait' = 'Class',
|
||
) {
|
||
const childId = generateId(childLabel, childName);
|
||
const parentId = generateId(parentLabel, parentName);
|
||
graph.addRelationship({
|
||
id: generateId('EXTENDS', `${childId}->${parentId}`),
|
||
sourceId: childId,
|
||
targetId: parentId,
|
||
type: 'EXTENDS',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
}
|
||
|
||
function addInterfaceExtends(
|
||
graph: KnowledgeGraph,
|
||
childName: string,
|
||
parentName: string,
|
||
childLabel: 'Interface' | 'Trait' = 'Interface',
|
||
parentLabel: 'Interface' | 'Trait' = 'Interface',
|
||
) {
|
||
const childId = generateId(childLabel, childName);
|
||
const parentId = generateId(parentLabel, parentName);
|
||
graph.addRelationship({
|
||
id: generateId('EXTENDS', `${childId}->${parentId}`),
|
||
sourceId: childId,
|
||
targetId: parentId,
|
||
type: 'EXTENDS',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
}
|
||
|
||
function addImplements(
|
||
graph: KnowledgeGraph,
|
||
childName: string,
|
||
parentName: string,
|
||
childLabel: 'Class' | 'Struct' = 'Class',
|
||
parentLabel: 'Interface' | 'Trait' = 'Interface',
|
||
) {
|
||
const childId = generateId(childLabel, childName);
|
||
const parentId = generateId(parentLabel, parentName);
|
||
graph.addRelationship({
|
||
id: generateId('IMPLEMENTS', `${childId}->${parentId}`),
|
||
sourceId: childId,
|
||
targetId: parentId,
|
||
type: 'IMPLEMENTS',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
}
|
||
|
||
// ---------------------------------------------------------------------------
|
||
// Tests
|
||
// ---------------------------------------------------------------------------
|
||
|
||
describe('computeMRO', () => {
|
||
// ---- C++ diamond --------------------------------------------------------
|
||
describe('C++ diamond inheritance', () => {
|
||
it('leftmost base wins when both B and C override foo', () => {
|
||
// Diamond: A <- B, A <- C, B <- D, C <- D
|
||
const graph = createKnowledgeGraph();
|
||
const aId = addClass(graph, 'A', 'cpp');
|
||
const bId = addClass(graph, 'B', 'cpp');
|
||
const cId = addClass(graph, 'C', 'cpp');
|
||
const dId = addClass(graph, 'D', 'cpp');
|
||
|
||
addExtends(graph, 'B', 'A');
|
||
addExtends(graph, 'C', 'A');
|
||
addExtends(graph, 'D', 'B'); // B is leftmost
|
||
addExtends(graph, 'D', 'C');
|
||
|
||
// A has foo, B overrides foo, C overrides foo
|
||
addMethod(graph, 'A', 'foo');
|
||
const bFoo = addMethod(graph, 'B', 'foo');
|
||
const cFoo = addMethod(graph, 'C', 'foo');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
// D should have an entry with ambiguity on foo
|
||
const dEntry = result.entries.find((e) => e.className === 'D');
|
||
expect(dEntry).toBeDefined();
|
||
expect(dEntry!.language).toBe('cpp');
|
||
|
||
const fooAmbiguity = dEntry!.ambiguities.find((a) => a.methodName === 'foo');
|
||
expect(fooAmbiguity).toBeDefined();
|
||
expect(fooAmbiguity!.definedIn.length).toBeGreaterThanOrEqual(2);
|
||
|
||
// Leftmost base (B) wins
|
||
expect(fooAmbiguity!.resolvedTo).toBe(bFoo);
|
||
expect(fooAmbiguity!.reason).toContain('leftmost base');
|
||
expect(fooAmbiguity!.reason).toContain('B');
|
||
|
||
// OVERRIDES edge emitted
|
||
expect(result.overrideEdges).toBeGreaterThanOrEqual(1);
|
||
const overrides = graph.relationships.filter((r) => r.type === 'METHOD_OVERRIDES');
|
||
expect(overrides.some((r) => r.sourceId === dId && r.targetId === bFoo)).toBe(true);
|
||
});
|
||
|
||
it('no ambiguity when foo only in A (diamond no override)', () => {
|
||
// Diamond: A <- B, A <- C, B <- D, C <- D, but only A has foo
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'A', 'cpp');
|
||
addClass(graph, 'B', 'cpp');
|
||
addClass(graph, 'C', 'cpp');
|
||
addClass(graph, 'D', 'cpp');
|
||
|
||
addExtends(graph, 'B', 'A');
|
||
addExtends(graph, 'C', 'A');
|
||
addExtends(graph, 'D', 'B');
|
||
addExtends(graph, 'D', 'C');
|
||
|
||
// Only A has foo
|
||
addMethod(graph, 'A', 'foo');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const dEntry = result.entries.find((e) => e.className === 'D');
|
||
expect(dEntry).toBeDefined();
|
||
// A::foo appears only once across ancestors — no collision
|
||
// (B and C don't have their own foo, the duplicate is A::foo seen through both paths)
|
||
const fooAmbiguity = dEntry!.ambiguities.find((a) => a.methodName === 'foo');
|
||
expect(fooAmbiguity).toBeUndefined();
|
||
});
|
||
});
|
||
|
||
// ---- C# class + interface -----------------------------------------------
|
||
describe('C# class + interface', () => {
|
||
it('class method beats interface default', () => {
|
||
const graph = createKnowledgeGraph();
|
||
const classId = addClass(graph, 'MyClass', 'csharp');
|
||
const baseId = addClass(graph, 'BaseClass', 'csharp');
|
||
const ifaceId = addClass(graph, 'IDoSomething', 'csharp', 'Interface');
|
||
|
||
addExtends(graph, 'MyClass', 'BaseClass');
|
||
addImplements(graph, 'MyClass', 'IDoSomething');
|
||
|
||
const baseDoIt = addMethod(graph, 'BaseClass', 'doIt');
|
||
const ifaceDoIt = addMethod(graph, 'IDoSomething', 'doIt', 'Interface');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const entry = result.entries.find((e) => e.className === 'MyClass');
|
||
expect(entry).toBeDefined();
|
||
|
||
const doItAmbiguity = entry!.ambiguities.find((a) => a.methodName === 'doIt');
|
||
expect(doItAmbiguity).toBeDefined();
|
||
// Class method wins
|
||
expect(doItAmbiguity!.resolvedTo).toBe(baseDoIt);
|
||
expect(doItAmbiguity!.reason).toContain('class method wins');
|
||
});
|
||
|
||
it('multiple interface methods with same name are ambiguous', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'MyClass', 'csharp');
|
||
addClass(graph, 'IFoo', 'csharp', 'Interface');
|
||
addClass(graph, 'IBar', 'csharp', 'Interface');
|
||
|
||
addImplements(graph, 'MyClass', 'IFoo');
|
||
addImplements(graph, 'MyClass', 'IBar');
|
||
|
||
addMethod(graph, 'IFoo', 'process', 'Interface');
|
||
addMethod(graph, 'IBar', 'process', 'Interface');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const entry = result.entries.find((e) => e.className === 'MyClass');
|
||
expect(entry).toBeDefined();
|
||
|
||
const processAmbiguity = entry!.ambiguities.find((a) => a.methodName === 'process');
|
||
expect(processAmbiguity).toBeDefined();
|
||
expect(processAmbiguity!.resolvedTo).toBeNull();
|
||
expect(processAmbiguity!.reason).toContain('ambiguous');
|
||
expect(result.ambiguityCount).toBeGreaterThanOrEqual(1);
|
||
});
|
||
});
|
||
|
||
// ---- Python C3 ----------------------------------------------------------
|
||
describe('Python C3 linearization', () => {
|
||
it('C3 order determines winner in diamond with overrides', () => {
|
||
// Diamond: A <- B, A <- C, B <- D, C <- D
|
||
// class D(B, C) → C3 MRO: B, C, A
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'A', 'python');
|
||
addClass(graph, 'B', 'python');
|
||
addClass(graph, 'C', 'python');
|
||
const dId = addClass(graph, 'D', 'python');
|
||
|
||
addExtends(graph, 'B', 'A');
|
||
addExtends(graph, 'C', 'A');
|
||
addExtends(graph, 'D', 'B'); // B first → leftmost in C3
|
||
addExtends(graph, 'D', 'C');
|
||
|
||
addMethod(graph, 'A', 'foo');
|
||
const bFoo = addMethod(graph, 'B', 'foo');
|
||
addMethod(graph, 'C', 'foo');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const dEntry = result.entries.find((e) => e.className === 'D');
|
||
expect(dEntry).toBeDefined();
|
||
|
||
const fooAmbiguity = dEntry!.ambiguities.find((a) => a.methodName === 'foo');
|
||
expect(fooAmbiguity).toBeDefined();
|
||
// C3 linearization for D(B, C): B comes first
|
||
expect(fooAmbiguity!.resolvedTo).toBe(bFoo);
|
||
expect(fooAmbiguity!.reason).toContain('C3 MRO');
|
||
});
|
||
});
|
||
|
||
// ---- Java class + interface ---------------------------------------------
|
||
describe('Java class + interface', () => {
|
||
it('class method beats interface default', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'Service', 'java');
|
||
addClass(graph, 'BaseService', 'java');
|
||
addClass(graph, 'Runnable', 'java', 'Interface');
|
||
|
||
addExtends(graph, 'Service', 'BaseService');
|
||
addImplements(graph, 'Service', 'Runnable');
|
||
|
||
const baseRun = addMethod(graph, 'BaseService', 'run');
|
||
addMethod(graph, 'Runnable', 'run', 'Interface');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const entry = result.entries.find((e) => e.className === 'Service');
|
||
expect(entry).toBeDefined();
|
||
|
||
const runAmbiguity = entry!.ambiguities.find((a) => a.methodName === 'run');
|
||
expect(runAmbiguity).toBeDefined();
|
||
expect(runAmbiguity!.resolvedTo).toBe(baseRun);
|
||
expect(runAmbiguity!.reason).toContain('class method wins');
|
||
});
|
||
});
|
||
|
||
// ---- Rust trait conflicts -----------------------------------------------
|
||
describe('Rust trait conflicts', () => {
|
||
it('trait conflicts result in null resolution with qualified syntax reason', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'MyStruct', 'rust', 'Struct');
|
||
addClass(graph, 'TraitA', 'rust', 'Trait');
|
||
addClass(graph, 'TraitB', 'rust', 'Trait');
|
||
|
||
addImplements(graph, 'MyStruct', 'TraitA', 'Struct', 'Trait');
|
||
addImplements(graph, 'MyStruct', 'TraitB', 'Struct', 'Trait');
|
||
|
||
addMethod(graph, 'TraitA', 'execute', 'Trait');
|
||
addMethod(graph, 'TraitB', 'execute', 'Trait');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const entry = result.entries.find((e) => e.className === 'MyStruct');
|
||
expect(entry).toBeDefined();
|
||
|
||
const execAmbiguity = entry!.ambiguities.find((a) => a.methodName === 'execute');
|
||
expect(execAmbiguity).toBeDefined();
|
||
expect(execAmbiguity!.resolvedTo).toBeNull();
|
||
expect(execAmbiguity!.reason).toContain('qualified syntax');
|
||
expect(result.ambiguityCount).toBeGreaterThanOrEqual(1);
|
||
|
||
// No OVERRIDES edge emitted for Rust ambiguity
|
||
const overrides = graph.relationships.filter(
|
||
(r) => r.type === 'METHOD_OVERRIDES' && r.sourceId === generateId('Struct', 'MyStruct'),
|
||
);
|
||
expect(overrides).toHaveLength(0);
|
||
});
|
||
});
|
||
|
||
// ---- Property collisions don't trigger OVERRIDES ------------------------
|
||
describe('Property nodes excluded from OVERRIDES', () => {
|
||
it('property name collision across parents does not emit OVERRIDES edge', () => {
|
||
const graph = createKnowledgeGraph();
|
||
const parentA = addClass(graph, 'ParentA', 'typescript');
|
||
const parentB = addClass(graph, 'ParentB', 'typescript');
|
||
const child = addClass(graph, 'Child', 'typescript');
|
||
|
||
addExtends(graph, 'Child', 'ParentA');
|
||
addExtends(graph, 'Child', 'ParentB');
|
||
|
||
// Add Property nodes (same name 'name') to both parents via HAS_PROPERTY
|
||
const propA = generateId('Property', 'ParentA.name');
|
||
graph.addNode({
|
||
id: propA,
|
||
label: 'Property',
|
||
properties: { name: 'name', filePath: 'src/ParentA.ts' },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_PROPERTY', `${parentA}->${propA}`),
|
||
sourceId: parentA,
|
||
targetId: propA,
|
||
type: 'HAS_PROPERTY',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const propB = generateId('Property', 'ParentB.name');
|
||
graph.addNode({
|
||
id: propB,
|
||
label: 'Property',
|
||
properties: { name: 'name', filePath: 'src/ParentB.ts' },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_PROPERTY', `${parentB}->${propB}`),
|
||
sourceId: parentB,
|
||
targetId: propB,
|
||
type: 'HAS_PROPERTY',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
// No OVERRIDES edge should be emitted for properties
|
||
const overrides = graph.relationships.filter((r) => r.type === 'METHOD_OVERRIDES');
|
||
expect(overrides).toHaveLength(0);
|
||
expect(result.overrideEdges).toBe(0);
|
||
});
|
||
|
||
it('method collision still triggers OVERRIDES even when properties also collide', () => {
|
||
const graph = createKnowledgeGraph();
|
||
const parentA = addClass(graph, 'PA', 'cpp');
|
||
const parentB = addClass(graph, 'PB', 'cpp');
|
||
addClass(graph, 'Ch', 'cpp');
|
||
|
||
addExtends(graph, 'Ch', 'PA');
|
||
addExtends(graph, 'Ch', 'PB');
|
||
|
||
// Method collision (should trigger OVERRIDES)
|
||
const methodA = addMethod(graph, 'PA', 'doWork');
|
||
addMethod(graph, 'PB', 'doWork');
|
||
|
||
// Property collision (should NOT trigger OVERRIDES — properties use HAS_PROPERTY, not HAS_METHOD)
|
||
const propA = generateId('Property', 'PA.id');
|
||
graph.addNode({
|
||
id: propA,
|
||
label: 'Property',
|
||
properties: { name: 'id', filePath: 'src/PA.ts' },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_PROPERTY', `${parentA}->${propA}`),
|
||
sourceId: parentA,
|
||
targetId: propA,
|
||
type: 'HAS_PROPERTY',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const propB = generateId('Property', 'PB.id');
|
||
graph.addNode({
|
||
id: propB,
|
||
label: 'Property',
|
||
properties: { name: 'id', filePath: 'src/PB.ts' },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_PROPERTY', `${parentB}->${propB}`),
|
||
sourceId: parentB,
|
||
targetId: propB,
|
||
type: 'HAS_PROPERTY',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
// Only 1 OVERRIDES edge (for the method, not the property)
|
||
const overrides = graph.relationships.filter((r) => r.type === 'METHOD_OVERRIDES');
|
||
expect(overrides).toHaveLength(1);
|
||
expect(overrides[0].targetId).toBe(methodA); // leftmost base wins for C++
|
||
expect(result.overrideEdges).toBe(1);
|
||
});
|
||
});
|
||
|
||
// ---- No ambiguity: single parent ----------------------------------------
|
||
describe('single parent, no ambiguity', () => {
|
||
it('single parent with unique methods produces no ambiguities', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'Parent', 'typescript');
|
||
addClass(graph, 'Child', 'typescript');
|
||
|
||
addExtends(graph, 'Child', 'Parent');
|
||
|
||
addMethod(graph, 'Parent', 'foo');
|
||
addMethod(graph, 'Parent', 'bar');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const entry = result.entries.find((e) => e.className === 'Child');
|
||
expect(entry).toBeDefined();
|
||
expect(entry!.ambiguities).toHaveLength(0);
|
||
});
|
||
});
|
||
|
||
// ---- No parents: standalone class not in entries ------------------------
|
||
describe('standalone class', () => {
|
||
it('class with no parents is not included in entries', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'Standalone', 'typescript');
|
||
addMethod(graph, 'Standalone', 'doStuff');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const entry = result.entries.find((e) => e.className === 'Standalone');
|
||
expect(entry).toBeUndefined();
|
||
expect(result.overrideEdges).toBe(0);
|
||
expect(result.ambiguityCount).toBe(0);
|
||
});
|
||
});
|
||
|
||
// ---- Own method shadows ancestor ----------------------------------------
|
||
describe('own method shadows ancestor', () => {
|
||
it('class defining its own method suppresses ambiguity', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'Base1', 'cpp');
|
||
addClass(graph, 'Base2', 'cpp');
|
||
addClass(graph, 'Child', 'cpp');
|
||
|
||
addExtends(graph, 'Child', 'Base1');
|
||
addExtends(graph, 'Child', 'Base2');
|
||
|
||
addMethod(graph, 'Base1', 'foo');
|
||
addMethod(graph, 'Base2', 'foo');
|
||
addMethod(graph, 'Child', 'foo'); // own method
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const entry = result.entries.find((e) => e.className === 'Child');
|
||
expect(entry).toBeDefined();
|
||
// No ambiguity because Child defines its own foo
|
||
const fooAmbiguity = entry!.ambiguities.find((a) => a.methodName === 'foo');
|
||
expect(fooAmbiguity).toBeUndefined();
|
||
});
|
||
});
|
||
|
||
// ---- Empty graph --------------------------------------------------------
|
||
describe('empty graph', () => {
|
||
it('returns empty result for graph with no classes', () => {
|
||
const graph = createKnowledgeGraph();
|
||
const result = computeMRO(graph);
|
||
expect(result.entries).toHaveLength(0);
|
||
expect(result.overrideEdges).toBe(0);
|
||
expect(result.ambiguityCount).toBe(0);
|
||
});
|
||
});
|
||
|
||
// ---- Cyclic inheritance (P1 fix) ----------------------------------------
|
||
describe('cyclic inheritance', () => {
|
||
it('does not stack overflow on cyclic Python hierarchy', () => {
|
||
// A extends B, B extends A — cyclic
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'A', 'python');
|
||
addClass(graph, 'B', 'python');
|
||
addExtends(graph, 'A', 'B');
|
||
addExtends(graph, 'B', 'A');
|
||
addMethod(graph, 'A', 'foo');
|
||
addMethod(graph, 'B', 'foo');
|
||
|
||
// Should NOT throw — c3Linearize returns null, falls back to BFS
|
||
const result = computeMRO(graph);
|
||
expect(result).toBeDefined();
|
||
// Both A and B have parents, so both get entries
|
||
expect(result.entries.length).toBeGreaterThanOrEqual(1);
|
||
});
|
||
|
||
it('handles 3-node cycle gracefully', () => {
|
||
// A → B → C → A
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'X', 'python');
|
||
addClass(graph, 'Y', 'python');
|
||
addClass(graph, 'Z', 'python');
|
||
addExtends(graph, 'X', 'Y');
|
||
addExtends(graph, 'Y', 'Z');
|
||
addExtends(graph, 'Z', 'X');
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result).toBeDefined();
|
||
});
|
||
|
||
it('handles very deep single-inheritance chain without stack overflow', () => {
|
||
// Chain of 2000 classes: C0 ← C1 ← C2 ← ... ← C1999
|
||
// The iterative c3Linearize handles this without blowing the stack.
|
||
// (The recursive version overflows at ~1K–5K levels depending on platform.)
|
||
const graph = createKnowledgeGraph();
|
||
const DEPTH = 2000;
|
||
for (let i = 0; i < DEPTH; i++) {
|
||
addClass(graph, `C${i}`, 'python');
|
||
}
|
||
for (let i = 1; i < DEPTH; i++) {
|
||
addExtends(graph, `C${i}`, `C${i - 1}`);
|
||
}
|
||
// Add a method on the root so MRO produces an entry
|
||
addMethod(graph, 'C0', 'baseMethod');
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result).toBeDefined();
|
||
// The deepest class should have all ancestors in its MRO
|
||
const deepest = result.entries.find((e) => e.className === `C${DEPTH - 1}`);
|
||
if (deepest) {
|
||
expect(deepest.mro.length).toBe(DEPTH - 1);
|
||
}
|
||
}, 15_000);
|
||
});
|
||
|
||
// ---- METHOD_IMPLEMENTS edges -----------------------------------------------
|
||
describe('METHOD_IMPLEMENTS edges', () => {
|
||
it('emits METHOD_IMPLEMENTS for class implementing interface method', () => {
|
||
// IAnimal { speak() } <-- Dog { speak() }
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IAnimal', 'java', 'Interface');
|
||
addClass(graph, 'Dog', 'java');
|
||
addImplements(graph, 'Dog', 'IAnimal');
|
||
const ifaceMethod = addMethod(graph, 'IAnimal', 'speak', 'Interface');
|
||
const classMethod = addMethod(graph, 'Dog', 'speak');
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result.methodImplementsEdges).toBe(1);
|
||
|
||
// Verify the edge exists: ConcreteMethod → InterfaceMethod
|
||
const edges: any[] = [];
|
||
graph.forEachRelationship((rel) => {
|
||
if (rel.type === 'METHOD_IMPLEMENTS') edges.push(rel);
|
||
});
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(classMethod);
|
||
expect(edges[0].targetId).toBe(ifaceMethod);
|
||
// Both sides have parameterCount=0 (arity match) → confidence 1.0
|
||
expect(edges[0].confidence).toBe(1.0);
|
||
});
|
||
|
||
it('emits METHOD_IMPLEMENTS for Rust struct implementing trait', () => {
|
||
// Drawable { draw() } <-- Circle { draw() }
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'Drawable', 'rust', 'Trait');
|
||
addClass(graph, 'Circle', 'rust', 'Struct');
|
||
addImplements(graph, 'Circle', 'Drawable', 'Struct', 'Trait');
|
||
const traitMethod = addMethod(graph, 'Drawable', 'draw', 'Trait');
|
||
const structMethod = addMethod(graph, 'Circle', 'draw', 'Struct');
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result.methodImplementsEdges).toBe(1);
|
||
|
||
const edges: any[] = [];
|
||
graph.forEachRelationship((rel) => {
|
||
if (rel.type === 'METHOD_IMPLEMENTS') edges.push(rel);
|
||
});
|
||
expect(edges[0].sourceId).toBe(structMethod);
|
||
expect(edges[0].targetId).toBe(traitMethod);
|
||
});
|
||
|
||
it('matches overloaded interface methods by parameterTypes', () => {
|
||
// IRepo { find(String), find(String, int) } <-- SqlRepo { find(String), find(String, int) }
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IRepo', 'java', 'Interface');
|
||
addClass(graph, 'SqlRepo', 'java');
|
||
addImplements(graph, 'SqlRepo', 'IRepo');
|
||
|
||
// Use manual IDs to avoid overloaded-name collision (same name, different types)
|
||
const ifaceFind1 = generateId('Method', 'IRepo.find#1');
|
||
graph.addNode({
|
||
id: ifaceFind1,
|
||
label: 'Method',
|
||
properties: {
|
||
name: 'find',
|
||
filePath: 'src/IRepo.ts',
|
||
parameterTypes: ['String'],
|
||
parameterCount: 1,
|
||
},
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Interface', 'IRepo')}->${ifaceFind1}`),
|
||
sourceId: generateId('Interface', 'IRepo'),
|
||
targetId: ifaceFind1,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const ifaceFind2 = generateId('Method', 'IRepo.find#2');
|
||
graph.addNode({
|
||
id: ifaceFind2,
|
||
label: 'Method',
|
||
properties: {
|
||
name: 'find',
|
||
filePath: 'src/IRepo.ts',
|
||
parameterTypes: ['String', 'int'],
|
||
parameterCount: 2,
|
||
},
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Interface', 'IRepo')}->${ifaceFind2}`),
|
||
sourceId: generateId('Interface', 'IRepo'),
|
||
targetId: ifaceFind2,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const sqlFind1Id = generateId('Method', 'SqlRepo.find#1');
|
||
graph.addNode({
|
||
id: sqlFind1Id,
|
||
label: 'Method',
|
||
properties: {
|
||
name: 'find',
|
||
filePath: 'src/SqlRepo.ts',
|
||
parameterTypes: ['String'],
|
||
parameterCount: 1,
|
||
},
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Class', 'SqlRepo')}->${sqlFind1Id}`),
|
||
sourceId: generateId('Class', 'SqlRepo'),
|
||
targetId: sqlFind1Id,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const sqlFind2Id = generateId('Method', 'SqlRepo.find#2');
|
||
graph.addNode({
|
||
id: sqlFind2Id,
|
||
label: 'Method',
|
||
properties: {
|
||
name: 'find',
|
||
filePath: 'src/SqlRepo.ts',
|
||
parameterTypes: ['String', 'int'],
|
||
parameterCount: 2,
|
||
},
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Class', 'SqlRepo')}->${sqlFind2Id}`),
|
||
sourceId: generateId('Class', 'SqlRepo'),
|
||
targetId: sqlFind2Id,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result.methodImplementsEdges).toBe(2);
|
||
|
||
const edges: any[] = [];
|
||
graph.forEachRelationship((rel) => {
|
||
if (rel.type === 'METHOD_IMPLEMENTS') edges.push(rel);
|
||
});
|
||
expect(edges).toHaveLength(2);
|
||
// find(String) → find(String) and find(String, int) → find(String, int)
|
||
const edge1 = edges.find((e) => e.targetId === ifaceFind1);
|
||
const edge2 = edges.find((e) => e.targetId === ifaceFind2);
|
||
expect(edge1).toBeDefined();
|
||
expect(edge1!.sourceId).toBe(sqlFind1Id);
|
||
expect(edge2).toBeDefined();
|
||
expect(edge2!.sourceId).toBe(sqlFind2Id);
|
||
});
|
||
|
||
it('includes default interface methods (not just abstract)', () => {
|
||
// Java 8 default method: IFoo { bar() } <-- Baz { bar() }
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IFoo', 'java', 'Interface');
|
||
addClass(graph, 'Baz', 'java');
|
||
addImplements(graph, 'Baz', 'IFoo');
|
||
// Default method (has body, not abstract) — should still get METHOD_IMPLEMENTS
|
||
addMethod(graph, 'IFoo', 'bar', 'Interface');
|
||
addMethod(graph, 'Baz', 'bar');
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result.methodImplementsEdges).toBe(1);
|
||
});
|
||
|
||
it('does not emit METHOD_IMPLEMENTS for class extending another class', () => {
|
||
// Animal { speak() } <-- Dog { speak() } — EXTENDS, not IMPLEMENTS
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'Animal', 'java');
|
||
addClass(graph, 'Dog', 'java');
|
||
addExtends(graph, 'Dog', 'Animal');
|
||
addMethod(graph, 'Animal', 'speak');
|
||
addMethod(graph, 'Dog', 'speak');
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result.methodImplementsEdges).toBe(0);
|
||
});
|
||
|
||
it('does not emit METHOD_IMPLEMENTS when class has no matching method', () => {
|
||
// IAnimal { speak() } <-- Dog { bark() } — no name match
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IAnimal', 'java', 'Interface');
|
||
addClass(graph, 'Dog', 'java');
|
||
addImplements(graph, 'Dog', 'IAnimal');
|
||
addMethod(graph, 'IAnimal', 'speak', 'Interface');
|
||
addMethod(graph, 'Dog', 'bark');
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result.methodImplementsEdges).toBe(0);
|
||
});
|
||
|
||
it('skips Property nodes on interface', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IFoo', 'csharp', 'Interface');
|
||
addClass(graph, 'Bar', 'csharp');
|
||
addImplements(graph, 'Bar', 'IFoo');
|
||
|
||
// Add a Property to the interface (not a Method)
|
||
const propId = generateId('Property', 'IFoo.name');
|
||
graph.addNode({
|
||
id: propId,
|
||
label: 'Property',
|
||
properties: { name: 'name', filePath: 'src/IFoo.ts' },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Interface', 'IFoo')}->${propId}`),
|
||
sourceId: generateId('Interface', 'IFoo'),
|
||
targetId: propId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
addMethod(graph, 'Bar', 'name');
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result.methodImplementsEdges).toBe(0);
|
||
});
|
||
|
||
describe('METHOD_IMPLEMENTS transitive ancestors', () => {
|
||
it('transitive interface chain: C.foo links to both B.foo and A.foo', () => {
|
||
// A (Interface) has foo, B (Interface) has foo extends A, C (Class) implements B
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'A', 'java', 'Interface');
|
||
addClass(graph, 'B', 'java', 'Interface');
|
||
addClass(graph, 'C', 'java');
|
||
|
||
addInterfaceExtends(graph, 'B', 'A');
|
||
addImplements(graph, 'C', 'B');
|
||
|
||
const aFoo = addMethod(graph, 'A', 'foo', 'Interface');
|
||
const bFoo = addMethod(graph, 'B', 'foo', 'Interface');
|
||
addMethod(graph, 'C', 'foo');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const edges: any[] = [];
|
||
graph.forEachRelationship((rel) => {
|
||
if (rel.type === 'METHOD_IMPLEMENTS') edges.push(rel);
|
||
});
|
||
|
||
// C.foo should link to both B.foo and A.foo
|
||
expect(edges.some((e) => e.targetId === bFoo)).toBe(true);
|
||
expect(edges.some((e) => e.targetId === aFoo)).toBe(true);
|
||
expect(result.methodImplementsEdges).toBeGreaterThanOrEqual(2);
|
||
});
|
||
|
||
it('inherited contract method only on grandparent: C.bar links to A.bar', () => {
|
||
// A (Interface) has bar, B (Interface) extends A but has NO bar, C implements B
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'A', 'java', 'Interface');
|
||
addClass(graph, 'B', 'java', 'Interface');
|
||
addClass(graph, 'C', 'java');
|
||
|
||
addInterfaceExtends(graph, 'B', 'A');
|
||
addImplements(graph, 'C', 'B');
|
||
|
||
const aBar = addMethod(graph, 'A', 'bar', 'Interface');
|
||
// B has no bar method
|
||
addMethod(graph, 'C', 'bar');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const edges: any[] = [];
|
||
graph.forEachRelationship((rel) => {
|
||
if (rel.type === 'METHOD_IMPLEMENTS') edges.push(rel);
|
||
});
|
||
|
||
// C.bar should link to A.bar even though A is not a direct parent
|
||
expect(edges.some((e) => e.targetId === aBar)).toBe(true);
|
||
expect(result.methodImplementsEdges).toBeGreaterThanOrEqual(1);
|
||
});
|
||
|
||
it('diamond deduplication: E.foo gets exactly one edge to A.foo', () => {
|
||
// A (Interface) has foo
|
||
// B (Interface) has foo, extends A
|
||
// D (Interface) has foo, extends A
|
||
// E (Class) implements B and D
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'A', 'java', 'Interface');
|
||
addClass(graph, 'B', 'java', 'Interface');
|
||
addClass(graph, 'D', 'java', 'Interface');
|
||
addClass(graph, 'E', 'java');
|
||
|
||
addInterfaceExtends(graph, 'B', 'A');
|
||
addInterfaceExtends(graph, 'D', 'A');
|
||
addImplements(graph, 'E', 'B');
|
||
addImplements(graph, 'E', 'D');
|
||
|
||
const aFoo = addMethod(graph, 'A', 'foo', 'Interface');
|
||
const bFoo = addMethod(graph, 'B', 'foo', 'Interface');
|
||
const dFoo = addMethod(graph, 'D', 'foo', 'Interface');
|
||
addMethod(graph, 'E', 'foo');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const eFoo = generateId('Method', 'E.foo#0');
|
||
const edges: any[] = [];
|
||
graph.forEachRelationship((rel) => {
|
||
if (rel.type === 'METHOD_IMPLEMENTS') edges.push(rel);
|
||
});
|
||
|
||
// Filter to only edges FROM E.foo
|
||
const eFooEdges = edges.filter((e) => e.sourceId === eFoo);
|
||
|
||
// E.foo should link to B.foo, D.foo, and exactly ONE A.foo (deduplicated)
|
||
expect(eFooEdges.filter((e) => e.targetId === bFoo)).toHaveLength(1);
|
||
expect(eFooEdges.filter((e) => e.targetId === dFoo)).toHaveLength(1);
|
||
expect(eFooEdges.filter((e) => e.targetId === aFoo)).toHaveLength(1);
|
||
// Total from E.foo: 3 edges (B.foo + D.foo + A.foo), not 4
|
||
expect(eFooEdges).toHaveLength(3);
|
||
});
|
||
|
||
it('no transitive through class-only chain', () => {
|
||
// A (Class) has foo, B (Class) extends A has foo, C (Class) extends B has foo
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'A', 'java');
|
||
addClass(graph, 'B', 'java');
|
||
addClass(graph, 'C', 'java');
|
||
|
||
addExtends(graph, 'B', 'A');
|
||
addExtends(graph, 'C', 'B');
|
||
|
||
addMethod(graph, 'A', 'foo');
|
||
addMethod(graph, 'B', 'foo');
|
||
addMethod(graph, 'C', 'foo');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
// All class-extends, no interface involved → 0 METHOD_IMPLEMENTS edges
|
||
expect(result.methodImplementsEdges).toBe(0);
|
||
});
|
||
});
|
||
|
||
it('is queryable via MATCH pattern', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IRepo', 'typescript', 'Interface');
|
||
addClass(graph, 'SqlRepo', 'typescript');
|
||
addImplements(graph, 'SqlRepo', 'IRepo');
|
||
addMethod(graph, 'IRepo', 'fetch', 'Interface');
|
||
const concreteId = addMethod(graph, 'SqlRepo', 'fetch');
|
||
|
||
computeMRO(graph);
|
||
|
||
// Simulate MATCH (m)-[:METHOD_IMPLEMENTS]->(i) RETURN m
|
||
const implementingMethods: string[] = [];
|
||
graph.forEachRelationship((rel) => {
|
||
if (rel.type === 'METHOD_IMPLEMENTS') {
|
||
implementingMethods.push(rel.sourceId);
|
||
}
|
||
});
|
||
expect(implementingMethods).toContain(concreteId);
|
||
});
|
||
|
||
describe('METHOD_IMPLEMENTS inherited + arity matching', () => {
|
||
it('inherited implementation: Base.foo satisfies I.foo when C has no own foo', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'Base', 'java');
|
||
addClass(graph, 'I', 'java', 'Interface');
|
||
addClass(graph, 'C', 'java');
|
||
|
||
addExtends(graph, 'C', 'Base');
|
||
addImplements(graph, 'C', 'I');
|
||
|
||
const baseFoo = addMethod(graph, 'Base', 'foo');
|
||
const iFoo = addMethod(graph, 'I', 'foo', 'Interface');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const edges: any[] = [];
|
||
graph.forEachRelationship((rel) => {
|
||
if (rel.type === 'METHOD_IMPLEMENTS') edges.push(rel);
|
||
});
|
||
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(baseFoo);
|
||
expect(edges[0].targetId).toBe(iFoo);
|
||
expect(result.methodImplementsEdges).toBe(1);
|
||
});
|
||
|
||
it('class has own method — no inherited lookup needed', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'Base2', 'java');
|
||
addClass(graph, 'I2', 'java', 'Interface');
|
||
addClass(graph, 'C2', 'java');
|
||
|
||
addExtends(graph, 'C2', 'Base2');
|
||
addImplements(graph, 'C2', 'I2');
|
||
|
||
const baseFoo = addMethod(graph, 'Base2', 'foo');
|
||
const iFoo = addMethod(graph, 'I2', 'foo', 'Interface');
|
||
const cFoo = addMethod(graph, 'C2', 'foo');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const edges: any[] = [];
|
||
graph.forEachRelationship((rel) => {
|
||
if (rel.type === 'METHOD_IMPLEMENTS') edges.push(rel);
|
||
});
|
||
|
||
// Should use C2.foo, not Base2.foo
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(cFoo);
|
||
expect(edges[0].targetId).toBe(iFoo);
|
||
});
|
||
|
||
it('deep inheritance chain: GrandBase.foo satisfies I.foo', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'GrandBase', 'java');
|
||
addClass(graph, 'Base3', 'java');
|
||
addClass(graph, 'I3', 'java', 'Interface');
|
||
addClass(graph, 'C3', 'java');
|
||
|
||
addExtends(graph, 'Base3', 'GrandBase');
|
||
addExtends(graph, 'C3', 'Base3');
|
||
addImplements(graph, 'C3', 'I3');
|
||
|
||
const grandFoo = addMethod(graph, 'GrandBase', 'foo');
|
||
// Base3 has NO foo
|
||
const iFoo = addMethod(graph, 'I3', 'foo', 'Interface');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const edges: any[] = [];
|
||
graph.forEachRelationship((rel) => {
|
||
if (rel.type === 'METHOD_IMPLEMENTS') edges.push(rel);
|
||
});
|
||
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(grandFoo);
|
||
expect(edges[0].targetId).toBe(iFoo);
|
||
expect(result.methodImplementsEdges).toBe(1);
|
||
});
|
||
|
||
it('arity mismatch prevents false match', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IArity', 'java', 'Interface');
|
||
addClass(graph, 'CArity', 'java');
|
||
addImplements(graph, 'CArity', 'IArity');
|
||
|
||
// Interface method: parameterCount=2, no parameterTypes
|
||
const iMethodId = generateId('Method', 'IArity.process#2');
|
||
graph.addNode({
|
||
id: iMethodId,
|
||
label: 'Method',
|
||
properties: { name: 'process', filePath: 'src/IArity.ts', parameterCount: 2 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Interface', 'IArity')}->${iMethodId}`),
|
||
sourceId: generateId('Interface', 'IArity'),
|
||
targetId: iMethodId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
// Class method: parameterCount=3, no parameterTypes
|
||
const cMethodId = generateId('Method', 'CArity.process#3');
|
||
graph.addNode({
|
||
id: cMethodId,
|
||
label: 'Method',
|
||
properties: { name: 'process', filePath: 'src/CArity.ts', parameterCount: 3 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Class', 'CArity')}->${cMethodId}`),
|
||
sourceId: generateId('Class', 'CArity'),
|
||
targetId: cMethodId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result.methodImplementsEdges).toBe(0);
|
||
});
|
||
|
||
it('arity match when types missing', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IArityOk', 'java', 'Interface');
|
||
addClass(graph, 'CArityOk', 'java');
|
||
addImplements(graph, 'CArityOk', 'IArityOk');
|
||
|
||
// Interface method: parameterCount=2, no parameterTypes
|
||
const iMethodId = generateId('Method', 'IArityOk.process#2');
|
||
graph.addNode({
|
||
id: iMethodId,
|
||
label: 'Method',
|
||
properties: { name: 'process', filePath: 'src/IArityOk.ts', parameterCount: 2 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Interface', 'IArityOk')}->${iMethodId}`),
|
||
sourceId: generateId('Interface', 'IArityOk'),
|
||
targetId: iMethodId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
// Class method: parameterCount=2, no parameterTypes
|
||
const cMethodId = generateId('Method', 'CArityOk.process#2');
|
||
graph.addNode({
|
||
id: cMethodId,
|
||
label: 'Method',
|
||
properties: { name: 'process', filePath: 'src/CArityOk.ts', parameterCount: 2 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Class', 'CArityOk')}->${cMethodId}`),
|
||
sourceId: generateId('Class', 'CArityOk'),
|
||
targetId: cMethodId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result.methodImplementsEdges).toBe(1);
|
||
});
|
||
|
||
it('multiple same-arity candidates = ambiguous, no edge emitted', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IAmbig', 'java', 'Interface');
|
||
addClass(graph, 'CAmbig', 'java');
|
||
addImplements(graph, 'CAmbig', 'IAmbig');
|
||
|
||
// Interface method: parameterCount=1, no parameterTypes
|
||
const iMethodId = generateId('Method', 'IAmbig.handle#1');
|
||
graph.addNode({
|
||
id: iMethodId,
|
||
label: 'Method',
|
||
properties: { name: 'handle', filePath: 'src/IAmbig.ts', parameterCount: 1 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Interface', 'IAmbig')}->${iMethodId}`),
|
||
sourceId: generateId('Interface', 'IAmbig'),
|
||
targetId: iMethodId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
// Two class methods named handle, both with parameterCount=1
|
||
const cMethod1 = generateId('Method', 'CAmbig.handle.1#1');
|
||
graph.addNode({
|
||
id: cMethod1,
|
||
label: 'Method',
|
||
properties: { name: 'handle', filePath: 'src/CAmbig.ts', parameterCount: 1 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Class', 'CAmbig')}->${cMethod1}`),
|
||
sourceId: generateId('Class', 'CAmbig'),
|
||
targetId: cMethod1,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const cMethod2 = generateId('Method', 'CAmbig.handle.2#1');
|
||
graph.addNode({
|
||
id: cMethod2,
|
||
label: 'Method',
|
||
properties: { name: 'handle', filePath: 'src/CAmbig.ts', parameterCount: 1 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Class', 'CAmbig')}->${cMethod2}`),
|
||
sourceId: generateId('Class', 'CAmbig'),
|
||
targetId: cMethod2,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const result = computeMRO(graph);
|
||
expect(result.methodImplementsEdges).toBe(0);
|
||
});
|
||
});
|
||
});
|
||
|
||
// ---- findInheritedMethod ambiguity detection ------------------------------
|
||
describe('findInheritedMethod ambiguity', () => {
|
||
it('returns null when two EXTENDS parents both provide matching method', () => {
|
||
// I { foo() }, B { foo() }, M { foo() }, C extends B + M, C implements I
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'I', 'cpp', 'Interface');
|
||
addClass(graph, 'B', 'cpp');
|
||
addClass(graph, 'M', 'cpp');
|
||
addClass(graph, 'C', 'cpp');
|
||
addImplements(graph, 'C', 'I');
|
||
addExtends(graph, 'C', 'B');
|
||
addExtends(graph, 'C', 'M');
|
||
addMethod(graph, 'I', 'foo', 'Interface');
|
||
addMethod(graph, 'B', 'foo');
|
||
addMethod(graph, 'M', 'foo');
|
||
// C has NO own foo — must walk EXTENDS chain
|
||
|
||
const result = computeMRO(graph);
|
||
// Ambiguous: B.foo and M.foo both match — no METHOD_IMPLEMENTS edge
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
const fooEdges = mi.filter((e) => graph.getNode(e.targetId)?.properties.name === 'foo');
|
||
expect(fooEdges).toHaveLength(0);
|
||
});
|
||
|
||
it('diamond dedup: same method via two paths is NOT ambiguous', () => {
|
||
// I { foo() }, GrandBase { foo() }, B extends GrandBase, M extends GrandBase
|
||
// C extends B + M, C implements I
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'I', 'cpp', 'Interface');
|
||
addClass(graph, 'GrandBase', 'cpp');
|
||
addClass(graph, 'B', 'cpp');
|
||
addClass(graph, 'M', 'cpp');
|
||
addClass(graph, 'C', 'cpp');
|
||
addImplements(graph, 'C', 'I');
|
||
addExtends(graph, 'C', 'B');
|
||
addExtends(graph, 'C', 'M');
|
||
addExtends(graph, 'B', 'GrandBase');
|
||
addExtends(graph, 'M', 'GrandBase');
|
||
addMethod(graph, 'I', 'foo', 'Interface');
|
||
const gbFoo = addMethod(graph, 'GrandBase', 'foo');
|
||
// B and M have NO own foo — both inherit from GrandBase
|
||
|
||
const result = computeMRO(graph);
|
||
// Not ambiguous: same GrandBase.foo via both paths
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
const fooEdge = mi.find((e) => e.sourceId === gbFoo);
|
||
expect(fooEdge).toBeDefined();
|
||
});
|
||
|
||
it('C extends B extends A, B and A both have foo → returns B.foo (nearest)', () => {
|
||
// I { foo() }, A { foo() }, B extends A { foo() }, C extends B implements I { no foo }
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'I', 'java', 'Interface');
|
||
addClass(graph, 'A', 'java');
|
||
addClass(graph, 'B', 'java');
|
||
addClass(graph, 'C', 'java');
|
||
addImplements(graph, 'C', 'I');
|
||
addExtends(graph, 'C', 'B');
|
||
addExtends(graph, 'B', 'A');
|
||
addMethod(graph, 'I', 'foo', 'Interface');
|
||
addMethod(graph, 'A', 'foo');
|
||
const bFoo = addMethod(graph, 'B', 'foo');
|
||
// C has NO own foo — nearest is B.foo at depth 1
|
||
|
||
computeMRO(graph);
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
const fooEdge = mi.find((e) => e.sourceId === bFoo);
|
||
expect(fooEdge).toBeDefined();
|
||
// A.foo should NOT be reached
|
||
const aFooId = generateId('Method', 'A.foo#0');
|
||
const aFooEdge = mi.find((e) => e.sourceId === aFooId);
|
||
expect(aFooEdge).toBeUndefined();
|
||
});
|
||
|
||
it('C extends B extends A, only A has foo → returns A.foo (single match at depth 2)', () => {
|
||
// I { foo() }, A { foo() }, B extends A { no foo }, C extends B implements I { no foo }
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'I', 'java', 'Interface');
|
||
addClass(graph, 'A', 'java');
|
||
addClass(graph, 'B', 'java');
|
||
addClass(graph, 'C', 'java');
|
||
addImplements(graph, 'C', 'I');
|
||
addExtends(graph, 'C', 'B');
|
||
addExtends(graph, 'B', 'A');
|
||
addMethod(graph, 'I', 'foo', 'Interface');
|
||
const aFoo = addMethod(graph, 'A', 'foo');
|
||
// B has NO foo, C has NO foo — only A.foo at depth 2
|
||
|
||
computeMRO(graph);
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
const fooEdge = mi.find((e) => e.sourceId === aFoo);
|
||
expect(fooEdge).toBeDefined();
|
||
});
|
||
});
|
||
|
||
// ---- METHOD_IMPLEMENTS concrete-source guard ----------------------------
|
||
describe('METHOD_IMPLEMENTS concrete-source guard', () => {
|
||
it('interface B extends interface A, B redeclares foo → 0 METHOD_IMPLEMENTS', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'A', 'java', 'Interface');
|
||
addClass(graph, 'B', 'java', 'Interface');
|
||
addInterfaceExtends(graph, 'B', 'A');
|
||
addMethod(graph, 'A', 'foo', 'Interface');
|
||
addMethod(graph, 'B', 'foo', 'Interface');
|
||
|
||
computeMRO(graph);
|
||
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(mi).toHaveLength(0);
|
||
});
|
||
|
||
it('abstract class C implements I, C has abstract foo → 0 METHOD_IMPLEMENTS for foo', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'I', 'java', 'Interface');
|
||
addClass(graph, 'C', 'java', 'Class');
|
||
addImplements(graph, 'C', 'I');
|
||
addMethod(graph, 'I', 'foo', 'Interface');
|
||
|
||
// Add abstract method manually with isAbstract flag
|
||
const classId = generateId('Class', 'C');
|
||
const methodId = generateId('Method', 'C.foo#0');
|
||
graph.addNode({
|
||
id: methodId,
|
||
label: 'Method',
|
||
properties: { name: 'foo', filePath: 'src/C.ts', isAbstract: true, parameterCount: 0 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${classId}->${methodId}`),
|
||
sourceId: classId,
|
||
targetId: methodId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
computeMRO(graph);
|
||
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(mi).toHaveLength(0);
|
||
});
|
||
|
||
it('abstract class C implements I, C has concrete bar → 1 METHOD_IMPLEMENTS for bar', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'I', 'java', 'Interface');
|
||
addClass(graph, 'C', 'java', 'Class');
|
||
addImplements(graph, 'C', 'I');
|
||
addMethod(graph, 'I', 'bar', 'Interface');
|
||
const cBar = addMethod(graph, 'C', 'bar');
|
||
|
||
computeMRO(graph);
|
||
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(mi).toHaveLength(1);
|
||
expect(mi[0].sourceId).toBe(cBar);
|
||
});
|
||
|
||
it('concrete class implements interface → 1 METHOD_IMPLEMENTS (regression)', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'I', 'java', 'Interface');
|
||
addClass(graph, 'C', 'java', 'Class');
|
||
addImplements(graph, 'C', 'I');
|
||
addMethod(graph, 'I', 'foo', 'Interface');
|
||
const cFoo = addMethod(graph, 'C', 'foo');
|
||
|
||
computeMRO(graph);
|
||
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(mi).toHaveLength(1);
|
||
expect(mi[0].sourceId).toBe(cFoo);
|
||
});
|
||
});
|
||
|
||
describe('default interface method resolution', () => {
|
||
it('interface default method satisfies grandparent interface contract', () => {
|
||
// I1 has abstract bar, I2 extends I1 and provides concrete bar,
|
||
// C implements I2 with no own bar → edge from I2.bar → I1.bar
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'I1Def', 'java', 'Interface');
|
||
addClass(graph, 'I2Def', 'java', 'Interface');
|
||
addClass(graph, 'CDef', 'java');
|
||
|
||
addInterfaceExtends(graph, 'I2Def', 'I1Def');
|
||
addImplements(graph, 'CDef', 'I2Def');
|
||
|
||
const i1Bar = addMethod(graph, 'I1Def', 'bar', 'Interface', undefined, { isAbstract: true });
|
||
const i2Bar = addMethod(graph, 'I2Def', 'bar', 'Interface', undefined, {
|
||
isAbstract: false,
|
||
});
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const edges = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(i2Bar);
|
||
expect(edges[0].targetId).toBe(i1Bar);
|
||
});
|
||
|
||
it('own method takes priority over interface default', () => {
|
||
// I has concrete default bar, C implements I and has own bar
|
||
// → edge from C.bar → I.bar (own method wins, no IMPLEMENTS fallback needed)
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IOwn', 'java', 'Interface');
|
||
addClass(graph, 'COwn', 'java');
|
||
|
||
addImplements(graph, 'COwn', 'IOwn');
|
||
|
||
const iBar = addMethod(graph, 'IOwn', 'bar', 'Interface', undefined, { isAbstract: false });
|
||
const cBar = addMethod(graph, 'COwn', 'bar');
|
||
|
||
computeMRO(graph);
|
||
|
||
const edges = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(cBar);
|
||
expect(edges[0].targetId).toBe(iBar);
|
||
});
|
||
|
||
it('transitive interface default: C implements I2, I2 extends I1, I1 has abstract bar, I2 has default bar → I2.bar satisfies I1.bar', () => {
|
||
// I1 (Interface) has abstract bar
|
||
// I2 (Interface) extends I1, has concrete default bar
|
||
// C (Class) implements I2, has NO bar
|
||
// The main emitter processes CImpl's ancestor I1 (transitive via I2).
|
||
// I1.bar is abstract → CImpl has no own bar → findInheritedMethod runs.
|
||
// EXTENDS BFS: nothing. IMPLEMENTS BFS: walks I2 → finds concrete I2.bar.
|
||
// Edge: I2.bar → I1.bar
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'I1', 'java', 'Interface');
|
||
addClass(graph, 'I2', 'java', 'Interface');
|
||
addClass(graph, 'CImpl', 'java');
|
||
|
||
// I1 has abstract bar
|
||
const i1Bar = addMethod(graph, 'I1', 'bar', 'Interface', undefined, { isAbstract: true });
|
||
// I2 has concrete default bar
|
||
const i2Bar = addMethod(graph, 'I2', 'bar', 'Interface');
|
||
|
||
// I2 extends I1
|
||
const i2Id = generateId('Interface', 'I2');
|
||
const i1Id = generateId('Interface', 'I1');
|
||
graph.addRelationship({
|
||
id: generateId('EXTENDS', `${i2Id}->${i1Id}`),
|
||
sourceId: i2Id,
|
||
targetId: i1Id,
|
||
type: 'EXTENDS',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
// CImpl implements I2
|
||
addImplements(graph, 'CImpl', 'I2');
|
||
|
||
const result = computeMRO(graph);
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
// I2.bar (concrete default) satisfies I1.bar (abstract contract)
|
||
const barEdge = mi.find((e) => e.targetId === i1Bar && e.sourceId === i2Bar);
|
||
expect(barEdge).toBeDefined();
|
||
});
|
||
|
||
it('EXTENDS method takes priority over interface default', () => {
|
||
// I has concrete default foo, Base has concrete foo,
|
||
// C extends Base and implements I with no own foo
|
||
// → edge from Base.foo → I.foo (EXTENDS wins over IMPLEMENTS default)
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IExtPri', 'java', 'Interface');
|
||
addClass(graph, 'BaseExtPri', 'java');
|
||
addClass(graph, 'CExtPri', 'java');
|
||
|
||
addExtends(graph, 'CExtPri', 'BaseExtPri');
|
||
addImplements(graph, 'CExtPri', 'IExtPri');
|
||
|
||
const iFoo = addMethod(graph, 'IExtPri', 'foo', 'Interface', undefined, {
|
||
isAbstract: false,
|
||
});
|
||
const baseFoo = addMethod(graph, 'BaseExtPri', 'foo');
|
||
|
||
computeMRO(graph);
|
||
|
||
const edges = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(baseFoo);
|
||
expect(edges[0].targetId).toBe(iFoo);
|
||
});
|
||
|
||
it('Dart implements Class — does NOT inherit concrete method bodies', () => {
|
||
// Dart: class C implements AbstractBase (labeled Class, not Interface)
|
||
// AbstractBase has concrete method foo
|
||
// C has NO foo — but Dart implements does NOT inherit bodies
|
||
// → 0 METHOD_IMPLEMENTS edges from the IMPLEMENTS fallback
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'AbstractBase', 'dart'); // Class label, not Interface
|
||
addClass(graph, 'DartImpl', 'dart');
|
||
|
||
// AbstractBase has concrete foo
|
||
addMethod(graph, 'AbstractBase', 'foo');
|
||
|
||
// DartImpl implements AbstractBase (IMPLEMENTS edge to a Class)
|
||
addImplements(graph, 'DartImpl', 'AbstractBase', 'Class', 'Interface');
|
||
// But we need AbstractBase to be a Class, not Interface — fix the label
|
||
// Actually addImplements creates the edge, but AbstractBase was added as Class.
|
||
// The IMPLEMENTS edge target needs to match the actual node ID.
|
||
// Let's do this manually:
|
||
const dartImplId = generateId('Class', 'DartImpl');
|
||
const absBaseId = generateId('Class', 'AbstractBase');
|
||
graph.addRelationship({
|
||
id: generateId('IMPLEMENTS', `${dartImplId}->${absBaseId}`),
|
||
sourceId: dartImplId,
|
||
targetId: absBaseId,
|
||
type: 'IMPLEMENTS',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
computeMRO(graph);
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
// No edges — IMPLEMENTS fallback skips Class-labeled parents
|
||
expect(mi).toHaveLength(0);
|
||
});
|
||
|
||
it('Interface default still works after Dart label gate', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IDefault', 'java', 'Interface');
|
||
addClass(graph, 'Impl', 'java');
|
||
addImplements(graph, 'Impl', 'IDefault');
|
||
// IDefault has abstract contract method
|
||
const iFoo = addMethod(graph, 'IDefault', 'foo', 'Interface', undefined, {
|
||
isAbstract: true,
|
||
});
|
||
// IDefault also has concrete default bar
|
||
const iBar = addMethod(graph, 'IDefault', 'bar', 'Interface');
|
||
// Impl has foo but not bar
|
||
addMethod(graph, 'Impl', 'foo');
|
||
|
||
computeMRO(graph);
|
||
const mi = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
// foo: own method matches → edge from Impl.foo → IDefault.foo
|
||
const fooEdge = mi.find((e) => e.targetId === iFoo);
|
||
expect(fooEdge).toBeDefined();
|
||
// bar: no own method, IMPLEMENTS fallback finds IDefault.bar (Interface label OK)
|
||
const barEdge = mi.find((e) => e.sourceId === iBar && e.targetId === iBar);
|
||
// Actually bar is the same method — it's the default implementation satisfying itself.
|
||
// The emitter processes IDefault.bar as an ancestor method, Impl has no bar,
|
||
// findInheritedMethod runs, walks IMPLEMENTS → finds IDefault.bar (non-abstract).
|
||
// But excludeMethodId = ancestorMethodId = iBar → skipped to prevent self-edge!
|
||
// So no bar edge. This is correct — the default satisfies the contract inherently.
|
||
});
|
||
});
|
||
|
||
describe('METHOD_IMPLEMENTS confidence tiering', () => {
|
||
it('fully-typed match gets confidence 1.0', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'ITyped', 'java', 'Interface');
|
||
addClass(graph, 'CTyped', 'java');
|
||
addImplements(graph, 'CTyped', 'ITyped');
|
||
|
||
const iFoo = addMethod(graph, 'ITyped', 'foo', 'Interface', ['int', 'String']);
|
||
const cFoo = addMethod(graph, 'CTyped', 'foo', 'Class', ['int', 'String']);
|
||
|
||
computeMRO(graph);
|
||
|
||
const edges = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(cFoo);
|
||
expect(edges[0].targetId).toBe(iFoo);
|
||
expect(edges[0].confidence).toBe(1.0);
|
||
});
|
||
|
||
it('arity-only match (both have parameterCount, no types) gets confidence 1.0', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IArity', 'java', 'Interface');
|
||
addClass(graph, 'CArity', 'java');
|
||
addImplements(graph, 'CArity', 'IArity');
|
||
|
||
// Manually add methods with parameterCount but no parameterTypes
|
||
const iBarId = generateId('Method', 'IArity.bar#2');
|
||
graph.addNode({
|
||
id: iBarId,
|
||
label: 'Method',
|
||
properties: { name: 'bar', filePath: 'src/IArity.ts', parameterCount: 2 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Interface', 'IArity')}->${iBarId}`),
|
||
sourceId: generateId('Interface', 'IArity'),
|
||
targetId: iBarId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const cBarId = generateId('Method', 'CArity.bar#2');
|
||
graph.addNode({
|
||
id: cBarId,
|
||
label: 'Method',
|
||
properties: { name: 'bar', filePath: 'src/CArity.ts', parameterCount: 2 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Class', 'CArity')}->${cBarId}`),
|
||
sourceId: generateId('Class', 'CArity'),
|
||
targetId: cBarId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
computeMRO(graph);
|
||
|
||
const edges = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(cBarId);
|
||
expect(edges[0].targetId).toBe(iBarId);
|
||
expect(edges[0].confidence).toBe(1.0);
|
||
});
|
||
|
||
it('lenient match (no types, no parameterCount on both sides) gets confidence 0.7', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'ILenient', 'java', 'Interface');
|
||
addClass(graph, 'CLenient', 'java');
|
||
addImplements(graph, 'CLenient', 'ILenient');
|
||
|
||
// Manually create methods WITHOUT parameterCount to simulate legacy/missing arity
|
||
const iBazId = generateId('Method', 'ILenient.baz');
|
||
graph.addNode({
|
||
id: iBazId,
|
||
label: 'Method',
|
||
properties: { name: 'baz', filePath: 'src/ILenient.ts' },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Interface', 'ILenient')}->${iBazId}`),
|
||
sourceId: generateId('Interface', 'ILenient'),
|
||
targetId: iBazId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
const cBazId = generateId('Method', 'CLenient.baz');
|
||
graph.addNode({
|
||
id: cBazId,
|
||
label: 'Method',
|
||
properties: { name: 'baz', filePath: 'src/CLenient.ts' },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Class', 'CLenient')}->${cBazId}`),
|
||
sourceId: generateId('Class', 'CLenient'),
|
||
targetId: cBazId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
computeMRO(graph);
|
||
|
||
const edges = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(cBazId);
|
||
expect(edges[0].targetId).toBe(iBazId);
|
||
expect(edges[0].confidence).toBe(0.7);
|
||
});
|
||
|
||
it('one side has parameterCount, other does not → confidence 0.7', () => {
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IHalf', 'java', 'Interface');
|
||
addClass(graph, 'CHalf', 'java');
|
||
addImplements(graph, 'CHalf', 'IHalf');
|
||
|
||
// Interface method has parameterCount but no parameterTypes
|
||
const iQuxId = generateId('Method', 'IHalf.qux#2');
|
||
graph.addNode({
|
||
id: iQuxId,
|
||
label: 'Method',
|
||
properties: { name: 'qux', filePath: 'src/IHalf.ts', parameterCount: 2 },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Interface', 'IHalf')}->${iQuxId}`),
|
||
sourceId: generateId('Interface', 'IHalf'),
|
||
targetId: iQuxId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
// Class method has neither parameterTypes nor parameterCount (manually constructed)
|
||
const cQuxId = generateId('Method', 'CHalf.qux');
|
||
graph.addNode({
|
||
id: cQuxId,
|
||
label: 'Method',
|
||
properties: { name: 'qux', filePath: 'src/CHalf.ts' },
|
||
});
|
||
graph.addRelationship({
|
||
id: generateId('HAS_METHOD', `${generateId('Class', 'CHalf')}->${cQuxId}`),
|
||
sourceId: generateId('Class', 'CHalf'),
|
||
targetId: cQuxId,
|
||
type: 'HAS_METHOD',
|
||
confidence: 1.0,
|
||
reason: '',
|
||
});
|
||
|
||
computeMRO(graph);
|
||
|
||
const edges = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
expect(edges).toHaveLength(1);
|
||
expect(edges[0].sourceId).toBe(cQuxId);
|
||
expect(edges[0].targetId).toBe(iQuxId);
|
||
expect(edges[0].confidence).toBe(0.7);
|
||
});
|
||
});
|
||
|
||
// ---- IMPLEMENTS BFS ambiguity for default methods -------------------------
|
||
describe('IMPLEMENTS BFS ambiguity for default methods', () => {
|
||
it('does not emit METHOD_IMPLEMENTS when two interfaces provide same default method (ambiguous)', () => {
|
||
// IAncestor (Interface) has abstract process()
|
||
// IAlpha (Interface) extends IAncestor, has concrete process()
|
||
// IBeta (Interface) extends IAncestor, has concrete process()
|
||
// CImpl (Class) implements IAlpha, implements IBeta
|
||
// CImpl has NO process() method
|
||
//
|
||
// findInheritedMethod walks IMPLEMENTS BFS and finds process() in BOTH
|
||
// IAlpha and IBeta => ambiguous => null => no METHOD_IMPLEMENTS edge.
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IAncestor', 'java', 'Interface');
|
||
addClass(graph, 'IAlpha', 'java', 'Interface');
|
||
addClass(graph, 'IBeta', 'java', 'Interface');
|
||
addClass(graph, 'CImpl', 'java');
|
||
|
||
addInterfaceExtends(graph, 'IAlpha', 'IAncestor');
|
||
addInterfaceExtends(graph, 'IBeta', 'IAncestor');
|
||
addImplements(graph, 'CImpl', 'IAlpha');
|
||
addImplements(graph, 'CImpl', 'IBeta');
|
||
|
||
addMethod(graph, 'IAncestor', 'process', 'Interface', undefined, { isAbstract: true });
|
||
addMethod(graph, 'IAlpha', 'process', 'Interface');
|
||
addMethod(graph, 'IBeta', 'process', 'Interface');
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
// IAlpha.process -> IAncestor.process and IBeta.process -> IAncestor.process
|
||
// are legitimate edges from sub-interface processing. The ambiguity check
|
||
// ensures that NO additional edge is emitted on behalf of CImpl (which has
|
||
// no own process()). Since CImpl has no methods, no edge should be sourced
|
||
// from a CImpl method. Verify by checking that the only METHOD_IMPLEMENTS
|
||
// edges for process() are the two interface-to-interface ones.
|
||
const edges = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
const processEdges = edges.filter((e) => {
|
||
const target = graph.getNode(e.targetId);
|
||
return target?.properties.name === 'process';
|
||
});
|
||
// IAlpha.process->IAncestor.process and IBeta.process->IAncestor.process
|
||
// are emitted from the sub-interface processing (each concrete method
|
||
// implements the ancestor's abstract method). No additional edge should be
|
||
// emitted on behalf of CImpl because findInheritedMethod returns null
|
||
// (ambiguous: two candidates at the same BFS depth).
|
||
expect(processEdges).toHaveLength(2);
|
||
const alphaProcess = generateId('Method', 'IAlpha.process#0');
|
||
const betaProcess = generateId('Method', 'IBeta.process#0');
|
||
const sourceIds = processEdges.map((e) => e.sourceId).sort();
|
||
expect(sourceIds).toEqual([alphaProcess, betaProcess].sort());
|
||
});
|
||
|
||
it('emits METHOD_IMPLEMENTS when only one interface provides the default method (unambiguous)', () => {
|
||
// IAncestor (Interface) has abstract process()
|
||
// IAlpha (Interface) extends IAncestor, has concrete process()
|
||
// IBeta (Interface) extends IAncestor, does NOT have process()
|
||
// CImpl (Class) implements IAlpha, implements IBeta
|
||
// CImpl has NO process() method
|
||
//
|
||
// findInheritedMethod walks IMPLEMENTS BFS and finds process() only in
|
||
// IAlpha => unambiguous => emits 1 METHOD_IMPLEMENTS edge.
|
||
const graph = createKnowledgeGraph();
|
||
addClass(graph, 'IAncestor', 'java', 'Interface');
|
||
addClass(graph, 'IAlpha', 'java', 'Interface');
|
||
addClass(graph, 'IBeta', 'java', 'Interface');
|
||
addClass(graph, 'CImpl', 'java');
|
||
|
||
addInterfaceExtends(graph, 'IAlpha', 'IAncestor');
|
||
addInterfaceExtends(graph, 'IBeta', 'IAncestor');
|
||
addImplements(graph, 'CImpl', 'IAlpha');
|
||
addImplements(graph, 'CImpl', 'IBeta');
|
||
|
||
const ancestorProcess = addMethod(graph, 'IAncestor', 'process', 'Interface', undefined, {
|
||
isAbstract: true,
|
||
});
|
||
const alphaProcess = addMethod(graph, 'IAlpha', 'process', 'Interface');
|
||
// IBeta has no process() method
|
||
|
||
const result = computeMRO(graph);
|
||
|
||
const edges = graph.relationships.filter((r) => r.type === 'METHOD_IMPLEMENTS');
|
||
const processEdges = edges.filter((e) => {
|
||
const target = graph.getNode(e.targetId);
|
||
return target?.properties.name === 'process';
|
||
});
|
||
// At minimum, IAlpha.process -> IAncestor.process is emitted (from IAlpha's
|
||
// own processing). CImpl's findInheritedMethod also finds IAlpha.process as
|
||
// the sole unambiguous match, potentially emitting the same edge again.
|
||
expect(processEdges.length).toBeGreaterThanOrEqual(1);
|
||
// Every process edge should point from IAlpha.process to IAncestor.process
|
||
for (const edge of processEdges) {
|
||
expect(edge.sourceId).toBe(alphaProcess);
|
||
expect(edge.targetId).toBe(ancestorProcess);
|
||
}
|
||
});
|
||
});
|
||
});
|