dotnet/msbuild

Add explicit tests around CompareAssemblyIdentity implementation

Open

#5,974 opened on 2020年12月16日

GitHub で見る
 (0 comments) (1 reaction) (0 assignees)C# (5,062 stars) (1,364 forks)batch import
help wantedtestingtriaged

説明

Before the .NET Core port, ResolveAssemblyReferences compared assembly identities using a native method call:

https://github.com/Microsoft/msbuild/blob/a939f1547a3f8b91512d0d2d270a8d0f0111e7e7/src/XMakeTasks/NativeMethods.cs#L976-L985

Now, there's a managed implementation that optionally calls that one on Windows:

https://github.com/dotnet/msbuild/blob/a8efaf15afcaf5fce9cb6e1c3b7c94e0c88d804b/src/Tasks/NativeMethods.cs#L1114-L1136

We should have tests that validate that implementation using the native method as a reference. This is a nice unit-testing problem since it's functional and idempotent, so we can probably hit 100% code coverage.

Related: #3930, #5973

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

技術スタック
csharp
領域
build system
Issue 種別
test
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
C#Unit testingMSBuild codebase
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70
調査方針
Examine the managed CompareAssemblyIdentity implementation in src/Tasks/NativeMethods.cs (lines 1114-1136) and the native method reference in src/XMakeTasks/NativeMethods.cs (lines 976-985). Study existing test patterns in the repository to write unit tests that compare outputs. Also review related issues #3930 and #5973 for context.