musora-content-services 2.152.1 → 2.154.0

package/src/services/multi-user-accounts/multi-user-accounts.ts

@@ -14,11 +14,16 @@ export interface User {
 }
 
 export interface InviteResponse {
-  email: string
   id: number
   created_at: string
   expires_at: string
-  existing_user_details: User
+  can_be_accepted: boolean
+  is_account_valid: boolean
+  is_invite_active: boolean
+  can_user_join: boolean
+  // These fields leak user information and are excluded entirely for the public endpoint
+  existing_user_details?: User
+  email?: string
 }
 
 export interface UsersMultiAccountResponse {
@@ -27,7 +32,7 @@ export interface UsersMultiAccountResponse {
   last_cancelled_multi_user_account: MultiUserAccountResponse
   is_active_primary: boolean
   is_active_sub: boolean
-  active_invite: InviteResponse
+  active_invites: InviteResponse[]
 }
 
 export interface MultiUserAccountResponse {
@@ -35,13 +40,15 @@ export interface MultiUserAccountResponse {
   product_name: string
   is_active: boolean
   primary_user: User
-  active_invited_emails: string[]
-  available_seats: number
-  available_invites: number
   total_seats: number
-  active_subs: User[]
   end_time: string
   is_primary_account_holder: boolean
+  // The following fields are not included for public or subaccount users
+  active_invited_emails?: InviteResponse[]
+  available_seats?: number
+  available_invites?: InviteResponse[]
+  active_subs?: User[]
+  show_welcome?: boolean
 }
 
 export interface CreateAccountParams {
@@ -53,6 +60,10 @@ export interface CreateInvitesParams {
   emails: string[]
 }
 
+export interface UpdateMultiUserAccountParams {
+  show_welcome: boolean
+}
+
 
 /**
  * Creates a new multi-user account with optional invites and seat count.
@@ -78,6 +89,19 @@ export async function fetchUsersMultiAccountDetails(userId: number): Promise<Use
   return httpClient.get<UsersMultiAccountResponse>(`${baseUrl}/${userId}/details`)
 }
 
+
+/**
+ * Fetch invite details
+ *
+ * @param {number} inviteId - The ID of the invite to check
+ * @returns {Promise<InviteResponse>} - A promise that resolves to the invite details.
+ * @throws {HttpError} - If the HTTP request fails.
+ */
+export async function fetchInvite(inviteId: number): Promise<InviteResponse> {
+  const httpClient = new HttpClient(globalConfig.baseUrl)
+  return httpClient.get<InviteResponse>(`${baseUrl}/invites/${inviteId}`)
+}
+
 /**
  * Creates invitations for an existing multi-user account.
  *
@@ -125,3 +149,15 @@ export async function rescindInvite(inviteId: number): Promise<void> {
 export async function removeUserFromActiveMultiUserAccount(userId: number): Promise<MultiUserAccountResponse|void> {
   return DELETE(`${globalConfig.baseUrl}${baseUrl}/${userId}/remove`, {})
 }
+
+/**
+ * Updates specified fields on a multi-user account. Authorized user must be the primary account owner
+ *
+ * @param {UpdateMultiUserAccountParams} params - The parameters for updating the account.
+ * @returns {Promise<MultiUserAccountResponse>} - Updated MultiUserAccountResponse if account owner
+ * @throws {HttpError} - If the request fails.
+ */
+export async function updateMultiUserAccount(params: UpdateMultiUserAccountParams): Promise<MultiUserAccountResponse> {
+  const httpClient = new HttpClient(globalConfig.baseUrl)
+  return httpClient.patch(`${globalConfig.baseUrl}${baseUrl}/update`, params)
+}

package/src/services/reporting/reporting.ts

@@ -11,9 +11,8 @@ import { HttpClient } from '../../infrastructure/http/HttpClient'
 import { globalConfig } from '../config.js'
 import { ReportResponse, ReportableType, IssueTypeMap, ReportIssueOption } from './types'
 import { Brands } from '../../lib/brands'
-import { generateContentUrl, generatePlaylistUrl, generateForumPostUrl, generateCommentUrl } from '../urlBuilder.ts'
-import {fetchByRailContentId} from "../../index";
-import {fetchByRailContentIds} from "../sanity";
+import { generateContentUrl, generatePlaylistUrl, generateForumPostUrl, generateCommentUrl } from '../urlBuilder'
+import {fetchByRailContentId, fetchByRailContentIds} from "../sanity";
 import {addContextToContent} from "../contentAggregator";
 
 /**
@@ -122,7 +121,7 @@ export async function report<T extends ReportableType>(
       id: params.id
     })
   } else if (params.type === 'forum_post') {
-    const { fetchPost } = await import('../forums/posts.ts')
+    const { fetchPost } = await import('../forums/posts')
     const post = await fetchPost(params.id, params.brand)
 
     if (post?.thread) {

package/src/services/sanity.js

@@ -33,8 +33,9 @@ import {
   SONG_TYPES_WITH_CHILDREN,
   liveFields,
   postProcessBadge,
-  parentField,
-  grandParentField, parentRecentTypes,
+  parentRecentTypes,
+  parentReferenceField,
+  grandParentReferenceField,
 } from '../contentTypeConfig.js'
 import { fetchSimilarItems } from './recommendations.js'
 import { getSongType, processMetadata, ALWAYS_VISIBLE_TABS, CONTENT_STATUSES } from '../contentMetaData.js'
@@ -952,7 +953,7 @@ export async function fetchLessonContent(railContentId, { addParent = false } =
   }
 
   const parentQuery = addParent
-    ? `"parent_content_data": *[railcontent_id in [...(^.parent_content_data[].id)]]{
+    ? `"parent_content_data": parent_content_reference[]->{
         "id": railcontent_id,
         title,
         slug,
@@ -960,11 +961,12 @@ export async function fetchLessonContent(railContentId, { addParent = false } =
         "logo" : logo_image_url.asset->url,
         "dark_mode_logo": dark_mode_logo_url.asset->url,
         "light_mode_logo": light_mode_logo_url.asset->url,
-        "badge": *[references(^._id) && _type == 'content-award'][0].badge.asset->url,
-        "badge_rear": *[references(^._id) && _type == 'content-award'][0].badge_rear.asset->url,
-        "badge_logo": *[references(^._id) && _type == 'content-award'][0].logo.asset->url,
-        'parentCount': coalesce(count(parent_content_data), 0)
-      } | order(parentCount desc),`
+        ...*[references(^._id) && _type == 'content-award'][0]{
+          "badge": badge.asset->url,
+          "badge_rear": badge_rear.asset->url,
+          "badge_logo": logo.asset->url,
+        }
+      },`
     : ''
 
   const fields = `${getFieldsForContentType()}
@@ -1106,19 +1108,6 @@ async function fetchRelatedByLicense(railcontentId, brand, onlyUseSongTypes, cou
  * @returns {Promise<Array<Object>|null>} - The fetched related lessons data or null if not found.
  */
 export async function fetchSiblingContent(railContentId, brand = null) {
-  const filterGetParent = await new FilterBuilder(`references(^._id) && _type == ^.parent_type`, {
-    pullFutureContent: true,
-    showMembershipRestrictedContent: true, // Show parent even without permissions
-  }).buildFilter()
-  const filterForParentList = await new FilterBuilder(
-    `references(^._id) && _type == ^.parent_type`,
-    {
-      pullFutureContent: true,
-      isParentFilter: true,
-      showMembershipRestrictedContent: true, // Show parent even without permissions
-    }
-  ).buildFilter()
-
   const childrenFilter = await new FilterBuilder(``, {
     isChildrenFilter: true,
     showMembershipRestrictedContent: true, // Show all lessons in sidebar, need_access applied on individual page
@@ -1126,20 +1115,20 @@ export async function fetchSiblingContent(railContentId, brand = null) {
 
   const brandString = brand ? ` && brand == "${brand}"` : ''
   const queryFields = getFieldsForContentType()
-
+  const courseCollectionFields = await getFieldsForContentTypeWithFilteredChildren('course-collection')
   const query = `*[railcontent_id == ${railContentId}${brandString}]{
     _type,
     parent_type,
     railcontent_id,
-    'parent_id': ${parentField}.id,
-    'grandparent_id':${grandParentField}.id,
-    'for-calculations': *[${filterGetParent}][0]{
-    'siblings-list': child[]->railcontent_id,
-    'parents-list': *[${filterForParentList}][0].child[]->railcontent_id
+    'parent_id': ${parentReferenceField}->railcontent_id,
+    'grandparent_id': ${grandParentReferenceField}->railcontent_id,
+    'collection_data': ${grandParentReferenceField}->{${courseCollectionFields}},
+    'for-calculations': ${parentReferenceField}->{
+      'siblings-list': child[]->railcontent_id,
+      'parents-list': ${parentReferenceField}->child[]->railcontent_id
     },
-    "related_lessons" : *[${filterGetParent}][0].child[${childrenFilter}]->{${queryFields}}
+    "related_lessons" : ${parentReferenceField}->child[${childrenFilter}]->{${queryFields}}
   }`
-
   let result = await fetchSanity(query, false, { processNeedAccess: true })
 
   //there's no way in sanity to retrieve the index of an array, so we must calculate after fetch
@@ -1152,10 +1141,6 @@ export async function fetchSiblingContent(railContentId, brand = null) {
 
     delete result['for-calculations']
 
-    if (result['grandparent_id']) {
-      result['collection_data'] = await fetchCourseCollectionData(result['grandparent_id'])
-    }
-
     result = { ...result, parentCount, currentParentIndex, siblingCount, currentSiblingIndex }
     return result
   } else {
@@ -1199,7 +1184,7 @@ export async function fetchRelatedLessons(railContentId) {
 }
 
 export async function fetchLiveEvent(brand, forcedContentId = null) {
-  const LIVE_EXTRA_MINUTES = 30
+  const LIVE_EXTRA_MINUTES = 15
   //calendarIDs taken from addevent.php
   // TODO import instructor calendars to Sanity
   let defaultCalendarID = ''
@@ -1320,16 +1305,12 @@ export async function fetchByReference(
  * @returns {Promise<int|null>}
  */
 export async function fetchTopLevelParentId(railcontentId) {
-  const parentFilter = 'railcontent_id in [...(^.parent_content_data[].id)] && (!defined(parent_content_data) || count(parent_content_data) == 0)'
-  const statusFilter = "&& status in ['scheduled', 'published', 'archived', 'unlisted']"
-
   const query = `*[railcontent_id == ${railcontentId}]{
-      railcontent_id,
-      'top_parent': *[${parentFilter} ${statusFilter}][0].railcontent_id
+      'top_parent': coalesce(${grandParentReferenceField}->railcontent_id, ${parentReferenceField}->railcontent_id, railcontent_id),
     }`
   let response = await fetchSanity(query, false, { processNeedAccess: false })
   if (!response) return null
-  return response['top_parent'] ?? response['railcontent_id']
+  return response['top_parent'] ?? railcontentId
 }
 
 export async function getHierarchy(contentId, collection) {
@@ -1403,31 +1384,18 @@ async function fetchALaCarteHierarchyData(railcontentId) {
   const childrenFilter = await new FilterBuilder(``, { isChildrenFilter: true }).buildFilter()
   const query = `*[railcontent_id == ${topLevelId}]{
       railcontent_id,
-      'metadata': { brand, 'type': _type, 'parent_id':  coalesce(parent_content_data[0].id, 0) },
+      'metadata': { brand, 'type': _type, 'parent_id':  coalesce(${parentReferenceField}->railcontent_id, 0) },
       'assignments': assignment[]{railcontent_id},
       'children': child[${childrenFilter}]->{
         railcontent_id,
-        'metadata': {
-  brand, 'type': _type, 'parent_id':  coalesce(parent_content_data[0].id, 0) },
+        'metadata': { brand, 'type': _type, 'parent_id':  coalesce(${parentReferenceField}->railcontent_id, 0) },
         'assignments': assignment[]{railcontent_id},
         'children': child[${childrenFilter}]->{
             railcontent_id,
-            'metadata': {
-      brand, 'type': _type, 'parent_id':  coalesce(parent_content_data[0].id, 0) },
+            'metadata': { brand, 'type': _type, 'parent_id':  coalesce(${parentReferenceField}->railcontent_id, 0) },
             'assignments': assignment[]{railcontent_id},
-            'children': child[${childrenFilter}]->{
-               railcontent_id,
-               'metadata': {
-         brand, 'type': _type, 'parent_id':  coalesce(parent_content_data[0].id, 0) },
-               'assignments': assignment[]{railcontent_id},
-               'children': child[${childrenFilter}]->{
-                  railcontent_id,
-                  'metadata': {
-            brand, 'type': _type, 'parent_id':  coalesce(parent_content_data[0].id, 0) },
-            }
-          }
         }
-      },
+      }
     }`
   return await fetchSanity(query, false, { processNeedAccess: false })
 }
@@ -2000,7 +1968,7 @@ export async function fetchTabData(
     ? `&& !(railcontent_id in [${excludeIds.join(',')}])`
     : ''
 
-  const excludeCoursesInCourseCollectionsFilter = `&& !(_type == 'course' && defined(parent_content_data))`
+  const excludeCoursesInCourseCollectionsFilter = `&& !(_type == 'course' && defined(parent_content_reference) && count(parent_content_reference[]) > 0)`
 
   filter = `brand == "${brand}" && (defined(railcontent_id)) ${includedFieldsFilter} ${progressFilter} ${excludedIdsFilter} ${excludeCoursesInCourseCollectionsFilter}`
   const childrenFilter = await new FilterBuilder(``, {

package/src/services/sync/adapters/lokijs.ts

@@ -1,14 +1,17 @@
 import { SyncTelemetry } from '../telemetry'
 
-import LokiJSAdapter from '@nozbe/watermelondb/adapters/lokijs'
+import _LokiJSAdapter from '@nozbe/watermelondb/adapters/lokijs'
 
 import { deleteDatabase, lokiFatalError } from '@nozbe/watermelondb/adapters/lokijs/worker/lokiExtensions'
 
+// Handle CJS/ESM interop: in Node.js ESM the default import is the exports object
+const LokiJSAdapter = (_LokiJSAdapter as any).default ?? _LokiJSAdapter
+
 export type LokiExtensions = {
   onPersistenceError?: (err: Error) => void
 }
 
-export default class LokiPersistenceErrorAwareAdapter extends LokiJSAdapter {
+export default class LokiPersistenceErrorAwareAdapter extends (LokiJSAdapter as typeof _LokiJSAdapter) {
   constructor(options: any, extensions: LokiExtensions = {}) {
     super(options);
     const that = this;
@@ -162,7 +165,7 @@ export function simulateIndexedDBQuotaExceeded() {
   })
 }
 
-export function abortWritesToDatabase(adapter: LokiJSAdapter) {
+export function abortWritesToDatabase(adapter: typeof LokiJSAdapter) {
   // acts as handy helper to disable loki's save methods entirely
   lokiFatalError(adapter._driver.loki)
   return Promise.resolve()
@@ -174,7 +177,7 @@ export function abortWritesToDatabase(adapter: LokiJSAdapter) {
  * Haven't encountered live issues related to this yet, but theoretically provides
  * the cleanest slate for a user to recover from schema issues?
  */
-export function destroyDatabase(dbName: string, adapter: LokiJSAdapter): Promise<void> {
+export function destroyDatabase(dbName: string, adapter: typeof LokiJSAdapter): Promise<void> {
   return new Promise(async (resolve, reject) => {
     if (adapter._driver) {
       try {

package/src/services/sync/fetch.ts

@@ -122,7 +122,7 @@ export interface SyncResponseBase {
 
 export type PushPayload = {
   entries: ({
-    record: BaseModel
+    record: Record<string, unknown>
     meta: {
       ids: {
         id: string
@@ -135,23 +135,11 @@ export type PushPayload = {
       ids: {
         id: string
       }
-      deleted_at: EpochMs
+      deleted_at: number
     }
   })[]
 }
 
-interface ServerPushPayload {
-  entries: {
-    record: BaseModel | null
-    meta: {
-      ids: {
-        id: string
-      },
-      deleted_at: EpochMs | null
-    }
-  }[]
-}
-
 export function makeFetchRequest(input: RequestInfo, init?: RequestInit) {
   return (userId: number, context: SyncContext) => new Request(globalConfig.baseUrl + input, {
     ...init,

package/src/services/sync/repositories/base.ts

@@ -19,6 +19,10 @@ export default class SyncRepository<TModel extends BaseModel> {
     this.store = store
   }
 
+  async getAll() {
+    return this.readAll()
+  }
+
   protected async readOne(id: RecordId) {
     return this._respondToRead(() => this.store.readOne(id))
   }

package/src/services/sync/repositories/content-progress.ts

@@ -23,7 +23,7 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
       Q.sortBy('updated_at', 'desc'),
 
       ...(limit ? [Q.take(limit)] : []),
-    ])
+    ].filter(Boolean) as Q.Clause[])
 
     return opts.onlyIds
         ? results.data.map((r) => r.content_id)
@@ -44,7 +44,7 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
       Q.sortBy('updated_at', 'desc'),
 
       ...(limit ? [Q.take(limit)] : []),
-    ])
+    ].filter(Boolean) as Q.Clause[])
 
     return opts.onlyIds
       ? results.data.map((r) => r.content_id)
@@ -78,7 +78,7 @@ export default class ProgressRepository extends SyncRepository<ContentProgress>
 
       Q.or(Q.where('state', STATE.STARTED), Q.where('state', STATE.COMPLETED)),
       Q.sortBy('updated_at', 'desc'),
-    ]
+    ].filter(Boolean) as Q.Clause[]
 
     if (opts.updatedAfter) {
       clauses.push(Q.where('updated_at', Q.gte(opts.updatedAfter)))

package/src/services/sync/store/index.ts

@@ -892,7 +892,7 @@ export default class SyncStore<TModel extends BaseModel = BaseModel> {
 
           default:
             this.telemetry.error(`[store:${this.model.table}] Unknown record status`, {
-              status: existing._raw._status,
+              extra: { status: existing._raw._status },
             })
         }
       } else {
@@ -999,6 +999,11 @@ export default class SyncStore<TModel extends BaseModel = BaseModel> {
         }, 'sync.cleanup')
       })
     }, SyncStore.CLEANUP_INTERVAL)
+
+    // in tests in node env, prevents the timer from keeping the process alive
+    if (typeof (this.cleanupTimer as any).unref === 'function') {
+      (this.cleanupTimer as any).unref()
+    }
   }
 
   private stopCleanupTimer() {

package/src/services/sync/strategies/base.ts

@@ -4,7 +4,7 @@ import SyncStore from "../store";
 
 type SyncCallback = (reason: string) => void
 
-type SyncCallbacks = {
+export type SyncCallbacks = {
   callback: SyncCallback
   requestSync: SyncCallback
   requestPull: SyncCallback

package/src/services/sync/telemetry/index.ts

@@ -10,7 +10,7 @@ export type SentryLike = {
   captureMessage: typeof InjectedSentry.captureMessage
   addBreadcrumb: typeof InjectedSentry.addBreadcrumb
   startSpan: typeof InjectedSentry.startSpan
-  logger: typeof InjectedSentry.logger
+  logger: { debug: (...args: any[]) => void; info: (...args: any[]) => void }
 }
 
 export type StartSpanOptions = Parameters<typeof InjectedSentry.startSpan>[0]

package/src/services/urlBuilder.ts

@@ -38,6 +38,7 @@ export interface ContentUrlParams {
   /** Navigation target (optional) */
   navigateTo?: {
     id: number
+    child?: { id: number }
   }
   /** Brand (drumeo, pianote, guitareo, singeo, playbass) */
   brand?: Brand

package/src/services/user/streakCalculator.ts

@@ -43,7 +43,7 @@ class StreakCalculator {
   }
 
   private async fetchAllPractices(): Promise<PracticeData> {
-    const query = await db.practices.queryAll()
+    const query = await db.practices.getAll()
 
     return query.data.reduce((acc, practice) => {
       acc[practice.date] = acc[practice.date] || []

package/test/SKIPPED_TESTS.md

@@ -0,0 +1,151 @@
+# Skipped Tests Reference
+
+This document tracks all skipped tests and why they are skipped. Tests are divided into two categories:
+
+1. **Skipped for CI** — were passing but depend on live external services; skipped to enable clean CI runs
+2. **Previously skipped — failing or unknown** — were already skipped before CI work; many confirmed failing
+
+The goal is to eventually move all Category 1 tests into a dedicated integration/live test suite, and to triage Category 2 tests as either fixable or retired.
+
+---
+
+## Category 1: Skipped for CI (were passing, have external dependencies)
+
+### `test/sanityQueryService.test.js` — Sanity CMS
+
+All tests in this file call real Sanity GROQ queries via `initializeTestService(true)`.
+
+| Test | Dependency |
+|---|---|
+| fetchSongById | Sanity |
+| fetchReturning | Sanity |
+| fetchLeaving | Sanity |
+| fetchComingSoon | Sanity |
+| fetchSanity-WithPostProcess | Sanity |
+| fetchSanityPostProcess | Sanity |
+| fetchByRailContentIds | Sanity |
+| fetchByRailContentIds_Order | Sanity |
+| fetchUpcomingNewReleases | Sanity |
+| fetchLessonContent | Sanity |
+| fetchAllSongsInProgress | Sanity |
+| fetchNewReleases | Sanity |
+| fetchAllWorkouts | Sanity |
+| fetchAllInstructorField | Sanity |
+| fetchAllInstructors | Sanity |
+| fetchAll-CustomFields | Sanity |
+| fetchRelatedLessons | Sanity |
+| fetchRelatedLessons-quick-tips | Sanity |
+| fetchRelatedLessons-in-rhythm | Sanity |
+| getSortOrder | Sanity (describe block requires live auth) |
+| fetchAll-WithProgress | Sanity |
+| fetchAllFilterOptions-WithProgress | Sanity |
+| fetchAll-IncludedFields | Sanity |
+| fetchAll-IncludedFields-rudiment-multiple-gear | Sanity |
+| fetchByReference | Sanity |
+| fetchScheduledReleases | Sanity |
+| fetchAll-GroupBy-Artists | Sanity |
+| fetchAll-GroupBy-Instructors | Sanity |
+| fetchMetadata | Sanity |
+| fetchMetadata-Coach-Lessons | Sanity |
+| invalidContentType | Sanity (describe block requires live auth) |
+| metaDataForLessons | Sanity |
+| metaDataForSongs | Sanity |
+| fetchAllFilterOptionsLessons | Sanity |
+| fetchAllFilterOptionsSongs | Sanity |
+| fetchLiveEvent | Sanity |
+| fetchRelatedLessons-pack-bundle-lessons | Sanity |
+| fetchRelatedLessons-course-parts | Sanity |
+| fetchRelatedLessons-song-tutorial-children | Sanity |
+| fetchMetadata (second) | Sanity |
+
+### `test/content.test.js` — Sanity CMS + Railcontent API
+
+| Test | Dependency |
+|---|---|
+| getTabResults-Singles | Sanity + Railcontent |
+| getTabResults-Courses | Sanity + Railcontent |
+| getTabResults-Type-Explore-All | Sanity + Railcontent |
+
+### `test/user/permissions.test.js` — Railcontent API
+
+| Test | Dependency |
+|---|---|
+| fetchUserPermissions | Railcontent `fetchUserPermissionsData` |
+
+---
+
+## Category 2: Previously Skipped — Failing or Unknown State
+
+These were already skipped before CI work. Status is noted where confirmed.
+
+### `test/sanityQueryService.test.js`
+
+| Test | Status | Failure Reason |
+|---|---|---|
+| fetchSongArtistCount | Unknown | — |
+| fetchUpcomingEvents | Unknown | — |
+| fetchLessonContent-PlayAlong-containts-array-of-videos | Unknown | — |
+| fetchAllSortField | Unknown | — |
+| fetchRelatedLessons-child | Unknown | — |
+| fetchPackAll | Unknown | — |
+| fetchAllPacks | Unknown | — |
+| fetchAll-IncludedFields-multiple | Unknown | — |
+| fetchAll-IncludedFields-playalong-multiple | Unknown | — |
+| fetchAll-IncludedFields-coaches-multiple-focus | Unknown | — |
+| fetchAll-IncludedFields-songs-multiple-instrumentless | Unknown | — |
+| fetchAll-GroupBy-Genre | Unknown | — |
+| fetchShowsData | Unknown | — |
+| fetchShowsData-OddTimes | Unknown | — |
+| fetchTopLevelParentId | Unknown | — |
+| fetchHierarchy | Unknown | — |
+| fetchTopLeveldrafts | Failing | Timeout (>5s) |
+| fetchCommentData | Failing | `null.forEach` — Sanity returns null for content IDs |
+| baseConstructor | Unknown | — |
+| withOnlyFilterAvailableStatuses | Unknown | — |
+| withContentStatusAndFutureScheduledContent | Unknown | — |
+| withUserPermissions | Unknown | — |
+| withUserPermissionsForPlusUser | Unknown | — |
+| withPermissionBypass | Unknown | — |
+| withPublishOnRestrictions | Unknown | — |
+| fetchAllFilterOptions | Unknown | — |
+| fetchAllFilterOptions-Rudiment | Unknown | — |
+| fetchAllFilterOptions-PlayAlong | Unknown | — |
+| fetchAllFilterOptions-Coaches | Unknown | — |
+| fetchAllFilterOptions-filter-selected | Failing | `null.meta` — API returns null for filter combination |
+| customBrandTypeExists | Unknown | — |
+| withCommon | Unknown | — |
+| fetchOtherSongVersions | Failing | 0 results — content is drafted/admin-only |
+| fetchLessonsFeaturingThisContent | Failing | 0 results — content is drafted/admin-only |
+| getRecommendedForYou | Failing | `SyncError: Intended user ID does not match` |
+| getRecommendedForYou-SeeAll | Failing | `SyncError: Intended user ID does not match` |
+
+### `test/content.test.js`
+
+| Test | Status | Failure Reason |
+|---|---|---|
+| getTabResults-Filters | Failing | Timeout (>5s) |
+| getTabResults-Type-Filter | Failing | `TypeError: null.entity` — Sanity returns null |
+| getContentRows | Unknown |Sanity & pw-recommender  |
+| getNewAndUpcoming | Failing | Timeout (>5s) |
+| getScheduleContentRows | Failing | Timeout (>5s) |
+| getSpecificScheduleContentRow | Failing | Timeout (>5s) |
+
+### `test/contentProgress.test.js`
+
+| Test | Status | Failure Reason |
+|---|---|---|
+| get-Songs-Tutorials | Unknown | Live Sanity call |
+| get-Songs-Transcriptions | Unknown | Live Sanity call |
+| get-Songs-Play-Alongs | Unknown | Live Sanity call |
+
+### `test/progressRows.test.js`
+
+| Test | Status | Failure Reason |
+|---|---|---|
+| check progress rows logic | Failing | Stale mock data — not a live API issue; mock data no longer reflects current data shape |
+
+### `test/learningPaths.test.js`
+
+| Test | Status | Failure Reason |
+|---|---|---|
+| learningPathCompletion | Unknown | Uses `initializeTestService(true)` — live API |

package/test/initializeTests.js

@@ -1,7 +1,7 @@
 import { globalConfig, initializeService } from '../src'
 import { LocalStorageMock } from './localStorageMock'
-import { initializeSyncManager } from './sync/initialize-sync-manager'
 const railContentModule = require('../src/services/railcontent.js')
+const awardDefsModule = require('../src/services/awards/internal/award-definitions.js')
 let token = null
 let userId = process.env.RAILCONTENT_USER_ID ?? null
 
@@ -51,11 +51,10 @@ export async function initializeTestService(useLive = false, isAdmin = false) {
     localStorage: new LocalStorageMock(),
     isMA: true,
   }
+  jest.spyOn(awardDefsModule.awardDefinitions, 'initialize').mockResolvedValue()
   initializeService(config)
   // Mock user permissions
   let permissionsMock = jest.spyOn(railContentModule, 'fetchUserPermissionsData')
   let permissionsData = { permissions: [108, 91, 92], isAdmin: isAdmin }
   permissionsMock.mockImplementation(() => permissionsData)
-
-  initializeSyncManager(userId)
 }