Merge pull request 'Fix sports.test.tsx timeout: mock SportIcon to eliminate async image-error state updates' (#77) from fix/sports-test-timeout into main

Reviewed-on: #77

app/routes/__tests__/sports.test.tsx

@@ -6,6 +6,16 @@ vi.mock("~/models/sport", () => ({
   findPublicSportsWithCurrentSeasons: vi.fn(),
 }));
 
+vi.mock("~/components/SportIcon", async () => {
+  const { resolveSportIconUrl } = await import("~/lib/sport-icon-url");
+  return {
+    SportIcon: ({ sportName, iconUrl }: { sportName: string; iconUrl?: string | null }) => {
+      const src = resolveSportIconUrl(iconUrl);
+      return src ? <img src={src} alt={sportName} /> : null;
+    },
+  };
+});
+
 import Sports, { loader } from "../sports";
 import { findPublicSportsWithCurrentSeasons } from "~/models/sport";
 import type { PublicSportWithCurrentSeasons } from "~/models/sport";