savant-code 0.0.19 → 0.0.20

package/cli/src/hooks/__tests__/use-activity-query.test.ts

@@ -1,810 +0,0 @@
-import { describe, test, expect, beforeEach, afterEach } from 'bun:test'
-
-import {
-  invalidateActivityQuery,
-  removeActivityQuery,
-  getActivityQueryData,
-  setActivityQueryData,
-  resetActivityQueryCache,
-  isEntryStale,
-  setErrorOnlyCacheEntry,
-  _retryTestHelpers,
-} from '../use-activity-query'
-describe('use-activity-query utilities', () => {
-  beforeEach(() => {
-    // Reset cache between tests
-    resetActivityQueryCache()
-  })
-  describe('setActivityQueryData', () => {
-    test('stores data in cache', () => {
-      setActivityQueryData(['test'], { value: 'hello' })
-      expect(
-        getActivityQueryData<{
-          value: string
-        }>(['test']),
-      ).toEqual({ value: 'hello' })
-    })
-    test('overwrites existing data', () => {
-      setActivityQueryData(['test'], { value: 'first' })
-      setActivityQueryData(['test'], { value: 'second' })
-      expect(
-        getActivityQueryData<{
-          value: string
-        }>(['test']),
-      ).toEqual({ value: 'second' })
-    })
-    test('handles complex query keys', () => {
-      setActivityQueryData(['users', 1], { name: 'John' })
-      expect(
-        getActivityQueryData<{
-          name: string
-        }>(['users', 1]),
-      ).toEqual({ name: 'John' })
-    })
-    test('handles query keys with objects', () => {
-      setActivityQueryData(['complex', { id: 1 }], { data: 'test' })
-      expect(
-        getActivityQueryData<{
-          data: string
-        }>(['complex', { id: 1 }]),
-      ).toEqual({
-        data: 'test',
-      })
-    })
-    test('different keys store different data', () => {
-      setActivityQueryData(['key1'], 'value1')
-      setActivityQueryData(['key2'], 'value2')
-      expect(getActivityQueryData<string>(['key1'])).toBe('value1')
-      expect(getActivityQueryData<string>(['key2'])).toBe('value2')
-    })
-  })
-  describe('getActivityQueryData', () => {
-    test('returns undefined for non-existent key', () => {
-      expect(getActivityQueryData(['nonexistent'])).toBeUndefined()
-    })
-    test('returns stored data for existing key', () => {
-      setActivityQueryData(['test'], 42)
-      expect(getActivityQueryData<number>(['test'])).toBe(42)
-    })
-    test('returns correct type', () => {
-      setActivityQueryData<string[]>(['test'], ['a', 'b', 'c'])
-      const data = getActivityQueryData<string[]>(['test'])
-      expect(data).toEqual(['a', 'b', 'c'])
-    })
-  })
-  describe('removeActivityQuery', () => {
-    test('removes existing cache entry', () => {
-      setActivityQueryData(['test'], 'value')
-      expect(getActivityQueryData<string>(['test'])).toBe('value')
-      removeActivityQuery(['test'])
-      expect(getActivityQueryData(['test'])).toBeUndefined()
-    })
-    test('does nothing for non-existent key', () => {
-      // Should not throw
-      removeActivityQuery(['nonexistent'])
-      expect(getActivityQueryData(['nonexistent'])).toBeUndefined()
-    })
-    test('only removes specified key', () => {
-      setActivityQueryData(['key1'], 'value1')
-      setActivityQueryData(['key2'], 'value2')
-      removeActivityQuery(['key1'])
-      expect(getActivityQueryData(['key1'])).toBeUndefined()
-      expect(getActivityQueryData<string>(['key2'])).toBe('value2')
-    })
-  })
-  describe('invalidateActivityQuery', () => {
-    test('marks query as stale by setting dataUpdatedAt to 0', () => {
-      setActivityQueryData(['test'], 'value')
-      // Before invalidation, data exists
-      expect(getActivityQueryData<string>(['test'])).toBe('value')
-      invalidateActivityQuery(['test'])
-      // Data should still exist after invalidation
-      expect(getActivityQueryData<string>(['test'])).toBe('value')
-    })
-    test('does nothing for non-existent key', () => {
-      // Should not throw
-      invalidateActivityQuery(['nonexistent'])
-    })
-  })
-  describe('query key serialization', () => {
-    test('same array values produce same cache key', () => {
-      setActivityQueryData(['test', 'key'], 'value')
-      expect(getActivityQueryData<string>(['test', 'key'])).toBe('value')
-    })
-    test('different array values produce different cache keys', () => {
-      setActivityQueryData(['test', 'key1'], 'value1')
-      setActivityQueryData(['test', 'key2'], 'value2')
-      expect(getActivityQueryData<string>(['test', 'key1'])).toBe('value1')
-      expect(getActivityQueryData<string>(['test', 'key2'])).toBe('value2')
-    })
-    test('object keys are serialized correctly', () => {
-      setActivityQueryData(['query', { page: 1, sort: 'asc' }], 'page1')
-      expect(
-        getActivityQueryData<string>(['query', { page: 1, sort: 'asc' }]),
-      ).toBe('page1')
-    })
-    test('nested objects in keys work correctly', () => {
-      setActivityQueryData(
-        ['query', { filter: { status: 'active', type: 'user' } }],
-        'filtered',
-      )
-      expect(
-        getActivityQueryData<string>([
-          'query',
-          { filter: { status: 'active', type: 'user' } },
-        ]),
-      ).toBe('filtered')
-    })
-  })
-})
-describe('useActivityQuery hook behavior', () => {
-  // These tests verify the hook's expected behavior patterns
-  // We can't easily test the actual hook without a React renderer,
-  // but we can test the underlying cache behavior
-  describe('cache entry structure', () => {
-    test('setActivityQueryData creates proper cache entry', () => {
-      const testData = { users: [1, 2, 3] }
-      setActivityQueryData(['users'], testData)
-      const retrieved = getActivityQueryData<typeof testData>(['users'])
-      expect(retrieved).toEqual(testData)
-    })
-    test('cache preserves data types', () => {
-      // Numbers
-      setActivityQueryData(['number'], 42)
-      expect(getActivityQueryData<number>(['number'])).toBe(42)
-      // Strings
-      setActivityQueryData(['string'], 'hello')
-      expect(getActivityQueryData<string>(['string'])).toBe('hello')
-      // Booleans
-      setActivityQueryData(['boolean'], true)
-      expect(getActivityQueryData<boolean>(['boolean'])).toBe(true)
-      // Arrays
-      setActivityQueryData(['array'], [1, 2, 3])
-      expect(getActivityQueryData<number[]>(['array'])).toEqual([1, 2, 3])
-      // Objects
-      setActivityQueryData(['object'], { a: 1, b: 2 })
-      expect(
-        getActivityQueryData<{
-          a: number
-          b: number
-        }>(['object']),
-      ).toEqual({ a: 1, b: 2 })
-      // Null
-      setActivityQueryData(['null'], null)
-      expect(getActivityQueryData<null>(['null'])).toBeNull()
-    })
-  })
-  describe('invalidation behavior', () => {
-    test('invalidation preserves existing data', () => {
-      const originalData = { id: 1, name: 'Test' }
-      setActivityQueryData(['preserve'], originalData)
-      invalidateActivityQuery(['preserve'])
-      // Data should still be accessible
-      expect(getActivityQueryData<typeof originalData>(['preserve'])).toEqual(
-        originalData,
-      )
-    })
-    test('multiple invalidations do not remove data', () => {
-      setActivityQueryData(['multi'], 'persistent')
-      invalidateActivityQuery(['multi'])
-      invalidateActivityQuery(['multi'])
-      invalidateActivityQuery(['multi'])
-      expect(getActivityQueryData<string>(['multi'])).toBe('persistent')
-    })
-  })
-  describe('remove behavior', () => {
-    test('remove completely clears the cache entry', () => {
-      setActivityQueryData(['remove-test'], 'data')
-      expect(getActivityQueryData<string>(['remove-test'])).toBe('data')
-      removeActivityQuery(['remove-test'])
-      expect(getActivityQueryData(['remove-test'])).toBeUndefined()
-      // Can set new data after removal
-      setActivityQueryData(['remove-test'], 'new-data')
-      expect(getActivityQueryData<string>(['remove-test'])).toBe('new-data')
-    })
-  })
-  describe('resetActivityQueryCache', () => {
-    test('clears all cache entries', () => {
-      setActivityQueryData(['key1'], 'value1')
-      setActivityQueryData(['key2'], 'value2')
-      setActivityQueryData(['key3'], 'value3')
-      expect(getActivityQueryData<string>(['key1'])).toBe('value1')
-      expect(getActivityQueryData<string>(['key2'])).toBe('value2')
-      expect(getActivityQueryData<string>(['key3'])).toBe('value3')
-      resetActivityQueryCache()
-      expect(getActivityQueryData(['key1'])).toBeUndefined()
-      expect(getActivityQueryData(['key2'])).toBeUndefined()
-      expect(getActivityQueryData(['key3'])).toBeUndefined()
-    })
-    test('allows setting new data after reset', () => {
-      setActivityQueryData(['test'], 'old')
-      resetActivityQueryCache()
-      setActivityQueryData(['test'], 'new')
-      expect(getActivityQueryData<string>(['test'])).toBe('new')
-    })
-  })
-})
-describe('staleness calculation', () => {
-  beforeEach(() => {
-    resetActivityQueryCache()
-  })
-  test('data is considered stale after staleTime has passed', () => {
-    const _staleTime = 100 // 100ms
-    const testKey = ['stale-test']
-    // Set data with a timestamp in the past
-    setActivityQueryData(testKey, 'test-value')
-    // Immediately after setting, data should be fresh
-    const dataImmediately = getActivityQueryData<string>(testKey)
-    expect(dataImmediately).toBe('test-value')
-  })
-  test('invalidated data should be refetchable', () => {
-    const testKey = ['invalidate-test']
-    // Set initial data
-    setActivityQueryData(testKey, 'initial')
-    expect(getActivityQueryData<string>(testKey)).toBe('initial')
-    // Invalidate - should mark as stale (dataUpdatedAt = 0)
-    invalidateActivityQuery(testKey)
-    // Data should still exist but be stale
-    expect(getActivityQueryData<string>(testKey)).toBe('initial')
-  })
-})
-describe('refetch interval staleness bug fix', () => {
-  // This test verifies the fix for the bug where refetch intervals stopped working
-  // because isStale was captured in a closure and never updated.
-  // The fix ensures staleness is computed dynamically by reading from cache.
-  beforeEach(() => {
-    resetActivityQueryCache()
-  })
-  test('setActivityQueryData sets dataUpdatedAt to current time', () => {
-    const _before = Date.now()
-    setActivityQueryData(['timing-test'], 'value')
-    const _after = Date.now()
-    // The data should exist
-    expect(getActivityQueryData<string>(['timing-test'])).toBe('value')
-    // We can't directly access dataUpdatedAt, but we can verify the data was set
-    // and invalidation resets it to 0
-    invalidateActivityQuery(['timing-test'])
-    // Data should still exist after invalidation
-    expect(getActivityQueryData<string>(['timing-test'])).toBe('value')
-  })
-  test('fresh data followed by stale time passage should allow refetch', () => {
-    // This simulates the scenario where:
-    // 1. Data is fetched (fresh)
-    // 2. staleTime passes
-    // 3. Interval should refetch (was broken before fix)
-    const testKey = ['refetch-bug-test']
-    // Step 1: Set "fresh" data
-    setActivityQueryData(testKey, 'fresh-data')
-    expect(getActivityQueryData<string>(testKey)).toBe('fresh-data')
-    // Step 2: Invalidate to simulate staleness (sets dataUpdatedAt to 0)
-    invalidateActivityQuery(testKey)
-    // The data should still exist but be considered stale
-    // (dataUpdatedAt is 0, so any staleTime > 0 would make it stale)
-    expect(getActivityQueryData<string>(testKey)).toBe('fresh-data')
-    // In the old buggy code, the interval tick would check closure-captured isStale
-    // which was false (computed when effect ran right after fetch).
-    // In the fixed code, staleness is computed dynamically from cache.
-    // We can't easily test the hook behavior without React, but we verify
-    // the cache manipulation works correctly for the staleness check
-  })
-  test('multiple data updates preserve latest data', () => {
-    const testKey = ['multi-update-test']
-    setActivityQueryData(testKey, 'first')
-    expect(getActivityQueryData<string>(testKey)).toBe('first')
-    setActivityQueryData(testKey, 'second')
-    expect(getActivityQueryData<string>(testKey)).toBe('second')
-    setActivityQueryData(testKey, 'third')
-    expect(getActivityQueryData<string>(testKey)).toBe('third')
-    // Invalidate and verify data is preserved
-    invalidateActivityQuery(testKey)
-    expect(getActivityQueryData<string>(testKey)).toBe('third')
-  })
-})
-/**
- * Tests for cache listener notification behavior.
- * These tests verify that cache updates properly notify subscribers,
- * which is critical for React components to re-render when data changes.
- */
-describe('cache listener notifications', () => {
-  beforeEach(() => {
-    resetActivityQueryCache()
-  })
-  test('setActivityQueryData notifies listeners', () => {
-    const testKey = ['listener-test']
-    let _notificationCount = 0
-    // First set up some data so the cache entry exists
-    setActivityQueryData(testKey, 'initial')
-    // Now update the data - we can't directly subscribe but we can verify
-    // the data is updated properly
-    setActivityQueryData(testKey, 'updated')
-    expect(getActivityQueryData<string>(testKey)).toBe('updated')
-  })
-  test('invalidateActivityQuery notifies listeners', () => {
-    const testKey = ['invalidate-listener-test']
-    // Set initial data
-    setActivityQueryData(testKey, 'data')
-    // Invalidate should trigger listeners
-    invalidateActivityQuery(testKey)
-    // Data should still be there but marked stale
-    expect(getActivityQueryData<string>(testKey)).toBe('data')
-  })
-  test('removeActivityQuery clears data and notifies listeners', () => {
-    const testKey = ['remove-listener-test']
-    setActivityQueryData(testKey, 'data')
-    expect(getActivityQueryData<string>(testKey)).toBe('data')
-    removeActivityQuery(testKey)
-    expect(getActivityQueryData<string>(testKey)).toBeUndefined()
-  })
-})
-/**
- * Tests simulating the polling behavior to verify refetch intervals work.
- * These tests mock Date.now() to simulate time passing.
- */
-describe('polling and staleness simulation', () => {
-  let originalDateNow: typeof Date.now
-  let mockNow: number
-  beforeEach(() => {
-    resetActivityQueryCache()
-    originalDateNow = Date.now
-    mockNow = 1000000
-    Date.now = () => mockNow
-  })
-  afterEach(() => {
-    Date.now = originalDateNow
-  })
-  test('data becomes stale after staleTime passes', () => {
-    const testKey = ['stale-time-test']
-    const serializedKey = JSON.stringify(testKey)
-    const staleTime = 30000 // 30 seconds
-    // Set data at t=0
-    setActivityQueryData(testKey, 'fresh-data')
-    // Data was set at mockNow (1000000), so dataUpdatedAt = 1000000
-    expect(getActivityQueryData<string>(testKey)).toBe('fresh-data')
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false) // Fresh
-    // Advance time by 25 seconds - still fresh
-    mockNow += 25000
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    // Advance time past staleTime
-    mockNow += 10000 // Now 35 seconds have passed
-    // Data should now be considered stale (35s > 30s staleTime)
-    expect(isEntryStale(serializedKey, staleTime)).toBe(true)
-    // The data is still accessible even when stale
-    expect(getActivityQueryData<string>(testKey)).toBe('fresh-data')
-  })
-  test('invalidated data is immediately stale', () => {
-    const testKey = ['invalidate-stale-test']
-    const serializedKey = JSON.stringify(testKey)
-    const staleTime = 30000
-    // Set fresh data
-    setActivityQueryData(testKey, 'data')
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    // Invalidate immediately makes it stale (dataUpdatedAt = 0)
-    invalidateActivityQuery(testKey)
-    expect(isEntryStale(serializedKey, staleTime)).toBe(true)
-    // Data still exists but would be refetched on next access
-    expect(getActivityQueryData<string>(testKey)).toBe('data')
-  })
-  test('updating data resets the staleness timer', () => {
-    const testKey = ['reset-timer-test']
-    const serializedKey = JSON.stringify(testKey)
-    const staleTime = 30000
-    // Set initial data
-    setActivityQueryData(testKey, 'initial')
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    // Advance time past staleTime
-    mockNow += 35000
-    expect(isEntryStale(serializedKey, staleTime)).toBe(true)
-    // Update data - should reset the timer
-    setActivityQueryData(testKey, 'updated')
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false) // Fresh again
-    // Data is fresh again
-    expect(getActivityQueryData<string>(testKey)).toBe('updated')
-    // Advance a little bit - should still be fresh
-    mockNow += 10000
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    expect(getActivityQueryData<string>(testKey)).toBe('updated')
-  })
-  test('staleTime of 0 means always stale', () => {
-    const testKey = ['zero-stale-test']
-    const serializedKey = JSON.stringify(testKey)
-    // Set data
-    setActivityQueryData(testKey, 'data')
-    // With staleTime=0, data is always considered stale
-    // (this means refetch should happen on every interval tick)
-    expect(isEntryStale(serializedKey, 0)).toBe(true)
-    expect(getActivityQueryData<string>(testKey)).toBe('data')
-  })
-  test('non-existent key is always stale', () => {
-    const serializedKey = JSON.stringify(['non-existent'])
-    expect(isEntryStale(serializedKey, 30000)).toBe(true)
-  })
-})
-/**
- * Tests for the refetch on activity feature.
- * Verifies that data is refetched when user becomes active after being idle.
- */
-describe('refetch on activity behavior', () => {
-  let originalDateNow: typeof Date.now
-  let mockNow: number
-  beforeEach(() => {
-    resetActivityQueryCache()
-    originalDateNow = Date.now
-    mockNow = 1000000
-    Date.now = () => mockNow
-  })
-  afterEach(() => {
-    Date.now = originalDateNow
-  })
-  test('data should be refetchable when user becomes active after idle', () => {
-    const testKey = ['activity-refetch-test']
-    const _staleTime = 30000
-    const _idleThreshold = 30000
-    // Set initial data
-    setActivityQueryData(testKey, 'initial')
-    // Simulate time passing beyond staleTime
-    mockNow += 35000
-    // At this point, if user was idle and becomes active,
-    // and data is stale, a refetch should occur
-    // Data should still be accessible
-    expect(getActivityQueryData<string>(testKey)).toBe('initial')
-    // Update with new data (simulating what refetch would do)
-    setActivityQueryData(testKey, 'refetched')
-    expect(getActivityQueryData<string>(testKey)).toBe('refetched')
-  })
-  test('pause when idle should prevent polling updates', () => {
-    const testKey = ['pause-idle-test']
-    // Set data
-    setActivityQueryData(testKey, 'before-idle')
-    // When pauseWhenIdle is true and user is idle:
-    // - Polling interval fires
-    // - But isUserActive returns false
-    // - So no refetch happens
-    // Data remains unchanged
-    expect(getActivityQueryData<string>(testKey)).toBe('before-idle')
-  })
-})
-/**
- * Tests for edge cases and error scenarios in the caching system.
- */
-describe('cache edge cases and error handling', () => {
-  beforeEach(() => {
-    resetActivityQueryCache()
-  })
-  test('setting undefined data should still create cache entry', () => {
-    const testKey = ['undefined-test']
-    setActivityQueryData(testKey, undefined)
-    // getActivityQueryData returns undefined for both "not in cache" and "data is undefined"
-    // This is expected behavior - undefined is a valid cached value
-    expect(getActivityQueryData(testKey)).toBeUndefined()
-  })
-  test('setting null data should store null', () => {
-    const testKey = ['null-test']
-    setActivityQueryData(testKey, null)
-    expect(getActivityQueryData(testKey)).toBeNull()
-  })
-  test('complex nested objects should be stored correctly', () => {
-    const testKey = ['complex-object-test']
-    const complexData = {
-      user: {
-        id: 1,
-        profile: {
-          name: 'Test',
-          settings: {
-            theme: 'dark',
-            notifications: [1, 2, 3],
-          },
-        },
-      },
-      timestamp: new Date('2024-01-01'),
-    }
-    setActivityQueryData(testKey, complexData)
-    const cached = getActivityQueryData<typeof complexData>(testKey)
-    expect(cached?.user.profile.settings.notifications).toEqual([1, 2, 3])
-    expect(cached?.timestamp).toEqual(new Date('2024-01-01'))
-  })
-  test('array data should be stored and retrieved correctly', () => {
-    const testKey = ['array-test']
-    const arrayData = [1, 2, 3, { nested: 'value' }]
-    setActivityQueryData(testKey, arrayData)
-    const cached = getActivityQueryData<typeof arrayData>(testKey)
-    expect(cached).toEqual(arrayData)
-    expect(cached?.[3]).toEqual({ nested: 'value' })
-  })
-  test('invalidating non-existent key should not throw', () => {
-    expect(() => {
-      invalidateActivityQuery(['non-existent-key'])
-    }).not.toThrow()
-  })
-  test('removing non-existent key should not throw', () => {
-    expect(() => {
-      removeActivityQuery(['non-existent-key'])
-    }).not.toThrow()
-  })
-  test('getting data after remove should return undefined', () => {
-    const testKey = ['remove-then-get-test']
-    setActivityQueryData(testKey, 'data')
-    removeActivityQuery(testKey)
-    expect(getActivityQueryData(testKey)).toBeUndefined()
-  })
-  test('setting data after remove should work', () => {
-    const testKey = ['remove-then-set-test']
-    setActivityQueryData(testKey, 'first')
-    removeActivityQuery(testKey)
-    setActivityQueryData(testKey, 'second')
-    expect(getActivityQueryData<string>(testKey)).toBe('second')
-  })
-})
-/**
- * Tests for error-only cache entries and persistent error scenarios.
- * This test suite was added to debug and fix an issue where fetchSubscriptionData
- * was being called every second when the endpoint returned errors.
- */
-describe('error-only entries and persistent error handling', () => {
-  let originalDateNow: typeof Date.now
-  let mockNow: number
-  beforeEach(() => {
-    resetActivityQueryCache()
-    originalDateNow = Date.now
-    mockNow = 1000000
-    Date.now = () => mockNow
-  })
-  afterEach(() => {
-    Date.now = originalDateNow
-  })
-  test('setErrorOnlyCacheEntry creates entry with no data and error', () => {
-    const testKey = ['error-entry-test']
-    const error = new Error('Network error')
-    setErrorOnlyCacheEntry(testKey, error)
-    // Data should be undefined (error-only entry)
-    expect(getActivityQueryData(testKey)).toBeUndefined()
-  })
-  test('error-only entry with recent errorUpdatedAt should NOT be stale', () => {
-    // This test verifies the fix for the infinite refetch loop bug.
-    //
-    // Scenario:
-    // 1. Fetch fails with no prior data
-    // 2. Error is stored with errorUpdatedAt = now
-    // 3. Polling tick fires
-    // 4. isEntryStale should return FALSE if errorUpdatedAt is recent
-    // 5. This prevents immediate refetch loop
-    const testKey = ['error-only-fresh-test']
-    const serializedKey = JSON.stringify(testKey)
-    const staleTime = 30000 // 30 seconds
-    const error = new Error('API error')
-    // Create error-only entry at current time (mockNow = 1000000)
-    setErrorOnlyCacheEntry(testKey, error, mockNow)
-    // Entry has errorUpdatedAt = 1000000, current time = 1000000
-    // Time since error: 0ms, staleTime: 30000ms
-    // Should NOT be stale because error is recent
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-  })
-  test('error-only entry becomes stale after staleTime passes', () => {
-    const testKey = ['error-stale-after-time-test']
-    const serializedKey = JSON.stringify(testKey)
-    const staleTime = 30000 // 30 seconds
-    const error = new Error('API error')
-    // Create error-only entry at current time
-    setErrorOnlyCacheEntry(testKey, error, mockNow)
-    // Initially not stale
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    // Advance time by 25 seconds - still fresh
-    mockNow += 25000
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    // Advance time past staleTime (now 35 seconds since error)
-    mockNow += 10000
-    expect(isEntryStale(serializedKey, staleTime)).toBe(true)
-  })
-  test('simulates subscription query polling with persistent errors', () => {
-    // This test simulates the exact bug scenario:
-    // - useSubscriptionQuery with staleTime=30s, refetchInterval=60s
-    // - Endpoint returns errors
-    // - Without fix: isEntryStale returns true immediately, causing rapid refetches
-    // - With fix: isEntryStale uses errorUpdatedAt, preventing rapid refetches
-    const subscriptionKey = ['subscription', 'current']
-    const serializedKey = JSON.stringify(subscriptionKey)
-    const staleTime = 30000 // 30 seconds (matches useSubscriptionQuery)
-    const _refetchInterval = 60000 // 60 seconds
-    const error = new Error('Failed to fetch subscription: 500')
-    // Simulate first fetch failure at t=0
-    setErrorOnlyCacheEntry(subscriptionKey, error, mockNow)
-    // Immediately after error, entry should NOT be stale
-    // This is the critical fix - prevents immediate refetch loop
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    // Simulate polling interval at t=1s (as reported in bug)
-    mockNow += 1000
-    // Entry should still NOT be stale (only 1s since error, staleTime is 30s)
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    // Simulate many 1-second intervals - none should trigger refetch until staleTime
-    for (let i = 0; i < 28; i++) {
-      mockNow += 1000
-      expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    }
-    // Now at t=29s - should still be fresh (29s is not > 30s)
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    // At t=30s - should still be fresh (30s is not > 30s, need strictly greater)
-    mockNow += 1000
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false)
-    // At t=31s - now stale, refetch should be allowed (31s > 30s)
-    mockNow += 1000
-    expect(isEntryStale(serializedKey, staleTime)).toBe(true)
-  })
-  test('staleTime of 0 means always stale even for error-only entries', () => {
-    const testKey = ['zero-stale-error-test']
-    const serializedKey = JSON.stringify(testKey)
-    const error = new Error('Some error')
-    setErrorOnlyCacheEntry(testKey, error, mockNow)
-    // With staleTime=0, entry is always considered stale
-    expect(isEntryStale(serializedKey, 0)).toBe(true)
-  })
-  test('error-only entry with null errorUpdatedAt is stale', () => {
-    // Edge case: if somehow errorUpdatedAt is null, entry should be stale
-    // This shouldn't happen in practice but tests defensive coding
-    const testKey = ['null-error-time-test']
-    const serializedKey = JSON.stringify(testKey)
-    const staleTime = 30000
-    // Create entry without errorUpdatedAt (using undefined which gets stored as null)
-    // Note: setErrorOnlyCacheEntry always sets errorUpdatedAt, so we test via regular data
-    // and then invalidate it
-    // Non-existent key is stale
-    expect(isEntryStale(serializedKey, staleTime)).toBe(true)
-  })
-  test('successful data takes precedence over errorUpdatedAt for staleness', () => {
-    const testKey = ['data-precedence-test']
-    const serializedKey = JSON.stringify(testKey)
-    const staleTime = 30000
-    // First, set an error-only entry
-    setErrorOnlyCacheEntry(testKey, new Error('Initial error'), mockNow)
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false) // Fresh error
-    // Now set successful data (this is what happens on successful retry)
-    setActivityQueryData(testKey, { subscription: 'active' })
-    // Staleness should now be based on dataUpdatedAt, not errorUpdatedAt
-    expect(isEntryStale(serializedKey, staleTime)).toBe(false) // Fresh data
-    // Advance time past staleTime
-    mockNow += 35000
-    expect(isEntryStale(serializedKey, staleTime)).toBe(true) // Stale based on dataUpdatedAt
-  })
-})
-/**
- * Tests for the retry infinite loop bug.
- *
- * BUG: When useSubscriptionQuery fetched /api/user/subscription and got a 401,
- * it would retry every ~1 second infinitely instead of respecting retry:1.
- *
- * ROOT CAUSE: In doFetch's catch block, when scheduling a retry:
- *   1. retryCounts.set(key, next)   // Sets count to 1
- *   2. clearRetryState(key)          // Deletes retryCounts → count back to 0!
- *   3. setTimeout to retry in 1s
- * When the retry fires, currentRetries reads as 0 again → thinks it still has
- * retries left → schedules another retry → infinite loop.
- *
- * FIX: Split clearRetryState into clearRetryTimeout (only clears timeout)
- * and clearRetryState (clears both). The retry scheduling block now uses
- * clearRetryTimeout so the retry count is preserved.
- */
-describe('retry infinite loop bug fix (subscription 401 scenario)', () => {
-  beforeEach(() => {
-    resetActivityQueryCache()
-  })
-  test('retry count is preserved after scheduling a retry', () => {
-    const queryKey = ['subscription', 'current']
-    const maxRetries = 1
-    // Simulate a mounted component (refCount > 0)
-    _retryTestHelpers.setRefCount(queryKey, 1)
-    // Initially, no retries have been attempted
-    expect(_retryTestHelpers.getRetryCount(queryKey)).toBe(0)
-    // First fetch fails → should schedule a retry
-    const result1 = _retryTestHelpers.simulateFailedFetch(queryKey, maxRetries)
-    expect(result1.retryScheduled).toBe(true)
-    expect(result1.retryCount).toBe(1)
-    // CRITICAL: Retry count must be preserved (not reset to 0)
-    expect(_retryTestHelpers.getRetryCount(queryKey)).toBe(1)
-  })
-  test('retries are exhausted after maxRetries attempts', () => {
-    const queryKey = ['subscription', 'current']
-    const maxRetries = 1
-    _retryTestHelpers.setRefCount(queryKey, 1)
-    // First fetch fails → retry scheduled (count becomes 1)
-    const result1 = _retryTestHelpers.simulateFailedFetch(queryKey, maxRetries)
-    expect(result1.retryScheduled).toBe(true)
-    expect(result1.retryCount).toBe(1)
-    // Retry fires, also fails → retries exhausted (count = 1, not < maxRetries=1)
-    const result2 = _retryTestHelpers.simulateFailedFetch(queryKey, maxRetries)
-    expect(result2.retryScheduled).toBe(false)
-    expect(result2.retryCount).toBe(0) // Reset after exhaustion
-  })
-  test('simulates full subscription 401 scenario: fetch + 1 retry + stop', () => {
-    // This reproduces the exact bug scenario:
-    // useSubscriptionQuery with retry:1 hitting a 401 on /api/user/subscription
-    const queryKey = ['subscription', 'current']
-    const maxRetries = 1
-    // Component is mounted
-    _retryTestHelpers.setRefCount(queryKey, 1)
-    // === Fetch #1: Initial fetch fails with 401 ===
-    const fetch1 = _retryTestHelpers.simulateFailedFetch(queryKey, maxRetries)
-    expect(fetch1.retryScheduled).toBe(true)
-    expect(fetch1.retryCount).toBe(1)
-    // === Fetch #2: Retry fires after 1s, also fails with 401 ===
-    const fetch2 = _retryTestHelpers.simulateFailedFetch(queryKey, maxRetries)
-    expect(fetch2.retryScheduled).toBe(false) // Retries exhausted!
-    expect(fetch2.retryCount).toBe(0)
-    // === Fetch #3: If the bug existed, this would schedule ANOTHER retry ===
-    // With the fix, the error is stored and no more retries are scheduled.
-    // A third call should also exhaust immediately since count was reset to 0
-    // BUT there's no retry scheduled, so this would only happen from polling.
-    const fetch3 = _retryTestHelpers.simulateFailedFetch(queryKey, maxRetries)
-    // Even if polling triggers another fetch, retry:1 means ONE retry per fetch cycle
-    expect(fetch3.retryScheduled).toBe(true) // New fetch cycle starts fresh
-    expect(fetch3.retryCount).toBe(1)
-    // The retry for fetch3 fires and fails
-    const fetch4 = _retryTestHelpers.simulateFailedFetch(queryKey, maxRetries)
-    expect(fetch4.retryScheduled).toBe(false) // Exhausted again
-  })
-  test('demonstrates the old bug: clearRetryState would reset count causing infinite loop', () => {
-    // This test documents the OLD buggy behavior.
-    // The old code called clearRetryState (which deletes retryCounts) right after
-    // setting the retry count, effectively resetting it to 0 every time.
-    const queryKey = ['subscription', 'current']
-    _retryTestHelpers.setRefCount(queryKey, 1)
-    // Step 1: Simulate first fetch failure setting retry count to 1
-    _retryTestHelpers.setRetryCount(queryKey, 1)
-    expect(_retryTestHelpers.getRetryCount(queryKey)).toBe(1)
-    // Step 2: OLD CODE would call clearRetryState here, which resets count to 0:
-    // clearRetryState(key) → retryCounts.delete(key) → count = 0
-    // Simulate the old bug by manually resetting:
-    _retryTestHelpers.setRetryCount(queryKey, 0)
-    expect(_retryTestHelpers.getRetryCount(queryKey)).toBe(0)
-    // Step 3: When the retry fires after 1s, it reads count as 0
-    // 0 < maxRetries(1) → true → schedules ANOTHER retry (should have been exhausted!)
-    const result = _retryTestHelpers.simulateFailedFetch(queryKey, 1)
-    expect(result.retryScheduled).toBe(true) // BUG: should have been false!
-    expect(result.retryCount).toBe(1) // Count set to 1 again...
-    // And the cycle repeats: count gets reset → retry fires → count is 0 → retry...
-    // With the fix (clearRetryTimeout instead of clearRetryState), count stays at 1
-    // so the next attempt correctly sees 1 >= maxRetries(1) → exhausted.
-  })
-  test('retry count resets to 0 when retries are exhausted', () => {
-    const queryKey = ['retry-reset-test']
-    const maxRetries = 2
-    _retryTestHelpers.setRefCount(queryKey, 1)
-    // First fail → retry scheduled, count=1
-    const r1 = _retryTestHelpers.simulateFailedFetch(queryKey, maxRetries)
-    expect(r1).toEqual({ retryScheduled: true, retryCount: 1 })
-    // Second fail → retry scheduled, count=2
-    const r2 = _retryTestHelpers.simulateFailedFetch(queryKey, maxRetries)
-    expect(r2).toEqual({ retryScheduled: true, retryCount: 2 })
-    // Third fail → retries exhausted, count reset to 0
-    const r3 = _retryTestHelpers.simulateFailedFetch(queryKey, maxRetries)
-    expect(r3).toEqual({ retryScheduled: false, retryCount: 0 })
-  })
-  test('no retries when retry is 0 or false', () => {
-    const queryKey = ['no-retry-test']
-    _retryTestHelpers.setRefCount(queryKey, 1)
-    // retry: 0 (equivalent to retry: false)
-    const result = _retryTestHelpers.simulateFailedFetch(queryKey, 0)
-    expect(result.retryScheduled).toBe(false)
-    expect(result.retryCount).toBe(0)
-  })
-  test('no retries when component is unmounted (refCount=0)', () => {
-    const queryKey = ['unmounted-test']
-    // Don't set refCount (defaults to 0 = no mounted components)
-    const result = _retryTestHelpers.simulateFailedFetch(queryKey, 1)
-    expect(result.retryScheduled).toBe(false)
-  })
-  test('error-only entry is created after retries exhausted', () => {
-    const queryKey = ['error-entry-after-retry']
-    _retryTestHelpers.setRefCount(queryKey, 1)
-    // First fail → retry
-    _retryTestHelpers.simulateFailedFetch(queryKey, 1)
-    // No cache entry yet during retry phase
-    expect(getActivityQueryData(queryKey)).toBeUndefined()
-    // Second fail → exhausted, error entry created
-    _retryTestHelpers.simulateFailedFetch(queryKey, 1)
-    // Error entry should exist (data is undefined but entry exists)
-    // The entry has error set, which we can verify via isEntryStale behavior
-    const serializedKey = JSON.stringify(queryKey)
-    // Entry exists (not stale due to "no entry" - stale due to other reasons)
-    // Since we just set errorUpdatedAt = Date.now(), it should not be stale
-    // for a reasonable staleTime
-    expect(isEntryStale(serializedKey, 30000)).toBe(false)
-  })
-})