dependabot-nuget 0.245.0 → 0.247.0

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Utilities/MSBuildHelper.cs

@@ -17,10 +17,14 @@ using Microsoft.Build.Exceptions;
 using Microsoft.Build.Locator;
 using Microsoft.Extensions.FileSystemGlobbing;
 
+using NuGet.Configuration;
+
 using NuGetUpdater.Core.Utilities;
 
 namespace NuGetUpdater.Core;
 
+using EvaluationResult = (MSBuildHelper.EvaluationResultType ResultType, string EvaluatedValue, string? ErrorMessage);
+
 internal static partial class MSBuildHelper
 {
     public static string MSBuildPath { get; private set; } = string.Empty;
@@ -57,7 +61,10 @@ internal static partial class MSBuildHelper
                 if (property.Name.Equals("TargetFramework", StringComparison.OrdinalIgnoreCase) ||
                     property.Name.Equals("TargetFrameworks", StringComparison.OrdinalIgnoreCase))
                 {
-                    targetFrameworkValues.Add(property.Value);
+                    foreach (var tfm in property.Value.Split(';', StringSplitOptions.RemoveEmptyEntries | StringSplitOptions.TrimEntries))
+                    {
+                        targetFrameworkValues.Add(tfm);
+                    }
                 }
                 else if (property.Name.Equals("TargetFrameworkVersion", StringComparison.OrdinalIgnoreCase))
                 {
@@ -75,8 +82,12 @@ internal static partial class MSBuildHelper
 
         foreach (var targetFrameworkValue in targetFrameworkValues)
         {
-            var tfms = targetFrameworkValue;
-            tfms = GetRootedValue(tfms, propertyInfo);
+            var (resultType, tfms, errorMessage) =
+                GetEvaluatedValue(targetFrameworkValue, propertyInfo, propertiesToIgnore: ["TargetFramework", "TargetFrameworks"]);
+            if (resultType != EvaluationResultType.Success)
+            {
+                continue;
+            }
 
             if (string.IsNullOrEmpty(tfms))
             {
@@ -101,10 +112,18 @@ internal static partial class MSBuildHelper
     public static IEnumerable<string> GetProjectPathsFromProject(string projFilePath)
     {
         var projectStack = new Stack<(string folderPath, ProjectRootElement)>();
-        var projectRootElement = ProjectRootElement.Open(projFilePath);
         var processedProjectFiles = new HashSet<string>(StringComparer.OrdinalIgnoreCase);
+        using var projectCollection = new ProjectCollection();
 
-        projectStack.Push((Path.GetFullPath(Path.GetDirectoryName(projFilePath)!), projectRootElement));
+        try
+        {
+            var projectRootElement = ProjectRootElement.Open(projFilePath, projectCollection);
+            projectStack.Push((Path.GetFullPath(Path.GetDirectoryName(projFilePath)!), projectRootElement));
+        }
+        catch (InvalidProjectFileException)
+        {
+            yield break; // Skip invalid project files
+        }
 
         while (projectStack.Count > 0)
         {
@@ -137,7 +156,7 @@ internal static partial class MSBuildHelper
                         // If there is some MSBuild logic that needs to run to fully resolve the path skip the project
                         if (File.Exists(file))
                         {
-                            var additionalProjectRootElement = ProjectRootElement.Open(file);
+                            var additionalProjectRootElement = ProjectRootElement.Open(file, projectCollection);
                             projectStack.Push((Path.GetFullPath(Path.GetDirectoryName(file)!), additionalProjectRootElement));
                             processedProjectFiles.Add(file);
                         }
@@ -222,9 +241,13 @@ internal static partial class MSBuildHelper
             }
 
             // Walk the property replacements until we don't find another one.
-            packageVersion = GetRootedValue(packageVersion, propertyInfo);
+            var evaluationResult = GetEvaluatedValue(packageVersion, propertyInfo);
+            if (evaluationResult.ResultType != EvaluationResultType.Success)
+            {
+                throw new InvalidDataException(evaluationResult.ErrorMessage);
+            }
 
-            packageVersion = packageVersion.TrimStart('[', '(').TrimEnd(']', ')');
+            packageVersion = evaluationResult.EvaluatedValue.TrimStart('[', '(').TrimEnd(']', ')');
 
             // We don't know the version for range requirements or wildcard
             // requirements, so return "" for these.
@@ -237,25 +260,32 @@ internal static partial class MSBuildHelper
     /// <summary>
     /// Given an MSBuild string and a set of properties, returns our best guess at the final value MSBuild will evaluate to.
     /// </summary>
-    /// <param name="msbuildString"></param>
-    /// <param name="propertyInfo"></param>
-    /// <returns></returns>
-    public static string GetRootedValue(string msbuildString, Dictionary<string, string> propertyInfo)
+    public static EvaluationResult GetEvaluatedValue(string msbuildString, Dictionary<string, string> propertyInfo, params string[] propertiesToIgnore)
     {
+        var ignoredProperties = new HashSet<string>(propertiesToIgnore, StringComparer.OrdinalIgnoreCase);
         var seenProperties = new HashSet<string>(StringComparer.OrdinalIgnoreCase);
+
         while (TryGetPropertyName(msbuildString, out var propertyName))
         {
+            if (ignoredProperties.Contains(propertyName))
+            {
+                return (EvaluationResultType.PropertyIgnored, msbuildString, $"Property '{propertyName}' is ignored.");
+            }
+
             if (!seenProperties.Add(propertyName))
             {
-                throw new InvalidDataException($"Property '{propertyName}' has a circular reference.");
+                return (EvaluationResultType.CircularReference, msbuildString, $"Property '{propertyName}' has a circular reference.");
             }
 
-            msbuildString = propertyInfo.TryGetValue(propertyName, out var propertyValue)
-                ? msbuildString.Replace($"$({propertyName})", propertyValue)
-                : throw new InvalidDataException($"Property '{propertyName}' was not found.");
+            if (!propertyInfo.TryGetValue(propertyName, out var propertyValue))
+            {
+                return (EvaluationResultType.PropertyNotFound, msbuildString, $"Property '{propertyName}' was not found.");
+            }
+
+            msbuildString = msbuildString.Replace($"$({propertyName})", propertyValue);
         }
 
-        return msbuildString;
+        return (EvaluationResultType.Success, msbuildString, null);
     }
 
     public static bool TryGetPropertyName(string versionContent, [NotNullWhen(true)] out string? propertyName)
@@ -303,6 +333,25 @@ internal static partial class MSBuildHelper
         return projectRoot;
     }
 
+    private static IEnumerable<PackageSource>? LoadPackageSources(string nugetConfigPath)
+    {
+        try
+        {
+            var nugetConfigDir = Path.GetDirectoryName(nugetConfigPath);
+            var settings = Settings.LoadSpecificSettings(nugetConfigDir, Path.GetFileName(nugetConfigPath));
+            var packageSourceProvider = new PackageSourceProvider(settings);
+            return packageSourceProvider.LoadPackageSources();
+        }
+        catch (NuGetConfigurationException ex)
+        {
+            Console.WriteLine("Error while parsing NuGet.config");
+            Console.WriteLine(ex.Message);
+
+            // Nuget.config is invalid. Won't be able to do anything with specific sources.
+            return null;
+        }
+    }
+
     private static async Task<string> CreateTempProjectAsync(
         DirectoryInfo tempDir,
         string repoRoot,
@@ -313,10 +362,27 @@ internal static partial class MSBuildHelper
         var projectDirectory = Path.GetDirectoryName(projectPath);
         projectDirectory ??= repoRoot;
         var topLevelFiles = Directory.GetFiles(repoRoot);
-        var nugetConfigPath = PathHelper.GetFileInDirectoryOrParent(projectDirectory, repoRoot, "NuGet.Config", caseSensitive: false);
+        var nugetConfigPath = PathHelper.GetFileInDirectoryOrParent(projectPath, repoRoot, "NuGet.Config", caseSensitive: false);
         if (nugetConfigPath is not null)
         {
+            // Copy nuget.config to temp project directory
             File.Copy(nugetConfigPath, Path.Combine(tempDir.FullName, "NuGet.Config"));
+            var nugetConfigDir = Path.GetDirectoryName(nugetConfigPath);
+
+            var packageSources = LoadPackageSources(nugetConfigPath);
+            if (packageSources is not null)
+            {
+                // We need to copy local package sources from the NuGet.Config file to the temp directory
+                foreach (var localSource in packageSources.Where(p => p.IsLocal))
+                {
+                    var subDir = localSource.Source.Split(nugetConfigDir)[1];
+                    var destPath = Path.Join(tempDir.FullName, subDir);
+                    if (Directory.Exists(localSource.Source))
+                    {
+                        PathHelper.CopyDirectory(localSource.Source, destPath);
+                    }
+                }
+            }
         }
 
         var packageReferences = string.Join(
@@ -492,4 +558,12 @@ internal static partial class MSBuildHelper
 
     [GeneratedRegex("^\\s*NuGetData::Package=(?<PackageName>[^,]+), Version=(?<PackageVersion>.+)$")]
     private static partial Regex PackagePattern();
+
+    internal enum EvaluationResultType
+    {
+        Success,
+        PropertyIgnored,
+        CircularReference,
+        PropertyNotFound,
+    }
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Utilities/NuGetHelper.cs

@@ -6,7 +6,7 @@ internal static class NuGetHelper
 {
     internal const string PackagesConfigFileName = "packages.config";
 
-    public static bool HasProjectConfigFile(string projectPath)
+    public static bool HasPackagesConfigFile(string projectPath)
     {
         var projectDirectory = Path.GetDirectoryName(projectPath);
         var packagesConfigPath = PathHelper.JoinPath(projectDirectory, PackagesConfigFileName);

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Utilities/PathHelper.cs

@@ -79,4 +79,31 @@ internal static class PathHelper
 
         return null;
     }
+
+    public static void CopyDirectory(string sourceDirectory, string destinationDirectory)
+    {
+        var sourceDirInfo = new DirectoryInfo(sourceDirectory);
+        var destinationDirInfo = new DirectoryInfo(destinationDirectory);
+
+        if (!sourceDirInfo.Exists)
+        {
+            throw new DirectoryNotFoundException($"Source directory does not exist or could not be found: {sourceDirectory}");
+        }
+
+        if (!destinationDirInfo.Exists)
+        {
+            destinationDirInfo.Create();
+        }
+
+        foreach (var file in sourceDirInfo.EnumerateFiles())
+        {
+            file.CopyTo(Path.Combine(destinationDirectory, file.Name), true);
+        }
+
+        foreach (var subDir in sourceDirInfo.EnumerateDirectories())
+        {
+            var newDestinationDir = Path.Combine(destinationDirectory, subDir.Name);
+            CopyDirectory(subDir.FullName, newDestinationDir);
+        }
+    }
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/Update/UpdateWorkerTestBase.cs

@@ -6,10 +6,61 @@ using System.Threading.Tasks;
 
 using Xunit;
 
+using TestFile = (string Path, string Content);
+using TestProject = (string Path, string Content, System.Guid ProjectId);
+
 namespace NuGetUpdater.Core.Test.Update;
 
 public abstract class UpdateWorkerTestBase
 {
+    protected static Task TestNoChange(
+        string dependencyName,
+        string oldVersion,
+        string newVersion,
+        bool useSolution,
+        string projectContents,
+        bool isTransitive = false,
+        (string Path, string Content)[]? additionalFiles = null,
+        string projectFilePath = "test-project.csproj")
+    {
+        return useSolution
+            ? TestNoChangeforSolution(dependencyName, oldVersion, newVersion, projectFiles: [(projectFilePath, projectContents)], isTransitive, additionalFiles)
+            : TestNoChangeforProject(dependencyName, oldVersion, newVersion, projectContents, isTransitive, additionalFiles, projectFilePath);
+    }
+
+    protected static Task TestUpdate(
+        string dependencyName,
+        string oldVersion,
+        string newVersion,
+        bool useSolution,
+        string projectContents,
+        string expectedProjectContents,
+        bool isTransitive = false,
+        TestFile[]? additionalFiles = null,
+        TestFile[]? additionalFilesExpected = null,
+        string projectFilePath = "test-project.csproj")
+    {
+        return useSolution
+            ? TestUpdateForSolution(dependencyName, oldVersion, newVersion, projectFiles: [(projectFilePath, projectContents)], projectFilesExpected: [(projectFilePath, expectedProjectContents)], isTransitive, additionalFiles, additionalFilesExpected)
+            : TestUpdateForProject(dependencyName, oldVersion, newVersion, projectFile: (projectFilePath, projectContents), expectedProjectContents, isTransitive, additionalFiles, additionalFilesExpected);
+    }
+
+    protected static Task TestUpdate(
+        string dependencyName,
+        string oldVersion,
+        string newVersion,
+        bool useSolution,
+        TestFile projectFile,
+        string expectedProjectContents,
+        bool isTransitive = false,
+        TestFile[]? additionalFiles = null,
+        TestFile[]? additionalFilesExpected = null)
+    {
+        return useSolution
+            ? TestUpdateForSolution(dependencyName, oldVersion, newVersion, projectFiles: [projectFile], projectFilesExpected: [(projectFile.Path, expectedProjectContents)], isTransitive, additionalFiles, additionalFilesExpected)
+            : TestUpdateForProject(dependencyName, oldVersion, newVersion, projectFile, expectedProjectContents, isTransitive, additionalFiles, additionalFilesExpected);
+    }
+
     protected static Task TestNoChangeforProject(
         string dependencyName,
         string oldVersion,
@@ -35,8 +86,8 @@ public abstract class UpdateWorkerTestBase
         string projectContents,
         string expectedProjectContents,
         bool isTransitive = false,
-        (string Path, string Content)[]? additionalFiles = null,
-        (string Path, string Content)[]? additionalFilesExpected = null,
+        TestFile[]? additionalFiles = null,
+        TestFile[]? additionalFilesExpected = null,
         string projectFilePath = "test-project.csproj")
         => TestUpdateForProject(
             dependencyName,
@@ -52,42 +103,92 @@ public abstract class UpdateWorkerTestBase
         string dependencyName,
         string oldVersion,
         string newVersion,
-        (string Path, string Content) projectFile,
+        TestFile projectFile,
         string expectedProjectContents,
         bool isTransitive = false,
-        (string Path, string Content)[]? additionalFiles = null,
-        (string Path, string Content)[]? additionalFilesExpected = null)
+        TestFile[]? additionalFiles = null,
+        TestFile[]? additionalFilesExpected = null)
     {
         additionalFiles ??= [];
         additionalFilesExpected ??= [];
 
         var projectFilePath = projectFile.Path;
-        var projectName = Path.GetFileNameWithoutExtension(projectFilePath);
+        var testFiles = new[] { projectFile }.Concat(additionalFiles).ToArray();
+
+        var actualResult = await RunUpdate(testFiles, async temporaryDirectory =>
+        {
+            var worker = new UpdaterWorker(new Logger(verbose: true));
+            await worker.RunAsync(temporaryDirectory, projectFilePath, dependencyName, oldVersion, newVersion, isTransitive);
+        });
+
+        var expectedResult = additionalFilesExpected.Prepend((projectFilePath, expectedProjectContents)).ToArray();
+
+        AssertContainsFiles(expectedResult, actualResult);
+    }
+
+    protected static Task TestNoChangeforSolution(
+        string dependencyName,
+        string oldVersion,
+        string newVersion,
+        TestFile[] projectFiles,
+        bool isTransitive = false,
+        TestFile[]? additionalFiles = null)
+        => TestUpdateForSolution(
+            dependencyName,
+            oldVersion,
+            newVersion,
+            projectFiles,
+            projectFilesExpected: projectFiles,
+            isTransitive,
+            additionalFiles,
+            additionalFilesExpected: additionalFiles);
+
+    protected static async Task TestUpdateForSolution(
+        string dependencyName,
+        string oldVersion,
+        string newVersion,
+        TestFile[] projectFiles,
+        TestFile[] projectFilesExpected,
+        bool isTransitive = false,
+        TestFile[]? additionalFiles = null,
+        TestFile[]? additionalFilesExpected = null)
+    {
+        additionalFiles ??= [];
+        additionalFilesExpected ??= [];
+
+        var testProjects = projectFiles.Select(file => new TestProject(file.Path, file.Content, Guid.NewGuid())).ToArray();
+        var projectDeclarations = testProjects.Select(project => $$"""
+            Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "{{Path.GetFileNameWithoutExtension(project.Path)}}", "{{project.Path}}", "{{project.ProjectId}}"
+            EndProject
+            """);
+        var debugConfiguration = testProjects.Select(project => $$"""
+                {{project.ProjectId}}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+                {{project.ProjectId}}.Debug|Any CPU.Build.0 = Debug|Any CPU
+                {{project.ProjectId}}..Release|Any CPU.ActiveCfg = Release|Any CPU
+                {{project.ProjectId}}..Release|Any CPU.Build.0 = Release|Any CPU
+            """);
+
         var slnName = "test-solution.sln";
         var slnContent = $$"""
             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}") = "{{projectName}}", "{{projectFilePath}}", "{782E0C0A-10D3-444D-9640-263D03D2B20C}"
-            EndProject
+            {{string.Join(Environment.NewLine, projectDeclarations)}}
             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
+            {{string.Join(Environment.NewLine, debugConfiguration)}}
               EndGlobalSection
               GlobalSection(SolutionProperties) = preSolution
                 HideSolutionNode = FALSE
               EndGlobalSection
             EndGlobal
             """;
-        var testFiles = new[] { (slnName, slnContent), projectFile }.Concat(additionalFiles).ToArray();
+        var testFiles = new[] { (slnName, slnContent) }.Concat(projectFiles).Concat(additionalFiles).ToArray();
 
         var actualResult = await RunUpdate(testFiles, async temporaryDirectory =>
         {
@@ -96,7 +197,7 @@ public abstract class UpdateWorkerTestBase
             await worker.RunAsync(temporaryDirectory, slnPath, dependencyName, oldVersion, newVersion, isTransitive);
         });
 
-        var expectedResult = additionalFilesExpected.Prepend((projectFilePath, expectedProjectContents)).ToArray();
+        var expectedResult = projectFilesExpected.Concat(additionalFilesExpected).ToArray();
 
         AssertContainsFiles(expectedResult, actualResult);
     }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/Update/{UpdateWorker.DirsProj.cs → UpdateWorkerTests.DirsProj.cs}

@@ -1,5 +1,3 @@
-using System;
-using System.Collections.Generic;
 using System.IO;
 using System.Linq;
 using System.Threading.Tasks;
@@ -24,7 +22,7 @@ public partial class UpdateWorkerTests
                 // initial
                 projectContents: """
                 <Project Sdk="Microsoft.Build.NoTargets">
-                
+
                   <ItemGroup>
                     <ProjectReference Include="src/test-project.csproj" />
                   </ItemGroup>
@@ -39,7 +37,7 @@ public partial class UpdateWorkerTests
                           <PropertyGroup>
                             <TargetFramework>netstandard2.0</TargetFramework>
                           </PropertyGroup>
-                        
+
                           <ItemGroup>
                             <PackageReference Include="Newtonsoft.Json" Version="9.0.1" />
                           </ItemGroup>
@@ -49,7 +47,7 @@ public partial class UpdateWorkerTests
                 // expected
                 expectedProjectContents: """
                 <Project Sdk="Microsoft.Build.NoTargets">
-                
+
                   <ItemGroup>
                     <ProjectReference Include="src/test-project.csproj" />
                   </ItemGroup>
@@ -64,7 +62,7 @@ public partial class UpdateWorkerTests
                           <PropertyGroup>
                             <TargetFramework>netstandard2.0</TargetFramework>
                           </PropertyGroup>
-                        
+
                           <ItemGroup>
                             <PackageReference Include="Newtonsoft.Json" Version="13.0.1" />
                           </ItemGroup>
@@ -101,7 +99,7 @@ public partial class UpdateWorkerTests
                 // initial
                 projectContents: """
                 <Project Sdk="Microsoft.Build.NoTargets">
-                
+
                   <ItemGroup>
                     <ProjectReference Include="src/dirs.proj" />
                   </ItemGroup>
@@ -113,7 +111,7 @@ public partial class UpdateWorkerTests
                     ("src/dirs.proj",
                         """
                         <Project Sdk="Microsoft.Build.NoTargets">
-                        
+
                           <ItemGroup>
                             <ProjectReference Include="test-project/test-project.csproj" />
                           </ItemGroup>
@@ -126,7 +124,7 @@ public partial class UpdateWorkerTests
                           <PropertyGroup>
                             <TargetFramework>netstandard2.0</TargetFramework>
                           </PropertyGroup>
-                        
+
                           <ItemGroup>
                             <PackageReference Include="Newtonsoft.Json" Version="9.0.1" />
                           </ItemGroup>
@@ -136,7 +134,7 @@ public partial class UpdateWorkerTests
                 // expected
                 expectedProjectContents: """
                 <Project Sdk="Microsoft.Build.NoTargets">
-                
+
                   <ItemGroup>
                     <ProjectReference Include="src/dirs.proj" />
                   </ItemGroup>
@@ -148,7 +146,7 @@ public partial class UpdateWorkerTests
                     ("src/dirs.proj",
                         """
                         <Project Sdk="Microsoft.Build.NoTargets">
-                        
+
                           <ItemGroup>
                             <ProjectReference Include="test-project/test-project.csproj" />
                           </ItemGroup>
@@ -161,7 +159,7 @@ public partial class UpdateWorkerTests
                           <PropertyGroup>
                             <TargetFramework>netstandard2.0</TargetFramework>
                           </PropertyGroup>
-                        
+
                           <ItemGroup>
                             <PackageReference Include="Newtonsoft.Json" Version="13.0.1" />
                           </ItemGroup>
@@ -177,7 +175,7 @@ public partial class UpdateWorkerTests
                 // initial
                 projectContents: """
                 <Project Sdk="Microsoft.Build.NoTargets">
-                
+
                   <ItemGroup>
                     <ProjectReference Include="src/*.proj" />
                   </ItemGroup>
@@ -189,7 +187,7 @@ public partial class UpdateWorkerTests
                     ("src/dirs.proj",
                         """
                         <Project Sdk="Microsoft.Build.NoTargets">
-                        
+
                           <ItemGroup>
                             <ProjectReference Include="test-project/test-project.csproj" />
                           </ItemGroup>
@@ -202,7 +200,7 @@ public partial class UpdateWorkerTests
                           <PropertyGroup>
                             <TargetFramework>netstandard2.0</TargetFramework>
                           </PropertyGroup>
-                        
+
                           <ItemGroup>
                             <PackageReference Include="Newtonsoft.Json" Version="9.0.1" />
                           </ItemGroup>
@@ -212,7 +210,7 @@ public partial class UpdateWorkerTests
                 // expected
                 expectedProjectContents: """
                 <Project Sdk="Microsoft.Build.NoTargets">
-                
+
                   <ItemGroup>
                     <ProjectReference Include="src/*.proj" />
                   </ItemGroup>
@@ -224,7 +222,7 @@ public partial class UpdateWorkerTests
                     ("src/dirs.proj",
                         """
                         <Project Sdk="Microsoft.Build.NoTargets">
-                        
+
                           <ItemGroup>
                             <ProjectReference Include="test-project/test-project.csproj" />
                           </ItemGroup>
@@ -237,7 +235,7 @@ public partial class UpdateWorkerTests
                           <PropertyGroup>
                             <TargetFramework>netstandard2.0</TargetFramework>
                           </PropertyGroup>
-                        
+
                           <ItemGroup>
                             <PackageReference Include="Newtonsoft.Json" Version="13.0.1" />
                           </ItemGroup>
@@ -253,7 +251,7 @@ public partial class UpdateWorkerTests
                 // initial
                 projectContents: """
                 <Project Sdk="Microsoft.Build.NoTargets">
-                
+
                   <ItemGroup>
                     <ProjectReference Include="**/*.proj" />
                   </ItemGroup>
@@ -265,7 +263,7 @@ public partial class UpdateWorkerTests
                     ("src/dirs.proj",
                         """
                         <Project Sdk="Microsoft.Build.NoTargets">
-                        
+
                           <ItemGroup>
                             <ProjectReference Include="test-project/test-project.csproj" />
                           </ItemGroup>
@@ -278,7 +276,7 @@ public partial class UpdateWorkerTests
                           <PropertyGroup>
                             <TargetFramework>netstandard2.0</TargetFramework>
                           </PropertyGroup>
-                        
+
                           <ItemGroup>
                             <PackageReference Include="Newtonsoft.Json" Version="9.0.1" />
                           </ItemGroup>
@@ -288,7 +286,7 @@ public partial class UpdateWorkerTests
                 // expected
                 expectedProjectContents: """
                 <Project Sdk="Microsoft.Build.NoTargets">
-                
+
                   <ItemGroup>
                     <ProjectReference Include="**/*.proj" />
                   </ItemGroup>
@@ -300,7 +298,7 @@ public partial class UpdateWorkerTests
                     ("src/dirs.proj",
                         """
                         <Project Sdk="Microsoft.Build.NoTargets">
-                        
+
                           <ItemGroup>
                             <ProjectReference Include="test-project/test-project.csproj" />
                           </ItemGroup>
@@ -313,7 +311,7 @@ public partial class UpdateWorkerTests
                           <PropertyGroup>
                             <TargetFramework>netstandard2.0</TargetFramework>
                           </PropertyGroup>
-                        
+
                           <ItemGroup>
                             <PackageReference Include="Newtonsoft.Json" Version="13.0.1" />
                           </ItemGroup>