docs: drift reporter double-counts CI retries
メンテナーはふだん 1 日以内に返信
@marcel-rbro がすでに取り組んでいます。
2026年8月24日 から。
評価
この issue はまだ評価されていません。
説明
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.
- 主要言語
- JavaScript
- スター
- 82
- フォーク
- 240
- 平均マージ
- 3日 15時間
- マージ済み PR（30日）
- 41
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
apify/apify-docs のほかの issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
apify/apify-docs#3071 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
apify/apify-docs#3049 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
apify/apify-docs#2742 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1〜3時間 初心者へのやさしさ 82/100
apify/apify-docs#2741 ·
メンテナーはふだん 1 日以内に返信
-
low priority t-unblocking tech enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
apify/apify-docs#2451 ·
メンテナーはふだん 1 日以内に返信
apify/apify-docs の issue をすべて見る
似ている issue
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
メンテナーはふだん 1 日以内に返信
-
enhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
SasanLabs/VulnerableApp#836 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
keyxmakerx/Chronicle#1469 ·
メンテナーはふだん 1 日以内に返信
-
Connection.prototype.bulkWrite() with ordered: false resolves null when a write fails対応中かも @maxymlyskov が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
Automattic/mongoose#16601 ·
メンテナーはふだん 2 日以内に返信
-
sandbox-vercel: persistent: false is dropped for live sandboxes created from a template, so every stop writes a snapshot対応中かも @jvg123 が今日担当しました。 オープンfactory-active task-identify-harness-labels-done task-identify-issue-type-done
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信