Cleanup

NuGet · Nov 14, 2024 · 2187307 · 2187307
1 parent 62ac4d1
commit 2187307
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/src/NuGet.Core/NuGet.Commands/RestoreCommand/Utility/MSBuildRestoreUtility.cs b/src/NuGet.Core/NuGet.Commands/RestoreCommand/Utility/MSBuildRestoreUtility.cs
@@ -739,10 +739,9 @@ internal static void AddPrunedPackageReferences(PackageSpec spec, IEnumerable<IM
                 var tfm = item.GetProperty("TargetFramework") ?? string.Empty;
 
                 bool enabled = IsPropertyTrue(item, "RestoreEnablePackagePruning");
-                isPruningEnabled.Add(tfm, enabled);
+                isPruningEnabled[tfm] = enabled;
             }
 
-
             foreach (var item in GetItemByType(items, "PrunedPackageReference"))
             {
                 var id = item.GetProperty("Id");

diff --git a/....PackageManagement.VisualStudio.Test/ProjectSystems/LegacyPackageReferenceProjectTests.cs b/....PackageManagement.VisualStudio.Test/ProjectSystems/LegacyPackageReferenceProjectTests.cs
@@ -1993,7 +1993,7 @@ public static readonly List<object[]> PrunedPackageReferenceData
             = new List<object[]>
             {
                 new object[] { "true", new (string, string[])[] { ("PackageA", ["1.0.0"]) }, true },
-                new object[] { "true", new (string, string[])[] { ("PackageA", ["1.0.0"]) }, false },
+                new object[] { "false", new (string, string[])[] { ("PackageA", ["1.0.0"]) }, false },
             };
 
         [Theory]

diff --git a/.../NuGet.Core.FuncTests/NuGet.Commands.FuncTest/RestoreCommand_AlgorithmEquivalencyTests.cs b/.../NuGet.Core.FuncTests/NuGet.Commands.FuncTest/RestoreCommand_AlgorithmEquivalencyTests.cs
@@ -2011,7 +2011,6 @@ await SimpleTestPackageUtility.CreateFolderFeedV3Async(
 
         // Add tests with ranges, in particular where the specified version is in the range, but not the one actually selected.
         // Add tests with ranges such as floating versions? This is interesting.
-        // When should the warning be raised? In NET 10 only?
         // TODO NK - Clarify that packages can be pruned from PackageReferences.
 
         // Here's why package driven dependencies should flow.