test(zoo-telemetry): align with privacy policy update - drop subscription gating expectation

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: James Mtendamema

src/services/__tests__/zoo-telemetry.test.ts

@@ -1,20 +1,11 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from "vitest"
 
-const {
-	mockCheckSubscriptionStatus,
-	mockGetCachedSubscriptionStatus,
-	mockGetCachedZooCodeToken,
-	mockGetZooCodeBaseUrl,
-} = vi.hoisted(() => ({
-	mockCheckSubscriptionStatus: vi.fn(),
-	mockGetCachedSubscriptionStatus: vi.fn(),
+const { mockGetCachedZooCodeToken, mockGetZooCodeBaseUrl } = vi.hoisted(() => ({
 	mockGetCachedZooCodeToken: vi.fn(),
 	mockGetZooCodeBaseUrl: vi.fn(),
 }))
 
 vi.mock("../zoo-code-auth", () => ({
-	checkSubscriptionStatus: mockCheckSubscriptionStatus,
-	getCachedSubscriptionStatus: mockGetCachedSubscriptionStatus,
 	getCachedZooCodeToken: mockGetCachedZooCodeToken,
 	getZooCodeBaseUrl: mockGetZooCodeBaseUrl,
 }))
@@ -52,21 +43,29 @@ describe("sendLlmTelemetry", () => {
 		expect(global.fetch).not.toHaveBeenCalled()
 	})
 
-	it("refreshes an unknown subscription status before sending", async () => {
-		mockGetCachedZooCodeToken.mockReturnValue("zoo_ext_test_token")
-		mockGetCachedSubscriptionStatus.mockReturnValue("unknown")
-		mockCheckSubscriptionStatus.mockResolvedValue("inactive")
-		global.fetch = vi.fn()
+	it("sends telemetry for authenticated users regardless of subscription tier", async () => {
+		// Privacy policy alignment: server-side retention (up to 90 days) and
+		// plan-gated dashboard visibility are enforced on zoocode.dev. The
+		// extension no longer filters telemetry by subscription status, so any
+		// authenticated user — free or paid — should reach the fetch call.
+		mockGetCachedZooCodeToken.mockReturnValue("zoo_ext_free_user_token")
+		global.fetch = vi.fn().mockResolvedValue({ ok: true })
 
 		await sendLlmTelemetry(payload)
 
-		expect(mockCheckSubscriptionStatus).toHaveBeenCalled()
-		expect(global.fetch).not.toHaveBeenCalled()
+		expect(global.fetch).toHaveBeenCalledWith(
+			"https://www.zoocode.dev/api/observability/events",
+			expect.objectContaining({
+				method: "POST",
+				headers: expect.objectContaining({
+					Authorization: "Bearer zoo_ext_free_user_token",
+				}),
+			}),
+		)
 	})
 
 	it("fires the observability request without waiting for it to settle", async () => {
 		mockGetCachedZooCodeToken.mockReturnValue("zoo_ext_test_token")
-		mockGetCachedSubscriptionStatus.mockReturnValue("active")
 
 		let resolveFetch: ((value: unknown) => void) | undefined
 		global.fetch = vi.fn(
@@ -104,7 +103,6 @@ describe("sendLlmTelemetry", () => {
 
 	it("sends cancelled status when provided in payload", async () => {
 		mockGetCachedZooCodeToken.mockReturnValue("zoo_ext_test_token")
-		mockGetCachedSubscriptionStatus.mockReturnValue("active")
 
 		global.fetch = vi.fn().mockResolvedValue({ ok: true })
 
@@ -119,7 +117,6 @@ describe("sendLlmTelemetry", () => {
 
 	it("defaults to completed status when not provided", async () => {
 		mockGetCachedZooCodeToken.mockReturnValue("zoo_ext_test_token")
-		mockGetCachedSubscriptionStatus.mockReturnValue("active")
 
 		global.fetch = vi.fn().mockResolvedValue({ ok: true })