musora-content-services 2.167.2 → 2.167.3

package/test/unit/content-progress/navigateTo.test.ts

@@ -1,392 +0,0 @@
-import { initializeTestService } from '../../initializeTests.js'
-import {
-  buildNavigateTo,
-  findIncompleteLesson,
-  getNavigateTo,
-  getNavigateToForMethod,
-} from '../../../src/services/contentProgress.js'
-import { COLLECTION_TYPE } from '../../../src/services/sync/models/ContentProgress'
-
-let mockProgressRecords: any[] = []
-let mockLastInteracted: number | null = null
-
-jest.mock('../../../src/services/sync/repository-proxy', () => {
-  const mockFns = {
-    contentProgress: {
-      getOneProgressByContentId: jest.fn().mockImplementation((contentId) => {
-        const record = mockProgressRecords.find(r => r.content_id === contentId)
-        return Promise.resolve({ data: record || null })
-      }),
-      getSomeProgressByContentIds: jest.fn().mockImplementation((contentIds) => {
-        const records = mockProgressRecords.filter(r => contentIds.includes(r.content_id))
-        return Promise.resolve({ data: records })
-      }),
-      mostRecentlyUpdatedId: jest.fn().mockImplementation(() => {
-        return Promise.resolve({ data: mockLastInteracted })
-      }),
-    },
-    practices: {
-      queryAll: jest.fn().mockResolvedValue({ data: [] }),
-      getAll: jest.fn().mockResolvedValue({ data: [] }),
-    },
-  }
-  return { default: mockFns, ...mockFns }
-})
-
-jest.mock('../../../src/services/content-org/learning-paths', () => ({
-  getDailySession: jest.fn().mockResolvedValue(null),
-  onLearningPathCompletedActions: jest.fn().mockResolvedValue(undefined),
-}))
-
-jest.mock('../../../src/services/sanity.js', () => ({
-  getHierarchy: jest.fn().mockResolvedValue({ metadata: {}, parents: {}, children: {} }),
-  getHierarchies: jest.fn().mockResolvedValue({ metadata: {}, parents: {}, children: {} }),
-  getSanityDate: jest.fn((date: Date) => date.toISOString()),
-}))
-
-const { getDailySession } = jest.requireMock('../../../src/services/content-org/learning-paths')
-
-const child = (id: number, type = 'lesson') => ({
-  id,
-  brand: 'drumeo',
-  thumbnail: '',
-  type,
-  published_on: null,
-  status: 'published',
-  children: null as any,
-})
-
-beforeEach(() => {
-  jest.clearAllMocks()
-  initializeTestService()
-  mockProgressRecords = []
-  mockLastInteracted = null
-})
-
-// ─── buildNavigateTo ──────────────────────────────────────────────────────────
-
-describe('buildNavigateTo', () => {
-  test('null content returns null', () => {
-    expect(buildNavigateTo(null)).toBeNull()
-  })
-
-  test('valid content returns correct shape', () => {
-    const content = {
-      id: 101,
-      brand: 'drumeo',
-      thumbnail: 'thumb.jpg',
-      type: 'lesson',
-      published_on: '2024-01-01',
-      status: 'published',
-    }
-    const result = buildNavigateTo(content)
-    expect(result).toEqual({
-      brand: 'drumeo',
-      thumbnail: 'thumb.jpg',
-      id: 101,
-      type: 'lesson',
-      published_on: '2024-01-01',
-      status: 'published',
-      child: null,
-      collection: null,
-    })
-  })
-
-  test('missing fields fall back to defaults', () => {
-    const result = buildNavigateTo({ id: 5 })
-    expect(result).toMatchObject({
-      brand: '',
-      thumbnail: '',
-      id: 5,
-      type: '',
-      published_on: null,
-      status: '',
-      child: null,
-      collection: null,
-    })
-  })
-
-  test('child and collection args pass through', () => {
-    const childObj = {
-      id: 200,
-      brand: 'drumeo',
-      thumbnail: '',
-      type: 'lesson',
-      published_on: null,
-      status: 'published',
-    }
-    const collection = { type: COLLECTION_TYPE.LEARNING_PATH, id: 999 }
-    const result = buildNavigateTo({ id: 100 }, childObj, collection)
-    expect(result?.child).toBe(childObj)
-    expect(result?.collection).toBe(collection)
-  })
-})
-
-// ─── findIncompleteLesson ─────────────────────────────────────────────────────
-
-describe('findIncompleteLesson', () => {
-  describe('course type', () => {
-    test('finds first incomplete after currentContentId', () => {
-      const progresses = new Map([
-        [101, 'completed'],
-        [102, 'completed'],
-        [103, 'started'],
-        [104, ''],
-      ])
-      expect(findIncompleteLesson(progresses, 102, 'course')).toBe(103)
-    })
-
-    // todo(BEHSTP-325): add a test "returns currentContentId if it's incomplete"
-
-    test('wraps to first when all after current are completed, even if some before are incomplete', () => {
-      const progresses = new Map([
-        [101, 'completed'],
-        [102, 'started'],
-        [103, 'completed'],
-        [104, 'completed'],
-      ])
-      expect(findIncompleteLesson(progresses, 103, 'course')).toBe(101)
-    })
-
-    test('returns null when currentContentId not in ids', () => {
-      const progresses = new Map([
-        [101, 'started'],
-        [102, 'completed'],
-      ])
-      expect(findIncompleteLesson(progresses, 999, 'course')).toBeNull()
-    })
-  })
-
-  describe('guided-course type', () => {
-    test('finds first incomplete regardless of position', () => {
-      const progresses = new Map([
-        [101, 'completed'],
-        [102, ''],
-        [103, 'started'],
-      ])
-      expect(findIncompleteLesson(progresses, 103, 'guided-course')).toBe(102)
-    })
-
-    test('returns first id when all completed', () => {
-      const progresses = new Map([
-        [101, 'completed'],
-        [102, 'completed'],
-        [103, 'completed'],
-      ])
-      expect(findIncompleteLesson(progresses, 102, 'guided-course')).toBe(101)
-    })
-  })
-
-  describe('learning-path-v2 type', () => {
-    test('finds first incomplete regardless of position', () => {
-      const progresses = new Map([
-        [101, 'completed'],
-        [102, ''],
-        [103, 'started'],
-      ])
-      expect(findIncompleteLesson(progresses, 103, 'learning-path-v2')).toBe(102)
-    })
-
-    test('returns first id when all completed', () => {
-      const progresses = new Map([
-        [101, 'completed'],
-        [102, 'completed'],
-        [103, 'completed'],
-      ])
-      expect(findIncompleteLesson(progresses, 102, 'learning-path-v2')).toBe(101)
-    })
-  })
-
-  describe('Map vs Object input', () => {
-    test('works with Map input', () => {
-      const progresses = new Map([
-        [101, 'completed'],
-        [102, ''],
-      ])
-      expect(findIncompleteLesson(progresses, 101, 'course')).toBe(102)
-    })
-
-    test('works with Object input', () => {
-      const progresses = { 101: 'completed', 102: '' }
-      expect(findIncompleteLesson(progresses, 101, 'course')).toBe(102)
-    })
-  })
-})
-
-// ─── getNavigateTo ────────────────────────────────────────────────────────────
-
-describe('getNavigateTo', () => {
-  test('null entry in data array skipped', async () => {
-    const result = await getNavigateTo([null as any, { id: 1, type: 'lesson', children: [] }])
-    expect(result[1]).toBeNull()
-    expect(Object.keys(result)).not.toContain('null')
-    expect(Object.keys(result)).not.toContain('undefined')
-  })
-
-  test('non-navigable type returns null', async () => {
-    const result = await getNavigateTo([{ id: 1, type: 'lesson', children: [child(101)] }])
-    expect(result[1]).toBeNull()
-  })
-
-  test('null children returns null', async () => {
-    const result = await getNavigateTo([{ id: 1, type: 'course', children: null }])
-    expect(result[1]).toBeNull()
-  })
-
-  test('empty children after filtering nulls returns null', async () => {
-    const result = await getNavigateTo([{ id: 1, type: 'course', children: [null, null] }])
-    expect(result[1]).toBeNull()
-  })
-
-  test('content not started navigates to first child', async () => {
-    const result = await getNavigateTo([{ id: 1, type: 'course', children: [child(101), child(102)] }])
-    expect(result[1]).toMatchObject({ id: 101 })
-  })
-
-  test('course started lastInteracted started navigates to lastInteracted child', async () => {
-    mockProgressRecords = [
-      { content_id: 1, state: 'started', progress_percent: 50, updated_at: 1000 },
-      { content_id: 101, state: 'started', progress_percent: 100, updated_at: 900 },
-      { content_id: 102, state: 'started', progress_percent: 30, updated_at: 1000 },
-    ]
-    mockLastInteracted = 101
-    const result = await getNavigateTo([{ id: 1, type: 'course', children: [child(101), child(102)] }])
-    expect(result[1]).toMatchObject({ id: 101 })
-  })
-
-  test('documentary started lastInteracted started navigates to lastInteracted child', async () => {
-    mockProgressRecords = [
-      { content_id: 1, state: 'started', progress_percent: 50, updated_at: 1000 },
-      { content_id: 101, state: 'started', progress_percent: 100, updated_at: 900 },
-      { content_id: 102, state: 'started', progress_percent: 30, updated_at: 1000 },
-    ]
-    mockLastInteracted = 101
-    const result = await getNavigateTo([{ id: 1, type: 'documentary', children: [child(101), child(102)] }])
-    expect(result[1]).toMatchObject({ id: 101 })
-  })
-
-  test('course started lastInteracted completed navigates to first incomplete after lastInteracted', async () => {
-    mockProgressRecords = [
-      { content_id: 1, state: 'started', progress_percent: 60, updated_at: 1000 },
-      { content_id: 101, state: 'completed', progress_percent: 100, updated_at: 900 },
-      { content_id: 102, state: 'completed', progress_percent: 100, updated_at: 1000 },
-      { content_id: 103, state: 'started', progress_percent: 20, updated_at: 800 },
-    ]
-    mockLastInteracted = 101
-    const result = await getNavigateTo([{ id: 1, type: 'course', children: [child(101), child(102), child(103)] }])
-    expect(result[1]).toMatchObject({ id: 103 })
-  })
-
-  test('course started all children completed wraps to first child', async () => {
-    mockProgressRecords = [
-      { content_id: 1, state: 'started', progress_percent: 100, updated_at: 1000 },
-      { content_id: 101, state: 'completed', progress_percent: 100, updated_at: 900 },
-      { content_id: 102, state: 'completed', progress_percent: 100, updated_at: 1000 },
-    ]
-    mockLastInteracted = 102
-    const result = await getNavigateTo([{ id: 1, type: 'course', children: [child(101), child(102)] }])
-    expect(result[1]).toMatchObject({ id: 101 })
-  })
-
-  test('guided-course started navigates to first incomplete child', async () => {
-    mockProgressRecords = [
-      { content_id: 1, state: 'started', progress_percent: 50, updated_at: 1000 },
-      { content_id: 101, state: 'completed', progress_percent: 100, updated_at: 900 },
-    ]
-    mockLastInteracted = 101
-    const result = await getNavigateTo([{
-      id: 1,
-      type: 'guided-course',
-      children: [child(101), child(102), child(103)],
-    }])
-    expect(result[1]).toMatchObject({ id: 102 })
-  })
-
-  describe.each(['course', 'skill-pack', 'song-tutorial'])('course-flow type %s', (type) => {
-    test('lastInteracted completed → first incomplete after lastInteracted', async () => {
-      mockProgressRecords = [
-        { content_id: 1, state: 'started', progress_percent: 50, updated_at: 1000 },
-        { content_id: 101, state: '', progress_percent: 0, updated_at: 0 },
-        { content_id: 102, state: 'completed', progress_percent: 100, updated_at: 1000 },
-        { content_id: 103, state: '', progress_percent: 0, updated_at: 0 },
-      ]
-      mockLastInteracted = 102
-      const result = await getNavigateTo([{ id: 1, type, children: [child(101), child(102), child(103)] }])
-      expect(result[1]).toMatchObject({ id: 103 })
-    })
-  })
-
-  describe.each(['guided-course', COLLECTION_TYPE.LEARNING_PATH])('guided-course-flow type %s', (type) => {
-    test('finds first incomplete regardless of lastInteracted position', async () => {
-      mockProgressRecords = [
-        { content_id: 1, state: 'started', progress_percent: 50, updated_at: 1000 },
-        { content_id: 101, state: '', progress_percent: 0, updated_at: 0 },
-        { content_id: 102, state: 'completed', progress_percent: 100, updated_at: 1000 },
-        { content_id: 103, state: '', progress_percent: 0, updated_at: 0 },
-      ]
-      mockLastInteracted = 102
-      const result = await getNavigateTo([{ id: 1, type, children: [child(101), child(102), child(103)] }])
-      expect(result[1]).toMatchObject({ id: 101 })
-    })
-  })
-
-  // need more tests to support other types and potentially other logic branches.
-})
-
-// ─── getNavigateToForMethod ───────────────────────────────────────────────────
-
-describe('getNavigateToForMethod', () => {
-  const lpContent = (id: number, children: any[]) => ({
-    id,
-    brand: 'drumeo',
-    thumbnail: '',
-    type: COLLECTION_TYPE.LEARNING_PATH,
-    published_on: null,
-    status: 'published',
-    children,
-    record_id: `${id}:${COLLECTION_TYPE.LEARNING_PATH}:${id}`,
-  })
-
-  const nonLpContent = (id: number) => ({
-    id,
-    brand: 'drumeo',
-    thumbnail: '',
-    type: 'course',
-    published_on: null,
-    status: 'published',
-    children: [child(201), child(202)],
-    record_id: `${id}:self:0`,
-  })
-
-  test('non-LP content type returns null', async () => {
-    const result = await getNavigateToForMethod([nonLpContent(1)])
-    expect(result[1]).toBeNull()
-  })
-
-  test('LP type with no daily session navigates to first incomplete child', async () => {
-    getDailySession.mockResolvedValueOnce(null)
-    const result = await getNavigateToForMethod([lpContent(10, [child(301), child(302), child(303)])])
-    expect(result[10]).toMatchObject({ id: 301 })
-  })
-
-  test('LP type with active learning path and daily session navigates using daily session', async () => {
-    mockProgressRecords = [
-      { content_id: 301, state: 'completed', progress_percent: 100, updated_at: 900 },
-    ]
-    getDailySession.mockResolvedValueOnce({
-      active_learning_path_id: 10,
-      daily_session: [{ content_ids: [301, 302] }],
-    })
-    const result = await getNavigateToForMethod([lpContent(10, [child(300), child(301), child(302)])])
-    expect(result[10]).toMatchObject({ id: 302 })
-  })
-
-  test('null content entries skipped', async () => {
-    getDailySession.mockResolvedValueOnce(null)
-    const validContent = lpContent(10, [child(301)])
-    const result = await getNavigateToForMethod([validContent, null as any])
-    expect(Object.keys(result)).toContain('10')
-    expect(Object.keys(result)).not.toContain('null')
-    expect(Object.keys(result)).not.toContain('undefined')
-  })
-})

