test(webapp): the RSVP re-ask test holds on a Tuesday #919

Merged

claude-bot merged 3 commits from fix/rsvp-horizon-test-tuesday into main 2026-09-29 04:31:03 +00:00

Conversation 0 Commits 3 Files changed 2 +4 -1

claude-bot commented 2026-09-29 02:20:17 +00:00

Collaborator

Copy link

tests/test_scheduling_lane4.py::test_needs_you_asks_again_after_a_move fails every Tuesday (UTC), on main and on every PR.

Why: it confirms a session for _monday(1) (7 to 13 days out, depending on the weekday) at 7 pm Chicago, then moves it one day. On a Tuesday that's 13 days + 1 day + the timezone offset, past RSVP_HORIZON (14 days) in attention_service. Needs you correctly doesn't ask about a session that far away, so the assertion len(asks) == 1 got 0.

Fix: the test's session is 3 days out, which stays inside the horizon on any weekday. Only this test depends on the horizon; the file's other _monday(1) uses are unaffected (all 34 pass today, the worst-case weekday).

Found while running lane 8 P6's full backend suite locally (5826 passed, this 1 failed).

🤖 Generated with Claude Code

`tests/test_scheduling_lane4.py::test_needs_you_asks_again_after_a_move` fails every Tuesday (UTC), on main and on every PR. **Why:** it confirms a session for `_monday(1)` (7 to 13 days out, depending on the weekday) at 7 pm Chicago, then moves it one day. On a Tuesday that's 13 days + 1 day + the timezone offset, past `RSVP_HORIZON` (14 days) in `attention_service`. Needs you correctly doesn't ask about a session that far away, so the assertion `len(asks) == 1` got 0. **Fix:** the test's session is 3 days out, which stays inside the horizon on any weekday. Only this test depends on the horizon; the file's other `_monday(1)` uses are unaffected (all 34 pass today, the worst-case weekday). Found while running lane 8 P6's full backend suite locally (5826 passed, this 1 failed). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

claude-bot added 1 commit 2026-09-29 02:20:18 +00:00

test(webapp): the RSVP re-ask test holds on a Tuesday ... f0fc2a81af

test_needs_you_asks_again_after_a_move confirmed a session for next
Monday (7 to 13 days out) and moved it a day. On a Tuesday (UTC) that
lands past RSVP_HORIZON (14 days), so Needs you rightly stays quiet and
the test failed. Three days out keeps it inside the horizon every day.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

claude-bot referenced this pull request 2026-09-29 02:21:56 +00:00

Lane 8 page P6: Handouts and Rules & bestiary; a reveal opens a handout to read (WIKI-14, WIKI-15, #916) #920

claude-bot referenced this pull request 2026-09-29 02:25:14 +00:00

Lane 8 page P4: a draft, Suggested wiki updates and Wiki health (WIKI-9, WIKI-10, WIKI-16) #921

rbrooks added 1 commit 2026-09-29 02:56:31 +00:00

ci: re-run (the runner failed to extract the Playwright image) ... a9546578e2

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

claude-bot referenced this pull request 2026-09-29 03:12:09 +00:00

Lane 8 page P7: the map; tiling that works; the trail kept in its campaign (WIKI-11, IS-124, IS-102) #923

claude-bot referenced this pull request 2026-09-29 03:25:11 +00:00

Lane 8 page P7: the map; tiling that works; the trail kept in its campaign (WIKI-11, IS-124, IS-102) #923

rbrooks added 1 commit 2026-09-29 03:46:00 +00:00

ci: re-run (the frontend job's runner died mid-test) ... 79ebe1d18c

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

claude-bot merged commit 2b34084a32 into main 2026-09-29 04:31:03 +00:00

claude-bot referenced this pull request from a commit 2026-09-29 04:31:04 +00:00

Merge PR #919: the RSVP re-ask test holds on a Tuesday

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

backend

Issue related to the backend

  

bot

Issue related to the bot

  

bug

Something is not working

  

duplicate

This issue or pull request already exists

  

enhancement

New feature

  

frontend

Issue related to the frontend

  

help wanted

Need some help

  

invalid

Something is wrong

  

question

More information is needed

  

recording

Issue related to the recording system

  

security

Security hardening / vulnerability work

  

wontfix

This won't be fixed

No labels

backend

bot

bug

duplicate

enhancement

frontend

help wanted

invalid

question

recording

security

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

rbrooks/Quest-Board!919

No description provided.