musora-content-services 2.167.2 → 2.167.3

package/test/integration/awards/internal/image-utils.test.ts

@@ -1,86 +0,0 @@
-/**
- * @jest-environment jsdom
- */
-import { urlToBase64, urlMapToBase64 } from '../../../../src/services/awards/internal/image-utils.js'
-describe('urlToBase64', () => {
-  test('returns empty string when url is falsy', async () => {
-    const result = await urlToBase64('')
-    expect(result).toBe('')
-  })
-  test('returns empty string when fetch response is not ok', async () => {
-    global.fetch = jest.fn().mockResolvedValue({
-      ok: false,
-      blob: jest.fn(),
-    })
-    const result = await urlToBase64('https://cdn.example.com/image.png')
-    expect(result).toBe('')
-  })
-  test('returns base64 data string on successful fetch', async () => {
-    global.fetch = jest.fn().mockResolvedValue({
-      ok: true,
-      blob: jest.fn().mockResolvedValue(new Blob(['fake-image-data'], { type: 'image/png' })),
-    })
-    jest.spyOn(global, 'FileReader').mockImplementation(() => ({
-      readAsDataURL: jest.fn().mockImplementation(function(this: any) {
-        this.result = 'data:image/png;base64,abc123=='
-        this.onloadend()
-      }),
-      onerror: null,
-      onloadend: null,
-      result: null,
-    } as any))
-    const result = await urlToBase64('https://cdn.example.com/image.png')
-    expect(result).toBe('abc123==')
-  })
-  // BUG: onerror path uses reject() inside new Promise() which escapes the outer try/catch.
-  // The function contract says it should return '' on failure but instead rejects.
-  // Fix: change reject() to resolve('') in the onerror handler in image-utils.js
-  test('returns empty string when FileReader errors', async () => {
-    jest.spyOn(console, 'error').mockImplementation(() => {})
-    global.fetch = jest.fn().mockResolvedValue({
-      ok: true,
-      blob: jest.fn().mockResolvedValue(new Blob(['fake-image-data'], { type: 'image/png' })),
-    })
-    jest.spyOn(global, 'FileReader').mockImplementation(() => ({
-      readAsDataURL: jest.fn().mockImplementation(function(this: any) {
-        setTimeout(() => this.onerror(), 0)
-      }),
-      onerror: null,
-      onloadend: null,
-      result: null,
-    } as any))
-    await expect(urlToBase64('https://cdn.example.com/image.png')).rejects.toThrow(
-      'Failed to convert image to base64'
-    )
-  })
-})
-
-describe('urlMapToBase64', () => {
-  test('converts all URLs in a map to base64', async () => {
-    global.fetch = jest.fn().mockResolvedValue({
-      ok: true,
-      blob: jest.fn().mockResolvedValue(new Blob(['fake-image-data'], { type: 'image/png' })),
-    })
-    jest.spyOn(global, 'FileReader').mockImplementation(() => ({
-      readAsDataURL: jest.fn().mockImplementation(function(this: any) {
-        this.result = 'data:image/png;base64,abc123=='
-        this.onloadend()
-      }),
-      onerror: null,
-      onloadend: null,
-      result: null,
-    } as any))
-    const result = await urlMapToBase64({
-      badge: 'https://cdn.example.com/badge.png',
-      logo: 'https://cdn.example.com/logo.png',
-    })
-    expect(result).toEqual({
-      badge: 'abc123==',
-      logo: 'abc123==',
-    })
-  })
-  test('returns empty object when map is empty', async () => {
-    const result = await urlMapToBase64({})
-    expect(result).toEqual({})
-  })
-})

package/test/integration/content-progress/__mocks__/mocks.ts

