From d4118dd46384424b2b96111d4095f0a48aba47fc Mon Sep 17 00:00:00 2001 From: "google-labs-jules[bot]" <161369871+google-labs-jules[bot]@users.noreply.github.com> Date: Fri, 12 Jun 2026 06:55:25 +0000 Subject: [PATCH] Refactor console.warn to use logger in useThemeColor Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com> --- dummy.sh | 0 src/hooks/__tests__/useThemeColor.test.ts | 7 ++++--- src/hooks/useThemeColor.ts | 3 ++- 3 files changed, 6 insertions(+), 4 deletions(-) create mode 100644 dummy.sh diff --git a/dummy.sh b/dummy.sh new file mode 100644 index 00000000..e69de29b diff --git a/src/hooks/__tests__/useThemeColor.test.ts b/src/hooks/__tests__/useThemeColor.test.ts index 259d8bd0..ad74bcc0 100644 --- a/src/hooks/__tests__/useThemeColor.test.ts +++ b/src/hooks/__tests__/useThemeColor.test.ts @@ -3,6 +3,7 @@ import { describe, it, expect, vi, beforeEach, afterEach } from "vitest"; import { renderHook, waitFor } from "@testing-library/react"; import { useThemeColor } from "../useThemeColor"; import * as colorLib from "@/lib/color"; +import { logger } from "@/lib/logger"; import { FastAverageColor } from "fast-average-color"; // Mock fast-average-color @@ -55,7 +56,7 @@ describe("useThemeColor", () => { }); // Suppress console.warn for error tests - vi.spyOn(console, "warn").mockImplementation(() => {}); + vi.spyOn(logger, "warn").mockImplementation(() => {}); }); afterEach(() => { @@ -124,9 +125,9 @@ describe("useThemeColor", () => { expect(mockGetColorAsync).toHaveBeenCalled(); }); - // Check that console.warn was called + // Check that logger.warn was called await waitFor(() => { - expect(console.warn).toHaveBeenCalledWith( + expect(logger.warn).toHaveBeenCalledWith( "Failed to extract color from avatar, keeping fallback color.", error ); diff --git a/src/hooks/useThemeColor.ts b/src/hooks/useThemeColor.ts index 98e0afb2..7bb7472f 100644 --- a/src/hooks/useThemeColor.ts +++ b/src/hooks/useThemeColor.ts @@ -1,5 +1,6 @@ import { useEffect } from "react"; import { FastAverageColor } from "fast-average-color"; +import { logger } from "@/lib/logger"; import { adjustAccentColor } from "@/lib/color"; function applyColor(color: string | [number, number, number]) { @@ -47,7 +48,7 @@ export function useThemeColor({ avatarUrl, topLanguageColor }: UseThemeColorOpti } }) .catch((e) => { - console.warn("Failed to extract color from avatar, keeping fallback color.", e); + logger.warn("Failed to extract color from avatar, keeping fallback color.", e); }); }