@terreno/rtk 0.26.0 → 0.27.0

package/src/useFeatureFlags.ts

@@ -7,6 +7,7 @@ interface FlagValues {
   [key: string]: boolean | string | null;
 }
 
+// noExplicitAny: RTK Query API generic typing is intentionally flexible here.
 // biome-ignore lint/suspicious/noExplicitAny: RTK Query API generic typing is intentionally flexible here.
 type FlagsApi = Api<any, any, any, any>;
 

package/src/useTerrenoFeatureFlags.test.ts

@@ -104,6 +104,22 @@ describe("useTerrenoFeatureFlags", () => {
     expect(result.current.error).toBeDefined();
   });
 
+  it("reports loading while the initial flag configuration query has no data yet", () => {
+    const {api} = buildApi({isLoading: true, isSuccess: false});
+    const {result} = renderHook(() =>
+      useTerrenoFeatureFlags(api as never, {domain: "feature-flags", userId: "u1"})
+    );
+    expect(result.current.isLoading).toBe(true);
+  });
+
+  it("reports loading while a refetch is in flight even after data has loaded", () => {
+    const {api} = buildApi({data: {alpha: boolDef("on")}, isFetching: true, isSuccess: true});
+    const {result} = renderHook(() =>
+      useTerrenoFeatureFlags(api as never, {domain: "feature-flags", userId: "u1"})
+    );
+    expect(result.current.isLoading).toBe(true);
+  });
+
   it("isolates domains so a custom domain does not read another domain's provider", async () => {
     const {api} = buildApi({data: {alpha: boolDef("on")}});
     const {result} = renderHook(() =>

package/src/useTerrenoFeatureFlags.ts

@@ -34,9 +34,11 @@ export interface UseTerrenoFeatureFlagsResult {
   refetch: () => unknown;
 }
 
+// noExplicitAny: RTK Query API generic typing is intentionally flexible here.
 // biome-ignore lint/suspicious/noExplicitAny: RTK Query API generic typing is intentionally flexible here.
 type FlagsApi = Api<any, any, any, any>;
 
+// noExplicitAny: Endpoint hook is injected dynamically by RTK Query.
 // biome-ignore lint/suspicious/noExplicitAny: Endpoint hook is injected dynamically by RTK Query.
 type EnhancedTerrenoFlagsApi = FlagsApi & {useTerrenoFlagConfigurationQuery: any};
 

package/src/useUpgradeCheckNative.test.ts

@@ -6,6 +6,7 @@ import React from "react";
 import {Provider} from "react-redux";
 
 const openedUrls: string[] = [];
+let openUrlShouldReject = false;
 
 // Keep this react-native mock a superset of the preload's mock so other test
 // files still resolve AppState / Linking / Platform after this file runs.
@@ -16,6 +17,9 @@ mock.module("react-native", () => ({
   },
   Linking: {
     openURL: async (url: string) => {
+      if (openUrlShouldReject) {
+        throw new Error("openURL failed");
+      }
       openedUrls.push(url);
       return true;
     },
@@ -74,6 +78,7 @@ describe("useUpgradeCheck (native)", () => {
   beforeEach(() => {
     originalFetch = globalThis.fetch;
     openedUrls.length = 0;
+    openUrlShouldReject = false;
     constantsWithExtra.expoConfig.extra.buildNumber = 100;
   });
 
@@ -103,6 +108,27 @@ describe("useUpgradeCheck (native)", () => {
     unmount();
   });
 
+  it("swallows errors when opening the update URL fails on native", async () => {
+    mockFetchWith({status: "required", updateUrl: "https://example.com/app-update"});
+    openUrlShouldReject = true;
+
+    const {result, unmount} = renderHook(() => useUpgradeCheck(), {
+      wrapper: createWrapper(createTestStore()),
+    });
+    await flush();
+
+    expect(result.current.canUpdate).toBe(true);
+
+    await act(async () => {
+      result.current.onUpdate();
+      await new Promise((resolve) => setTimeout(resolve, 10));
+    });
+
+    // The rejection is caught internally; no URL is recorded and nothing throws.
+    expect(openedUrls).toEqual([]);
+    unmount();
+  });
+
   it("does not open a URL on native when no update URL is available", async () => {
     mockFetchWith({status: "required"});