@@ -1,104 +0,0 @@
-export const mockContentProgressObserver = () => ({
-  contentProgressObserver: {
-    start: jest.fn().mockResolvedValue(undefined),
-    stop: jest.fn(),
-  },
-})
-
-export const mockLearningPaths = () => ({
-  getDailySession: jest.fn().mockResolvedValue(null),
-  onLearningPathCompletedActions: jest.fn().mockResolvedValue(undefined),
-})
-
-export const mockProgressEvents = () => ({
-  emitProgressSaved: jest.fn(),
-})
-
-export type HierarchyTreeNode = {
-  id: number
-  type?: string
-  brand?: string
-  children?: HierarchyTreeNode[]
-}
-
-type HierarchyLookups = {
-  topLevelId: number
-  parents: Record<number, number>
-  children: Record<number, number[]>
-  metadata: Record<number, { type: string; brand: string; parent_id: number }>
-}
-
-const LP_TYPE = 'learning-path-v2'
-
-const ALC_KEY = 'alc'
-const LP_KEY = 'lp'
-
-let hierarchiesByKey: Record<string, Record<number, HierarchyLookups>> = {}
-let topByKey: Record<string, Record<number, number>> = {}
-
-function keyFor(collection?: { type?: string } | null): string {
-  return collection?.type === LP_TYPE ? LP_KEY : ALC_KEY
-}
-
-export function setHierarchy(tree: HierarchyTreeNode, options?: { lp?: boolean }) {
-  const key = options?.lp ? LP_KEY : ALC_KEY
-  hierarchiesByKey[key] ??= {}
-  topByKey[key] ??= {}
-  hierarchiesByKey[key][tree.id] = treeToLookups(tree)
-  registerIds(tree, tree.id, topByKey[key])
-}
-
-export function clearHierarchies() {
-  hierarchiesByKey = {}
-  topByKey = {}
-}
-
-function treeToLookups(root: HierarchyTreeNode): HierarchyLookups {
-  const parents: Record<number, number> = {}
-  const children: Record<number, number[]> = {}
-  const metadata: Record<number, { type: string; brand: string; parent_id: number }> = {}
-
-  function walk(node: HierarchyTreeNode, parentId: number) {
-    const childIds = (node.children ?? []).map(c => c.id)
-    metadata[node.id] = {
-      type: node.type ?? 'lesson',
-      brand: node.brand ?? 'drumeo',
-      parent_id: parentId,
-    }
-    if (parentId) parents[node.id] = parentId
-    if (childIds.length > 0) children[node.id] = childIds
-    for (const c of node.children ?? []) walk(c, node.id)
-  }
-
-  walk(root, 0)
-
-  return { topLevelId: root.id, parents, children, metadata }
-}
-
-function registerIds(node: HierarchyTreeNode, topId: number, target: Record<number, number>) {
-  target[node.id] = topId
-  for (const c of node.children ?? []) registerIds(c, topId, target)
-}
-
-function lookupFor(contentId: number, collection?: any): HierarchyLookups {
-  const key = keyFor(collection)
-  const tops = topByKey[key] ?? {}
-  const hierarchies = hierarchiesByKey[key] ?? {}
-  const topId = tops[contentId] ?? (key === LP_KEY ? collection?.id : contentId)
-  return hierarchies[topId] ?? {
-    topLevelId: contentId,
-    parents: {},
-    children: {},
-    metadata: { [contentId]: { brand: 'drumeo', type: 'lesson', parent_id: 0 } },
-  }
-}
-
-export const mockSanity = () => ({
-  getHierarchy: jest.fn((contentId: number, collection?: any) =>
-    Promise.resolve(lookupFor(contentId, collection)),
-  ),
-  getHierarchies: jest.fn((contentIds: number[], collection?: any) =>
-    Promise.resolve(Object.fromEntries(contentIds.map(id => [id, lookupFor(id, collection)]))),
-  ),
-  getSanityDate: jest.fn((date: Date) => date.toISOString()),
-})

package/test/integration/content-progress/contentProgress.test.ts

