🧪 [testing improvement] 100% test coverage for yearInReviewUtils

src/lib/__tests__/yearInReviewUtils.test.ts

@@ -174,3 +174,61 @@ describe("getMostActiveDayFromCalendar", () => {
         expect(getMostActiveDayFromCalendar(calendar)).toBe("Thursday");
     });
 });
+
+describe("buildHourlyHeatmapFromCommitDates - edge cases", () => {
+    // Note: The following tests use mock objects cast to string to intentionally trigger specific
+    // internal parsing failures and verify the fallback logic without relying on invalid JS Date quirks
+    // that differ between environments.
+
+    it("falls back to full date parsing if cached day parsing results in NaN but full date is parseable", () => {
+        const mockDateStr = {
+            length: 19,
+            10: "T",
+            slice: () => "invalid___",
+            toString: () => "2023-01-01T10:00:00Z"
+        } as unknown as string;
+        const heatmap = buildHourlyHeatmapFromCommitDates([mockDateStr]);
+        expect(heatmap[0][10]).toBe(1);
+    });
+
+    it("falls back to full string parsing for unparseable hour data", () => {
+        const heatmap = buildHourlyHeatmapFromCommitDates(["2023-01-01TX0:00:00Z"]);
+        const totalCommits = heatmap.flat().reduce((sum, count) => sum + count, 0);
+        expect(totalCommits).toBe(0);
+    });
+
+    it("falls back to full string parsing for unparseable hour data but valid date", () => {
+        const heatmap = buildHourlyHeatmapFromCommitDates(["2023-01-01T10:00:00.1234Z"]);
+        expect(heatmap[0][10]).toBe(1);
+    });
+
+    it("falls back if caching fails and full date is also invalid", () => {
+        const mockDateStr = {
+            length: 19,
+            10: "T",
+            slice: () => "invalid___",
+            toString: () => "invalid-full-date"
+        } as unknown as string;
+        const heatmap = buildHourlyHeatmapFromCommitDates([mockDateStr]);
+        const totalCommits = heatmap.flat().reduce((sum, count) => sum + count, 0);
+        expect(totalCommits).toBe(0);
+    });
+
+    it("falls back to full date parsing when timezone offsets don't match fast-path ending conditions", () => {
+        const commitDates = [
+            "2023-01-01T10:00:00.000Z", // length 24, ends with .000Z
+            "2023-01-01T10:00:00.123+02:00", // length 29
+        ];
+        const heatmap = buildHourlyHeatmapFromCommitDates(commitDates);
+        expect(heatmap[0][10]).toBe(1);
+        expect(heatmap[0][8]).toBe(1); // 10:00:00+02:00 is 08:00 UTC
+    });
+
+    it("falls back to full date parsing when length is 24 but does not end with .000Z", () => {
+        const commitDates = [
+            "2023-01-01T10:00:00.123+", // length 24, invalid ending
+        ];
+        const heatmap = buildHourlyHeatmapFromCommitDates(commitDates);
+        expect(heatmap[0][10]).toBe(0);
+    });
+});