chore: upgrade Expo SDK 55 → 57 and React Native 0.83 → 0.86 (#54)

packages/core/test/availability.test.ts

@@ -11,7 +11,9 @@ import {
 } from "@sofa/test/db";
 
 const { getWatchProviders } = vi.hoisted(() => ({
-  getWatchProviders: vi.fn(async () => ({ results: {} as Record<string, unknown> })),
+  getWatchProviders: vi.fn<() => Promise<{ results: Record<string, unknown> }>>(async () => ({
+    results: {},
+  })),
 }));
 
 vi.mock("@sofa/tmdb/client", () => ({

packages/core/test/credits.test.ts

@@ -4,11 +4,15 @@ import { persons, titleCast } from "@sofa/db/schema";
 import { clearAllTables, eq, insertTitle, testDb } from "@sofa/test/db";
 
 const { mockGetMovieCredits, mockGetTvAggregateCredits } = vi.hoisted(() => ({
-  mockGetMovieCredits: vi.fn(async () => ({
+  mockGetMovieCredits: vi.fn<
+    () => Promise<{ cast: Record<string, unknown>[]; crew: Record<string, unknown>[] }>
+  >(async () => ({
     cast: [] as Record<string, unknown>[],
     crew: [] as Record<string, unknown>[],
   })),
-  mockGetTvAggregateCredits: vi.fn(async () => ({
+  mockGetTvAggregateCredits: vi.fn<
+    () => Promise<{ cast: Record<string, unknown>[]; crew: Record<string, unknown>[] }>
+  >(async () => ({
     cast: [] as Record<string, unknown>[],
     crew: [] as Record<string, unknown>[],
   })),

packages/core/test/import-processor.test.ts

@@ -370,7 +370,8 @@ describe("processImportJob — state transitions", () => {
       })
       .run();
 
-    expect(() =>
+    let duplicateActiveJobRejected = false;
+    try {
       testDb
         .insert(importJobs)
         .values({
@@ -390,8 +391,11 @@ describe("processImportJob — state transitions", () => {
           importRatings: true,
           createdAt,
         })
-        .run(),
-    ).toThrow("UNIQUE constraint");
+        .run();
+    } catch {
+      duplicateActiveJobRejected = true;
+    }
+    expect(duplicateActiveJobRejected).toBe(true);
 
     testDb.update(importJobs).set({ status: "success" }).where(eq(importJobs.id, "job-1")).run();
 

packages/core/test/lists.test.ts

@@ -9,9 +9,8 @@ import {
 } from "@sofa/test/db";
 
 const { mockGetTvExternalIds } = vi.hoisted(() => ({
-  mockGetTvExternalIds: vi.fn(
-    (): Promise<{ tvdb_id: number | null; imdb_id: string | null }> =>
-      Promise.resolve({ tvdb_id: 55555, imdb_id: "tt9999999" }),
+  mockGetTvExternalIds: vi.fn<() => Promise<{ tvdb_id: number | null; imdb_id: string | null }>>(
+    () => Promise.resolve({ tvdb_id: 55555, imdb_id: "tt9999999" }),
   ),
 }));
 

packages/core/test/metadata-refresh.test.ts

@@ -40,7 +40,7 @@ const defaultSeasonDetails: MockSeasonDetails = {
 };
 
 const { mockGetTvSeasonDetails } = vi.hoisted(() => ({
-  mockGetTvSeasonDetails: vi.fn(),
+  mockGetTvSeasonDetails: vi.fn<() => Promise<MockSeasonDetails>>(),
 }));
 
 vi.mock("@sofa/tmdb/client", () => ({

packages/core/test/person-detail.test.ts

@@ -4,7 +4,7 @@ import { personFilmography, persons } from "@sofa/db/schema";
 import { clearAllTables, eq, testDb } from "@sofa/test/db";
 
 const TINY_PNG = Buffer.from(
-  "iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVQIHWP4//8/AwAI/AL+X2NDNwAAAABJRU5ErkJggg==",
+  "iVBORw0KGgoAAAANSUhEUgAAAAIAAAACCAYAAABytg0kAAAACXBIWXMAAAPoAAAD6AG1e1JrAAAAEUlEQVQImWP4z8DwH4QZYAwAR8oH+Xm0fdIAAAAASUVORK5CYII=",
   "base64",
 );
 
@@ -43,8 +43,8 @@ const defaultCombinedCredits = {
 };
 
 const { mockGetPersonDetails, mockGetPersonCombinedCredits } = vi.hoisted(() => ({
-  mockGetPersonDetails: vi.fn(),
-  mockGetPersonCombinedCredits: vi.fn(),
+  mockGetPersonDetails: vi.fn<() => Promise<typeof defaultPersonDetails>>(),
+  mockGetPersonCombinedCredits: vi.fn<() => Promise<typeof defaultCombinedCredits>>(),
 }));
 
 vi.mock("@sofa/tmdb/client", () => ({

packages/core/test/thumbhash.test.ts

@@ -4,7 +4,7 @@ import { episodes, seasons, titles } from "@sofa/db/schema";
 import { clearAllTables, eq, testDb } from "@sofa/test/db";
 
 const TINY_PNG = Buffer.from(
-  "iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVQIHWP4//8/AwAI/AL+X2NDNwAAAABJRU5ErkJggg==",
+  "iVBORw0KGgoAAAANSUhEUgAAAAIAAAACCAYAAABytg0kAAAACXBIWXMAAAPoAAAD6AG1e1JrAAAAEUlEQVQImWP4z8DwH4QZYAwAR8oH+Xm0fdIAAAAASUVORK5CYII=",
   "base64",
 );
 

packages/core/test/webhooks.test.ts

@@ -18,8 +18,8 @@ import {
 } from "../src/webhooks";
 
 const { mockResolveMovieTmdbId, mockResolveShowTmdbId } = vi.hoisted(() => ({
-  mockResolveMovieTmdbId: vi.fn(async () => null as number | null),
-  mockResolveShowTmdbId: vi.fn(async () => null as number | null),
+  mockResolveMovieTmdbId: vi.fn<() => Promise<number | null>>(async () => null),
+  mockResolveShowTmdbId: vi.fn<() => Promise<number | null>>(async () => null),
 }));
 
 vi.mock("../src/imports/resolve", () => ({
@@ -28,8 +28,8 @@ vi.mock("../src/imports/resolve", () => ({
 }));
 
 const { mockGetOrFetchTitleByTmdbId, mockRefreshTvChildren } = vi.hoisted(() => ({
-  mockGetOrFetchTitleByTmdbId: vi.fn(async () => null as { id: string } | null),
-  mockRefreshTvChildren: vi.fn(async () => {}),
+  mockGetOrFetchTitleByTmdbId: vi.fn<() => Promise<{ id: string } | null>>(async () => null),
+  mockRefreshTvChildren: vi.fn<() => Promise<void>>(async () => {}),
 }));
 
 vi.mock("../src/metadata", () => ({
@@ -38,7 +38,9 @@ vi.mock("../src/metadata", () => ({
 }));
 
 const { mockGetTvDetails } = vi.hoisted(() => ({
-  mockGetTvDetails: vi.fn(async () => ({ number_of_seasons: 1 })),
+  mockGetTvDetails: vi.fn<() => Promise<{ number_of_seasons: number }>>(async () => ({
+    number_of_seasons: 1,
+  })),
 }));
 
 vi.mock("@sofa/tmdb/client", () => ({