test: model open -W in the UI lane and drop the pyc wall-clock bound

The native UI verifier now removes its fixture root only after `open -W`
exits on its own (67ce671d66). The lane test's `open` stand-in stayed
alive until the test released it, so every case waited out the grace
period, retained the root and failed on the Apple E2E job. The stand-in
now exits when its fixture exits, as `open -W` does.

The shared-descendant pyc case bounded wall-clock time at 5 s; it runs in
~0.5 s locally but 4-7 s on loaded CI shards and failed scheduled runs.
The retained-heap bound and exact object counts already detect the
per-wrapper copy regression, so the host-speed assertion is removed.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
morlutoandClaude Opus 5.5 committed 2026-11-14 05:36:39 -05:00
1 parent 9e24b36fa6
commit 2343e56925
2 files changed
+6 -6

No files matched your search

+2 -3
View File
@@ -1265,11 +1265,10 @@ it("keeps one descendant list when many wrappers share a container", () => {
marshalCode("py311", { constsObject: Buffer.concat(pieces) }),
]);
const before = process.memoryUsage().heapUsed;
const started = Date.now();
const report = parseWith(bytes, { max_objects: 65_536 });
const retained = process.memoryUsage().heapUsed - before;
// 1,800 children and 14,000 wrappers: 49,612 reads, previously ~181 MiB.
expect(Date.now() - started).toBeLessThan(5000);
// 1,800 children and 14,000 wrappers: 49,612 reads. A copied list per wrapper
// previously retained ~181 MiB; retention, not host speed, shows the sharing.
expect(retained).toBeLessThan(64 * 1024 * 1024);
expect(report.coverage.objects).toBe(49_612);
expect(report.code_objects).toHaveLength(children + 1);