NuGet/Home

Refactor UIActionEngineTests that use packagesFolder to avoid looking at the drive root

Open

#12,556 opened on 2023年4月24日

GitHub で見る
 (0 comments) (0 reactions) (0 assignees)HTML (1,459 stars) (292 forks)batch import
Category:Quality WeekFunctionality:VisualStudioUIPriority:2Type:Testgood first issue

説明

@"\packagesFolder" seems like a bad idea. A single leading '' usually means "from the root of the drive" on Windows. A temporary directory or a local directory (@".\packgesFolder") seems better.

https://github.com/NuGet/NuGet.Client/pull/5109/files#r1175482193

コントリビューターガイド

技術スタック
csharp
領域
testing
Issue 種別
refactor
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
under 1 hour
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
fresh
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
C# basicsunit testingNuGet client tests
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
90
調査方針
The issue is to refactor UIActionEngineTests that use a hardcoded path '\packagesFolder' to a temporary directory, as suggested in PR #5109. Look at the test file UIActionEngineTests (likely in the NuGet.Client repo) and replace occurrences of @"\packagesFolder" with a path like Path.Combine(Path.GetTempPath(), "packagesFolder") or use a test temp directory. Ensure the change is consistent with similar test patterns.