[Features][test-lib] Add message arg to the check function

オープン

#1,010 2025年5月28日に作成コメント 1 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 2/5
見積もり時間: 1〜3時間
初心者へのやさしさ: 50/100

issue の種類: 機能追加
明瞭さ: おおむね明確
活発さ: 停滞
技術スタック: shell, typescript
領域: testing-qa

調査の方向性

src/spec-node/featuresCLI/utils.ts#L75-L91 から始め、既存の feature テストで check 関数がどのように呼び出されているかを確認します。オプションのメッセージとコマンド引数をどのように区別するかを定義し、現在の呼び出し形式を維持したうえで、失敗した check に指定されたメッセージが表示されることを検証します。

索引モデルが issue の本文から書いたものです。

説明

I would like to propose a change to the check function of the dev-container-features-test-lib script.

The function should take a message to display when the test fails, like other testing frameworks have.

The current function code is:
https://github.com/devcontainers/cli/blob/c246645f97fba402c3b1b95983e573e92bcec464/src/spec-node/featuresCLI/utils.ts#L75-L91

I propose to add a message on $2 or the last argument.
Any thoughts on this?

I would suggest something like this:

check() { 
     LABEL=$1 
     MESSAGE = $2 or ${!#}
 
     #
     # the rest of the function
     #
 }

The new function should work like the examples below, and be backwards compatible

old: 
check "java version LTS installed as default" \
          grep "LTS" <(java --version)

new:
check "java version LTS installed as default" \
         grep "LTS" <(java --version) \
        "REASON: The installed version is: $(java --version)"

or 

check "java version LTS installed as default"  \
        "REASON: The installed version is: $(java --version)" \
        grep "LTS" <(java --version)

主要言語: TypeScript
スター: 3k
フォーク: 461
平均マージ: 18分
マージ済み PR（30日）: 5

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
issue 番号を参照したプルリクエストを送ります。

devcontainers/cli のほかの issue

README npm install section mentions Python and C/C++ but they are no longer needed オープン

難易度 1/5 1時間未満初心者へのやさしさ 92/100

devcontainers/cli#1203 · 2026年4月14日に作成
test: findUserArg has no unit tests オープン

難易度 1/5 1〜3時間初心者へのやさしさ 68/100

devcontainers/cli#1178 · コメント 1 件 · 2026年3月18日に作成
Feature parity: CLI has no equivalent to the host-integration capabilities IDEs provide (port forwarding, browser opening, credential/file mounts) オープン

難易度 5/5 1週間以上初心者へのやさしさ 25/100

devcontainers/cli#1308 · 2026年9月21日に作成
build rejects valid tag@digest image references during registry inspection オープン

難易度 3/5 1〜2日初心者へのやさしさ 78/100

devcontainers/cli#1307 · 2026年9月19日に作成
Feature request: devcontainer build --build-output-folder オープン

難易度 4/5 3〜5日初心者へのやさしさ 55/100

devcontainers/cli#1305 · 2026年9月18日に作成

devcontainers/cli の issue をすべて見る

似ている issue

⚡perf(web): replace <img> with next/image in WalletSelectionModal (3 warnings) オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

safetrustcr/dApp-SafeTrust#426 · 2026年9月23日に作成
fix: in-app bug reports carry the user-report label — ADR 0143's P1 door never fires オープン

area:workflow bug ready-for-agent

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

fil-donadoni/tolaria#4409 · 2026年9月23日に作成
Add GigaCode as a supported --tools target オープン

難易度 2/5 1〜3時間初心者へのやさしさ 70/100

Fission-AI/OpenSpec#1960 · 2026年9月23日に作成
Add dependabot オープン

難易度 2/5 1〜3時間初心者へのやさしさ 70/100

TanStack/ai#1462 · 2026年9月23日に作成
Missing security headers on corsair.dev (CSP, X-Frame-Options, etc.) オープン

難易度 2/5 1〜3時間初心者へのやさしさ 75/100

corsairdev/corsair#1764 · 2026年9月23日に作成

TypeScript の issue をもっと見る