package/test/unit/contentLikes.test.ts

@@ -1,62 +0,0 @@
-import {
-  isContentLiked,
-  likeContent,
-  unlikeContent,
-} from '../../src/services/contentLikes.js'
-import { initializeTestService } from '../initializeTests.js'
-
-let mockLikedIds: Set<number> = new Set()
-
-jest.mock('../../src/services/sync/repository-proxy.ts', () => {
-  const mockFns = {
-    likes: {
-      isLiked: jest.fn().mockImplementation((contentId) =>
-        Promise.resolve({ data: mockLikedIds.has(Number(contentId)) })
-      ),
-      areLiked: jest.fn().mockImplementation((contentIds) =>
-        Promise.resolve({ data: contentIds.map(id => mockLikedIds.has(Number(id))) })
-      ),
-      like: jest.fn().mockImplementation((contentId) => {
-        mockLikedIds.add(Number(contentId))
-        return Promise.resolve({ success: true })
-      }),
-      unlike: jest.fn().mockImplementation((contentId) => {
-        mockLikedIds.delete(Number(contentId))
-        return Promise.resolve({ success: true })
-      }),
-    }
-  }
-  return { default: mockFns, ...mockFns }
-})
-
-
-describe('contentLikesDataContext', function () {
-  beforeEach(() => {
-    initializeTestService()
-    mockLikedIds = new Set([308516, 308515, 308514, 308518])
-  })
-
-  test('contentLiked', async () => {
-    expect(await isContentLiked(308516)).toBe(true)
-  })
-
-  test('contentLikedStringInput', async () => {
-    expect(await isContentLiked('308516')).toBe(true)
-  })
-
-  test('contentNotLiked', async () => {
-    expect(await isContentLiked(121111)).toBe(false)
-  })
-
-  test('likeContent', async () => {
-    expect(await isContentLiked(111111)).toBe(false)
-    await likeContent(111111)
-    expect(await isContentLiked(111111)).toBe(true)
-  })
-
-  test('unlikeContent', async () => {
-    expect(await isContentLiked(308516)).toBe(true)
-    await unlikeContent(308516)
-    expect(await isContentLiked(308516)).toBe(false)
-  })
-})

