aep/sdlc-hub
1
0
Fork 0
Code Issues 24 Pull Requests Actions Packages Projects Releases Wiki Activity

Флейк shell/activity/activity.test.tsx:331 — гонка в ActivityProvider #116

New Issue
Closed
opened 2026-10-25 20:55:40 +03:00 by aep · 3 comments
aep commented 2026-10-25 20:55:40 +03:00
Owner
Copy Link

В полном npm test под нагрузкой тест web/src/shell/activity/activity.test.tsx:331 иногда падает, при повторе зелёный. Замечен трижды во время #112 (O-3, O-4, O-5 в ledger) и раньше в #108 (O-3 в #111).

Гипотеза финального ревью #112 (M-9): ответ опроса /batches/9, пришедший после смены проекта, снова ставит событие в ActivityProvider — гонка в самом провайдере, не только в тесте.

Что нужно: воспроизвести (повтор теста в цикле / под нагрузкой), подтвердить причину, починить провайдер (игнорировать ответы опроса прежнего проекта) или тест.

В полном `npm test` под нагрузкой тест `web/src/shell/activity/activity.test.tsx:331` иногда падает, при повторе зелёный. Замечен трижды во время #112 (O-3, O-4, O-5 в ledger) и раньше в #108 (O-3 в #111). Гипотеза финального ревью #112 (M-9): ответ опроса `/batches/9`, пришедший после смены проекта, снова ставит событие в ActivityProvider — гонка в самом провайдере, не только в тесте. Что нужно: воспроизвести (повтор теста в цикле / под нагрузкой), подтвердить причину, починить провайдер (игнорировать ответы опроса прежнего проекта) или тест.
aep commented 2026-10-25 21:18:13 +03:00
Author
Owner
Copy Link

Ещё один случай под нагрузкой (#118, O-1): в полном make check один тест src/shell/Sidebar.test.tsx упал, отдельно и в повторе полного vitest — зелёный; файл не менялся. Возможно, та же природа (параллельные прогоны/таймауты), стоит смотреть вместе.

Ещё один случай под нагрузкой (#118, O-1): в полном `make check` один тест `src/shell/Sidebar.test.tsx` упал, отдельно и в повторе полного vitest — зелёный; файл не менялся. Возможно, та же природа (параллельные прогоны/таймауты), стоит смотреть вместе.
aep commented 2026-10-25 21:31:54 +03:00
Author
Owner
Copy Link

Ещё один случай 25.10.2026 — make check ветки feat/113-sp-estimation-board (фронт не менялся): «M-2: пакет, отданный track для нового проекта, переживает смену проекта на него; чужой — сбрасывается», AssertionError: expected { kind: 'batch', id: 9, …(4) } to be null. Повтор файла 3×39/39, следующий make check зелёный. Дубль #122 закрыт сюда.

Ещё один случай 25.10.2026 — make check ветки feat/113-sp-estimation-board (фронт не менялся): «M-2: пакет, отданный track для нового проекта, переживает смену проекта на него; чужой — сбрасывается», AssertionError: expected { kind: 'batch', id: 9, …(4) } to be null. Повтор файла 3×39/39, следующий make check зелёный. Дубль #122 закрыт сюда.
aep commented 2026-10-26 00:56:38 +03:00
Author
Owner
Copy Link

Исправлено в PR #125 (merge 9b98420), коммит 231ca2f: ActivityProvider отбрасывает ответ опроса прежнего проекта (сверка с activeRef); тест #116 — 20/20 в цикле, без правки красный 5/5.

Исправлено в PR #125 (merge 9b98420), коммит 231ca2f: ActivityProvider отбрасывает ответ опроса прежнего проекта (сверка с `activeRef`); тест #116 — 20/20 в цикле, без правки красный 5/5.
aep closed this issue 2026-10-26 00:56:40 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aep
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aep/sdlc-hub#116
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?