Skip to content

Commit 6ca6d56

Browse files
committed
fix(bench): use qualified node name accessorTarget.accessMethod in JS expected-edges (#1351)
The graph registers object literal arrow properties under qualified names (e.g. 'accessorTarget.accessMethod' not bare 'accessMethod') since extractObjectLiteralFunctions was updated to avoid polluting the global definition index. The expected edge must match the stored node name so the precision check passes for both WASM and native engines.
1 parent 72632fb commit 6ca6d56

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

tests/benchmarks/resolution/fixtures/javascript/expected-edges.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -236,10 +236,10 @@
236236
},
237237
{
238238
"source": { "name": "accessorGetter", "file": "define-property-accessor.js" },
239-
"target": { "name": "accessMethod", "file": "define-property-accessor.js" },
239+
"target": { "name": "accessorTarget.accessMethod", "file": "define-property-accessor.js" },
240240
"kind": "calls",
241241
"mode": "defineProperty-accessor",
242-
"notes": "this.accessMethod() inside accessorGetter — this === accessorTarget (get accessor via Object.defineProperty); accessMethod is arrow function property of accessorTarget"
242+
"notes": "this.accessMethod() inside accessorGetter — this === accessorTarget (get accessor via Object.defineProperty); accessorTarget.accessMethod is the qualified node name for the arrow function property of accessorTarget"
243243
}
244244
]
245245
}

0 commit comments

Comments
 (0)