acdream/tests/AcDream.Content.Tests
Erik 619de97ad1 fix(test): evaluate BOTH deleted guards; correct AP-22's overstated coverage claim
From the AP-22 dual review (both lenses PASS). No production change.

THE RECORD WAS WRONG. bc4679cd claimed "Headless.Tests 89/89 exercises the
site-3 copy". The architecture review disproved it by sabotage: restoring the
invented cylinder in BOTH static sites left the entire suite green. No test
anywhere references PublishStaticCollision, and the headless suite's dummy DAT
proxy makes LandblockLoader.Load fail for every landblock, so CreatePublication
returns before reaching it. Two of the three deletions — including the
headless-only one — are pinned by the installed-DAT reachability proof ALONE.
The deletion is still correct; the evidence claim was not, and a successor
trusting it would think those sites had regression cover they do not have.

THE TEST NOW COVERS WHAT IT CLAIMED. Its comment said "the exact guard the
three deleted copies used", but site 1 guarded on `Radius > 0.0001f` while
sites 2 and 3 used the strictly wider `Radius > 0f`. Those are not the same
predicate: the review measured that they differ over the installed DAT by
exactly one Setup, 0x02001657, whose radius is the denormal 1.3e-39. The test
now evaluates BOTH and asserts each is empty, so the wider guard the
headless-reachable deletion actually used is no longer asserted by proxy.

Sabotage-verified: widening the new guard to `>= 0f` reddens it (1,652
zero-radius Setups appear), so the assertion is live rather than vacuously
empty over real DAT data.

AP-22's row also corrected for two precisions the reviews surfaced: the
load-bearing fact is that all 1,652 no-primitive Setups carry Radius exactly 0
(not the 1,294 first cited), and retail's `report_object_collision` DOES read
GetHeight for the quadrant field — recorded so a future reader does not mistake
it for a refutation of "never collision geometry", which is a claim about
FindObjCollisions' shape dispatch only.

Reachability now independently reproduced by four decoders — the contract's
sweep, the implementer's parser, and both reviewers' from-scratch parsers —
plus tools/SetupInspect agreeing bit-for-bit on the cited ids.

Content.Tests 125/125. No new skips; #302/#308/#321 did not fire.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-08-06 08:54:31 +02:00
..
Spells feat(headless): share immutable gameplay content 2026-07-27 09:00:48 +02:00
Vfx fix(tests): replace sleep-race concurrency proofs in RetailDatLoaderTests 2026-07-30 11:03:03 +02:00
AcDream.Content.Tests.csproj test(pipeline): MP1b - live-vs-pak equivalence suite (dat-gated) 2026-07-05 21:33:32 +02:00
BoundedDatObjectCacheTests.cs perf(diag): per-frame history export + checkpoint LOH/cache counters + soak capped mode (2026-07-24 audit review) 2026-07-24 12:00:30 +02:00
ContentConformanceDats.cs test(pipeline): MP1b - live-vs-pak equivalence suite (dat-gated) 2026-07-05 21:33:32 +02:00
Crc32Tests.cs fix(pipeline): MP1b review - pak format/reader hardening 2026-07-05 22:08:59 +02:00
DatResolutionPrecedenceTests.cs perf(content): remove exception-driven setup probes 2026-07-24 15:12:21 +02:00
DecodedTextureCacheTests.cs perf(diag): per-frame history export + checkpoint LOH/cache counters + soak capped mode (2026-07-24 audit review) 2026-07-24 12:00:30 +02:00
FlatCollisionAssetSerializerTests.cs feat(content): bake and read flat collision assets 2026-07-25 15:22:08 +02:00
InstalledPreparedCollisionCatalogTests.cs feat(content): bake and read flat collision assets 2026-07-25 15:22:08 +02:00
InstalledSetupCollisionReachabilityTests.cs fix(test): evaluate BOTH deleted guards; correct AP-22's overstated coverage claim 2026-08-06 08:54:31 +02:00
ObjectMeshDataEquality.cs perf(render): consume prepared mesh package at runtime 2026-07-24 15:07:25 +02:00
ObjectMeshDataSerializerTests.cs perf(render): consume prepared mesh package at runtime 2026-07-24 15:07:25 +02:00
PakAliasTests.cs feat(content): add physical blob aliases to pak writer 2026-07-24 13:33:53 +02:00
PakEquivalenceTests.cs feat(bake): publish validated pak artifacts atomically 2026-07-24 13:40:58 +02:00
PakFormatTests.cs feat(pipeline): MP1b - pak key + header/TOC primitives 2026-07-05 21:14:10 +02:00
PakKeyTests.cs feat(content): bake and read flat collision assets 2026-07-25 15:22:08 +02:00
PakRoundTripTests.cs feat(bake): publish validated pak artifacts atomically 2026-07-24 13:40:58 +02:00
PreparedAssetSourceTests.cs perf(content): add typed prepared asset source 2026-07-24 14:52:12 +02:00
SharedPreparedCollisionCacheTests.cs feat(headless): share immutable gameplay content 2026-07-27 09:00:48 +02:00
SolidColorTextureCacheTests.cs perf(content): CellStruct clone guard, solid-color texture cache, drop double ToList (2026-07-24 audit review) 2026-07-24 11:49:46 +02:00