Test harness stops at the first thrown error and prints a stale check count
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 45/100
- issue の種類
- リファクタリング
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- javascript, nodejs
- 領域
- testing-qa
調査の方向性
The harness is hand-rolled eq/has/throws helpers in site/checks.test.mjs (around lines 52-75 and the summary near 817-822) and site/lib/lib.test.mjs (lines 16-38 and the summary near 1156-1162). Read those helpers and run npm test to see how one thrown exception stops the run. Done means a thrown error in one section no longer hides later results, the printed check count matches the real number of sections, and a single test can be selected by name, ideally via node:test's --test-name-pattern while keeping npm test as the entry point.
索引モデルが issue の本文から書いたものです。
説明
Summary
Both test files hand-roll eq/has/throws. An exception inside any section (as opposed to a failed eq) kills the process before the failure report prints, hiding every later assertion.
Where
site/checks.test.mjs:52-75, 817-822site/lib/lib.test.mjs:16-38, 1156-1162
Details
checks.test.mjs prints a hard-coded "across 30 checks" while the file has 26 V-sections. lib.test.mjs is 1,163 lines in one scope and is the most-changed file in the repo.
Impact
One crash masks all other failures; no way to run a single test.
Suggested fix
Migrate section by section to the built-in node:test + node:assert (ships with Node 22, no dependency): per-test isolation, --test-name-pattern, real counts. Keep npm test as the entry point.
Acceptance criteria
- A thrown error in one test does not hide the others
-
npm testreports real counts - A single test can be run by name
Getting started
- Relevant files:
site/checks.test.mjs,site/lib/lib.test.mjs,package.json - How to run tests:
npm test(unit tests),npm run check(tests + validate + build). Node 22+, no dependencies. - Related docs:
site/README.md,CONTRIBUTING.md
Filed by repo-audit/full-audit at 982fcd9. Severity: low. Effort: M. audit-key: testing|site/lib/lib.test.mjs|harness
- 主要言語
- JavaScript
- スター
- 58
- フォーク
- 15
- 平均マージ
- 1日 15時間
- マージ済み PR（30日）
- 12
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
TMHSDigital/Free-Game-Dev-Assets のほかの issue
-
community marketing priority: low
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
TMHSDigital/Free-Game-Dev-Assets#151 ·
メンテナーはふだん 1 日以内に返信
-
documentation priority: medium
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
TMHSDigital/Free-Game-Dev-Assets#72 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 1/5 1〜3時間 初心者へのやさしさ 68/100
TMHSDigital/Free-Game-Dev-Assets#3 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
enhancement priority: medium
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
TMHSDigital/Free-Game-Dev-Assets#211 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
enhancement marketing priority: low
難易度 4/5 3〜5日 初心者へのやさしさ 55/100
TMHSDigital/Free-Game-Dev-Assets#189 ·
メンテナーはふだん 1 日以内に返信
TMHSDigital/Free-Game-Dev-Assets の issue をすべて見る
似ている issue
-
test-reliability
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
Azure/azure-sdk-for-js#40672 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
txn2/mcp-data-platform#2492 ·
メンテナーはふだん 1 日以内に返信
-
documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
liberation-data/drivine#639 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
accessibility ux
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
prophet-technocracy/civilizational-emergency-protocol#59 ·
メンテナーはふだん 1 日以内に返信
-
type: bug
難易度 1/5 1〜3時間 初心者へのやさしさ 72/100
RocketChat/Rocket.Chat#43554 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信