docs: drift reporter double-counts CI retries
Maintainer thường phản hồi trong vòng 1 ngày
@marcel-rbro đang làm issue này rồi.
Từ ngày 24/8/2026.
Đánh giá
Issue này chưa được đánh giá.
Mô tả
Part of #2671.
Problem
IssuesReporter.onTestEnd (docs-tests/reporters/issues-reporter.ts:59) pushes every test attempt into collected. With retries: 1 set for CI in playwright.config.ts, a failing assertion is recorded twice — so summary.failed, summary.total and the issues[] array all double, and every row appears twice in the auto-filed drift issue.
Evidence
#2908 was filed as "28 failing assertions (50/90 passed, 12 skipped)" and its table listed all 14 assertions twice.
The real numbers were 14 failures out of 76. Running the same suite locally with retries: 0 produced exactly 14 failures / 24 passed on the integrity project.
The total inflates the same way: 76 real results + 14 retried failures = the 90 reported.
Fix
Keep only the final attempt per TestCase before computing the summary and building issues[] — e.g. key collected by test ID and overwrite, so a retry replaces rather than appends.
Impact
Cosmetic but misleading. Drift issues read as roughly twice as severe as they are, and duplicate rows make triage slower. It also means a flaky-then-passing test could still be counted, since only failed attempts get collected while the eventual pass is recorded separately.
- Ngôn ngữ chính
- JavaScript
- Star
- 82
- Fork
- 240
- Merge trung bình
- 4 ngày 2 giờ
- Pull request đã merge (30 ngày)
- 44
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của apify/apify-docs
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
apify/apify-docs#3071 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
apify/apify-docs#3049 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
apify/apify-docs#2742 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 82/100
apify/apify-docs#2741 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
low priority t-unblocking tech enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
apify/apify-docs#2451 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của apify/apify-docs
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày
-
t0ggles
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
nolimits4web/swiper#8240 · 1 bình luận ·
-
netbox status: backlog topic: testing type: housekeeping
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
netbox-community/netbox#23395 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[scanner] check:spelling skips blog/ — posts and author metadata are never spell-checkedCó thể đã có người làm @hivecommons-hive đã nhận hôm nay. Đang mởagent/scanner bug hive/covered-by-pr hive/hosted-available-lke648397-260827-5n31
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
cncf/endusers#1452 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày