test: update mocks and add tests for new shared exports

- Add classifyError tests in format-utils.test.js
- Add buildProfileUrl tests in url-builder.test.js
- Add CONCURRENCY and classifyError to service-worker test mocks
- Add GITHUB_SITE_BASE to notification-renderer test mock

Author: euxx

tests/format-utils.test.js

@@ -5,6 +5,7 @@ import {
   formatState,
   getNotificationStatus,
   escapeHtml,
+  classifyError,
 } from "../src/lib/format-utils.js";
 
 describe("formatReason", () => {
@@ -138,3 +139,28 @@ describe("escapeHtml", () => {
     expect(escapeHtml(true)).toBe("true");
   });
 });
+
+describe("classifyError", () => {
+  it('should classify rate limit errors as "rate-limited"', () => {
+    expect(classifyError(new Error("Rate limited until 12:00"))).toBe("rate-limited");
+  });
+
+  it('should classify timeout errors as "timeout"', () => {
+    expect(classifyError(new Error("Request timeout after 30s"))).toBe("timeout");
+  });
+
+  it('should classify network errors as "offline"', () => {
+    expect(classifyError(new Error("NetworkError when attempting to fetch"))).toBe("offline");
+    expect(classifyError(new Error("Failed to fetch"))).toBe("offline");
+  });
+
+  it('should classify other errors as "unknown"', () => {
+    expect(classifyError(new Error("Something went wrong"))).toBe("unknown");
+  });
+
+  it("should handle null/undefined errors", () => {
+    expect(classifyError(null)).toBe("unknown");
+    expect(classifyError(undefined)).toBe("unknown");
+    expect(classifyError({})).toBe("unknown");
+  });
+});

tests/notification-renderer.test.js

@@ -48,6 +48,7 @@ beforeAll(() => {
 
 // Mock dependencies before importing
 vi.mock("../src/lib/constants.js", () => ({
+  GITHUB_SITE_BASE: "https://github.com",
   ANIMATION_DURATION: {
     FADE_OUT: 200,
     ERROR_BACKGROUND_FADE: 1000,

tests/service-worker.test.js

@@ -134,11 +134,23 @@ vi.mock("../src/lib/constants.js", () => ({
     Release: "release",
     CheckSuite: "actions",
   },
+  CONCURRENCY: {
+    PRIORITY: 5,
+    BACKGROUND: 3,
+    VISIBLE_COUNT: 10,
+  },
 }));
 
 // Mock format-utils
 vi.mock("../src/lib/format-utils.js", () => ({
   formatReason: vi.fn((reason) => reason || "Unknown"),
+  classifyError: vi.fn((error) => {
+    const msg = error?.message || "";
+    if (msg.includes("Rate limited")) return "rate-limited";
+    if (msg.includes("timeout")) return "timeout";
+    if (msg.includes("NetworkError") || msg.includes("Failed to fetch")) return "offline";
+    return "unknown";
+  }),
 }));
 
 // Mock url-builder

tests/url-builder.test.js

@@ -1,5 +1,5 @@
 import { describe, it, expect } from "vitest";
-import { buildNotificationUrl } from "../src/lib/url-builder.js";
+import { buildNotificationUrl, buildProfileUrl } from "../src/lib/url-builder.js";
 
 const GITHUB_BASE = "https://github.com";
 
@@ -182,3 +182,19 @@ describe("buildNotificationUrl", () => {
     });
   });
 });
+
+describe("buildProfileUrl", () => {
+  it("should build URL from login", () => {
+    expect(buildProfileUrl("octocat")).toBe("https://github.com/octocat");
+  });
+
+  it("should encode special characters in login", () => {
+    expect(buildProfileUrl("user name")).toBe("https://github.com/user%20name");
+  });
+
+  it("should return null for falsy login", () => {
+    expect(buildProfileUrl(null)).toBeNull();
+    expect(buildProfileUrl(undefined)).toBeNull();
+    expect(buildProfileUrl("")).toBeNull();
+  });
+});