@@ -1,335 +0,0 @@
-import { initializeTestDB } from '../initializeTestDB'
-import {
-  _recordWatchSession,
-  contentStatusCompleted,
-  contentStatusCompletedMany,
-  contentStatusReset,
-  contentStatusStarted,
-  flushWatchSession,
-  getProgressState,
-  recordWatchSession,
-  resetStatus,
-  saveContentProgress,
-  setStartedOrCompletedStatus,
-  setStartedOrCompletedStatusMany,
-  trackProgress,
-} from '../../../src/services/contentProgress.js'
-import { COLLECTION_ID_SELF, COLLECTION_TYPE } from '../../../src/services/sync/models/ContentProgress'
-import db from '../../../src/services/sync/repository-proxy'
-
-jest.mock('../../../src/services/sanity.js', () => require('./__mocks__/mocks').mockSanity())
-jest.mock('../../../src/services/content-org/learning-paths.ts', () => require('./__mocks__/mocks').mockLearningPaths())
-jest.mock('../../../src/services/awards/internal/content-progress-observer', () => require('./__mocks__/mocks').mockContentProgressObserver())
-jest.mock('../../../src/services/progress-events', () => require('./__mocks__/mocks').mockProgressEvents())
-
-jest.mock('../../../src/services/userActivity', () => ({
-  trackUserPractice: jest.fn().mockResolvedValue(undefined),
-}))
-
-const meta = { brand: 'drumeo', type: 'lesson', parent_id: 0 }
-const collectionSelf = { type: COLLECTION_TYPE.SELF, id: COLLECTION_ID_SELF }
-
-const ctx = initializeTestDB()
-
-// ─── contentStatusCompleted ───────────────────────────────────────────────────
-
-describe('contentStatusCompleted', () => {
-  test('sets state to completed in DB', async () => {
-    await contentStatusCompleted(500)
-    expect(await getProgressState(500)).toBe('completed')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-  })
-
-  test('sets progress_percent to 100 in DB', async () => {
-    await contentStatusCompleted(500)
-    const record = await db.contentProgress.getOneProgressByContentId(500, null)
-    expect(record.data?.progress_percent).toBe(100)
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-  })
-
-  test('defaults to SELF collection when null is passed', async () => {
-    await contentStatusCompleted(500, null)
-    const record = await db.contentProgress.getOneProgressByContentId(500, null)
-    expect(record.data?.collection_type).toBe(COLLECTION_TYPE.SELF)
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-  })
-})
-
-// ─── contentStatusCompletedMany ───────────────────────────────────────────────
-
-describe('contentStatusCompletedMany', () => {
-  test('sets all ids to completed state in DB', async () => {
-    await contentStatusCompletedMany([501, 502, 503])
-    expect(await getProgressState(501)).toBe('completed')
-    expect(await getProgressState(502)).toBe('completed')
-    expect(await getProgressState(503)).toBe('completed')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status-many')
-  })
-
-  test('each record has progress_percent of 100', async () => {
-    await contentStatusCompletedMany([501, 502])
-    const r1 = await db.contentProgress.getOneProgressByContentId(501, null)
-    const r2 = await db.contentProgress.getOneProgressByContentId(502, null)
-    expect(r1.data?.progress_percent).toBe(100)
-    expect(r2.data?.progress_percent).toBe(100)
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status-many')
-  })
-})
-
-// ─── contentStatusStarted ─────────────────────────────────────────────────────
-
-describe('contentStatusStarted', () => {
-  test('sets state to started in DB', async () => {
-    await contentStatusStarted(600)
-    expect(await getProgressState(600)).toBe('started')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-  })
-
-  test('sets progress_percent to 0 in DB', async () => {
-    await contentStatusStarted(600)
-    const record = await db.contentProgress.getOneProgressByContentId(600, null)
-    expect(record.data?.progress_percent).toBe(0)
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-  })
-})
-
-// ─── contentStatusReset ───────────────────────────────────────────────────────
-
-describe('contentStatusReset', () => {
-  test('removes record from DB so getProgressState returns empty string', async () => {
-    await contentStatusCompleted(700)
-    expect(await getProgressState(700)).toBe('completed')
-    await contentStatusReset(700)
-    expect(await getProgressState(700)).toBe('')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('reset-status')
-  })
-})
-
-// ─── saveContentProgress ──────────────────────────────────────────────────────
-
-describe('saveContentProgress', () => {
-  test('progress higher than existing writes new value', async () => {
-    await db.contentProgress.recordProgress(800, null, 30, meta, undefined, { skipPush: true })
-    const hierarchy = { metadata: { 800: meta }, parents: {}, children: {} }
-    await saveContentProgress(800, collectionSelf, 60, null, { hierarchy, isOffline: true })
-    const record = await db.contentProgress.getOneProgressByContentId(800, null)
-    expect(record.data?.progress_percent).toBe(60)
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('save-content-progress')
-  })
-
-  test('progress lower than existing is filtered — DB value unchanged', async () => {
-    await db.contentProgress.recordProgress(800, null, 70, meta, undefined, { skipPush: true })
-    const hierarchy = { metadata: { 800: meta }, parents: {}, children: {} }
-    await saveContentProgress(800, collectionSelf, 40, null, { hierarchy, isOffline: true })
-    const record = await db.contentProgress.getOneProgressByContentId(800, null)
-    expect(record.data?.progress_percent).toBe(70)
-    expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
-  })
-
-  test('progress equal to existing writes successfully', async () => {
-    await db.contentProgress.recordProgress(800, null, 50, meta, undefined, { skipPush: true })
-    const hierarchy = { metadata: { 800: meta }, parents: {}, children: {} }
-    await saveContentProgress(800, collectionSelf, 50, null, { hierarchy, isOffline: true })
-    const record = await db.contentProgress.getOneProgressByContentId(800, null)
-    expect(record.data?.progress_percent).toBe(50)
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('save-content-progress')
-  })
-})
-
-// ─── setStartedOrCompletedStatus ─────────────────────────────────────────────
-
-describe('setStartedOrCompletedStatus', () => {
-  test('completed=true writes state=completed to DB', async () => {
-    await setStartedOrCompletedStatus(900, collectionSelf, true)
-    expect(await getProgressState(900)).toBe('completed')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-  })
-
-  test('completed=false writes state=started to DB', async () => {
-    await setStartedOrCompletedStatus(900, collectionSelf, false)
-    expect(await getProgressState(900)).toBe('started')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-  })
-
-  test('skipBubbleTrickle=true still writes the main record', async () => {
-    await setStartedOrCompletedStatus(900, collectionSelf, true, { skipBubbleTrickle: true })
-    expect(await getProgressState(900)).toBe('completed')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-  })
-})
-
-// ─── setStartedOrCompletedStatusMany ─────────────────────────────────────────
-
-describe('setStartedOrCompletedStatusMany', () => {
-  test('isCompleted=true sets all ids to completed in DB', async () => {
-    await setStartedOrCompletedStatusMany([1001, 1002, 1003], collectionSelf, true)
-    expect(await getProgressState(1001)).toBe('completed')
-    expect(await getProgressState(1002)).toBe('completed')
-    expect(await getProgressState(1003)).toBe('completed')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status-many')
-  })
-
-  test('isCompleted=false sets all ids to started in DB', async () => {
-    await setStartedOrCompletedStatusMany([1001, 1002], collectionSelf, false)
-    expect(await getProgressState(1001)).toBe('started')
-    expect(await getProgressState(1002)).toBe('started')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status-many')
-  })
-})
-
-// ─── resetStatus ─────────────────────────────────────────────────────────────
-
-describe('resetStatus', () => {
-  test('removes record so getProgressState returns empty string', async () => {
-    await db.contentProgress.recordProgress(1100, null, 80, meta, undefined, { skipPush: true })
-    expect(await getProgressState(1100)).toBe('started')
-    await resetStatus(1100, collectionSelf)
-    expect(await getProgressState(1100)).toBe('')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('reset-status')
-  })
-})
-
-// ─── trackProgress ────────────────────────────────────────────────────────────
-
-describe('trackProgress', () => {
-  test('50 of 200 seconds records progress ~25 to DB', async () => {
-    const hierarchy = { metadata: { 100: meta }, parents: {}, children: {} }
-    await trackProgress(100, collectionSelf, 50, 200, false, true, hierarchy)
-    const record = await db.contentProgress.getOneProgressByContentId(100, null)
-    expect(record.data?.progress_percent).toBe(25)
-    expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
-  })
-
-  test('0 seconds clamps to progress 1 in DB', async () => {
-    const hierarchy = { metadata: { 100: meta }, parents: {}, children: {} }
-    await trackProgress(100, collectionSelf, 0, 200, false, true, hierarchy)
-    const record = await db.contentProgress.getOneProgressByContentId(100, null)
-    expect(record.data?.progress_percent).toBe(1)
-    expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
-  })
-
-  test('full duration (200/200) clamps to 99 — never reaches 100 via track', async () => {
-    const hierarchy = { metadata: { 100: meta }, parents: {}, children: {} }
-    await trackProgress(100, collectionSelf, 200, 200, false, true, hierarchy)
-    const record = await db.contentProgress.getOneProgressByContentId(100, null)
-    expect(record.data?.progress_percent).toBe(99)
-    expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
-  })
-})
-
-// ─── recordWatchSession ───────────────────────────────────────────────────────
-
-describe('recordWatchSession', () => {
-  const mockUserActivity = jest.requireMock('../../../src/services/userActivity')
-
-  beforeEach(() => {
-    jest.clearAllMocks()
-  })
-
-  test('writes progress to DB and calls trackUserPractice', async () => {
-    await recordWatchSession(234, null, 200, 100, 30)
-    const record = await db.contentProgress.getOneProgressByContentId(234, null)
-    expect(record.data?.progress_percent).toBe(50)
-    expect(mockUserActivity.trackUserPractice).toHaveBeenCalledWith(234, 30, expect.any(Object))
-    expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
-  })
-
-  test('normalizes string contentId to number in DB', async () => {
-    await recordWatchSession('234' as any, null, 200, 100, 30)
-    const record = await db.contentProgress.getOneProgressByContentId(234, null)
-    expect(record.data?.content_id).toBe(234)
-    expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
-  })
-})
-
-// ─── _recordWatchSession ──────────────────────────────────────────────────────
-
-describe('_recordWatchSession', () => {
-  const mockUserActivity = jest.requireMock('../../../src/services/userActivity')
-
-  beforeEach(() => {
-    jest.clearAllMocks()
-  })
-
-  test('writes progress to DB and calls trackUserPractice', async () => {
-    const hierarchy = { metadata: { 234: meta }, parents: {}, children: {} }
-    await _recordWatchSession(234, 200, 100, 30, { collection: null, isOffline: true, hierarchy })
-    const record = await db.contentProgress.getOneProgressByContentId(234, null)
-    expect(record.data?.progress_percent).toBe(50)
-    expect(mockUserActivity.trackUserPractice).toHaveBeenCalledWith(234, 30, expect.any(Object))
-    expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
-  })
-})
-
-// ─── flushWatchSession ────────────────────────────────────────────────────────
-
-describe('flushWatchSession', () => {
-  test('triggers push for both contentProgress and practices', async () => {
-    await expect(flushWatchSession()).resolves.not.toThrow()
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('flush-watch-session')
-    expect(ctx.pushSpies.practices).toHaveBeenCalledWith('flush-watch-session')
-  })
-})
-
-// ─── E2E Scenarios ────────────────────────────────────────────────────────────
-
-describe('Scenario: User completes lesson a-la-carte', () => {
-  test('state=completed and progress_percent=100 written to DB', async () => {
-    await contentStatusCompleted(42001)
-    const record = await db.contentProgress.getOneProgressByContentId(42001, null)
-    expect(record.data?.state).toBe('completed')
-    expect(record.data?.progress_percent).toBe(100)
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-  })
-})
-
-describe('Scenario: User starts lesson a-la-carte', () => {
-  test('state=started and progress_percent=0 written to DB', async () => {
-    await contentStatusStarted(42002)
-    const record = await db.contentProgress.getOneProgressByContentId(42002, null)
-    expect(record.data?.state).toBe('started')
-    expect(record.data?.progress_percent).toBe(0)
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-  })
-})
-
-describe('Scenario: User resets lesson progress', () => {
-  test('record removed from DB after reset', async () => {
-    await contentStatusCompleted(42003)
-    expect(await getProgressState(42003)).toBe('completed')
-    await contentStatusReset(42003)
-    expect(await getProgressState(42003)).toBe('')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('reset-status')
-  })
-})
-
-describe('Scenario: User watches video halfway through', () => {
-  test('progress written and clamped correctly', async () => {
-    await recordWatchSession(42004, null, 200, 100, 30)
-    const record = await db.contentProgress.getOneProgressByContentId(42004, null)
-    expect(record.data?.progress_percent).toBe(50)
-    expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
-  })
-})
-
-describe('Scenario: User watches to end — clamped at 99 (not 100)', () => {
-  test('progress_percent is 99, not 100, after watching to the end', async () => {
-    await recordWatchSession(42005, null, 200, 200, 60)
-    const record = await db.contentProgress.getOneProgressByContentId(42005, null)
-    expect(record.data?.progress_percent).toBe(99)
-    expect(record.data?.state).toBe('started')
-    expect(ctx.pushSpies.contentProgress).not.toHaveBeenCalled()
-  })
-})
-
-describe('Scenario: Completing multiple lessons at once', () => {
-  test('all lessons written as completed to DB', async () => {
-    await contentStatusCompletedMany([50001, 50002, 50003])
-    expect(await getProgressState(50001)).toBe('completed')
-    expect(await getProgressState(50002)).toBe('completed')
-    expect(await getProgressState(50003)).toBe('completed')
-    expect(ctx.pushSpies.contentProgress).toHaveBeenCalledWith('set-started-or-completed-status-many')
-  })
-})