From b27b37c11f20c26e6fc253ff4f313bb031de8ba9 Mon Sep 17 00:00:00 2001
From: yusufipk <yusufipek@protonmail.com>
Date: Wed, 5 Aug 2026 23:57:13 +0300
Subject: [PATCH] fix(tests): stop the scoreboard suite from depending on the
 weekday

The scoreboard groups by date_trunc('week'), which starts on Monday, but the
suite seeded its events with "three days ago". On a Wednesday that walks back
into the previous week, so a returning visitor was counted once in each of two
weeks and a subscription landed outside the week the assertions read. The suite
passed Monday and Thursday through Sunday, and failed Tuesday and Wednesday.

Seed from a week boundary instead: the visitor events go into last week, which
is whole whenever the suite runs, and the subscription pair goes into this one,
which is the week those assertions read.
---
 tests/api/analytics-scoreboard.test.ts | 36 ++++++++++++++++++++++----
 1 file changed, 31 insertions(+), 5 deletions(-)

diff --git a/tests/api/analytics-scoreboard.test.ts b/tests/api/analytics-scoreboard.test.ts
index d128614..27e936d 100644
--- a/tests/api/analytics-scoreboard.test.ts
+++ b/tests/api/analytics-scoreboard.test.ts
@@ -17,6 +17,28 @@ function daysAgo(days: number): Date {
   return date;
 }
 
+/**
+ * The Monday this week started, in UTC.
+ *
+ * Anything asserted per week has to be seeded from here rather than from
+ * `daysAgo`: weeks start on Monday, so "three days ago" is last week on a
+ * Wednesday and this week on a Saturday, and a suite written the second way
+ * fails on the days the calendar disagrees.
+ */
+function startOfThisWeek(): Date {
+  const now = new Date();
+  const start = new Date(Date.UTC(now.getUTCFullYear(), now.getUTCMonth(), now.getUTCDate()));
+  start.setUTCDate(start.getUTCDate() - ((start.getUTCDay() + 6) % 7));
+  return start;
+}
+
+/** `days` into the week beginning at `weekStart`. Negative walks back a week. */
+function intoWeek(weekStart: Date, days: number): Date {
+  const date = new Date(weekStart);
+  date.setUTCDate(date.getUTCDate() + days);
+  return date;
+}
+
 let sequence = 0;
 
 async function seedEvent(params: {
@@ -61,17 +83,19 @@ describe('getScoreboard', () => {
   it('counts a returning visitor once per week, not once per visit', async () => {
     // Landing views are deduped per visitor per day, so the same person on three
     // days is three rows. Weekly visitors is a distinct count over the id.
-    for (const days of [1, 2, 3]) {
+    // Seeded into last week, which is whole however the suite is scheduled.
+    const lastWeek = intoWeek(startOfThisWeek(), -7);
+    for (const day of [0, 1, 2]) {
       await seedEvent({
         name: 'LANDING_VIEW',
-        occurredAt: daysAgo(days),
+        occurredAt: intoWeek(lastWeek, day),
         anonymousId: 'visitor-one',
         channel: 'GITHUB',
       });
     }
     await seedEvent({
       name: 'LANDING_VIEW',
-      occurredAt: daysAgo(1),
+      occurredAt: intoWeek(lastWeek, 1),
       anonymousId: 'visitor-two',
       channel: 'GOOGLE',
     });
@@ -112,9 +136,11 @@ describe('getScoreboard', () => {
   });
 
   it('carries subscriptions started before the window into the running total', async () => {
+    // The pair has to land in the week the assertions read, which is this one.
+    const thisWeek = startOfThisWeek();
     await seedEvent({ name: 'SUBSCRIPTION_STARTED', occurredAt: daysAgo(120) });
-    await seedEvent({ name: 'SUBSCRIPTION_STARTED', occurredAt: daysAgo(3) });
-    await seedEvent({ name: 'SUBSCRIPTION_CANCELED', occurredAt: daysAgo(3) });
+    await seedEvent({ name: 'SUBSCRIPTION_STARTED', occurredAt: thisWeek });
+    await seedEvent({ name: 'SUBSCRIPTION_CANCELED', occurredAt: thisWeek });
 
     const scoreboard = await getScoreboard({ weeks: 2 });
     const last = scoreboard.weeks[scoreboard.weeks.length - 1];