mirror of
https://github.com/abhigyanpatwari/GitNexus.git
synced 2026-11-11 08:37:35 +00:00
test: consolidate covered integration cases (#3792)
Remove ten cases covered by stronger assertions on the same fixtures. Preserve three detector recall pairs and the unique COBOL source-value guard. Co-authored-by: Gergo Magyar <gergomagyar0@gmail.com>
This commit is contained in:
parent
02101fdfed
commit
d32ed0bf06
9 changed files with 0 additions and 68 deletions
|
|
@ -189,14 +189,6 @@ describe('COBOL parsing coverage (F17-F23)', () => {
|
|||
// F23: PERFORM TIMES + THRU on digit-leading targets
|
||||
// =====================================================================
|
||||
describe('F23 — PERFORM TIMES and THRU', () => {
|
||||
it('PERFORM 2000-PROCESS THRU 2100-CLEANUP resolves THRU target', () => {
|
||||
const perfThruEdges = getRelationships(result, 'CALLS').filter(
|
||||
(e) => e.rel.reason === 'cobol-perform-thru',
|
||||
);
|
||||
const thruTargets = perfThruEdges.map((e) => e.target);
|
||||
expect(thruTargets).toContain('2100-CLEANUP');
|
||||
});
|
||||
|
||||
it('PERFORM with VARYING does NOT create spurious CALLS edge', () => {
|
||||
// PERFTIMS has PERFORM VARYING WS-COUNT FROM 1 BY 1...
|
||||
// VARYING is in PERFORM_KEYWORD_SKIP, so it should be skipped.
|
||||
|
|
|
|||
|
|
@ -290,13 +290,6 @@ describe.skipIf(!dartAvailable)('Dart method enrichment', () => {
|
|||
expect(speak!.properties.isAbstract).toBe(true);
|
||||
});
|
||||
|
||||
it('marks breathe as NOT isAbstract', () => {
|
||||
const methods = getNodesByLabelFull(result, 'Method');
|
||||
const breathe = methods.find((n) => n.name === 'breathe');
|
||||
expect(breathe).toBeDefined();
|
||||
expect(breathe!.properties.isAbstract).toBe(false);
|
||||
});
|
||||
|
||||
it('marks classify as isStatic', () => {
|
||||
const methods = getNodesByLabelFull(result, 'Method');
|
||||
const classify = methods.find((n) => n.name === 'classify');
|
||||
|
|
|
|||
|
|
@ -1705,17 +1705,9 @@ describe('Go pointer-receiver field chains (#2766)', () => {
|
|||
expect(calls()).toContain('RunLocal → DoWork');
|
||||
});
|
||||
|
||||
it('keeps resolving a value receiver', () => {
|
||||
expect(calls()).toContain('RunFromValueReceiver → DoWork');
|
||||
});
|
||||
|
||||
// U8: the same-package field receiver that previously produced an ACCESSES
|
||||
// edge to the method and no CALLS edge. Typing the base is what emits CALLS;
|
||||
// the ACCESSES now correctly targets the PROPERTY being read instead.
|
||||
it('emits CALLS for a same-package field receiver, not ACCESSES alone', () => {
|
||||
expect(calls()).toContain('RunSamePackage → Work');
|
||||
});
|
||||
|
||||
it('retargets the field ACCESSES to the property, not the method', () => {
|
||||
// Unlabeled on purpose: the negative must reject `→ Work` under ANY target
|
||||
// kind, which the kind-qualified rows below cannot express.
|
||||
|
|
|
|||
|
|
@ -1511,14 +1511,6 @@ describe('Write access tracking (Kotlin)', () => {
|
|||
expect(scoreWrite!.source).toBe('updateUser');
|
||||
});
|
||||
|
||||
it('emits ACCESSES write edge for compound assignment (+=)', () => {
|
||||
const accesses = getRelationships(result, 'ACCESSES');
|
||||
const writes = accesses.filter((e) => e.rel.reason === 'write');
|
||||
const scoreWrite = writes.find((e) => e.target === 'score');
|
||||
expect(scoreWrite).toBeDefined();
|
||||
expect(scoreWrite!.source).toBe('updateUser');
|
||||
});
|
||||
|
||||
it('write ACCESSES edges have confidence 1.0', () => {
|
||||
const accesses = getRelationships(result, 'ACCESSES');
|
||||
const writes = accesses.filter((e) => e.rel.reason === 'write');
|
||||
|
|
|
|||
|
|
@ -52,12 +52,6 @@ describe('caller-derived parameter types (W2-2)', () => {
|
|||
)
|
||||
.map((e) => e.rel.targetId);
|
||||
|
||||
it('types a bare parameter from its single caller', () => {
|
||||
const targets = preciseTargetsOf('readSpike', 'consumer.js');
|
||||
expect(targets).toHaveLength(1);
|
||||
expect(targets[0]).toContain('makeSpike.wickRatio');
|
||||
});
|
||||
|
||||
it('does not reach the OTHER producer of the same field name', () => {
|
||||
// `makeCandle.wickRatio` exists and is a candidate for any name-based join.
|
||||
expect(preciseTargetsOf('readSpike', 'consumer.js')[0]).not.toContain('makeCandle');
|
||||
|
|
|
|||
|
|
@ -1493,14 +1493,6 @@ describe('Write access tracking (PHP)', () => {
|
|||
expect(countWrite!.source).toBe('updateUser');
|
||||
});
|
||||
|
||||
it('emits ACCESSES write edge for static property assignment', () => {
|
||||
const accesses = getRelationships(result, 'ACCESSES');
|
||||
const writes = accesses.filter((e) => e.rel.reason === 'write');
|
||||
const countWrite = writes.find((e) => e.target === 'count');
|
||||
expect(countWrite).toBeDefined();
|
||||
expect(countWrite!.source).toBe('updateUser');
|
||||
});
|
||||
|
||||
it('write ACCESSES edges have confidence 1.0', () => {
|
||||
const accesses = getRelationships(result, 'ACCESSES');
|
||||
const writes = accesses.filter((e) => e.rel.reason === 'write');
|
||||
|
|
|
|||
|
|
@ -1024,14 +1024,6 @@ describe('Write access tracking (Ruby)', () => {
|
|||
expect(scoreWrite!.source).toBe('update_user');
|
||||
});
|
||||
|
||||
it('emits ACCESSES write edge for compound assignment (operator_assignment)', () => {
|
||||
const accesses = getRelationships(result, 'ACCESSES');
|
||||
const writes = accesses.filter((e) => e.rel.reason === 'write');
|
||||
const scoreWrite = writes.find((e) => e.target === 'score');
|
||||
expect(scoreWrite).toBeDefined();
|
||||
expect(scoreWrite!.source).toBe('update_user');
|
||||
});
|
||||
|
||||
it('write ACCESSES edges have confidence 1.0', () => {
|
||||
const accesses = getRelationships(result, 'ACCESSES');
|
||||
const writes = accesses.filter((e) => e.rel.reason === 'write');
|
||||
|
|
|
|||
|
|
@ -93,17 +93,6 @@ describe('TypeScript HOC-wrapped variable declarations', () => {
|
|||
expect(targets).toContain('fmt');
|
||||
});
|
||||
|
||||
it('useCallback: handleSubmit → doStuff (sibling const, separate caller)', () => {
|
||||
// Two useCallback consts in the same file — each must be its own
|
||||
// caller anchor (no leakage to the first sibling, the analogue of
|
||||
// the multi-action-store regression in PR #1175).
|
||||
const calls = getRelationships(result, 'CALLS').filter(
|
||||
(c) => c.sourceFilePath === 'src/use-callback.ts' && c.target === 'doStuff',
|
||||
);
|
||||
const fromHandleSubmit = calls.filter((c) => c.source === 'handleSubmit');
|
||||
expect(fromHandleSubmit.length, 'handleSubmit must call doStuff').toBeGreaterThan(0);
|
||||
});
|
||||
|
||||
it('useMemo: computed → doStuff (returns-a-value variant)', () => {
|
||||
// `const computed = useMemo(() => { return doStuff(42); }, [])`.
|
||||
// The arrow's body has a `return` statement — irrelevant to call
|
||||
|
|
|
|||
|
|
@ -422,10 +422,6 @@ describe.skipIf(!zigAvailable)('Zig idioms (zig-idioms fixture)', () => {
|
|||
]);
|
||||
});
|
||||
|
||||
it('applies the callable gate through a HUB too', () => {
|
||||
expect(valueRefs).not.toContain('JsApi → DEFAULT_NS');
|
||||
});
|
||||
|
||||
it('applies the callable gate to a MODULE owner too', () => {
|
||||
// `dom_utils.DEFAULT_NS` is a module-scope constant. Widening the owner
|
||||
// channel must not widen what counts as a registration, or every
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue