nessar/vac-optimizer

0 Fork 0

fix: stabilize two CI test failures (Paris today, worker cancel race) #307

Closed

nessar wants to merge 3 commits from feature/ci-test-stability into develop

pull from: feature/ci-test-stability

Author	SHA1	Message	Date
nessarandClaude Sonnet 5.5	036b13613c	test(calendar): pin the clock for the Paris today marker CI / policy (push) Successful in 26s Details CI / validate (push) Successful in 28s Details CI / build (push) Successful in 23s Details CI / benchmark (push) Successful in 1m26s Details CI / security (push) Successful in 1m14s Details CI / test (push) Successful in 1m58s Details CI / images (push) Successful in 57s Details CI / required (push) Successful in 3s Details CI / policy (pull_request) Failing after 10s Details CI / validate (pull_request) Successful in 26s Details CI / build (pull_request) Successful in 21s Details CI / benchmark (pull_request) Successful in 1m0s Details CI / security (pull_request) Successful in 58s Details CI / test (pull_request) Successful in 1m52s Details CI / images (pull_request) Skipped Details CI / required (pull_request) Failing after 1s Details The test of the today marker read the wall clock, so it only exercised the Paris/local date mismatch between 22:00 and 24:00 UTC and passed with or without the fix the rest of the day. Pin the clock to 22:16 UTC, already the next day in Paris, so a UTC runner fails without the fix at any time. Refs #301 Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>	2026-10-19 10:21:54 +02:00
nessarandClaude Sonnet 5.5	af20c96f01	test(planning): hold the worker until the preparation cancel arrives CI / policy (push) Successful in 16s Details CI / validate (push) Successful in 29s Details CI / build (push) Successful in 20s Details CI / benchmark (push) Successful in 1m27s Details CI / security (push) Successful in 1m0s Details CI / test (push) Successful in 1m57s Details CI / images (push) Successful in 1m20s Details CI / required (push) Successful in 1s Details CI / policy (pull_request) Successful in 8s Details CI / validate (pull_request) Successful in 23s Details CI / test (pull_request) Successful in 1m51s Details CI / build (pull_request) Successful in 38s Details CI / security (pull_request) Successful in 1m1s Details CI / benchmark (pull_request) Successful in 46s Details CI / images (pull_request) Successful in 51s Details CI / required (pull_request) Successful in 2s Details The worker reads its queue only between event-loop turns, so the calendar preparation could finish before the cancel sent on its second progress update arrived, and the test received a complete plan. Use the real-worker harness hold already applied to the checkpoint cancellation test, and cancel on the first preparation update so the hold is released. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>	2026-10-19 09:52:15 +02:00
nessarandClaude Sonnet 5.5	f458a83599	fix(calendar): mark today on the Europe/Paris date The calendar highlighted today from the browser's local date while the default planning horizon starts on the Europe/Paris date, so the two disagreed for a few hours a day (and always, for users in another time zone). The horizon's first day was then not marked as today. This also failed the CI test run in UTC containers between 22:00 and 24:00 UTC. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>	2026-10-19 09:52:03 +02:00