From 3ac986882c46451c7270dc2fb133bea43cb2b15d Mon Sep 17 00:00:00 2001
From: quanticsoul4772 <71096528+quanticsoul4772@users.noreply.github.com>
Date: Sun, 25 Oct 2026 16:56:47 -0700
Subject: [PATCH] test(terrain): bound the 429 pause test's settle loop

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 src/tooling/environmentProviders.test.mjs | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/tooling/environmentProviders.test.mjs b/src/tooling/environmentProviders.test.mjs
index dc0bc6ab7..52dcd8351 100644
--- a/src/tooling/environmentProviders.test.mjs
+++ b/src/tooling/environmentProviders.test.mjs
@@ -451,10 +451,12 @@ test('a far-off Retry-After pauses the terrain upstream for at most 10 minutes',
     settled = true;
     return res;
   });
-  while (!settled) {
+  // Bounded: the retry budget is 10 s, so 60 simulated seconds is plenty.
+  for (let step = 0; step < 60 && !settled; step += 1) {
     await new Promise((resolve) => setImmediate(resolve));
     t.mock.timers.tick(1_000);
   }
+  assert.ok(settled, 'the first request settles within its retry budget');
   assert.equal((await first).status, 502);
   limited = false;
   const pausedAt = Date.now();