musora-content-services 2.167.2 → 2.167.3

package/test/integration/sync/store/store.test.ts

@@ -1,361 +0,0 @@
-import { Database } from '@nozbe/watermelondb'
-import { makeTelemetry, makeContext, makeUserScope, makePullMock, makePushMock } from '../../../unit/sync/helpers/index'
-import TestModel, { makeTestDatabase } from '../../../unit/sync/helpers/TestModel'
-import SyncStore from '@/services/sync/store/index'
-import SyncRetry from '@/services/sync/retry'
-import SyncRunScope from '@/services/sync/run-scope'
-
-let db: Database
-
-function makeStore(overrides: { pull?: any; push?: any } = {}) {
-  const userScope = makeUserScope()
-  const context = makeContext()
-  const telemetry = makeTelemetry(userScope)
-  const retry = new SyncRetry(context, telemetry)
-  const runScope = new SyncRunScope()
-
-  const store = new SyncStore<TestModel>(
-    {
-      model: TestModel,
-      pull: overrides.pull ?? makePullMock(),
-      push: overrides.push ?? makePushMock(),
-    },
-    userScope,
-    context,
-    db,
-    retry,
-    runScope,
-    telemetry
-  )
-
-  return store
-}
-
-beforeEach(() => {
-  db = makeTestDatabase()
-})
-
-afterEach(async () => {
-  await db.write(async () => db.unsafeResetDatabase())
-})
-
-// ---
-
-describe('upsert', () => {
-  test('insertOne creates record', async () => {
-    const store = makeStore()
-    const result = await store.insertOne(r => { r.value = 'hello'; r.score = 5 })
-    store.destroy()
-
-    expect(result.value).toBe('hello')
-    expect(result.score).toBe(5)
-    expect(typeof result.id).toBe('string')
-  })
-
-  test('upsertOne creates when absent', async () => {
-    const store = makeStore()
-    await store.upsertOne('rec-1', r => { r.value = 'first'; r.score = 1 })
-    const record = await store.readOne('rec-1')
-    store.destroy()
-
-    expect(record).not.toBeNull()
-    expect(record!.value).toBe('first')
-  })
-
-  test('upsertOne updates existing record', async () => {
-    const store = makeStore()
-    await store.upsertOne('rec-2', r => { r.value = 'original'; r.score = 10 })
-    await store.upsertOne('rec-2', r => { r.value = 'updated'; r.score = 20 })
-    const record = await store.readOne('rec-2')
-    store.destroy()
-
-    expect(record!.value).toBe('updated')
-    expect(record!.score).toBe(20)
-  })
-
-  test('upsertSome creates multiple records in one batch', async () => {
-    const store = makeStore()
-    await store.upsertSome({
-      'a': r => { r.value = 'a'; r.score = 1 },
-      'b': r => { r.value = 'b'; r.score = 2 },
-      'c': r => { r.value = 'c'; r.score = 3 },
-    })
-    const all = await store.readAll()
-    store.destroy()
-
-    expect(all).toHaveLength(3)
-    expect(all.map(r => r.value).sort()).toEqual(['a', 'b', 'c'])
-  })
-
-  test('upsertSome no-ops on empty object', async () => {
-    const store = makeStore()
-    const result = await store.upsertSome({})
-    store.destroy()
-
-    expect(result).toEqual([])
-  })
-
-  test('upsertOne over deleted tombstone recreates the record', async () => {
-    const store = makeStore()
-    await store.upsertOne('tomb-1', r => { r.value = 'original'; r.score = 1 })
-    await store.deleteOne('tomb-1')
-    expect(await store.readOne('tomb-1')).toBeNull()
-
-    await store.upsertOne('tomb-1', r => { r.value = 'resurrected'; r.score = 2 })
-    const record = await store.readOne('tomb-1')
-    store.destroy()
-
-    expect(record).not.toBeNull()
-    expect(record!.value).toBe('resurrected')
-    expect(record!.score).toBe(2)
-  })
-})
-
-describe('updateOneId', () => {
-  test('updates existing record fields', async () => {
-    const store = makeStore()
-    await store.upsertOne('upd-1', r => { r.value = 'original'; r.score = 10 })
-    await store.updateOneId('upd-1', r => { r.value = 'changed'; r.score = 99 })
-    const record = await store.readOne('upd-1')
-    store.destroy()
-
-    expect(record!.value).toBe('changed')
-    expect(record!.score).toBe(99)
-  })
-
-  test('throws SyncError when record does not exist', async () => {
-    const store = makeStore()
-
-    await expect(store.updateOneId('ghost', r => { r.value = 'x' })).rejects.toThrow('Record not found')
-    store.destroy()
-  })
-
-  test('fires upserted event', async () => {
-    const store = makeStore()
-    await store.upsertOne('upd-evt', r => { r.value = 'v'; r.score = 0 })
-    const handler = jest.fn()
-    store.on('upserted', handler)
-    await store.updateOneId('upd-evt', r => { r.value = 'updated' })
-    store.destroy()
-
-    expect(handler).toHaveBeenCalledTimes(1)
-    expect(handler.mock.calls[0][0][0].value).toBe('updated')
-  })
-})
-
-describe('delete / restore', () => {
-  test('deleteOne hides record from readAll', async () => {
-    const store = makeStore()
-    await store.upsertSome({
-      'keep': r => { r.value = 'keep'; r.score = 1 },
-      'delete-me': r => { r.value = 'gone'; r.score = 2 },
-    })
-    await store.deleteOne('delete-me')
-    const all = await store.readAll()
-    store.destroy()
-
-    expect(all).toHaveLength(1)
-    expect(all[0].value).toBe('keep')
-  })
-
-  test('deleteOne on non-existent record creates tombstone without throwing', async () => {
-    const store = makeStore()
-    await expect(store.deleteOne('ghost')).resolves.toBe('ghost')
-    store.destroy()
-  })
-
-  test('restoreOne makes deleted record visible again', async () => {
-    const store = makeStore()
-    await store.upsertOne('restore-me', r => { r.value = 'alive'; r.score = 7 })
-    await store.deleteOne('restore-me')
-    expect(await store.readOne('restore-me')).toBeNull()
-
-    await store.restoreOne('restore-me')
-    const restored = await store.readOne('restore-me')
-    store.destroy()
-
-    expect(restored).not.toBeNull()
-    expect(restored!.value).toBe('alive')
-  })
-
-  test('deleteSome removes multiple records', async () => {
-    const store = makeStore()
-    await store.upsertSome({
-      'x': r => { r.value = 'x'; r.score = 1 },
-      'y': r => { r.value = 'y'; r.score = 2 },
-      'z': r => { r.value = 'z'; r.score = 3 },
-    })
-    await store.deleteSome(['x', 'z'])
-    const all = await store.readAll()
-    store.destroy()
-
-    expect(all).toHaveLength(1)
-    expect(all[0].value).toBe('y')
-  })
-})
-
-describe('read / query', () => {
-  test('readOne returns null for missing id', async () => {
-    const store = makeStore()
-    const result = await store.readOne('missing')
-    store.destroy()
-
-    expect(result).toBeNull()
-  })
-
-  test('readSome returns only matched ids', async () => {
-    const store = makeStore()
-    await store.upsertSome({
-      'p': r => { r.value = 'p'; r.score = 1 },
-      'q': r => { r.value = 'q'; r.score = 2 },
-      'r': r => { r.value = 'r'; r.score = 3 },
-    })
-    const some = await store.readSome(['p', 'r'])
-    store.destroy()
-
-    expect(some).toHaveLength(2)
-    expect(some.map(s => s.value).sort()).toEqual(['p', 'r'])
-  })
-
-  test('readAll excludes deleted records', async () => {
-    const store = makeStore()
-    await store.upsertSome({
-      'v1': r => { r.value = 'v1'; r.score = 1 },
-      'v2': r => { r.value = 'v2'; r.score = 2 },
-    })
-    await store.deleteOne('v1')
-    const all = await store.readAll()
-    store.destroy()
-
-    expect(all).toHaveLength(1)
-    expect(all[0].value).toBe('v2')
-  })
-})
-
-describe('events', () => {
-  test('upserted fires on insertOne', async () => {
-    const store = makeStore()
-    const handler = jest.fn()
-    store.on('upserted', handler)
-    await store.insertOne(r => { r.value = 'event-test'; r.score = 0 })
-    store.destroy()
-
-    expect(handler).toHaveBeenCalledTimes(1)
-    expect(handler.mock.calls[0][0][0].value).toBe('event-test')
-  })
-
-  test('upserted fires on upsertOne', async () => {
-    const store = makeStore()
-    const handler = jest.fn()
-    store.on('upserted', handler)
-    await store.upsertOne('evt-1', r => { r.value = 'v'; r.score = 1 })
-    store.destroy()
-
-    expect(handler).toHaveBeenCalledTimes(1)
-  })
-
-  test('deleted fires on deleteOne', async () => {
-    const store = makeStore()
-    await store.upsertOne('del-evt', r => { r.value = 'bye'; r.score = 0 })
-    const handler = jest.fn()
-    store.on('deleted', handler)
-    await store.deleteOne('del-evt')
-    store.destroy()
-
-    expect(handler).toHaveBeenCalledTimes(1)
-    expect(handler.mock.calls[0][0]).toContain('del-evt')
-  })
-
-  test('off unsubscribes handler', async () => {
-    const store = makeStore()
-    const handler = jest.fn()
-    const unsub = store.on('upserted', handler)
-    unsub()
-    await store.upsertOne('no-event', r => { r.value = 'quiet'; r.score = 0 })
-    store.destroy()
-
-    expect(handler).not.toHaveBeenCalled()
-  })
-})
-
-describe('pull token', () => {
-  test('second pull sends token from first pull as previousFetchToken', async () => {
-    const syncToken = 1700000001234
-    const pullMock = jest.fn().mockResolvedValue({
-      ok: true,
-      entries: [],
-      token: syncToken,
-      previousToken: null,
-      intendedUserId: 1,
-    })
-    const store = makeStore({ pull: pullMock })
-
-    await store.pull('first')
-    await store.pull('second')
-    store.destroy()
-
-    const secondCallArgs = pullMock.mock.calls[1]
-    const previousFetchToken = secondCallArgs[secondCallArgs.length - 1]
-    expect(previousFetchToken).toBe(syncToken)
-  })
-
-  test('first pull sends null previousFetchToken', async () => {
-    const pullMock = jest.fn().mockResolvedValue({
-      ok: true,
-      entries: [],
-      token: Date.now(),
-      previousToken: null,
-      intendedUserId: 1,
-    })
-    const store = makeStore({ pull: pullMock })
-
-    await store.pull('first')
-    store.destroy()
-
-    const firstCallArgs = pullMock.mock.calls[0]
-    const previousFetchToken = firstCallArgs[firstCallArgs.length - 1]
-    expect(previousFetchToken).toBeNull()
-  })
-})
-
-describe('push coalescing', () => {
-  test('concurrent pushes for same record resolve to same response', async () => {
-    const pushMock = jest.fn().mockResolvedValue({ ok: true, results: [] })
-    const store = makeStore({ push: pushMock })
-    await store.upsertOne('coalesce-me', r => { r.value = 'v'; r.score = 0 })
-
-    const p1 = store.pushRecordIdsImpatiently(['coalesce-me'])
-    const p2 = store.pushRecordIdsImpatiently(['coalesce-me'])
-    const [r1, r2] = await Promise.all([p1, p2])
-    store.destroy()
-
-    expect(r1).toBe(r2)
-  })
-})
-
-describe('importUpsert / importDeletion', () => {
-  test('importUpsert creates records from raw data', async () => {
-    const store = makeStore()
-    const now = Date.now()
-
-    await store.importUpsert([
-      { id: 'import-1', value: 'imported', score: 99, server_record_id: 0, created_at: now, updated_at: now, _status: 'synced', _changed: '' } as unknown as TestModel['_raw'],
-    ])
-
-    const record = await store.readOne('import-1')
-    store.destroy()
-
-    expect(record).not.toBeNull()
-    expect(record!.value).toBe('imported')
-  })
-
-  test('importDeletion removes records', async () => {
-    const store = makeStore()
-    await store.upsertOne('to-delete', r => { r.value = 'x'; r.score = 0 })
-    await store.importDeletion(['to-delete'])
-    const record = await store.readOne('to-delete')
-    store.destroy()
-
-    expect(record).toBeNull()
-  })
-})

