fix: improve the fetch handling (and testing) of requests to the Clou…

…dflare API

.changeset/modern-books-leave.md

@@ -0,0 +1,17 @@
+---
+"wrangler": patch
+---
+
+Refactor raw value extraction from Cloudflare APIs
+
+Most API responses are JSON of the form:
+
+```
+{ result, success, errors, messages, result_info }
+```
+
+where the `result` contains the actual response value.
+
+But some API responses only contain the result value.
+
+This change refactors the client-side fetch API to allow callers to specify what kind of response they expect.

.changeset/thick-beans-thank.md

@@ -0,0 +1,9 @@
+---
+"wrangler": patch
+---
+
+Fix pagination handling of list requests to the Cloudflare API
+
+When doing a list request to the API, the server may respond with only a single page of results.
+In this case, it will also provide a `cursor` value in the `result_info` part of the response, which can be used to request the next page.
+This change implements this on the client-side so that we get all the results by requesting further pages when there is a cursor.

packages/wrangler/src/__tests__/jest.setup.ts

@@ -1,6 +1,10 @@
+import { mockFetchInternal } from "./mock-cfetch";
 import { confirm, prompt } from "../dialogs";
+import { fetchInternal } from "../cfetch/internal";
+
+jest.mock("../cfetch/internal");
+(fetchInternal as jest.Mock).mockImplementation(mockFetchInternal);
 
-jest.mock("../cfetch", () => jest.requireActual("./mock-cfetch"));
 jest.mock("../dialogs");
 
 // By default (if not configured by mockConfirm()) calls to `confirm()` should throw.