fix(suspense): always return a promise (#75)

src/vue/__tests__/useQuery.test.ts

@@ -151,24 +151,7 @@ describe("useQuery", () => {
   });
 
   describe("suspense", () => {
-    test("should return undefined by default", () => {
-      const query = useQuery("suspense1", simpleFetcher);
-      const result = query.suspense();
-
-      expect(result).toEqual(undefined);
-    });
-
-    test("should return undefined when not used in a vue component", () => {
-      const getCurrentInstanceSpy = getCurrentInstance as jest.Mock;
-      getCurrentInstanceSpy.mockImplementation(() => null);
-
-      const query = useQuery("suspense2", simpleFetcher);
-      const result = query.suspense();
-
-      expect(result).toEqual(undefined);
-    });
-
-    test("should return a Promise when inside a Suspense boundary", () => {
+    test("should return a Promise", () => {
       const getCurrentInstanceSpy = getCurrentInstance as jest.Mock;
       getCurrentInstanceSpy.mockImplementation(() => ({ suspense: {} }));
 

src/vue/useBaseQuery.ts

@@ -5,7 +5,6 @@ import {
   ToRefs,
   reactive,
   watchEffect,
-  getCurrentInstance,
 } from "vue-demi";
 
 import type { QueryObserver } from "react-query/core";
@@ -30,7 +29,7 @@ export type UseQueryReturnType<
   TError,
   Result = UseQueryResult<TData, TError>
 > = ToRefs<Readonly<Result>> & {
-  suspense: () => Promise<Result> | void;
+  suspense: () => Promise<Result>;
 };
 
 export function useBaseQuery<TQueryFnData, TError, TData, TQueryData>(
@@ -59,12 +58,7 @@ export function useBaseQuery<TQueryFnData, TError, TData, TQueryData>(
   >;
 
   // Suspense
-  const currentInstance = getCurrentInstance();
-  // @ts-expect-error Suspense is considered experimental and not exposed
-  const isSuspense = Boolean(currentInstance?.suspense);
-  const suspense = isSuspense
-    ? () => observer.fetchOptimistic(defaultedOptions)
-    : () => undefined;
+  const suspense = () => observer.fetchOptimistic(defaultedOptions);
 
   return {
     ...resultRefs,