musora-content-services 2.167.2 → 2.167.4

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (140) hide show
  1. package/.agent/decisions/2026-07-20-mu2-1724-songs-lessons-counts.md +33 -0
  2. package/.agent/decisions/2026-07-20-query-builder-tostring.md +37 -0
  3. package/.idea/codeStyles/Project.xml +61 -0
  4. package/.idea/codeStyles/codeStyleConfig.xml +5 -0
  5. package/.idea/modules.xml +8 -0
  6. package/.idea/musora-content-services.iml +8 -0
  7. package/.idea/php.xml +19 -0
  8. package/.idea/prettier.xml +6 -0
  9. package/.idea/vcs.xml +7 -0
  10. package/.yarn/install-state.gz +0 -0
  11. package/CHANGELOG.md +10 -0
  12. package/package.json +2 -1
  13. package/src/contentTypeConfig.js +1 -0
  14. package/src/lib/sanity/query.ts +5 -0
  15. package/src/services/config.js +15 -0
  16. package/src/services/content/artist.ts +1 -3
  17. package/src/services/content/counts.ts +47 -0
  18. package/src/services/content/genre.ts +1 -3
  19. package/src/services/content/instructor.ts +1 -3
  20. package/src/services/multi-user-accounts/multi-user-accounts.ts +1 -0
  21. package/src/services/userActivity.js +2 -0
  22. package/src/version-info.js +1 -0
  23. package/.claude/settings.local.json +0 -53
  24. package/test/initializeTests.js +0 -60
  25. package/test/integration/awards/award-exclusion-handling.test.ts +0 -275
  26. package/test/integration/awards/internal/image-utils.test.ts +0 -86
  27. package/test/integration/content-progress/__mocks__/mocks.ts +0 -104
  28. package/test/integration/content-progress/contentProgress.test.ts +0 -335
  29. package/test/integration/content-progress/e2eOfflineProgress.test.ts +0 -352
  30. package/test/integration/content-progress/e2eProgress.test.ts +0 -612
  31. package/test/integration/content-progress/getters.test.ts +0 -334
  32. package/test/integration/content-progress/helpers.test.ts +0 -263
  33. package/test/integration/content-progress/offlineContentProgress.test.ts +0 -226
  34. package/test/integration/forums.test.ts +0 -209
  35. package/test/integration/infrastructure/FetchRequestExecutor.test.ts +0 -88
  36. package/test/integration/initializeTestDB.ts +0 -80
  37. package/test/integration/learning-paths.test.ts +0 -854
  38. package/test/integration/notifications.test.ts +0 -178
  39. package/test/integration/progress-row/content-card.test.ts +0 -283
  40. package/test/integration/sync/adapters/idb-errors.test.ts +0 -144
  41. package/test/integration/sync/adapters/sqlite-errors.test.ts +0 -173
  42. package/test/integration/sync/fetch.test.ts +0 -224
  43. package/test/integration/sync/repositories/content-likes.test.ts +0 -99
  44. package/test/integration/sync/repositories/practices.test.ts +0 -179
  45. package/test/integration/sync/repositories/progress.test.ts +0 -245
  46. package/test/integration/sync/repositories/user-award-progress.static.test.ts +0 -68
  47. package/test/integration/sync/repositories/user-award-progress.test.ts +0 -387
  48. package/test/integration/sync/store/cross-user-protection.test.ts +0 -217
  49. package/test/integration/sync/store/store-idb.test.ts +0 -180
  50. package/test/integration/sync/store/store.test.ts +0 -361
  51. package/test/integration/user/permissions.test.js +0 -20
  52. package/test/integration/userActivity.test.ts +0 -102
  53. package/test/live/README.md +0 -29
  54. package/test/live/content.test.js +0 -116
  55. package/test/localStorageMock.js +0 -21
  56. package/test/log.js +0 -5
  57. package/test/mockData/award-definitions.js +0 -171
  58. package/test/mockData/mockData_fetchByRailContentIds_one_content.json +0 -35
  59. package/test/mockData/mockData_progress_content.json +0 -182
  60. package/test/mockData/mockData_sanity_progress_content.json +0 -1451
  61. package/test/mockData/mockData_user_practices.json +0 -9
  62. package/test/setupConsole.js +0 -6
  63. package/test/setupNetworkGuard.js +0 -3
  64. package/test/setupTimers.js +0 -13
  65. package/test/unit/HttpClient.test.ts +0 -267
  66. package/test/unit/awards/award-alacarte-observer.test.ts +0 -199
  67. package/test/unit/awards/award-auto-refresh.test.ts +0 -84
  68. package/test/unit/awards/award-calculations.test.ts +0 -34
  69. package/test/unit/awards/award-callbacks.test.ts +0 -144
  70. package/test/unit/awards/award-certificate-display.test.ts +0 -330
  71. package/test/unit/awards/award-collection-edge-cases.test.ts +0 -212
  72. package/test/unit/awards/award-collection-filtering.test.ts +0 -287
  73. package/test/unit/awards/award-completion-flow.test.ts +0 -306
  74. package/test/unit/awards/award-multi-lesson.test.ts +0 -243
  75. package/test/unit/awards/award-observer-integration.test.ts +0 -327
  76. package/test/unit/awards/award-query-messages.test.ts +0 -448
  77. package/test/unit/awards/award-user-collection.test.ts +0 -416
  78. package/test/unit/awards/duplicate-prevention.test.ts +0 -120
  79. package/test/unit/awards/helpers/completion-mock.ts +0 -56
  80. package/test/unit/awards/helpers/index.ts +0 -3
  81. package/test/unit/awards/helpers/mock-setup.ts +0 -79
  82. package/test/unit/awards/helpers/progress-emitter.ts +0 -45
  83. package/test/unit/awards/message-generator.test.ts +0 -162
  84. package/test/unit/content-progress/bubbleTrickle.test.ts +0 -322
  85. package/test/unit/content-progress/helpers.test.ts +0 -329
  86. package/test/unit/content-progress/navigateTo.test.ts +0 -392
  87. package/test/unit/contentLikes.test.ts +0 -62
  88. package/test/unit/contentMetaData.test.ts +0 -58
  89. package/test/unit/dataContext.test.ts +0 -38
  90. package/test/unit/dateUtils.test.ts +0 -188
  91. package/test/unit/endScreen.test.js +0 -712
  92. package/test/unit/eventsAPI.test.ts +0 -159
  93. package/test/unit/imageSRCBuilder.test.ts +0 -37
  94. package/test/unit/imageSRCVerify.test.ts +0 -155
  95. package/test/unit/infrastructure/DefaultHeaderProvider.test.ts +0 -39
  96. package/test/unit/infrastructure/sanity/ContentClient.test.ts +0 -168
  97. package/test/unit/infrastructure/sanity/DefaultConfigProvider.test.ts +0 -93
  98. package/test/unit/infrastructure/sanity/FetchQueryExecutor.test.ts +0 -174
  99. package/test/unit/infrastructure/sanity/SanityClient.test.ts +0 -140
  100. package/test/unit/lib/__snapshots__/filter.test.ts.snap +0 -5
  101. package/test/unit/lib/filter.test.ts +0 -1154
  102. package/test/unit/lib/lastUpdated.test.ts +0 -21
  103. package/test/unit/lib/query.test.ts +0 -1469
  104. package/test/unit/lib/sanity/decorators/base.test.ts +0 -368
  105. package/test/unit/lib/sanity/decorators/navigate-to.test.ts +0 -361
  106. package/test/unit/lib/sanity/decorators/need-access.test.ts +0 -89
  107. package/test/unit/lib/sanity/decorators/need-lifetime-upgrade.test.ts +0 -85
  108. package/test/unit/lib/sanity/decorators/page-type.test.ts +0 -81
  109. package/test/unit/playerSettings.test.ts +0 -73
  110. package/test/unit/progress-row/content-card.test.ts +0 -314
  111. package/test/unit/progress-row/playlist-card.test.ts +0 -104
  112. package/test/unit/progressRows.test.ts +0 -243
  113. package/test/unit/recommendations.test.js +0 -47
  114. package/test/unit/sanityQueryService.test.ts +0 -200
  115. package/test/unit/search.test.ts +0 -44
  116. package/test/unit/sentry.test.ts +0 -62
  117. package/test/unit/services/progress-internal/bubble.test.ts +0 -279
  118. package/test/unit/services/progress.test.ts +0 -396
  119. package/test/unit/streakMessage.test.ts +0 -251
  120. package/test/unit/sync/context.test.ts +0 -51
  121. package/test/unit/sync/effects/logout-warning.test.ts +0 -158
  122. package/test/unit/sync/effects/push-failure-notification.test.ts +0 -196
  123. package/test/unit/sync/errors/sync-errors.test.ts +0 -106
  124. package/test/unit/sync/errors/validators.test.ts +0 -61
  125. package/test/unit/sync/helpers/TestModel.ts +0 -44
  126. package/test/unit/sync/helpers/index.ts +0 -172
  127. package/test/unit/sync/manager.test.ts +0 -303
  128. package/test/unit/sync/models/user-award-progress.test.ts +0 -82
  129. package/test/unit/sync/resolver.test.ts +0 -229
  130. package/test/unit/sync/retry.test.ts +0 -314
  131. package/test/unit/sync/run-scope.test.ts +0 -23
  132. package/test/unit/sync/stale-record-cleanup.test.ts +0 -107
  133. package/test/unit/sync/store/push-coalescer.test.ts +0 -156
  134. package/test/unit/sync/store-configs.test.ts +0 -37
  135. package/test/unit/sync/telemetry/sync-telemetry.test.ts +0 -118
  136. package/test/unit/sync/utils/event-emitter.test.ts +0 -64
  137. package/test/unit/sync/utils/throttle.test.ts +0 -245
  138. package/test/unit/url-builder.test.ts +0 -72
  139. package/test/unit/user/account.test.ts +0 -66
  140. package/test/unit/user/permissions.test.js +0 -191
