fathom-multi-key.test.ts

apps/dashboard/test/fathom-multi-key.test.ts

1. /**
2. * Several keys, and what happens when one of them is dark.
3. *
4. * Fathom has no org keys; each key returns its owner's recordings plus
5. * whatever Fathom shares with that user. Adding keys buys coverage, never
6. * gating (config/fathom-keys.ts).
7. *
8. * With more than one key there are two ways to lie, and both break the prime
9. * invariant:
10. *
11. * - all-or-nothing failure. One revoked key blanks meetings for everyone,
12. * throwing away the other keys' real data to report one.
13. * - a swallowed per-key failure, which is worse. A key that cannot reach
14. * someone's meetings then looks exactly like a person who recorded none,
15. * and one of those two means free capacity.
16. *
17. * So the rule asserted here: a partial read stays partial. The data survives,
18. * and every surface that shows it says which key is dark, by owner (report
19. * what happened, name no cause).
20. */
22. import { describe, expect, it } from 'vitest'
23. import { FIXTURE_SNAPSHOT } from '../src/fixtures.js'
24. import { headline, peopleBoard, type Snapshot, type FathomSourceStatus } from '../src/collect.js'
25. import { sourceStatuses, myDay, personViews } from '../src/view-model.js'
26. import { render } from '../src/render.js'
27. import { viewerFor } from '../src/viewer.js'
28. import { FATHOM_KEYS } from '../../../config/fathom-keys.js'
30. /**
31. * `mode: 'live'` because sourceStatuses() deliberately overrides every pill to
32. * "sample data" on a fixture snapshot, so a fixture never renders `live` pills
33. * over frozen data. These tests are about which live state the
34. * Fathom pill reaches, so they have to be on the live side of that override.
35. */
36. const snap: Snapshot = { ...FIXTURE_SNAPSHOT, mode: 'live', fixtureCapturedAt: null }
37. const withSources = (fathomSources: FathomSourceStatus[]): Snapshot => ({ ...snap, fathomSources })
38. const fathomPill = (s: Snapshot) => sourceStatuses(s).find((r) => r.name === 'Fathom')!
40. const DARK_INES: FathomSourceStatus = {
41. owner: 'Ines Carvalho',
42. ok: false,
43. error: 'HTTP 401',
44. returned: 0,
45. truncated: false,
46. }
48. describe('the config still says what this suite assumes', () => {
49. it('lists Ines and Theo alongside Sam', () => {
50. const owners = FATHOM_KEYS.map((k) => k.owner)
51. expect(owners).toContain('Sam Whitaker')
52. expect(owners).toContain('Ines Carvalho')
53. expect(owners).toContain('Theo Bassett')
54. })
56. it('is a hand-written list, not a loop over the roster', () => {
57. // Four people have handed over no key. If this ever equals the roster
58. // length, somebody has generated the list and the deliberate gap is gone.
59. expect(FATHOM_KEYS.length).toBe(3)
60. })
62. it('names an environment variable per owner and never an inline secret', () => {
63. for (const k of FATHOM_KEYS) {
64. expect(k.secret).toMatch(/^FATHOM_KEY_[A-Z]+$/)
65. }
66. })
67. })
69. describe('every key clean', () => {
70. it('reads live and names the keys it read', () => {
71. const pill = fathomPill(snap)
72. expect(pill.status).toBe('live')
73. expect(pill.tone).toBe('live')
74. expect(pill.note).toContain('Ines Carvalho')
75. expect(pill.note).toContain('Theo Bassett')
76. })
78. it('says a person at zero meetings is a floor, not a fact', () => {
79. // A gap that stands in for knowledge is surfaced every run (rule 2).
80. expect(fathomPill(snap).note).toMatch(/floor, not a fact/)
81. })
82. })
84. describe('one dark key degrades the source without erasing it', () => {
85. const partial = withSources([
86. { owner: 'Sam Whitaker', ok: true, error: null, returned: 8, truncated: false },
87. DARK_INES,
88. { owner: 'Theo Bassett', ok: true, error: null, returned: 2, truncated: false },
89. ])
91. it('is partial, not live and not failed', () => {
92. expect(fathomPill(partial).status).toBe('partial')
93. expect(fathomPill(partial).tone).toBe('degraded')
94. })
96. it('names the owner whose key failed, and the failure verbatim', () => {
97. const note = fathomPill(partial).note
98. expect(note).toContain('Ines Carvalho')
99. expect(note).toContain('HTTP 401')
100. })
102. it('states no cause for the failure', () => {
103. // The 401 handler once said a token had "most likely expired" about a token
104. // with eleven months left. Report the status; do not pick a reason.
105. expect(fathomPill(partial).note).not.toMatch(/expired|revoked|most likely/i)
106. })
108. it('keeps every meeting the working keys returned', () => {
109. // The whole point. Two keys' data must not be discarded to report a third.
110. expect(partial.meetings.ok).toBe(true)
111. expect(partial.meetings.data).toEqual(snap.meetings.data)
112. })
113. })
115. describe('a truncated sweep is reported, not silently short', () => {
116. it('reads partial and says the window is incomplete', () => {
117. // Keep what was fetched AND say it was capped.
118. const capped = withSources([
119. { owner: 'Sam Whitaker', ok: true, error: null, returned: 400, truncated: true },
120. { owner: 'Ines Carvalho', ok: true, error: null, returned: 3, truncated: false },
121. { owner: 'Theo Bassett', ok: true, error: null, returned: 2, truncated: false },
122. ])
123. expect(fathomPill(capped).status).toBe('partial')
124. expect(fathomPill(capped).note).toMatch(/page cap/)
125. expect(fathomPill(capped).note).toContain('Sam Whitaker')
126. })
127. })
129. describe("a person's own card says a key is dark", () => {
130. const partial = withSources([
131. { owner: 'Sam Whitaker', ok: true, error: null, returned: 8, truncated: false },
132. DARK_INES,
133. { owner: 'Theo Bassett', ok: true, error: null, returned: 2, truncated: false },
134. ])
135. const viewFor = (s: Snapshot, name: string) =>
136. personViews(peopleBoard(s), s.tasks.data).find((v) => v.name === name)!
138. it('sets commitmentsPartial when a key failed', () => {
139. const d = myDay(partial, viewFor(partial, 'Sam Whitaker'), viewerFor('hello@meridiandisplay.example'))
140. expect(d.commitmentsTracked).toBe(true)
141. expect(d.commitmentsPartial).toContain('Ines Carvalho')
142. })
144. it('leaves it null when every key is clean', () => {
145. const d = myDay(snap, viewFor(snap, 'Sam Whitaker'), viewerFor('hello@meridiandisplay.example'))
146. expect(d.commitmentsPartial).toBeNull()
147. })
149. it('reaches the rendered page, not just the view-model', () => {
150. // A card that is short by one key looks exactly like a light fortnight, and
151. // nobody scrolls to the source band to find out which. The finished document
152. // is the only thing that proves the sentence is actually on it.
153. const html = render(
154. partial,
155. headline(partial),
156. [],
157. '2026-08-12',
158. null,
159. viewerFor('hello@meridiandisplay.example'),
160. )
161. expect(html).toContain('Ines Carvalho')
162. expect(html).toMatch(/could not be read this run/)
163. })
164. })