package/test/integration/user/permissions.test.js

@@ -1,20 +0,0 @@
-const { fetchUserPermissions, reset } = require('../../../src/services/permissions/permissions.ts')
-const { initializeTestService } = require('../../initializeTests.js')
-
-describe('user.permissions', function () {
-  beforeEach(async () => {
-    await reset()
-    initializeTestService()
-  })
-
-  test('fetchUserPermissions', async () => {
-    let result = await fetchUserPermissions() //fetch from server
-    let result2 = await fetchUserPermissions() //fetch locally
-
-    //This breaks when running tests in parallel
-    //expect(railContentModule.fetchUserPermissionsData).toHaveBeenCalledTimes(1);
-    expect(result.permissions).toStrictEqual([108, 91, 92])
-    expect(result.isAdmin).toStrictEqual(false)
-    expect(result).toBe(result2)
-  })
-})

package/test/integration/userActivity.test.ts

@@ -1,102 +0,0 @@
-import { initializeTestService } from '../initializeTests.js'
-import { getUserMonthlyStats, getUserWeeklyStats, recordUserPractice } from '../../src/services/userActivity.js'
-
-let mockPracticesData: {
-  date: string;
-  duration_seconds: number
-}[] = []
-
-jest.mock('../../src/services/sync/repository-proxy.ts', () => {
-  const mockFns = {
-    practices: {
-      queryAll: jest.fn().mockImplementation(() => Promise.resolve({ data: mockPracticesData })),
-      getAll: jest.fn().mockImplementation(() => Promise.resolve({ data: mockPracticesData })),
-      recordManualPractice: jest.fn().mockResolvedValue({ success: true }),
-    }
-  }
-  return { default: mockFns, ...mockFns }
-})
-
-jest.mock('../../src/services/user/streakCalculator.ts', () => ({
-  streakCalculator: {
-    getStreakData: jest.fn().mockResolvedValue({
-      currentDailyStreak: 0,
-      currentWeeklyStreak: 0,
-      streakMessage: '',
-      calculatedAt: Date.now(),
-      lastPracticeDate: null,
-    }),
-    invalidate: jest.fn(),
-  }
-}))
-
-jest.mock('../../src/services/railcontent.js', () => ({
-  ...jest.requireActual('../../src/services/railcontent'),
-  logUserPractice: jest.fn(() => Promise.resolve()),
-  fetchUserPermissionsData: jest.fn(() => ({ permissions: [78, 91, 92], isAdmin: false }))
-}))
-
-const repositoryProxy: any = require('../../src/services/sync/repository-proxy.ts')
-
-describe('User Activity API Tests', function () {
-  beforeEach(() => {
-    initializeTestService()
-    mockPracticesData = [
-      { date: '2025-02-10', duration_seconds: 190 },
-      { date: '2025-02-11', duration_seconds: 340 },
-      { date: '2025-02-19', duration_seconds: 340 },
-      { date: '2025-03-01', duration_seconds: 360 },
-      { date: '2025-03-03', duration_seconds: 360 },
-      { date: '2025-03-05', duration_seconds: 100 },
-      { date: '2025-03-11', duration_seconds: 190 },
-      { date: '2025-03-14', duration_seconds: 456 },
-      { date: '2025-03-15', duration_seconds: 124 },
-      { date: '2025-03-16', duration_seconds: 452 },
-      { date: '2025-03-16', duration_seconds: 456 },
-      { date: '2025-03-17', duration_seconds: 122 },
-    ]
-  })
-
-  test('fetches user practices successfully', async () => {
-    const practices = await getUserMonthlyStats()
-    const dailyStats = practices.data.dailyActiveStats
-    const currentDateString = new Date().toISOString().split('T')[0]
-
-    const current = dailyStats.find(stat => stat.label === currentDateString)
-    expect(current).toBeTruthy()
-    expect(current.isActive).toBe(true)
-    expect(current.type).toBe('active')
-    expect(current.inStreak).toBe(false)
-  })
-
-  test('fetches user practices from past', async () => {
-    const practices = await getUserMonthlyStats({ year: 2025, month: 1 })
-    const dailyStats = practices.data.dailyActiveStats
-
-    const feb10 = dailyStats.find(stat => stat.label === '2025-02-10')
-    expect(feb10.inStreak).toBe(true)
-    expect(feb10.type).toBe('tracked')
-    expect(feb10.isActive).toBe(false)
-  })
-
-  test('fetches user practices for current week', async () => {
-    const practices = await getUserWeeklyStats()
-    const dailyStats = practices.data.dailyActiveStats
-
-    const monday = dailyStats.find(stat => stat.label === 'M')
-    expect(monday).toBeDefined()
-    const tuesday = dailyStats.find(stat => stat.label === 'T')
-    expect(tuesday).toBeDefined()
-  })
-
-  test('should add a new practice entry', async () => {
-    const mockPractice = { duration_seconds: 300, content_id: 415183 }
-    await recordUserPractice(mockPractice)
-
-    expect(repositoryProxy.default.practices.recordManualPractice).toHaveBeenCalledWith(
-      expect.any(String),
-      300,
-      { title: null, category_id: null, thumbnail_url: null, instrument_id: null }
-    )
-  })
-})