package/test/unit/contentMetaData.test.ts

@@ -1,58 +0,0 @@
-import { processMetadata } from '@/contentMetaData'
-
-describe('processMetadata', () => {
-  test('returns null for unknown brand and type', () => {
-    expect(processMetadata('unknown', 'unknown')).toBeNull()
-  })
-
-  test('returns null for unknown type on known brand', () => {
-    expect(processMetadata('drumeo', 'unknown-type')).toBeNull()
-  })
-
-  test('returns expected shape for known brand and type', () => {
-    const result = processMetadata('drumeo', 'lessons')
-
-    expect(result).not.toBeNull()
-    expect(result.type).toBe('lessons')
-    expect(result.name).toBeDefined()
-    expect(result.sort).toBeDefined()
-    expect(result.tabs).toBeDefined()
-    expect(Array.isArray(result.tabs)).toBe(true)
-  })
-
-  test('does not include filters when withFilters is false', () => {
-    const result = processMetadata('drumeo', 'lessons')
-
-    expect(result.filters).toBeUndefined()
-  })
-
-  test('includes filters when withFilters is true', () => {
-    const result = processMetadata('drumeo', 'lessons', true)
-
-    expect(result.filters).toBeDefined()
-  })
-
-  test('brand-specific filters differ between brands for the same type', () => {
-    const drumeo = processMetadata('drumeo', 'lessons', true)
-    const pianote = processMetadata('pianote', 'lessons', true)
-
-    expect(JSON.stringify(drumeo.filters)).not.toBe(JSON.stringify(pianote.filters))
-  })
-
-  test('shared structure is consistent across brands for the same type', () => {
-    const drumeo = processMetadata('drumeo', 'lessons')
-    const pianote = processMetadata('pianote', 'lessons')
-
-    expect(drumeo.type).toBe(pianote.type)
-    expect(drumeo.name).toBe(pianote.name)
-    expect(JSON.stringify(drumeo.tabs)).toBe(JSON.stringify(pianote.tabs))
-  })
-
-  test('songs tabs are brand-specific based on song types config', () => {
-    const result = processMetadata('drumeo', 'songs')
-
-    expect(result).not.toBeNull()
-    expect(result.tabs.length).toBeGreaterThan(0)
-    expect(result.tabs[0].name).toBe('For You')
-  })
-})

