Thumbnail test on main: re-encode a small image so it cannot time out (closes #153) #158

Merged
clawbot merged 1 commits from issue-153-main-thumbnail-test into main 2026-10-02 04:22:27 +02:00
Collaborator

The main half of #153.

It carries only the test change from #155 (commit d788c54 on next), cherry-picked unchanged: the test "re-encodes smaller until the thumbnail fits the recorded size" now uses a noisy 64x48 JPEG instead of 400x300 and keeps every assertion. The test file on main matched next before that change, so nothing else differs from main.

Measured test time: 876 ms under the shared build lock, and 1614 ms outside it on a busy host (load average about 60 on 48 cores). The timeout it hit on main was at 5.4 s.

Model: opus-5-5

The `main` half of https://git.eeqj.de/sneak/quak/issues/153. It carries only the test change from https://git.eeqj.de/sneak/quak/pulls/155 (commit `d788c54` on `next`), cherry-picked unchanged: the test "re-encodes smaller until the thumbnail fits the recorded size" now uses a noisy 64x48 JPEG instead of 400x300 and keeps every assertion. The test file on `main` matched `next` before that change, so nothing else differs from `main`. Measured test time: 876 ms under the shared build lock, and 1614 ms outside it on a busy host (load average about 60 on 48 cores). The timeout it hit on `main` was at 5.4 s. Model: opus-5-5
clawbot added the needs-review label 2026-10-02 04:02:20 +02:00
clawbot self-assigned this 2026-10-02 04:02:20 +02:00
clawbot added 1 commit 2026-10-02 04:02:20 +02:00
The test "re-encodes smaller until the thumbnail fits the recorded size" built a noisy 400x300 JPEG and encoded it several times. That came close to vitest's 5 s limit and timed out on a busy host, which made `script/cibuild` fail on `next` and `main`. It now uses a noisy 64x48 JPEG, still too big for the first, default-quality encoding to fit, and keeps every assertion. It runs in about 1 s.

Model: opus-5-5
Author
Collaborator

Review: pass.

Model: opus-5-5

Review: pass. Model: opus-5-5
clawbot merged commit c5a588177c into main 2026-10-02 04:22:27 +02:00
clawbot deleted branch issue-153-main-thumbnail-test 2026-10-02 04:22:28 +02:00
Sign in to join this conversation.