Stamp the release in the page for the drag check in e2e 41 (#282)

The first CI run after 9cff528 failed e2e 41 again, at the release this
time (task 405): the hold was fine for 54 frames, but the first sample
after the release was 6.9 units below the hold, over the bound of 3.
The test took its release time after the round trip of the mouse event
and one more evaluate. The frames in between were missing, so the
first counted sample was already about 100 ms into the fall.

- The page stamps the release itself with a capturing `pointerup`
  listener. Held and released samples are split at that stamp.
- The fixed bound on the first sample after the release is gone. How
  far that frame has fallen follows the frame time. Instead every
  sample in the first 200 ms must stay below the hold and keep
  falling, which is what "let go at rest" means: no jump, no kick up.
- The 200 ms fall still has to be more than 1 and less than 40 units.
  The comment now says about 25, as measured, instead of under 20.

Tests:
- e2e 41 on melektaus-atx: 8 of 8 runs ALL PASSED, 21 checks. The
  first sample came 6 to 42 ms after the stamp.
- Counter-check: a release velocity of y -2 instead of 0 in
  PhysicsSystem makes the body rise to -3510 and the check fails.
- Gate on melektaus-atx: 195 files, 2829 tests, CHECK EXIT 0.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
adminandClaude Opus 5.5 committed 2026-09-29 22:48:23 +02:00
1 parent 6d4247ebf8
commit 543b052136
1 file changed
+17 -7
+17 -7
View File
@@ -302,6 +302,8 @@ export async function physics(t) {
() => false, () => false,
); );
// Every frame recorded in the page, so the count follows the frames and not the round trips. // Every frame recorded in the page, so the count follows the frames and not the round trips.
// The page also stamps the release itself: a stamp taken after the round trip came frames late,
// and the first sample after it was already 7 units into the fall on the WSL runner.
await evaluate(`(() => { await evaluate(`(() => {
const out = (window.__dragSamples = []); const out = (window.__dragSamples = []);
const sample = () => { const sample = () => {
@@ -309,29 +311,36 @@ export async function physics(t) {
if (!window.__dragSamplesDone) requestAnimationFrame(sample); if (!window.__dragSamplesDone) requestAnimationFrame(sample);
}; };
window.__dragSamplesDone = false; window.__dragSamplesDone = false;
window.__dragLetGo = undefined;
window.addEventListener('pointerup', () => (window.__dragLetGo = performance.now()), {
capture: true,
once: true,
});
requestAnimationFrame(sample); requestAnimationFrame(sample);
})()`); })()`);
await sleep(1000); await sleep(1000);
await shot('41-drag-while-playing'); await shot('41-drag-while-playing');
const dragInfo = { grab, selected: (await state()).selection?.objectId }; const dragInfo = { grab, selected: (await state()).selection?.objectId };
const holdEnd = await evaluate('performance.now()');
await mouse('mouseReleased', to.x, to.y, 'left'); await mouse('mouseReleased', to.x, to.y, 'left');
const letGo = await evaluate('performance.now()');
await sleep(300); await sleep(300);
await evaluate('window.__dragSamplesDone = true'); await evaluate('window.__dragSamplesDone = true');
const dragSamples = await json('window.__dragSamples'); const dragSamples = await json('window.__dragSamples');
const letGo = await evaluate('window.__dragLetGo');
const log = (await json('__gm.world()'))?.log ?? []; const log = (await json('__gm.world()'))?.log ?? [];
await stop(); await stop();
const held = dragSamples.filter((s) => s.t < holdEnd).map((s) => s.y); const held = dragSamples.filter((s) => s.t < letGo).map((s) => s.y);
const released = dragSamples.filter((s) => s.t >= letGo); const released = dragSamples.filter((s) => s.t >= letGo);
// 200 ms of free fall from rest is under 20 below the hold, a body let go at speed is not. // 200 ms of free fall from rest is about 25 below the hold, a body let go at speed is not.
const fell = released.filter((s) => s.t - letGo <= 200).at(-1)?.y; const falling = released.filter((s) => s.t - letGo <= 200).map((s) => s.y);
const fell = falling.at(-1);
check( check(
'a body dragged while playing hangs where the pointer holds it and is let go at rest', 'a body dragged while playing hangs where the pointer holds it and is let go at rest',
arrived && arrived &&
held.length >= 30 && held.length >= 30 &&
held.every((y) => Math.abs(y - target) < 1.5) && held.every((y) => Math.abs(y - target) < 1.5) &&
Math.abs(released[0]?.y - target) < 3 && // Let go it only falls: no jump above the hold, no kick up. How far the first frame after
// the release has already fallen follows the frame time, so that is not a bound here.
falling.every((y, i) => y > target - 1.5 && (i === 0 || y >= falling[i - 1] - 0.01)) &&
fell > target + 1 && fell > target + 1 &&
fell < target + 40 && fell < target + 40 &&
!log.some((line) => line.includes('moved directly')), !log.some((line) => line.includes('moved directly')),
@@ -339,7 +348,8 @@ export async function physics(t) {
target, target,
arrived, arrived,
held: { frames: held.length, first: held[0], last: held.at(-1) }, held: { frames: held.length, first: held[0], last: held.at(-1) },
released: released.slice(0, 3).map((s) => s.y), stamped: typeof letGo === 'number',
released: released.slice(0, 3).map((s) => [Math.round(s.t - letGo), s.y]),
fell, fell,
log, log,
dragInfo, dragInfo,