Skip to content

Commit 62d3ec8

Browse files
committed
test(plugin-dev): correct the header's measured after-figure and the mock count
The scaled repro's after-side is 1 of 10, not 0 of 10, and the one residual timeout lands on the single-org case — which resolves nothing. That control is worth more than a clean zero, so it is recorded as measured. The block now also says "twelve OTHER packages", since organizations joined the list. Claude-Session: https://claude.ai/code/session_01AhQASwqJr2Z7XfGWUdvnbF Co-authored-by: Claude <noreply@anthropic.com>
1 parent 2d81f58 commit 62d3ec8

1 file changed

Lines changed: 13 additions & 7 deletions

File tree

‎packages/plugins/plugin-dev/src/dev-plugin-tenancy-posture.test.ts‎

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@
2727
// fake enterprise package, i.e. stubbing the very thing under test.
2828
//
2929
// `@objectstack/organizations` is SIMULATED absent here, by the same throwing
30-
// factory the twelve packages below get. That is a change: this file used to let
30+
// factory the twelve OTHER packages in the block below get. That is a change: this file used to let
3131
// the REAL dynamic import fail, which it does — ADR-0132's entitlement boundary
3232
// forbids any framework package declaring the name
3333
// (`no-framework-dependents.pin.test.ts`), so it is genuinely unresolvable from
@@ -50,12 +50,18 @@
5050
// So each walled case drew from an unbounded, machine-load-dependent
5151
// distribution, and a 5000ms per-test budget bounded the DRAW rather than any
5252
// work this file is about. Under a parallel shard it timed out on a case that
53-
// asserts a back-compat env-var reading. Scaled repro, same command before and
54-
// after, timeout scaled down instead of load scaled up: with the per-test budget
55-
// at 25ms under filesystem contention, 10 runs of this file failed 6 times
56-
// before this change and 0 times after — and every one of those 10 timeouts
57-
// landed on a WALLED case, never on the single-org case, which runs three
58-
// `init()`s and resolves nothing. That control is what names the construct.
53+
// asserts a back-compat env-var reading.
54+
//
55+
// Scaled repro — same command and same filesystem contention on both sides,
56+
// with the per-test budget scaled DOWN to 25ms rather than the load scaled up,
57+
// N=10 runs each side: 6 of 10 runs failed before this change, 1 of 10 after.
58+
// The CASE-level reading is what names the construct: all 10 timeouts before
59+
// landed on a WALLED case and none on the single-org case; after, the single
60+
// remaining timeout WAS the single-org case — which resolves nothing and runs
61+
// three `init()`s, i.e. the scaled harness's own CPU floor at a 25ms budget,
62+
// not this construct. ⛔ Do not read that residual as a reason to reach for the
63+
// budget: at the 5000ms CI actually uses, the whole window is now pure
64+
// in-process CPU measured at 1–9ms.
5965
//
6066
// ⛔ Nothing about the absence is given up by the SUITE, only by this file: the
6167
// real, unmocked resolution failure is still the signal in

0 commit comments

Comments
 (0)