TestRunCommandHelper_RejectsPathEscapingRoot/absolute_path fails on Windows
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
internal/tools/tools_test.go の TestRunCommandHelper_RejectsPathEscapingRoot を読み、run_command が Go の filepath.IsAbs を使って引数 path をどのように検証するかを追ってください。Windows のルート付きパスをどう扱うべきかを決め、そのテストを Windows と別のプラットフォームで実行してください。テストの期待値がすべてのプラットフォームで一貫した意図的なものとなり、成功すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
TestRunCommandHelper_RejectsPathEscapingRoot/absolute_path fails on Windows.
What happens
The test passes "/etc" as the path argument to run_command and expects an error result before any confirm prompt. On Windows, /etc is not an absolute path to filepath.IsAbs, so the path is not rejected there. The command reaches the interactive y/n confirm instead, and the test gets a different result.
The other three cases in the same test (../outside, a/../../etc, sub/../../secrets) pass on Windows.
Where this showed up
- Seen on Windows in #21 and #23 (each PR's test run). Both PR authors noted that the same failure happens on the base branch (
1bff3dfin #23). - The test is in
internal/tools/tools_test.goatTestRunCommandHelper_RejectsPathEscapingRoot.
Suggested direction
Decide what "absolute" means for path arguments on Windows. Either treat a leading / or \ as rooted when checking path (so /etc is rejected on every platform), or change the test to use a Windows-absolute path such as C:\Windows on Windows only. The first keeps one expectation across platforms.
I have not run this on the current main (335144d); the report above is from the PR runs.
- 主要言語
- Go
- スター
- 1
- フォーク
- 2
- 平均マージ
- 13時間 15分
- マージ済み PR（30日）
- 9
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
Edwin-metcalf/MetCode のほかの issue
-
documentation enhancement help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
Edwin-metcalf/MetCode#19 ·
メンテナーはふだん 1 日以内に返信
-
testing
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
Edwin-metcalf/MetCode#9 ·
メンテナーはふだん 1 日以内に返信
-
Add a Makefileオープンenhancement good first issue help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
Edwin-metcalf/MetCode#8 ·
メンテナーはふだん 1 日以内に返信
-
Add a --host flagオープンenhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
Edwin-metcalf/MetCode#7 ·
メンテナーはふだん 1 日以内に返信
-
Add a --version flag対応中かも @jay23606 が 2 日前に担当しました。 オープンenhancement good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
Edwin-metcalf/MetCode#6 ·
メンテナーはふだん 1 日以内に返信
Edwin-metcalf/MetCode の issue をすべて見る
似ている issue
-
plugins: symlinked plugin directories are skipped対応中かも @lab1207 が今日担当しました。 オープンbug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
eugenioenko/ttt#1029 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1時間未満 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
needs-operator
難易度 2/5 1〜3時間 初心者へのやさしさ 67/100
keyxmakerx/Chronicle#1654 ·
メンテナーはふだん 1 日以内に返信
-
area/security kind/bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
agent-substrate/substrate#2935 ·
メンテナーはふだん 1 日以内に返信
-
100 points complexity: trivial documentation drips-wave good first issue help wanted
難易度 2/5 1〜3時間 初心者へのやさしさ 86/100
メンテナーはふだん 1 日以内に返信