From 2965706b7edcc76b1d20b808e7cfdda0c3e56166 Mon Sep 17 00:00:00 2001 From: Enzo Marioni Date: Thu, 1 Oct 2026 18:24:08 +0200 Subject: [PATCH] Test de texture : une assertion par classe au lieu de trois par octet MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 245 760 appels à expect() (10 classes × 8 192 octets × 3) dépassaient le délai de 5 s par défaut sur le runner Gitea (1,4 s ici) ; le test tombe à 11 ms. Même vérification, même résultat. Co-Authored-By: Claude Opus 5.5 (1M context) --- .../rendering/procedural-planet-texture.spec.ts | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/src/app/shared/rendering/procedural-planet-texture.spec.ts b/src/app/shared/rendering/procedural-planet-texture.spec.ts index 4695d9d..f232882 100644 --- a/src/app/shared/rendering/procedural-planet-texture.spec.ts +++ b/src/app/shared/rendering/procedural-planet-texture.spec.ts @@ -115,11 +115,12 @@ describe('renderPlanetTexture', () => { it('produces no NaN or out-of-range bytes for any class', () => { for (const planetClass of ALL_CLASSES) { const pixels = renderPlanetTexture(appearanceOf(planetClass, { polarCapExtentDeg: 30 }), SIZE); - for (const value of pixels) { - expect(Number.isInteger(value)).toBe(true); - expect(value).toBeGreaterThanOrEqual(0); - expect(value).toBeLessThanOrEqual(255); - } + // One assertion per class, not three per byte: 245 760 expect() calls ran past the + // default 5 s timeout on a slower CI machine. + const firstBad = pixels.findIndex( + (value) => !Number.isInteger(value) || value < 0 || value > 255, + ); + expect(firstBad, planetClass).toBe(-1); } }); });