package/test/live/README.md

@@ -1,29 +0,0 @@
-# test/live
-
-This directory is reserved for live tests that require real external services.
-
-## Why is this empty?
-
-Tests that previously lived here were written before the WatermelonDB sync rewrite.
-Progress tracking now goes through the local sync store rather than calling Railcontent
-directly — they were never updated, none were passing, and were removed rather than
-left as misleading dead code.
-
-## Where live verification actually belongs
-
-**`mcs-cli`** is the right tool for manually verifying behaviour against real services.
-Once the WatermelonDB ESM interop issues are resolved it can run commands like
-`mcs progress reset`, `mcs progress get` against a local BE and staging Sanity
-without needing a test framework.
-
-**Postman collections** are better suited for verifying Railcontent API contracts —
-checking that endpoint response shapes haven't changed in ways that would break MCS.
-
-**Jest live tests** are a last resort — only for logic that genuinely cannot be
-verified any other way. If added, they require:
-
-- WatermelonDB test harness from TP-1192
-- Dedicated test user account (not a real user)
-- Valid `.env` credentials — see 1Password
-- Manual trigger only: `npx jest --config=jest.live.config.js --no-coverage --forceExit`
-- Must never run in CI

package/test/live/content.test.js

@@ -1,116 +0,0 @@
-import { initializeTestService } from '../initializeTests.js'
-import {getContentRows, getNewAndUpcoming, getScheduleContentRows, getTabResults} from '../../src/index.js'
-
-// Mock fetchContentProgress before other modules load
-jest.mock('../../src/services/railcontent.js', () => ({
-  ...jest.requireActual('../../src/services/railcontent.js'),
-  fetchContentProgress: jest.fn().mockResolvedValue({ version: 1, data: {} })
-}))
-
-const railContentModule = require('../../src/services/railcontent.js')
-
-describe('content', function () {
-  beforeAll(async () => {
-    await initializeTestService(true)
-  })
-
-  test('getTabResults-Singles', async () => {
-    const results = await getTabResults('drumeo','lessons','Individuals', {selectedFilters:['difficulty,All','difficulty,Beginner'], sort:'-published_on'})
-    console.log(results)
-    expect(results.type).toBeDefined()
-    expect(results.type).toBe('catalog')
-    expect(results.data).toBeDefined()
-    expect(results.meta).toBeDefined()
-    expect(results.meta.filters).toBeDefined()
-    expect(results.meta.sort).toBeDefined()
-  })
-
-  test('getTabResults-Courses', async () => {
-    const results = await getTabResults('pianote','lessons','Collections', {selectedFilters:['difficulty,Expert'], sort:'slug'})
-    console.log(results)
-    expect(results.type).toBeDefined()
-    expect(results.type).toBe('catalog')
-    expect(results.data).toBeDefined()
-    expect(results.meta).toBeDefined()
-    expect(results.meta.filters).toBeDefined()
-    expect(results.meta.sort).toBeDefined()
-  })
-
-  test('getTabResults-Filters', async () => {
-    const results = await getTabResults('pianote','lessons','Explore All', {selectedFilters:['difficulty,Expert'], sort:'slug'})
-    console.log(results)
-    expect(results.type).toBeDefined()
-    expect(results.data).toBeDefined()
-    expect(results.meta).toBeDefined()
-    expect(results.meta.filters).toBeDefined()
-    expect(results.meta.sort).toBeDefined()
-  })
-
-  test('getTabResults-Type-Filter', async () => {
-    const results = await getTabResults('drumeo','lessons','Explore All', {selectedFilters:['type,Courses', 'type,Documentaries'], sort:'slug'})
-    console.log(results)
-    expect(results.type).toBeDefined()
-    expect(results.data).toBeDefined()
-    expect(results.meta).toBeDefined()
-    expect(results.meta.filters).toBeDefined()
-    expect(results.meta.sort).toBeDefined()
-  })
-
-  test('getTabResults-Type-Explore-All', async () => {
-    const results = await getTabResults('drumeo','lessons','Explore All', {selectedFilters:[], sort:'slug'})
-    console.log(results)
-    expect(results.type).toBeDefined()
-    expect(results.data).toBeDefined()
-    expect(results.meta).toBeDefined()
-    expect(results.meta.filters).toBeDefined()
-    expect(results.meta.sort).toBeDefined()
-  })
-
-  test('getContentRows', async () => {
-    const results = await getContentRows('drumeo', 'lessons', 'Your-Daily-Warmup')
-    console.log(results)
-  })
-
-//   test('getTabResults-Songs-For-You', async () => {
-//     const results = await getTabResults('drumeo','songs','For You')
-//     console.log(results)
-//     expect(results.type).toBeDefined()
-//     expect(results.type).toBe('sections')
-//     expect(results.data).toBeDefined()
-//     expect(results.meta).toBeDefined()
-//     expect(results.meta.filters).toBeDefined()
-//     expect(results.meta.sort).toBeDefined()
-//   })
-  test('getNewAndUpcoming', async () => {
-    const results = await getNewAndUpcoming('drumeo')
-    expect(results).not.toBeNull()
-    expect(Array.isArray(results.data)).toBe(true)
-
-    const childAwareParentTypes = ['course', 'song-tutorial', 'skill-pack']
-    const parentItems = results.data.filter((item) => childAwareParentTypes.includes(item.type))
-
-    parentItems.forEach((item) => {
-      expect(item.navigateTo).not.toBeNull()
-      expect(item.navigateTo.child).not.toBeNull()
-      expect(typeof item.navigateTo.child.id).toBe('number')
-    })
-  })
-
-  test('getScheduleContentRows', async () => {
-    const results = await getScheduleContentRows('drumeo')
-    console.log(results.data[1])
-    expect(results.type).toBeDefined()
-    expect(results.type).toBe('sections')
-    expect(results.data).toBeDefined()
-    expect(results.meta).toBeDefined()
-  })
-
-  test('getSpecificScheduleContentRow', async () => {
-    const results = await getScheduleContentRows('drumeo', 'Leaving-Soon')
-    console.log(results)
-    expect(results.type).toBeDefined()
-    expect(results.type).toBe('catalog')
-    expect(results.data).toBeDefined()
-    expect(results.meta).toBeDefined()
-  })
-})

package/test/localStorageMock.js

@@ -1,21 +0,0 @@
-export class LocalStorageMock {
-  constructor() {
-    this.store = {}
-  }
-
-  clear() {
-    this.store = {}
-  }
-
-  getItem(key) {
-    return this.store[key] || null
-  }
-
-  setItem(key, value) {
-    this.store[key] = String(value)
-  }
-
-  removeItem(key) {
-    delete this.store[key]
-  }
-}

package/test/log.js

@@ -1,5 +0,0 @@
-export function log(message, forceEnabled = false) {
-  if (process.env.DEBUG === 'true' || forceEnabled) {
-    console.log(message)
-  }
-}