@@ -1,226 +0,0 @@
1
- import { initializeTestDB } from '../initializeTestDB'
2
- import {
3
- contentStatusCompletedManyOffline,
4
- contentStatusCompletedOffline,
5
- contentStatusResetOffline,
6
- contentStatusStartedOffline,
7
- duplicateProgressToALaCarteOffline,
8
- recordWatchSessionOffline,
9
- } from '../../../src/services/offline/progress'
10
- import { getProgressState } from '../../../src/services/contentProgress.js'
11
- import { COLLECTION_ID_SELF, COLLECTION_TYPE } from '../../../src/services/sync/models/ContentProgress'
12
- import db from '../../../src/services/sync/repository-proxy'
13
-
14
- jest.mock('../../../src/services/sanity.js', () => require('./__mocks__/mocks').mockSanity())
15
- jest.mock('../../../src/services/content-org/learning-paths.ts', () => require('./__mocks__/mocks').mockLearningPaths())
16
- jest.mock('../../../src/services/awards/internal/content-progress-observer', () => require('./__mocks__/mocks').mockContentProgressObserver())
17
- jest.mock('../../../src/services/progress-events', () => require('./__mocks__/mocks').mockProgressEvents())
18
-
19
- jest.mock('../../../src/services/userActivity', () => ({
20
- trackUserPractice: jest.fn().mockResolvedValue(undefined),
21
- }))
22
-
23
- const meta = { brand: 'drumeo', type: 'lesson', parent_id: 0 }
24
- const collectionSelf = { type: COLLECTION_TYPE.SELF, id: COLLECTION_ID_SELF }
25
- const collectionLP = (id: number) => ({ type: COLLECTION_TYPE.LEARNING_PATH, id })
26
-
27
- const flushPromises = async () => {
28
- for (let i = 0; i < 200; i++) {
29
- await new Promise(resolve => setImmediate(resolve))
30
- }
31
- }
32
-
33
- const hierarchyFor = (contentId: number) => ({
34
- topLevelId: contentId,
35
- parents: {},
36
- children: {},
37
- metadata: { [contentId]: meta },
38
- })
39
-
40
- const ctx = initializeTestDB()
41
-
42
- // ─── contentStatusCompletedOffline ────────────────────────────────────────────
43
-
44
- describe('contentStatusCompletedOffline', () => {
45
- test('sets state to completed in DB', async () => {
46
- await contentStatusCompletedOffline(500, null, hierarchyFor(500))
47
- expect(await getProgressState(500)).toBe('completed')
48
- expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('save-content-progress')
49
- })
50
-
51
- test('sets progress_percent to 100 in DB', async () => {
52
- await contentStatusCompletedOffline(500, null, hierarchyFor(500))
53
- const record = await db.contentProgress.getOneProgressByContentId(500, null)
54
- expect(record.data?.progress_percent).toBe(100)
55
- expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('save-content-progress')
56
- })
57
-
58
- test('defaults to SELF collection when null is passed', async () => {
59
- await contentStatusCompletedOffline(500, null, hierarchyFor(500))
60
- const record = await db.contentProgress.getOneProgressByContentId(500, null)
61
- expect(record.data?.collection_type).toBe(COLLECTION_TYPE.SELF)
62
- expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('save-content-progress')
63
- })
64
- })
65
-
66
- // ─── contentStatusCompletedManyOffline ────────────────────────────────────────
67
-
68
- describe('contentStatusCompletedManyOffline', () => {
69
- test('sets all ids to completed state in DB', async () => {
70
- const hierarchy = {
71
- topLevelId: 501,
72
- parents: {},
73
- children: {},
74
- metadata: { 501: meta, 502: meta, 503: meta },
75
- }
76
- await contentStatusCompletedManyOffline([501, 502, 503], null, hierarchy)
77
- expect(await getProgressState(501)).toBe('completed')
78
- expect(await getProgressState(502)).toBe('completed')
79
- expect(await getProgressState(503)).toBe('completed')
80
- expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('save-content-progress')
81
- })
82
-
83
- test('each record has progress_percent of 100', async () => {
84
- const hierarchy = {
85
- topLevelId: 501,
86
- parents: {},
87
- children: {},
88
- metadata: { 501: meta, 502: meta },
89
- }
90
- await contentStatusCompletedManyOffline([501, 502], null, hierarchy)
91
- const r1 = await db.contentProgress.getOneProgressByContentId(501, null)
92
- const r2 = await db.contentProgress.getOneProgressByContentId(502, null)
93
- expect(r1.data?.progress_percent).toBe(100)
94
- expect(r2.data?.progress_percent).toBe(100)
95
- expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('save-content-progress')
96
- })
97
- })
98
-
99
- // ─── contentStatusStartedOffline ──────────────────────────────────────────────
100
-
101
- describe('contentStatusStartedOffline', () => {
102
- test('sets state to started in DB', async () => {
103
- await contentStatusStartedOffline(600, null, hierarchyFor(600))
104
- expect(await getProgressState(600)).toBe('started')
105
- expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('save-content-progress')
106
- })
107
-
108
- test('sets progress_percent to 0 in DB', async () => {
109
- await contentStatusStartedOffline(600, null, hierarchyFor(600))
110
- const record = await db.contentProgress.getOneProgressByContentId(600, null)
111
- expect(record.data?.progress_percent).toBe(0)
112
- expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('save-content-progress')
113
- })
114
- })
115
-
116
- // ─── contentStatusResetOffline ────────────────────────────────────────────────
117
-
118
- describe('contentStatusResetOffline', () => {
119
- test('removes record from DB so getProgressState returns empty string', async () => {
120
- await contentStatusCompletedOffline(700, null, hierarchyFor(700))
121
- expect(await getProgressState(700)).toBe('completed')
122
- await contentStatusResetOffline(700)
123
- expect(await getProgressState(700)).toBe('')
124
- expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('save-content-progress')
125
- expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('reset-status')
126
- })
127
-
128
- test('defaults to SELF collection when called without collection', async () => {
129
- await db.contentProgress.recordProgress(701, null, 80, meta, undefined, { skipPush: true })
130
- await contentStatusResetOffline(701)
131
- expect(await getProgressState(701)).toBe('')
132
- expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('reset-status')
133
- })
134
- })
135
-
136
- // ─── recordWatchSessionOffline ────────────────────────────────────────────────
137
-
138
- describe('recordWatchSessionOffline', () => {
139
- const mockUserActivity = jest.requireMock('../../../src/services/userActivity')
140
-
141
- beforeEach(() => {
142
- jest.clearAllMocks()
143
- })
144
-
145
- test('writes progress to DB and calls trackUserPractice', async () => {
146
- await recordWatchSessionOffline(234, 200, 100, 30, hierarchyFor(234))
147
- const record = await db.contentProgress.getOneProgressByContentId(234, null)
148
- expect(record.data?.progress_percent).toBe(50)
149
- expect(mockUserActivity.trackUserPractice).toHaveBeenCalledWith(234, 30, expect.any(Object))
150
- expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
151
- })
152
-
153
- test('normalizes string contentId to number in DB', async () => {
154
- await recordWatchSessionOffline('234' as any, 200, 100, 30, hierarchyFor(234))
155
- const record = await db.contentProgress.getOneProgressByContentId(234, null)
156
- expect(record.data?.content_id).toBe(234)
157
- expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
158
- })
159
-
160
- test('full duration (200/200) clamps to 99 — never reaches 100 via watch', async () => {
161
- await recordWatchSessionOffline(234, 200, 200, 60, hierarchyFor(234))
162
- const record = await db.contentProgress.getOneProgressByContentId(234, null)
163
- expect(record.data?.progress_percent).toBe(99)
164
- expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
165
- })
166
-
167
- test('passes instrumentId and categoryId through to trackUserPractice', async () => {
168
- await recordWatchSessionOffline(234, 200, 100, 30, hierarchyFor(234), {
169
- instrumentId: 7,
170
- categoryId: 9,
171
- })
172
- expect(mockUserActivity.trackUserPractice).toHaveBeenCalledWith(
173
- 234,
174
- 30,
175
- expect.objectContaining({ instrumentId: 7, categoryId: 9 }),
176
- )
177
- })
178
- })
179
-
180
- // ─── duplicateProgressToALaCarteOffline ───────────────────────────────────────
181
-
182
- describe('duplicateProgressToALaCarteOffline', () => {
183
- test('writes SELF records for a-la-carte collection', async () => {
184
- await duplicateProgressToALaCarteOffline({ 101: 50 }, { 101: meta }, collectionSelf)
185
- await flushPromises()
186
- const record = await db.contentProgress.getOneProgressByContentId(101, null)
187
- expect(record.data?.collection_type).toBe(COLLECTION_TYPE.SELF)
188
- expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
189
- })
190
-
191
- test('LP collection excludes the LP id itself from duplication', async () => {
192
- await duplicateProgressToALaCarteOffline({ 200: 50, 101: 75 }, { 200: meta, 101: meta }, collectionLP(200))
193
- await flushPromises()
194
- const lpRecord = await db.contentProgress.getOneProgressByContentId(200, null)
195
- const lessonRecord = await db.contentProgress.getOneProgressByContentId(101, null)
196
- expect(lpRecord.data).toBeNull()
197
- expect(lessonRecord.data?.progress_percent).toBe(75)
198
- expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
199
- })
200
-
201
- test('lower progress than existing is filtered — existing stays unchanged', async () => {
202
- await db.contentProgress.recordProgress(101, null, 80, meta, undefined, { skipPush: true })
203
- await duplicateProgressToALaCarteOffline({ 101: 50 }, { 101: meta }, collectionSelf)
204
- await flushPromises()
205
- const record = await db.contentProgress.getOneProgressByContentId(101, null)
206
- expect(record.data?.progress_percent).toBe(80)
207
- expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
208
- })
209
-
210
- test('equal progress passes and record is updated', async () => {
211
- await db.contentProgress.recordProgress(101, null, 50, meta, undefined, { skipPush: true })
212
- await duplicateProgressToALaCarteOffline({ 101: 50 }, { 101: meta }, collectionSelf)
213
- await flushPromises()
214
- const record = await db.contentProgress.getOneProgressByContentId(101, null)
215
- expect(record.data?.progress_percent).toBe(50)
216
- expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
217
- })
218
-
219
- test('empty progresses results in no new records', async () => {
220
- await duplicateProgressToALaCarteOffline({}, {}, collectionSelf)
221
- await flushPromises()
222
- const record = await db.contentProgress.getOneProgressByContentId(101, null)
223
- expect(record.data).toBeNull()
224
- expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
225
- })
226
- })
@@ -1,209 +0,0 @@
1
- import { getActiveDiscussions } from '@/services/forums/forums'
2
-
3
- const mockGet = jest.fn()
4
-
5
- jest.mock('@/infrastructure/http/HttpClient', () => ({
6
- HttpClient: jest.fn().mockImplementation(() => ({
7
- get: mockGet,
8
- })),
9
- }))
10
-
11
- function makeThread(overrides = {}) {
12
- return {
13
- id: 1,
14
- slug: 'test-thread',
15
- title: 'Test Thread',
16
- locked: false,
17
- pinned: false,
18
- state: 'open',
19
- category_id: 10,
20
- post_count: 1,
21
- is_read: false,
22
- author: {
23
- id: 100,
24
- display_name: 'Jane Doe',
25
- profile_picture_url: 'https://example.com/avatar.jpg',
26
- access_level: 'member',
27
- signature: null,
28
- },
29
- last_post: {
30
- id: 200,
31
- content: 'Hello world',
32
- created_at: '2024-01-01T00:00:00Z',
33
- created_at_human: '1 day ago',
34
- author: null,
35
- like_count: 0,
36
- is_liked: false,
37
- },
38
- ...overrides,
39
- }
40
- }
41
-
42
- function makePaginatedResponse(threads = [makeThread()]) {
43
- return {
44
- data: threads,
45
- meta: {
46
- current_page: 1,
47
- per_page: 10,
48
- total: threads.length,
49
- last_page: 1,
50
- },
51
- }
52
- }
53
-
54
- describe('getActiveDiscussions', () => {
55
- test('returns transformed data and meta', async () => {
56
- mockGet.mockResolvedValue(makePaginatedResponse())
57
-
58
- const result = await getActiveDiscussions('drumeo')
59
-
60
- expect(result.data).toHaveLength(1)
61
- expect(result.meta).toBeDefined()
62
- expect(result.meta.current_page).toBe(1)
63
- })
64
-
65
- test('maps thread fields correctly', async () => {
66
- mockGet.mockResolvedValue(makePaginatedResponse())
67
-
68
- const result = await getActiveDiscussions('drumeo')
69
- const thread = result.data[0]
70
-
71
- expect(thread.id).toBe(1)
72
- expect(thread.url).toBe('forums/threads/10/1')
73
- expect(thread.title).toBe('Test Thread')
74
- expect(thread.post).toBe('Hello world')
75
- expect(thread.author.id).toBe(100)
76
- expect(thread.author.name).toBe('Jane Doe')
77
- expect(thread.author.avatar).toBe('https://example.com/avatar.jpg')
78
- })
79
-
80
- test('maps author avatar to empty string when profile_picture_url is null', async () => {
81
- const thread = makeThread({
82
- author: { id: 1, display_name: 'No Avatar', profile_picture_url: null, access_level: 'member', signature: null },
83
- })
84
- mockGet.mockResolvedValue(makePaginatedResponse([thread]))
85
-
86
- const result = await getActiveDiscussions('drumeo')
87
-
88
- expect(result.data[0].author.avatar).toBe('')
89
- })
90
-
91
- test('filters out threads with null last_post', async () => {
92
- const threads = [makeThread({ last_post: null }), makeThread({ id: 2 })]
93
- mockGet.mockResolvedValue(makePaginatedResponse(threads))
94
-
95
- const result = await getActiveDiscussions('drumeo')
96
-
97
- expect(result.data).toHaveLength(1)
98
- expect(result.data[0].id).toBe(2)
99
- })
100
-
101
- test('filters out threads with null author', async () => {
102
- const threads = [makeThread({ author: null }), makeThread({ id: 2 })]
103
- mockGet.mockResolvedValue(makePaginatedResponse(threads))
104
-
105
- const result = await getActiveDiscussions('drumeo')
106
-
107
- expect(result.data).toHaveLength(1)
108
- expect(result.data[0].id).toBe(2)
109
- })
110
-
111
- test('passes brand, page and limit to the HTTP call', async () => {
112
- mockGet.mockResolvedValue(makePaginatedResponse())
113
-
114
- await getActiveDiscussions('pianote', { page: 3, limit: 25 })
115
-
116
- expect(mockGet).toHaveBeenCalledWith(
117
- '/api/forums/v1/threads/latest?brand=pianote&page=3&limit=25'
118
- )
119
- })
120
-
121
- test('defaults page to 1 and limit to 10', async () => {
122
- mockGet.mockResolvedValue(makePaginatedResponse())
123
-
124
- await getActiveDiscussions('drumeo')
125
-
126
- expect(mockGet).toHaveBeenCalledWith(
127
- '/api/forums/v1/threads/latest?brand=drumeo&page=1&limit=10'
128
- )
129
- })
130
-
131
- describe('stripHtml', () => {
132
- test('removes blockquote and its content', async () => {
133
- const thread = makeThread({
134
- last_post: { id: 1, content: 'Before<blockquote>quoted text</blockquote>After', created_at: '', created_at_human: '', author: null, like_count: 0, is_liked: false },
135
- })
136
- mockGet.mockResolvedValue(makePaginatedResponse([thread]))
137
-
138
- const result = await getActiveDiscussions('drumeo')
139
-
140
- expect(result.data[0].post).toBe('BeforeAfter')
141
- })
142
-
143
- test('removes iframes and their content', async () => {
144
- const thread = makeThread({
145
- last_post: { id: 1, content: 'Before<iframe src="x">content</iframe>After', created_at: '', created_at_human: '', author: null, like_count: 0, is_liked: false },
146
- })
147
- mockGet.mockResolvedValue(makePaginatedResponse([thread]))
148
-
149
- const result = await getActiveDiscussions('drumeo')
150
-
151
- expect(result.data[0].post).toBe('BeforeAfter')
152
- })
153
-
154
- test('converts <br> to space to prevent word concatenation', async () => {
155
- const thread = makeThread({
156
- last_post: { id: 1, content: 'word1<br>word2<br/>word3', created_at: '', created_at_human: '', author: null, like_count: 0, is_liked: false },
157
- })
158
- mockGet.mockResolvedValue(makePaginatedResponse([thread]))
159
-
160
- const result = await getActiveDiscussions('drumeo')
161
-
162
- expect(result.data[0].post).toBe('word1 word2 word3')
163
- })
164
-
165
- test('converts closing block-level tags to spaces', async () => {
166
- const thread = makeThread({
167
- last_post: { id: 1, content: '<p>word1</p><div>word2</div>', created_at: '', created_at_human: '', author: null, like_count: 0, is_liked: false },
168
- })
169
- mockGet.mockResolvedValue(makePaginatedResponse([thread]))
170
-
171
- const result = await getActiveDiscussions('drumeo')
172
-
173
- expect(result.data[0].post).toBe('word1 word2')
174
- })
175
-
176
- test('decodes HTML entities', async () => {
177
- const thread = makeThread({
178
- last_post: { id: 1, content: 'a &amp; b &lt;c&gt; &quot;d&quot; &#39;e&#39; f&nbsp;g', created_at: '', created_at_human: '', author: null, like_count: 0, is_liked: false },
179
- })
180
- mockGet.mockResolvedValue(makePaginatedResponse([thread]))
181
-
182
- const result = await getActiveDiscussions('drumeo')
183
-
184
- expect(result.data[0].post).toBe("a & b <c> \"d\" 'e' f g")
185
- })
186
-
187
- test('removes remaining HTML tags', async () => {
188
- const thread = makeThread({
189
- last_post: { id: 1, content: '<strong>bold</strong> and <em>italic</em>', created_at: '', created_at_human: '', author: null, like_count: 0, is_liked: false },
190
- })
191
- mockGet.mockResolvedValue(makePaginatedResponse([thread]))
192
-
193
- const result = await getActiveDiscussions('drumeo')
194
-
195
- expect(result.data[0].post).toBe('bold and italic')
196
- })
197
-
198
- test('replaces img tags with spaces', async () => {
199
- const thread = makeThread({
200
- last_post: { id: 1, content: 'word1<img src="x.jpg">word2', created_at: '', created_at_human: '', author: null, like_count: 0, is_liked: false },
201
- })
202
- mockGet.mockResolvedValue(makePaginatedResponse([thread]))
203
-
204
- const result = await getActiveDiscussions('drumeo')
205
-
206
- expect(result.data[0].post).toBe('word1 word2')
207
- })
208
- })
209
- })
@@ -1,88 +0,0 @@
1
- import { FetchRequestExecutor } from '../../../src/infrastructure/http'
2
- describe('FetchRequestExecutor', () => {
3
- let executor: FetchRequestExecutor
4
- beforeEach(() => {
5
- executor = new FetchRequestExecutor()
6
- })
7
- afterEach(() => {
8
- jest.restoreAllMocks()
9
- })
10
- describe('successful responses', () => {
11
- test('returns parsed JSON when content-type is application/json', async () => {
12
- const mockData = { id: 1, name: 'test' }
13
- global.fetch = jest.fn().mockResolvedValue({
14
- ok: true,
15
- headers: { get: jest.fn().mockReturnValue('application/json') },
16
- json: jest.fn().mockResolvedValue(mockData),
17
- text: jest.fn(),
18
- })
19
- const result = await executor.execute('https://api.example.com/test', { method: 'GET', headers: {} })
20
- expect(result).toEqual(mockData)
21
- })
22
- test('returns text when content-type is not application/json', async () => {
23
- global.fetch = jest.fn().mockResolvedValue({
24
- ok: true,
25
- headers: { get: jest.fn().mockReturnValue('text/plain') },
26
- json: jest.fn(),
27
- text: jest.fn().mockResolvedValue('plain text response'),
28
- })
29
- const result = await executor.execute('https://api.example.com/test', { method: 'GET', headers: {} })
30
- expect(result).toBe('plain text response')
31
- })
32
- })
33
- describe('error responses', () => {
34
- test('throws HttpError with correct shape when response is not ok', async () => {
35
- global.fetch = jest.fn().mockResolvedValue({
36
- ok: false,
37
- status: 404,
38
- statusText: 'Not Found',
39
- headers: { get: jest.fn().mockReturnValue('application/json') },
40
- json: jest.fn().mockResolvedValue({ message: 'not found' }),
41
- text: jest.fn(),
42
- })
43
- await expect(
44
- executor.execute('https://api.example.com/test', { method: 'GET', headers: {} })
45
- ).rejects.toMatchObject({
46
- status: 404,
47
- statusText: 'Not Found',
48
- url: 'https://api.example.com/test',
49
- method: 'GET',
50
- body: { message: 'not found' },
51
- })
52
- })
53
- test('error body is parsed as JSON when response body is valid JSON', async () => {
54
- const errorBody = { message: 'validation failed', code: 422 }
55
- global.fetch = jest.fn().mockResolvedValue({
56
- ok: false,
57
- status: 422,
58
- statusText: 'Unprocessable Entity',
59
- headers: { get: jest.fn() },
60
- json: jest.fn().mockResolvedValue(errorBody),
61
- text: jest.fn(),
62
- })
63
- await expect(
64
- executor.execute('https://api.example.com/test', { method: 'POST', headers: {} })
65
- ).rejects.toMatchObject({
66
- body: errorBody,
67
- })
68
- })
69
- test('error body falls back to text when response body is not JSON', async () => {
70
- global.fetch = jest.fn().mockResolvedValue({
71
- ok: false,
72
- status: 500,
73
- statusText: 'Internal Server Error',
74
- headers: { get: jest.fn() },
75
- json: jest.fn().mockRejectedValue(new Error('invalid json')),
76
- text: jest.fn().mockResolvedValue('Internal Server Error'),
77
- })
78
- await expect(
79
- executor.execute('https://api.example.com/test', { method: 'GET', headers: {} })
80
- ).rejects.toMatchObject({
81
- body: 'Internal Server Error',
82
- })
83
- })
84
- })
85
- })
86
-
87
-
88
-
@@ -1,80 +0,0 @@
1
- import { initializeTestService } from '../initializeTests.js'
2
- import SyncManager from '@/services/sync/manager'
3
- import { SyncTelemetry } from '@/services/sync/telemetry'
4
- import { makeContext, makeDatabase, makeTelemetry, makeUserScope } from '../unit/sync/helpers'
5
- import db from '@/services/sync/repository-proxy'
6
-
7
- export interface PushSpies {
8
- contentProgress: jest.SpyInstance
9
- practices: jest.SpyInstance
10
- likes: jest.SpyInstance
11
- userAwardProgress: jest.SpyInstance
12
- practiceDayNotes: jest.SpyInstance
13
- }
14
-
15
- export interface TestDBContext {
16
- pushSpies: PushSpies
17
- }
18
-
19
- export const waitForPushCall = async (
20
- spy: jest.SpyInstance,
21
- cause: string,
22
- timeoutMs = 2000,
23
- ): Promise<void> => {
24
- const start = Date.now()
25
- while (Date.now() - start < timeoutMs) {
26
- if (spy.mock.calls.some(c => c[0] === cause)) return
27
- await new Promise(resolve => setImmediate(resolve))
28
- }
29
- }
30
-
31
- export function initializeMockPushes(): PushSpies {
32
- return {
33
- contentProgress: jest.spyOn(db.contentProgress, 'requestPushUnsynced').mockImplementation(() => {
34
- }),
35
- practices: jest.spyOn(db.practices, 'requestPushUnsynced').mockImplementation(() => {
36
- }),
37
- likes: jest.spyOn(db.likes, 'requestPushUnsynced').mockImplementation(() => {
38
- }),
39
- userAwardProgress: jest.spyOn(db.userAwardProgress, 'requestPushUnsynced').mockImplementation(() => {
40
- }),
41
- practiceDayNotes: jest.spyOn(db.practiceDayNotes, 'requestPushUnsynced').mockImplementation(() => {
42
- }),
43
- }
44
- }
45
-
46
- export function initializeTestDB(): TestDBContext {
47
- const ctx: TestDBContext = { pushSpies: {} as PushSpies }
48
-
49
- beforeAll(() => {
50
- global.fetch = jest.fn().mockResolvedValue({
51
- ok: true,
52
- headers: { get: (key: string) => key === 'X-Sync-Intended-User-Id' ? '1' : null },
53
- json: () => Promise.resolve({
54
- meta: { since: null, max_updated_at: Date.now(), timestamp: Date.now() },
55
- entries: [],
56
- }),
57
- } as any)
58
- })
59
-
60
- let teardown: ((mode?: any) => Promise<void>) | null = null
61
-
62
- beforeEach(() => {
63
- initializeTestService()
64
- const userScope = makeUserScope()
65
- SyncTelemetry.setInstance(makeTelemetry(userScope))
66
- const manager = new SyncManager(userScope, makeContext(), () => makeDatabase())
67
- teardown = SyncManager.assignAndSetupInstance(manager)
68
- ctx.pushSpies = initializeMockPushes()
69
- })
70
-
71
- afterEach(async () => {
72
- for (let i = 0; i < 200; i++) await new Promise(resolve => setImmediate(resolve))
73
- await teardown?.('reset')
74
- teardown = null
75
- SyncTelemetry.clearInstance()
76
- ;(SyncManager as any).instance = null
77
- })
78
-
79
- return ctx
80
- }