add tests

Author: thomaswhyyou

packages/client/test/clients/guide/guide.test.ts

@@ -288,6 +288,61 @@ describe("KnockGuideClient", () => {
       );
     });
 
+    test("stores ineligible_guides from fetch response into the store", async () => {
+      const mockIneligibleGuides = [
+        {
+          __typename: "GuideIneligibilityMarker" as const,
+          key: "guide_123",
+          reason: "marked_as_archived",
+          message: "User has archived this guide already",
+        },
+        {
+          __typename: "GuideIneligibilityMarker" as const,
+          key: "guide_456",
+          reason: "not_in_target_audience",
+          message: "User is not a member of the target audience",
+        },
+      ];
+
+      const mockResponse = {
+        entries: [
+          {
+            __typename: "Guide",
+            channel_id: channelId,
+            id: "guide_789",
+            key: "active_guide",
+            type: "test",
+            semver: "1.0.0",
+            active: true,
+            steps: [],
+            activation_url_rules: [],
+            activation_url_patterns: [],
+            bypass_global_group_limit: false,
+            inserted_at: new Date().toISOString(),
+            updated_at: new Date().toISOString(),
+          },
+        ],
+        guide_groups: [],
+        guide_group_display_logs: {},
+        ineligible_guides: mockIneligibleGuides,
+      };
+
+      vi.mocked(mockKnock.user.getGuides).mockResolvedValueOnce(mockResponse);
+
+      const client = new KnockGuideClient(
+        mockKnock,
+        channelId,
+        defaultTargetParams,
+      );
+
+      await client.fetch();
+
+      expect(client.store.state.ineligibleGuides).toEqual({
+        guide_123: mockIneligibleGuides[0],
+        guide_456: mockIneligibleGuides[1],
+      });
+    });
+
     test("handles fetch errors", async () => {
       const mockError = new Error("Network error");
       vi.mocked(mockKnock.user.getGuides).mockRejectedValueOnce(mockError);
@@ -938,7 +993,7 @@ describe("KnockGuideClient", () => {
     });
   });
 
-  describe("select", () => {
+  describe("selectGuide", () => {
     const mockStep = {
       ref: "step_1",
       schema_key: "foo",
@@ -1972,6 +2027,73 @@ describe("KnockGuideClient", () => {
       expect(result2).toBeDefined();
       expect(result2!.type).toBe("banner");
     });
+
+    test("skips ineligible guides during selection", () => {
+      const stateWithGuides = {
+        guideGroups: [mockDefaultGroup],
+        guideGroupDisplayLogs: {},
+        guides: mockGuides,
+        ineligibleGuides: {
+          feature_tour: {
+            __typename: "GuideIneligibilityMarker" as const,
+            key: "feature_tour",
+            reason: "target_conditions_not_met",
+            message: "User does not match the targeting conditions",
+          },
+        },
+        previewGuides: {},
+        queries: {},
+        location: undefined,
+        counter: 0,
+        debug: { forcedGuideKey: null, previewSessionId: null },
+      };
+
+      const client = new KnockGuideClient(mockKnock, channelId);
+      const result = client["_selectGuide"](stateWithGuides);
+
+      // feature_tour is first in display_sequence but is ineligible,
+      // so it should be skipped and onboarding should be selected next
+      expect(result!.key).toBe("onboarding");
+    });
+
+    test("skips all ineligible guides and returns undefined when all are ineligible", () => {
+      const stateWithGuides = {
+        guideGroups: [mockDefaultGroup],
+        guideGroupDisplayLogs: {},
+        guides: mockGuides,
+        ineligibleGuides: {
+          feature_tour: {
+            __typename: "GuideIneligibilityMarker" as const,
+            key: "feature_tour",
+            reason: "marked_as_archived",
+            message: "User has archived this guide already",
+          },
+          onboarding: {
+            __typename: "GuideIneligibilityMarker" as const,
+            key: "onboarding",
+            reason: "marked_as_archived",
+            message: "User has archived this guide already",
+          },
+          system_status: {
+            __typename: "GuideIneligibilityMarker" as const,
+            key: "system_status",
+            reason: "marked_as_archived",
+            message: "User has archived this guide already",
+          },
+        },
+        previewGuides: {},
+        queries: {},
+        location: undefined,
+        counter: 0,
+        debug: { forcedGuideKey: null, previewSessionId: null },
+      };
+
+      const client = new KnockGuideClient(mockKnock, channelId);
+      const result = client["_selectGuide"](stateWithGuides);
+
+      // All guides are ineligible, so should return undefined
+      expect(result).toBeUndefined();
+    });
   });
 
   describe("selectGuides", () => {