package/test/unit/dataContext.test.ts

@@ -1,38 +0,0 @@
-import { initializeTestService } from '../initializeTests.js'
-import { DataContext, verifyLocalDataContext } from '../../src/services/dataContext.js'
-
-describe('dataContext', function () {
-  let mock: jest.SpyInstance
-  let testVersion = 1
-  const dataVersionKey = 1
-  const dataContext = new DataContext(dataVersionKey, null)
-
-  beforeEach(() => {
-    initializeTestService()
-    mock = jest.spyOn(dataContext, 'fetchData')
-    mock.mockImplementation(() =>
-      JSON.parse(`{"version":${testVersion},"data":[308516,308515,308514,308518]}`)
-    )
-  })
-
-  test('verifyLocalData', async () => {
-    //force load data into context and verify version 1 loaded
-    await dataContext.getData()
-    expect(dataContext.version()).toBe(1)
-
-    //increment source version and verify data context still uses old version
-    testVersion++
-    await dataContext.getData()
-    expect(dataContext.version()).toBe(1)
-
-    //verifyLocalData with old source version and verify context still uses old version
-    await verifyLocalDataContext(1, 1)
-    await dataContext.getData()
-    expect(dataContext.version()).toBe(1)
-
-    //verifyLocalData with new source version and verify context loads new version
-    await verifyLocalDataContext(1, 2)
-    await dataContext.getData()
-    expect(dataContext.version()).toBe(2)
-  })
-})