Add test + changeset

Author: brophdawg11

.changeset/six-ways-carry.md

@@ -0,0 +1,5 @@
+---
+"react-router": patch
+---
+
+Preserve `statusText` on the `ErrorResponse` instance when throwing `data()` from a route handler

packages/react-router/__tests__/router/router-test.ts

@@ -5,7 +5,7 @@ import type {
   AgnosticDataRouteObject,
   AgnosticRouteObject,
 } from "../../lib/router/utils";
-import { ErrorResponseImpl } from "../../lib/router/utils";
+import { data, ErrorResponseImpl } from "../../lib/router/utils";
 
 import { urlMatch } from "./utils/custom-matchers";
 import {
@@ -1915,6 +1915,33 @@ describe("a router", () => {
       });
     });
 
+    it("handles thrown data() values as ErrorResponse's", async () => {
+      let t = setup({
+        routes: TASK_ROUTES,
+        initialEntries: ["/"],
+        hydrationData: {
+          loaderData: {
+            root: "ROOT_DATA",
+            index: "INDEX_DATA",
+          },
+        },
+      });
+
+      let nav = await t.navigate("/tasks");
+      await nav.loaders.tasks.reject(
+        data("broken", { status: 400, statusText: "Bad Request" }),
+      );
+      expect(t.router.state).toMatchObject({
+        navigation: IDLE_NAVIGATION,
+        loaderData: {
+          root: "ROOT_DATA",
+        },
+        errors: {
+          tasks: new ErrorResponseImpl(400, "Bad Request", "broken"),
+        },
+      });
+    });
+
     it("sends proper arguments to actions", async () => {
       let t = setup({
         routes: TASK_ROUTES,

packages/react-router/lib/router/router.ts

@@ -6212,11 +6212,7 @@ async function convertDataStrategyResultToDataResult(
       // Convert thrown data() to ErrorResponse instances
       return {
         type: ResultType.error,
-        error: new ErrorResponseImpl(
-          result.init?.status || 500,
-          result.init?.statusText,
-          result.data,
-        ),
+        error: dataWithResponseInitToErrorResponse(result),
         statusCode: isRouteErrorResponse(result) ? result.status : undefined,
         headers: result.init?.headers
           ? new Headers(result.init.headers)