Include subpath to the URL during downloading question results (#33856)

* Include subpath to the URL during downloading question results

* Fix test

* Simplify logic, drop tests

* Fix test

* Use api.basename

frontend/src/metabase/query_builder/actions/downloading.ts

 import { t } from "ttag";
 import _ from "underscore";
 import * as Urls from "metabase/lib/urls";
+import api from "metabase/lib/api";
 import { getCardKey } from "metabase/visualizations/lib/utils";
 import { saveChartImage } from "metabase/visualizations/lib/save-chart-image";
 import type {
@@ -132,10 +133,12 @@ const getDatasetResponse = ({
   method,
   params,
 }: DownloadQueryResultsParams) => {
+  const requestUrl = new URL(api.basename + url, location.origin);
+
   if (method === "POST") {
-    return fetch(url, { method, body: params });
+    return fetch(requestUrl.href, { method, body: params });
   } else {
-    return fetch(`${url}?${params}`);
+    return fetch(`${requestUrl.href}?${params}`);
   }
 };
 

frontend/src/metabase/query_builder/containers/QueryBuilder.unit.spec.tsx

@@ -490,7 +490,7 @@ describe("QueryBuilder", () => {
     });
 
     it("should allow downloading results for a native query using the current result even the query has changed but not rerun (metabase#28834)", async () => {
-      const mockDownloadEndpoint = fetchMock.post("/api/dataset/csv", {});
+      const mockDownloadEndpoint = fetchMock.post("path:/api/dataset/csv", {});
       await setup({
         card: TEST_NATIVE_CARD,
         dataset: TEST_NATIVE_CARD_DATASET,
@@ -517,8 +517,10 @@ describe("QueryBuilder", () => {
             urlSearchParams instanceof URLSearchParams
               ? JSON.parse(urlSearchParams.get("query") ?? "{}")
               : {};
+
           return (
-            url === "/api/dataset/csv" && query?.native.query === "SELECT 1"
+            url.includes("/api/dataset/csv") &&
+            query?.native.query === "SELECT 1"
           );
         }),
       ).toBe(true);