[Feature] Update the VSIX path used by smoke tests
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 76/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- typescript
- 領域
- testing
調査の方向性
問題のあるパスは、test/smoke 配下の VSIX スモークテストコードにある path.resolve(__dirname, "..", "..", "resources", "extension") の呼び出しです。issue ではファイル名が示されていないため、この呼び出しを検索して特定してください。コンパイル済みテストから test/smoke/out 内に解決後のディレクトリをログ出力し、test/resources/extension を指していることを確認します。解決先が test/smoke/resources/extension になり、VSIX テストがスキップされずファイルを見つけられるようになり、パスが再び移動した場合に回帰チェックが失敗すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Description
The VSIX smoke tests resolve the extension resource directory using two parent-directory segments:
path.resolve(__dirname, "..", "..", "resources", "extension");
At runtime, the compiled test is located under:
test/smoke/out
As a result, the current path resolves to:
test/resources/extension
Instead of the intended directory:
test/smoke/resources/extension
This is not detected because the existing tests are skipped when the resolved directory or VSIX does not exist.
Expected behavior
The smoke tests should resolve and validate VSIX files from:
test/smoke/resources/extension
Proposed fix
- Remove the extra parent-directory segment:
path.resolve(__dirname, "..", "resources", "extension");
- Ensure the smoke tests use the intended VSIX resource directory.
- Add a regression check to prevent the path from resolving to the wrong directory.
- 主要言語
- TypeScript
- スター
- 2.7k
- フォーク
- 298
- 平均マージ
- 14時間 54分
- マージ済み PR（30日）
- 28
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
microsoft/vscode-react-native のほかの issue
-
[Feature] Avoid duplicate error notifications対応中かも @erenbati が 4 日前に担当しました。 オープンenhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
microsoft/vscode-react-native#2963 ·
メンテナーはふだん 1 日以内に返信
-
[Feature] Stop Expo Go installation when Quick Pick is canceled対応中かも @erenbati が 19 日前に担当しました。 オープンenhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
microsoft/vscode-react-native#2959 ·
メンテナーはふだん 1 日以内に返信
-
enhancement follow-up
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
microsoft/vscode-react-native#2951 ·
メンテナーはふだん 1 日以内に返信
-
[Feature] Add smoke coverage for release VSIX artifact handling対応中かも @EmmaYuan1015 が今日担当しました。 オープンenhancement
難易度 3/5 半日 初心者へのやさしさ 35/100
microsoft/vscode-react-native#3004 ·
メンテナーはふだん 1 日以内に返信
-
[Feature] Expo Doctor output is lost when diagnostics fail対応中かも @erenbati が 19 日前に担当しました。 オープンenhancement
難易度 3/5 1〜2日 初心者へのやさしさ 72/100
microsoft/vscode-react-native#2957 ·
メンテナーはふだん 1 日以内に返信
microsoft/vscode-react-native の issue をすべて見る
似ている issue
-
test: make the #3408 Claude OAuth token fixture obviously fake so secret scanners stop flagging itオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
code-yeongyu/senpi#3436 ·
メンテナーはふだん 1 日以内に返信
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
難易度 1/5 1時間未満 初心者へのやさしさ 78/100
lingdojo/kana-dojo#32505 · コメント 1 件 · リアクション 5 件 ·
メンテナーはふだん 1 日以内に返信
-
code-quality refactoring
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
github/gh-aw-firewall#10040 ·
メンテナーはふだん 1 日以内に返信
-
review-followup
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
griffinwork40/agent-afk#3868 ·
メンテナーはふだん 1 日以内に返信
-
area: import-export bug status: needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
amatya-aditya/obsidian-rss-dashboard#1191 ·
メンテナーはふだん 1 日以内に返信