dependabot-nuget 0.321.3 → 0.322.1

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/Utilities/MSBuildHelperTests.cs

@@ -1,4 +1,3 @@
-using System.Collections.Immutable;
 using System.Text.Json;
 
 using NuGetUpdater.Core.Run;
@@ -13,185 +12,6 @@ using TestFile = (string Path, string Content);
 
 public class MSBuildHelperTests : TestBase
 {
-    [Fact]
-    public void GetRootedValue_FindsValue()
-    {
-        // Arrange
-        var projectContents = """
-            <Project>
-                <PropertyGroup>
-                    <TargetFramework>net8.0</TargetFramework>
-                </PropertyGroup>
-                <ItemGroup>
-                    <PackageReference Include="Some.Package" Version="$(PackageVersion1)" />
-                </ItemGroup>
-            </Project>
-            """;
-        var propertyInfo = new Dictionary<string, Property>
-        {
-            { "PackageVersion1", new("PackageVersion1", "1.1.1", "Packages.props") },
-        };
-
-        // Act
-        var (resultType, _, evaluatedValue, _, _) = MSBuildHelper.GetEvaluatedValue(projectContents, propertyInfo);
-
-        Assert.Equal(EvaluationResultType.Success, resultType);
-
-        // Assert
-        Assert.Equal("""
-            <Project>
-                <PropertyGroup>
-                    <TargetFramework>net8.0</TargetFramework>
-                </PropertyGroup>
-                <ItemGroup>
-                    <PackageReference Include="Some.Package" Version="1.1.1" />
-                </ItemGroup>
-            </Project>
-            """, evaluatedValue);
-    }
-
-    [Fact(Timeout = 1000)]
-    public async Task GetRootedValue_DoesNotRecurseAsync()
-    {
-        // Arrange
-        var projectContents = """
-            <Project>
-                <PropertyGroup>
-                    <TargetFramework>net8.0</TargetFramework>
-                </PropertyGroup>
-                <ItemGroup>
-                    <PackageReference Include="Some.Package" Version="$(PackageVersion1)" />
-                </ItemGroup>
-            </Project>
-            """;
-        var propertyInfo = new Dictionary<string, Property>
-        {
-            { "PackageVersion1", new("PackageVersion1", "$(PackageVersion2)", "Packages.props") },
-            { "PackageVersion2", new("PackageVersion2", "$(PackageVersion1)", "Packages.props") }
-        };
-        // This is needed to make the timeout work. Without that we could get caugth in an infinite loop.
-        await Task.Delay(1);
-
-        // Act
-        var (resultType, _, _, _, errorMessage) = MSBuildHelper.GetEvaluatedValue(projectContents, propertyInfo);
-
-        // Assert
-        Assert.Equal(EvaluationResultType.CircularReference, resultType);
-        Assert.Equal("Property 'PackageVersion1' has a circular reference.", errorMessage);
-    }
-
-    [Theory]
-    [MemberData(nameof(SolutionProjectPathTestData))]
-    public void ProjectPathsCanBeParsedFromSolutionFiles(string solutionContent, string[] expectedProjectSubPaths)
-    {
-        var solutionPath = Path.GetTempFileName();
-        var solutionDirectory = Path.GetDirectoryName(solutionPath)!;
-        try
-        {
-            File.WriteAllText(solutionPath, solutionContent);
-            var actualProjectSubPaths = MSBuildHelper.GetProjectPathsFromSolution(solutionPath).ToArray();
-            var expectedPaths = expectedProjectSubPaths.Select(path => Path.Combine(solutionDirectory, path)).ToArray();
-            if (Environment.OSVersion.Platform == PlatformID.Win32NT)
-            {
-                // make the test happy when running on Windows
-                expectedPaths = expectedPaths.Select(p => p.Replace("/", "\\")).ToArray();
-            }
-
-            AssertEx.Equal(expectedPaths, actualProjectSubPaths);
-        }
-        finally
-        {
-            File.Delete(solutionPath);
-        }
-    }
-
-    [Theory]
-    [InlineData("<Project><PropertyGroup><TargetFramework>netstandard2.0</TargetFramework></PropertyGroup></Project>", "netstandard2.0", null)]
-    [InlineData("<Project><PropertyGroup><TargetFrameworks>netstandard2.0</TargetFrameworks></PropertyGroup></Project>", "netstandard2.0", null)]
-    [InlineData("<Project><PropertyGroup><TargetFrameworks>  ; netstandard2.0 ; </TargetFrameworks></PropertyGroup></Project>", "netstandard2.0", null)]
-    [InlineData("<Project><PropertyGroup><TargetFrameworks>netstandard2.0 ; netstandard2.1 ; </TargetFrameworks></PropertyGroup></Project>", "netstandard2.0", "netstandard2.1")]
-    [InlineData("<Project><PropertyGroup><TargetFramework>netstandard2.0</TargetFramework><TargetFrameworkVersion Condition='False'>v4.7.2</TargetFrameworkVersion></PropertyGroup></Project>", "netstandard2.0", null)]
-    [InlineData("<Project><PropertyGroup><TargetFramework>$(PropertyThatCannotBeResolved)</TargetFramework></PropertyGroup></Project>", null, null)]
-    public async Task TfmsCanBeDeterminedFromProjectContents(string projectContents, string? expectedTfm1, string? expectedTfm2)
-    {
-        var projectPath = Path.GetTempFileName();
-        try
-        {
-            File.WriteAllText(projectPath, projectContents);
-            var expectedTfms = new[] { expectedTfm1, expectedTfm2 }.Where(tfm => tfm is not null).ToArray();
-            var (_buildFiles, actualTfms) = await MSBuildHelper.LoadBuildFilesAndTargetFrameworksAsync(Path.GetDirectoryName(projectPath)!, projectPath);
-            AssertEx.Equal(expectedTfms, actualTfms);
-        }
-        finally
-        {
-            File.Delete(projectPath);
-        }
-    }
-
-    [Fact]
-    public async Task IntermediatePropsAndTargetsAreExcludedFromBuildFileDiscovery()
-    {
-        // arrange
-        var repoFiles = new[]
-        {
-            ("project.csproj", """
-                <Project Sdk="Microsoft.NET.Sdk">
-                  <PropertyGroup>
-                    <TargetFramework>net9.0</TargetFramework>
-                  </PropertyGroup>
-                  <Import Project="SomeFile.props" />
-                  <ItemGroup>
-                    <PackageReference Include="Some.Package" Version="1.0.0" />
-                  </ItemGroup>
-                </Project>
-                """),
-            ("global.json", "{}"),
-            ("Directory.Build.props", "<Project />"),
-            ("Directory.Build.targets", "<Project />"),
-            ("SomeFile.props", "<Project />"),
-            // these simulate a direct discovery operation having previously been performed
-            ("obj/project.csproj.nuget.g.props", "<Project />"),
-            ("obj/project.csproj.nuget.g.targets", "<Project />"),
-        };
-        using var tempDir = await TemporaryDirectory.CreateWithContentsAsync(repoFiles);
-        var fullProjectPath = Path.Combine(tempDir.DirectoryPath, "project.csproj");
-        await UpdateWorkerTestBase.MockNuGetPackagesInDirectory([], tempDir.DirectoryPath);
-
-        // act
-        var (buildFiles, _tfms) = await MSBuildHelper.LoadBuildFilesAndTargetFrameworksAsync(tempDir.DirectoryPath, fullProjectPath);
-
-        // assert
-        var actualBuildFilePaths = buildFiles.Select(f => Path.GetRelativePath(tempDir.DirectoryPath, f.Path).NormalizePathToUnix()).ToArray();
-        var expectedBuildFilePaths = new[]
-        {
-            "project.csproj",
-            "Directory.Build.props",
-            "SomeFile.props",
-            "Directory.Build.targets",
-        };
-        AssertEx.Equal(expectedBuildFilePaths, actualBuildFilePaths);
-    }
-
-    [Theory]
-    [MemberData(nameof(GetTopLevelPackageDependencyInfosTestData))]
-    public async Task TopLevelPackageDependenciesCanBeDetermined(TestFile[] buildFileContents, Dependency[] expectedTopLevelDependencies, MockNuGetPackage[] testPackages)
-    {
-        using var testDirectory = new TemporaryDirectory();
-        var buildFiles = new List<ProjectBuildFile>();
-
-        await UpdateWorkerTestBase.MockNuGetPackagesInDirectory(testPackages, testDirectory.DirectoryPath);
-
-        foreach (var (path, content) in buildFileContents)
-        {
-            var fullPath = Path.Combine(testDirectory.DirectoryPath, path);
-            await File.WriteAllTextAsync(fullPath, content);
-            buildFiles.Add(ProjectBuildFile.Parse(testDirectory.DirectoryPath, fullPath, content));
-        }
-
-        var actualTopLevelDependencies = MSBuildHelper.GetTopLevelPackageDependencyInfos(buildFiles.ToImmutableArray());
-        AssertEx.Equal(expectedTopLevelDependencies, actualTopLevelDependencies);
-    }
-
     [Theory]
     [InlineData(true)]
     [InlineData(false)]
@@ -209,8 +29,7 @@ public class MSBuildHelperTests : TestBase
 
         Dependency[] expectedDependencies =
         [
-            new("NETStandard.Library", "2.0.3", DependencyType.Unknown, TargetFrameworks: ["netstandard2.0"], IsTransitive: true),
-            new("Package.A", "1.0.0", DependencyType.Unknown, TargetFrameworks: ["netstandard2.0"]),
+            new("Package.A", "1.0.0", DependencyType.PackageReference, TargetFrameworks: ["netstandard2.0"], IsDirect: true),
             new("Package.B", "2.0.0", DependencyType.Unknown, TargetFrameworks: ["netstandard2.0"], IsTransitive: true),
             new("Package.C", "3.0.0", DependencyType.Unknown, TargetFrameworks: ["netstandard2.0"], IsTransitive: true),
             new("Package.D", "4.0.0", DependencyType.Unknown, TargetFrameworks: ["netstandard2.0"], IsTransitive: true),
@@ -291,7 +110,7 @@ public class MSBuildHelperTests : TestBase
 
         Dependency[] expectedDependencies =
         [
-            new("Package.1A", "1.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"]),
+            new("Package.1A", "1.0.0", DependencyType.PackageReference, TargetFrameworks: ["net8.0"], IsDirect: true),
             new("Package.1B", "2.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.1C", "3.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.1D", "4.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
@@ -308,7 +127,7 @@ public class MSBuildHelperTests : TestBase
             new("Package.1O", "15.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.1P", "16.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.1Q", "17.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
-            new("Package.1R", "18.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"]),
+            new("Package.1R", "18.0.0", DependencyType.PackageReference, TargetFrameworks: ["net8.0"], IsDirect: true),
             new("Package.1S", "19.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.1T", "20.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.1U", "21.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
@@ -317,7 +136,7 @@ public class MSBuildHelperTests : TestBase
             new("Package.1X", "24.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.1Y", "25.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.1Z", "26.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
-            new("Package.2A", "1.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"]),
+            new("Package.2A", "1.0.0", DependencyType.PackageReference, TargetFrameworks: ["net8.0"], IsDirect: true),
             new("Package.2B", "2.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.2C", "3.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.2D", "4.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
@@ -334,7 +153,7 @@ public class MSBuildHelperTests : TestBase
             new("Package.2O", "15.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.2P", "16.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.2Q", "17.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
-            new("Package.2R", "18.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"]),
+            new("Package.2R", "18.0.0", DependencyType.PackageReference, TargetFrameworks: ["net8.0"], IsDirect : true),
             new("Package.2S", "19.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.2T", "20.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
             new("Package.2U", "21.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
@@ -385,8 +204,8 @@ public class MSBuildHelperTests : TestBase
 
         Dependency[] expectedDependencies =
         [
-            new("Package.A", "1.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"]),
-            new("Package.B", "2.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"]),
+            new("Package.A", "1.0.0", DependencyType.PackageReference, TargetFrameworks: ["net8.0"], IsDirect: true),
+            new("Package.B", "2.0.0", DependencyType.PackageReference, TargetFrameworks: ["net8.0"], IsDirect: true),
         ];
         var packages = new[]
         {
@@ -422,7 +241,7 @@ public class MSBuildHelperTests : TestBase
                 <add key="contoso" value="https://contoso.com/v3/index.json"
                 </packageSources>
             </configuration>
-            """);
+            """, TestContext.Current.CancellationToken);
 
         // Asserting it didn't throw
         var actualDependencies = await MSBuildHelper.GetAllPackageDependenciesAsync(
@@ -460,11 +279,11 @@ public class MSBuildHelperTests : TestBase
                 <add key="localSource2" value="local/source2" />
                 </packageSources>
             </configuration>
-            """);
+            """, TestContext.Current.CancellationToken);
 
         Dependency[] expectedDependencies =
         [
-            new("Package.A", "1.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"]),
+            new("Package.A", "1.0.0", DependencyType.PackageReference, TargetFrameworks: ["net8.0"], IsDirect : true),
             new("Package.B", "2.0.0", DependencyType.Unknown, TargetFrameworks: ["net8.0"], IsTransitive: true),
         ];
 
@@ -480,108 +299,6 @@ public class MSBuildHelperTests : TestBase
         AssertEx.Equal(expectedDependencies, actualDependencies);
     }
 
-    [Theory]
-    [InlineData(true)]
-    [InlineData(false)]
-    public async Task DependencyConflictsCanBeResolvedWithBruteForce(bool useExistingSdks)
-    {
-        var repoRoot = Directory.CreateTempSubdirectory($"test_{nameof(DependencyConflictsCanBeResolvedWithBruteForce)}_");
-        MockNuGetPackage[] testPackages =
-        [
-            // some base packages
-            MockNuGetPackage.CreateSimplePackage("Some.Package", "1.0.0", "net8.0"),
-            MockNuGetPackage.CreateSimplePackage("Some.Package", "1.1.0", "net8.0"),
-            MockNuGetPackage.CreateSimplePackage("Some.Package", "1.2.0", "net8.0"),
-            // some packages that are hard-locked to specific versions of the previous package
-            MockNuGetPackage.CreateSimplePackage("Some.Other.Package", "1.0.0", "net8.0", [(null, [("Some.Package", "[1.0.0]")])]),
-            MockNuGetPackage.CreateSimplePackage("Some.Other.Package", "1.1.0", "net8.0", [(null, [("Some.Package", "[1.1.0]")])]),
-            MockNuGetPackage.CreateSimplePackage("Some.Other.Package", "1.2.0", "net8.0", [(null, [("Some.Package", "[1.2.0]")])]),
-        ];
-        await UpdateWorkerTestBase.MockNuGetPackagesInDirectory(testPackages, repoRoot.FullName);
-
-        // the package `Some.Package` was already updated from 1.0.0 to 1.2.0, but this causes a conflict with
-        // `Some.Other.Package` that needs to be resolved
-        try
-        {
-            var projectPath = Path.Join(repoRoot.FullName, "project.csproj");
-            await File.WriteAllTextAsync(projectPath, """
-                <Project Sdk="Microsoft.NET.Sdk">
-                  <PropertyGroup>
-                    <TargetFramework>net8.0</TargetFramework>
-                  </PropertyGroup>
-                  <ItemGroup>
-                    <PackageReference Include="Some.Package" Version="1.2.0" />
-                    <PackageReference Include="Some.Other.Package" Version="1.0.0" />
-                  </ItemGroup>
-                </Project>
-                """);
-            var dependencies = new[]
-            {
-                new Dependency("Some.Package", "1.2.0", DependencyType.PackageReference),
-                new Dependency("Some.Other.Package", "1.0.0", DependencyType.PackageReference),
-            }.ToImmutableArray();
-            var update = new[]
-            {
-                new Dependency("Some.Other.Package", "1.2.0", DependencyType.PackageReference),
-            };
-            var resolvedDependencies = await MSBuildHelper.ResolveDependencyConflictsWithBruteForce(
-                repoRoot.FullName,
-                projectPath,
-                "net8.0",
-                dependencies,
-                new ExperimentsManager() { InstallDotnetSdks = useExistingSdks },
-                new TestLogger()
-            );
-            Assert.NotNull(resolvedDependencies);
-            Assert.Equal(2, resolvedDependencies.Value.Length);
-            Assert.Equal("Some.Package", resolvedDependencies.Value[0].Name);
-            Assert.Equal("1.2.0", resolvedDependencies.Value[0].Version);
-            Assert.Equal("Some.Other.Package", resolvedDependencies.Value[1].Name);
-            Assert.Equal("1.2.0", resolvedDependencies.Value[1].Version);
-        }
-        finally
-        {
-            repoRoot.Delete(recursive: true);
-        }
-    }
-
-    [Fact]
-    public void UpdateWithWorkloadsTargetFrameworks()
-    {
-        // Arrange
-        var projectContents = """
-            <Project>
-                <PropertyGroup>
-                    <TargetFrameworks>net8.0-ios;net8.0-android;net8.0-macos;net8.0-maccatalyst;</TargetFrameworks>
-                </PropertyGroup>
-                <ItemGroup>
-                    <PackageReference Include="Some.Package" Version="$(PackageVersion1)" />
-                </ItemGroup>
-            </Project>
-            """;
-        var propertyInfo = new Dictionary<string, Property>
-        {
-            { "PackageVersion1", new("PackageVersion1", "1.1.1", "Packages.props") },
-        };
-
-        // Act
-        var (resultType, _, evaluatedValue, _, _) = MSBuildHelper.GetEvaluatedValue(projectContents, propertyInfo);
-
-        Assert.Equal(EvaluationResultType.Success, resultType);
-
-        // Assert
-        Assert.Equal("""
-            <Project>
-                <PropertyGroup>
-                    <TargetFrameworks>net8.0-ios;net8.0-android;net8.0-macos;net8.0-maccatalyst;</TargetFrameworks>
-                </PropertyGroup>
-                <ItemGroup>
-                    <PackageReference Include="Some.Package" Version="1.1.1" />
-                </ItemGroup>
-            </Project>
-            """, evaluatedValue);
-    }
-
     [Theory]
     [MemberData(nameof(GetTargetFrameworkValuesFromProjectData))]
     public async Task GetTargetFrameworkValuesFromProject(string projectContents, string[] expectedTfms)
@@ -778,6 +495,14 @@ public class MSBuildHelperTests : TestBase
             new PrivateSourceBadResponse(["http://localhost/test-feed"]),
         ];
 
+        yield return
+        [
+            // output
+            "  The HTTP request to 'GET some-source' has timed out after 100000ms.",
+            // expectedError
+            new PrivateSourceTimedOut("some-source"),
+        ];
+
         yield return
         [
             // output
@@ -826,6 +551,14 @@ public class MSBuildHelperTests : TestBase
             new DependencyNotFound("Some.Package"),
         ];
 
+        yield return
+        [
+            // output
+            "Unable to resolve dependency 'Some.Package'. Source(s) used: 'nuget.org'.",
+            // expectedError
+            new DependencyNotFound("Some.Package"),
+        ];
+
         yield return
         [
             // output
@@ -893,371 +626,4 @@ public class MSBuildHelperTests : TestBase
             new DependencyFileNotParseable("/path/to/NuGet.Config", "Some error message."),
         ];
     }
-
-    public static IEnumerable<object[]> GetTopLevelPackageDependencyInfosTestData()
-    {
-        // simple case
-        yield return
-        [
-            // build file contents
-            new[]
-            {
-                ("project.csproj", """
-                    <Project Sdk="Microsoft.NET.Sdk">
-                      <ItemGroup>
-                        <PackageReference Include="Some.Package" Version="12.0.1" />
-                      </ItemGroup>
-                    </Project>
-                    """)
-            },
-            // expected dependencies
-            new Dependency[]
-            {
-                new(
-                    "Some.Package",
-                    "12.0.1",
-                    DependencyType.PackageReference,
-                    EvaluationResult: new(EvaluationResultType.Success, "12.0.1", "12.0.1", null, null))
-            },
-            new MockNuGetPackage[]
-            {
-                MockNuGetPackage.CreateSimplePackage("Some.Package", "12.0.1", "net8.0")
-            }
-        ];
-
-        // version is a child-node of the package reference
-        yield return
-        [
-            // build file contents
-            new[]
-            {
-                ("project.csproj", """
-                    <Project Sdk="Microsoft.NET.Sdk">
-                      <ItemGroup>
-                        <PackageReference Include="Some.Package">
-                            <Version>12.0.1</Version>
-                        </PackageReference>
-                      </ItemGroup>
-                    </Project>
-                    """)
-            },
-            // expected dependencies
-            new Dependency[]
-            {
-                new(
-                    "Some.Package",
-                    "12.0.1",
-                    DependencyType.PackageReference,
-                    EvaluationResult: new(EvaluationResultType.Success, "12.0.1", "12.0.1", null, null))
-            },
-            new MockNuGetPackage[]
-            {
-                MockNuGetPackage.CreateSimplePackage("Some.Package", "12.0.1", "net8.0")
-            }
-        ];
-
-        // version is in property in same file
-        yield return
-        [
-            // build file contents
-            new[]
-            {
-                ("project.csproj", """
-                    <Project Sdk="Microsoft.NET.Sdk">
-                      <PropertyGroup>
-                        <SomePackageVersion>12.0.1</SomePackageVersion>
-                      </PropertyGroup>
-                      <ItemGroup>
-                        <PackageReference Include="Some.Package" Version="$(SomePackageVersion)" />
-                      </ItemGroup>
-                    </Project>
-                    """)
-            },
-            // expected dependencies
-            new Dependency[]
-            {
-                new(
-                    "Some.Package",
-                    "12.0.1",
-                    DependencyType.PackageReference,
-                    new(EvaluationResultType.Success, "$(SomePackageVersion)", "12.0.1", "SomePackageVersion", null))
-            },
-            new MockNuGetPackage[]
-            {
-                MockNuGetPackage.CreateSimplePackage("Some.Package", "12.0.1", "net8.0")
-            }
-        ];
-
-        // version is a property not triggered by a condition
-        yield return
-        [
-            // build file contents
-            new[]
-            {
-                ("project.csproj", """
-                    <Project Sdk="Microsoft.NET.Sdk">
-                      <PropertyGroup>
-                        <TargetFramework>netstandard2.0</TargetFramework>
-                        <SomePackageVersion>12.0.1</SomePackageVersion>
-                        <SomePackageVersion Condition="$(PropertyThatDoesNotExist) == 'true'">13.0.1</SomePackageVersion>
-                      </PropertyGroup>
-                      <ItemGroup>
-                        <PackageReference Include="Some.Package" Version="$(SomePackageVersion)" />
-                      </ItemGroup>
-                    </Project>
-                    """)
-            },
-            // expected dependencies
-            new Dependency[]
-            {
-                new(
-                    "Some.Package",
-                    "12.0.1",
-                    DependencyType.PackageReference,
-                    new(EvaluationResultType.Success, "$(SomePackageVersion)", "12.0.1", "SomePackageVersion", null))
-            },
-            new MockNuGetPackage[]
-            {
-                MockNuGetPackage.CreateSimplePackage("Some.Package", "12.0.1", "net8.0")
-            }
-        ];
-
-        // version is a property not triggered by a quoted condition
-        yield return new object[]
-        {
-            // build file contents
-            new[]
-            {
-                ("project.csproj", """
-                    <Project Sdk="Microsoft.NET.Sdk">
-                      <PropertyGroup>
-                        <TargetFramework>netstandard2.0</TargetFramework>
-                        <SomePackageVersion>12.0.1</SomePackageVersion>
-                        <SomePackageVersion Condition="'$(PropertyThatDoesNotExist)' == 'true'">13.0.1</SomePackageVersion>
-                      </PropertyGroup>
-                      <ItemGroup>
-                        <PackageReference Include="Some.Package" Version="$(SomePackageVersion)" />
-                      </ItemGroup>
-                    </Project>
-                    """)
-            },
-            // expected dependencies
-            new Dependency[]
-            {
-                new(
-                    "Some.Package",
-                    "12.0.1",
-                    DependencyType.PackageReference,
-                    new(EvaluationResultType.Success, "$(SomePackageVersion)", "12.0.1", "SomePackageVersion", null))
-            },
-            new MockNuGetPackage[]
-            {
-                MockNuGetPackage.CreateSimplePackage("Some.Package", "12.0.1", "net8.0")
-            }
-        };
-
-        // version is a property with a condition checking for an empty string
-        yield return
-        [
-            // build file contents
-            new[]
-            {
-                ("project.csproj", """
-                    <Project Sdk="Microsoft.NET.Sdk">
-                      <PropertyGroup>
-                        <TargetFramework>netstandard2.0</TargetFramework>
-                        <SomePackageVersion Condition="$(SomePackageVersion) == ''">12.0.1</SomePackageVersion>
-                        <SomePackageVersion Condition="$(PropertyThatDoesNotExist) == 'true'">13.0.1</SomePackageVersion>
-                      </PropertyGroup>
-                      <ItemGroup>
-                        <PackageReference Include="Some.Package" Version="$(SomePackageVersion)" />
-                      </ItemGroup>
-                    </Project>
-                    """)
-            },
-            // expected dependencies
-            new Dependency[]
-            {
-                new(
-                    "Some.Package",
-                    "12.0.1",
-                    DependencyType.PackageReference,
-                    new(EvaluationResultType.Success, "$(SomePackageVersion)", "12.0.1", "SomePackageVersion", null))
-            },
-            new MockNuGetPackage[]
-            {
-                MockNuGetPackage.CreateSimplePackage("Some.Package", "12.0.1", "net8.0")
-            }
-        ];
-
-        // version is a property with a quoted condition checking for an empty string
-        yield return new object[]
-        {
-            // build file contents
-            new[]
-            {
-                ("project.csproj", """
-                    <Project Sdk="Microsoft.NET.Sdk">
-                      <PropertyGroup>
-                        <TargetFramework>netstandard2.0</TargetFramework>
-                        <SomePackageVersion Condition="'$(SomePackageVersion)' == ''">12.0.1</SomePackageVersion>
-                        <SomePackageVersion Condition="'$(PropertyThatDoesNotExist)' == 'true'">13.0.1</SomePackageVersion>
-                      </PropertyGroup>
-                      <ItemGroup>
-                        <PackageReference Include="Some.Package" Version="$(SomePackageVersion)" />
-                      </ItemGroup>
-                    </Project>
-                    """)
-            },
-            // expected dependencies
-            new Dependency[]
-            {
-                new(
-                    "Some.Package",
-                    "12.0.1",
-                    DependencyType.PackageReference,
-                    new(EvaluationResultType.Success, "$(SomePackageVersion)", "12.0.1", "SomePackageVersion", null))
-            },
-            new MockNuGetPackage[]
-            {
-                MockNuGetPackage.CreateSimplePackage("Some.Package", "12.0.1", "net8.0")
-            }
-        };
-
-        // version is set in one file, used in another
-        yield return
-        [
-            // build file contents
-            new[]
-            {
-                ("Packages.props", """
-                        <Project>
-                          <ItemGroup>
-                            <PackageReference Update="Package.A" Version="1.6.0" />
-                            <PackageReference Update="Package.B" Version="5.1.4" />
-                          </ItemGroup>
-                        </Project>
-                    """),
-                ("project.csproj", """
-                    <Project Sdk="Microsoft.NET.Sdk">
-                      <PropertyGroup>
-                        <TargetFramework>net8.0</TargetFramework>
-                      </PropertyGroup>
-                      <ItemGroup>
-                        <PackageReference Include="Package.A" Version="1.6.1" />
-                      </ItemGroup>
-                    </Project>
-                    """)
-            },
-            // expected dependencies
-            new Dependency[]
-            {
-                new(
-                    "Package.A",
-                    "1.6.0",
-                    DependencyType.PackageReference,
-                    EvaluationResult: new(EvaluationResultType.Success, "1.6.0", "1.6.0", null, null)),
-                new(
-                    "Package.B",
-                    "5.1.4",
-                    DependencyType.PackageReference,
-                    EvaluationResult: new(EvaluationResultType.Success, "5.1.4", "5.1.4", null, null),
-                    IsUpdate: true),
-            },
-            new MockNuGetPackage[]
-            {
-                MockNuGetPackage.CreateSimplePackage("Package.A", "1.6.0", "net8.0"),
-                MockNuGetPackage.CreateSimplePackage("Package.A", "1.6.1", "net8.0"),
-                MockNuGetPackage.CreateSimplePackage("Package.B", "5.1.4", "net8.0"),
-            }
-        ];
-
-        // version is set in one file, used in another
-        yield return
-        [
-            // build file contents
-            new[]
-            {
-                ("project.csproj", """
-                    <Project Sdk="Microsoft.NET.Sdk">
-                      <PropertyGroup>
-                        <TargetFramework>net8.0</TargetFramework>
-                      </PropertyGroup>
-                      <ItemGroup>
-                        <PackageReference Include="Package.A" />
-                      </ItemGroup>
-                    </Project>
-                    """),
-                ("Packages.props", """
-                        <Project>
-                          <ItemGroup>
-                            <PackageReference Update="Package.A" Version="1.6.0" />
-                            <PackageReference Update="Package.B" Version="5.1.4" />
-                          </ItemGroup>
-                        </Project>
-                    """)
-            },
-            // expected dependencies
-            new Dependency[]
-            {
-                new(
-                    "Package.A",
-                    "1.6.0",
-                    DependencyType.PackageReference,
-                    EvaluationResult: new(EvaluationResultType.Success, "1.6.0", "1.6.0", null, null)),
-                new(
-                    "Package.B",
-                    "5.1.4",
-                    DependencyType.PackageReference,
-                    EvaluationResult: new(EvaluationResultType.Success, "5.1.4", "5.1.4", null, null),
-                    IsUpdate: true),
-            },
-            new MockNuGetPackage[]
-            {
-                MockNuGetPackage.CreateSimplePackage("Package.A", "1.6.0", "net8.0"),
-                MockNuGetPackage.CreateSimplePackage("Package.B", "5.1.4", "net8.0"),
-            }
-        ];
-    }
-
-    public static IEnumerable<object[]> SolutionProjectPathTestData()
-    {
-        yield return
-        [
-            """
-            Microsoft Visual Studio Solution File, Format Version 12.00
-            # Visual Studio 14
-            VisualStudioVersion = 14.0.22705.0
-            MinimumVisualStudioVersion = 10.0.40219.1
-            Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Some.Project", "src\Some.Project\SomeProject.csproj", "{782E0C0A-10D3-444D-9640-263D03D2B20C}"
-            EndProject
-            Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "Some.Project.Test", "src\Some.Project.Test\Some.Project.Test.csproj", "{5C15FD5B-1975-4CEA-8F1B-C0C9174C60A9}"
-            EndProject
-            Global
-            	GlobalSection(SolutionConfigurationPlatforms) = preSolution
-            		Debug|Any CPU = Debug|Any CPU
-            		Release|Any CPU = Release|Any CPU
-            	EndGlobalSection
-            	GlobalSection(ProjectConfigurationPlatforms) = postSolution
-            		{782E0C0A-10D3-444D-9640-263D03D2B20C}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
-            		{782E0C0A-10D3-444D-9640-263D03D2B20C}.Debug|Any CPU.Build.0 = Debug|Any CPU
-            		{782E0C0A-10D3-444D-9640-263D03D2B20C}.Release|Any CPU.ActiveCfg = Release|Any CPU
-            		{782E0C0A-10D3-444D-9640-263D03D2B20C}.Release|Any CPU.Build.0 = Release|Any CPU
-            		{5C15FD5B-1975-4CEA-8F1B-C0C9174C60A9}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
-            		{5C15FD5B-1975-4CEA-8F1B-C0C9174C60A9}.Debug|Any CPU.Build.0 = Debug|Any CPU
-            		{5C15FD5B-1975-4CEA-8F1B-C0C9174C60A9}.Release|Any CPU.ActiveCfg = Release|Any CPU
-            		{5C15FD5B-1975-4CEA-8F1B-C0C9174C60A9}.Release|Any CPU.Build.0 = Release|Any CPU
-            	EndGlobalSection
-            	GlobalSection(SolutionProperties) = preSolution
-            		HideSolutionNode = FALSE
-            	EndGlobalSection
-            EndGlobal
-            """,
-            new[]
-            {
-                "src/Some.Project/SomeProject.csproj",
-                "src/Some.Project.Test/Some.Project.Test.csproj",
-            }
-        ];
-    }
 }