fix cache is part of query

Author: hannosgit

packages/cubejs-client-core/src/index.ts

@@ -50,10 +50,6 @@ export type LoadMethodOptions = {
    * AbortSignal to cancel requests
    */
   signal?: AbortSignal;
-  /**
-   * Cache mode for query execution
-   */
-  cache?: CacheMode;
 };
 
 export type DeeplyReadonly<T> = {
@@ -112,6 +108,10 @@ export type CubeSqlOptions = LoadMethodOptions & {
    * Query timeout in milliseconds
    */
   timeout?: number;
+  /**
+   * Cache mode for query execution
+   */
+  cache?: CacheMode;
 };
 
 export type CubeSqlSchemaColumn = {
@@ -574,7 +574,6 @@ class CubeApi {
       () => this.request('load', {
         query,
         queryType: 'multi',
-        cache: options?.cache,
         signal: options?.signal,
       }),
       (response: any) => this.loadResponseInternal(response, options),

packages/cubejs-client-core/src/types.ts

@@ -115,6 +115,7 @@ export interface Query {
   timezone?: string;
   // @deprecated
   renewQuery?: boolean;
+  cache?: CacheMode;
   ungrouped?: boolean;
   responseFormat?: 'compact' | 'default';
   total?: boolean;

packages/cubejs-client-core/test/CubeApi.test.ts

@@ -217,30 +217,6 @@ describe('CubeApi with Abort Signal', () => {
     expect(requestSpy.mock.calls[0]?.[1]?.signal).toBe(signal);
   });
 
-  test('should pass cache from options to request', async () => {
-    // Mock for this specific test
-    const requestSpy = jest.spyOn(HttpTransport.prototype, 'request').mockImplementation(() => ({
-      subscribe: (cb) => Promise.resolve(cb({
-        status: 200,
-        text: () => Promise.resolve('{"results":[]}'),
-        json: () => Promise.resolve({ results: [] })
-      } as any,
-      async () => undefined as any))
-    }));
-
-    const cubeApi = new CubeApi('token', {
-      apiUrl: 'http://localhost:4000/cubejs-api/v1'
-    });
-
-    await cubeApi.load(
-      { measures: ['Orders.count'] },
-      { cache: 'stale-if-slow' }
-    );
-
-    expect(requestSpy).toHaveBeenCalled();
-    expect(requestSpy.mock.calls[0]?.[1]?.cache).toBe('stale-if-slow');
-  });
-
   test('options signal should override constructor signal', async () => {
     const constructorController = new AbortController();
     const optionsController = new AbortController();