From feab92424aa84ced1141af00090baebc01735342 Mon Sep 17 00:00:00 2001
From: Francesco Novy <francesco.novy@sentry.io>
Date: Thu, 1 Oct 2026 11:24:04 +0200
Subject: [PATCH] test(cloudflare): Widen the wrangler readiness retry window
 in integration tests

The `cache-client` dedupe test failed on its very first request after ~5s of
500s: wrangler had reported "Ready" but could not serve the Durable Object RPC
yet. The retry budget was last widened to ~5s for the same race; give it ~15s,
and include the response body when a request unexpectedly fails so the next
occurrence shows what wrangler returned.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 dev-packages/cloudflare-integration-tests/runner.ts | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/dev-packages/cloudflare-integration-tests/runner.ts b/dev-packages/cloudflare-integration-tests/runner.ts
index 1ce7493466bd..69ab9e20fc2e 100644
--- a/dev-packages/cloudflare-integration-tests/runner.ts
+++ b/dev-packages/cloudflare-integration-tests/runner.ts
@@ -85,7 +85,7 @@ type RetryOptions = { maxRetries?: number; retryDelayMs?: number };
 async function fetchWithRetry(
   url: string,
   init: RequestInit,
-  { maxRetries = 25, retryDelayMs = 200 }: RetryOptions = {},
+  { maxRetries = 75, retryDelayMs = 200 }: RetryOptions = {},
 ): Promise<Response> {
   for (let attempt = 0; attempt < maxRetries; attempt++) {
     try {
@@ -569,7 +569,12 @@ export function createRunner(...paths: string[]) {
 
             if (!res.ok) {
               if (!expectError) {
-                reject(new Error(`Expected request to "${path}" to succeed, but got a ${res.status} response`));
+                const responseBody = await res.text().catch(() => '');
+                reject(
+                  new Error(
+                    `Expected request to "${path}" to succeed, but got a ${res.status} response: ${responseBody.slice(0, 500)}`,
+                  ),
+                );
               }
 
               return;