dependabot-nuget 0.285.0 → 0.286.0

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/Run/SerializationTests.cs

@@ -59,6 +59,162 @@ public class SerializationTests
         Assert.Equal("specific-sdk", jobWrapper.Job.Source.Directory);
     }
 
+    [Fact]
+    public void DeserializeJob_FieldsNotYetSupported()
+    {
+        // the `source` field is required in the C# model; the remaining fields might exist in the JSON file, but are
+        // not yet supported in the C# model (some keys missing, others deserialize to `object?`; deserialization
+        // should not fail
+        var jobWrapper = RunWorker.Deserialize("""
+            {
+              "job": {
+                "source": {
+                  "provider": "github",
+                  "repo": "some-org/some-repo",
+                  "directory": "specific-sdk",
+                  "hostname": null,
+                  "api-endpoint": null
+                },
+                "id": "some-id",
+                "allowed-updates": [
+                  {
+                    "dependency-type": "direct",
+                    "update-type": "all"
+                  }
+                ],
+                "credentials": [
+                  {
+                    "name": "some-cred",
+                    "token": "abc123"
+                  }
+                ],
+                "existing-pull-requests": [
+                  [
+                    {
+                      "dependency-name": "Some.Package",
+                      "dependency-version": "1.2.3"
+                    }
+                  ]
+                ],
+                "repo-contents-path": "/path/to/repo",
+                "token": "abc123"
+              }
+            }
+            """);
+        Assert.Equal("github", jobWrapper.Job.Source.Provider);
+        Assert.Equal("some-org/some-repo", jobWrapper.Job.Source.Repo);
+        Assert.Equal("specific-sdk", jobWrapper.Job.Source.Directory);
+    }
+
+    [Fact]
+    public void DeserializeExperimentsManager()
+    {
+        var jobWrapper = RunWorker.Deserialize("""
+            {
+              "job": {
+                "package-manager": "nuget",
+                "allowed-updates": [
+                  {
+                    "update-type": "all"
+                  }
+                ],
+                "source": {
+                  "provider": "github",
+                  "repo": "some-org/some-repo",
+                  "directory": "some-dir"
+                },
+                "experiments": {
+                  "nuget_legacy_dependency_solver": true,
+                  "unexpected_bool": true,
+                  "unexpected_number": 42,
+                  "unexpected_null": null,
+                  "unexpected_string": "abc",
+                  "unexpected_array": [1, "two", 3.0],
+                  "unexpected_object": {
+                    "a": 1,
+                    "b": "two"
+                  }
+                }
+              }
+            }
+            """);
+        var experimentsManager = ExperimentsManager.GetExperimentsManager(jobWrapper.Job.Experiments);
+        Assert.True(experimentsManager.UseLegacyDependencySolver);
+    }
+
+    [Fact]
+    public void DeserializeExperimentsManager_EmptyExperiments()
+    {
+        var jobWrapper = RunWorker.Deserialize("""
+            {
+              "job": {
+                "package-manager": "nuget",
+                "allowed-updates": [
+                  {
+                    "update-type": "all"
+                  }
+                ],
+                "source": {
+                  "provider": "github",
+                  "repo": "some-org/some-repo",
+                  "directory": "some-dir"
+                },
+                "experiments": {
+                }
+              }
+            }
+            """);
+        var experimentsManager = ExperimentsManager.GetExperimentsManager(jobWrapper.Job.Experiments);
+        Assert.False(experimentsManager.UseLegacyDependencySolver);
+    }
+
+    [Fact]
+    public void DeserializeExperimentsManager_NoExperiments()
+    {
+        var jobWrapper = RunWorker.Deserialize("""
+            {
+              "job": {
+                "package-manager": "nuget",
+                "allowed-updates": [
+                  {
+                    "update-type": "all"
+                  }
+                ],
+                "source": {
+                  "provider": "github",
+                  "repo": "some-org/some-repo",
+                  "directory": "some-dir"
+                }
+              }
+            }
+            """);
+        var experimentsManager = ExperimentsManager.GetExperimentsManager(jobWrapper.Job.Experiments);
+        Assert.False(experimentsManager.UseLegacyDependencySolver);
+    }
+
+    [Fact]
+    public async Task DeserializeExperimentsManager_UnsupportedJobFileShape_InfoIsReportedAndEmptyExperimentSetIsReturned()
+    {
+        // arrange
+        using var tempDir = new TemporaryDirectory();
+        var jobFilePath = Path.Combine(tempDir.DirectoryPath, "job.json");
+        var jobContent = """
+            {
+              "this-is-not-a-job-and-parsing-will-fail-but-an-empty-experiment-set-should-sill-be-returned": {
+              }
+            }
+            """;
+        await File.WriteAllTextAsync(jobFilePath, jobContent);
+        var capturingTestLogger = new CapturingTestLogger();
+
+        // act - this is the entrypoint the update command uses to parse the job file
+        var experimentsManager = await ExperimentsManager.FromJobFileAsync(jobFilePath, capturingTestLogger);
+
+        // assert
+        Assert.False(experimentsManager.UseLegacyDependencySolver);
+        Assert.Single(capturingTestLogger.Messages.Where(m => m.Contains("Error deserializing job file")));
+    }
+
     [Fact]
     public void SerializeError()
     {
@@ -67,4 +223,16 @@ public class SerializationTests
         var expected = """{"data":{"error-type":"job_repo_not_found","error-details":{"message":"some message"}}}""";
         Assert.Equal(expected, actual);
     }
+
+    private class CapturingTestLogger : ILogger
+    {
+        private readonly List<string> _messages = new();
+
+        public IReadOnlyList<string> Messages => _messages;
+
+        public void Log(string message)
+        {
+            _messages.Add(message);
+        }
+    }
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/TestAnalyzeWorker.cs

@@ -0,0 +1,37 @@
+using NuGetUpdater.Core.Analyze;
+using NuGetUpdater.Core.Discover;
+
+namespace NuGetUpdater.Core.Test;
+
+internal class TestAnalyzeWorker : IAnalyzeWorker
+{
+    private readonly Func<(string, WorkspaceDiscoveryResult, DependencyInfo), Task<AnalysisResult>> _getResult;
+
+    public TestAnalyzeWorker(Func<(string, WorkspaceDiscoveryResult, DependencyInfo), Task<AnalysisResult>> getResult)
+    {
+        _getResult = getResult;
+    }
+
+    public Task<AnalysisResult> RunAsync(string repoRoot, WorkspaceDiscoveryResult discovery, DependencyInfo dependencyInfo)
+    {
+        return _getResult((repoRoot, discovery, dependencyInfo));
+    }
+
+    public static TestAnalyzeWorker FromResults(params (string RepoRoot, WorkspaceDiscoveryResult Discovery, DependencyInfo DependencyInfo, AnalysisResult Result)[] results)
+    {
+        return new TestAnalyzeWorker(((string RepoRoot, WorkspaceDiscoveryResult Discovery, DependencyInfo DependencyInfo) input) =>
+        {
+            foreach (var set in results)
+            {
+                if (set.RepoRoot == input.RepoRoot &&
+                    set.Discovery == input.Discovery &&
+                    set.DependencyInfo == input.DependencyInfo)
+                {
+                    return Task.FromResult(set.Result);
+                }
+            }
+
+            throw new NotImplementedException($"No saved response for {input}");
+        });
+    }
+}

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/TestDiscoveryWorker.cs

@@ -0,0 +1,35 @@
+using NuGetUpdater.Core.Discover;
+
+namespace NuGetUpdater.Core.Test;
+
+internal class TestDiscoveryWorker : IDiscoveryWorker
+{
+    private readonly Func<(string, string), Task<WorkspaceDiscoveryResult>> _getResult;
+
+    public TestDiscoveryWorker(Func<(string, string), Task<WorkspaceDiscoveryResult>> getResult)
+    {
+        _getResult = getResult;
+    }
+
+    public Task<WorkspaceDiscoveryResult> RunAsync(string repoRootPath, string workspacePath)
+    {
+        return _getResult((repoRootPath, workspacePath));
+    }
+
+    public static TestDiscoveryWorker FromResults(params (string RepoRootPath, string WorkspacePath, WorkspaceDiscoveryResult Result)[] results)
+    {
+        return new TestDiscoveryWorker(((string RepoRootPath, string WorkspacePath) input) =>
+        {
+            foreach (var set in results)
+            {
+                if (set.RepoRootPath == input.RepoRootPath &&
+                    set.WorkspacePath == input.WorkspacePath)
+                {
+                    return Task.FromResult(set.Result);
+                }
+            }
+
+            throw new NotImplementedException($"No saved response for {input}");
+        });
+    }
+}

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/TestUpdaterWorker.cs

@@ -0,0 +1,39 @@
+using NuGetUpdater.Core.Updater;
+
+namespace NuGetUpdater.Core.Test;
+
+internal class TestUpdaterWorker : IUpdaterWorker
+{
+    private readonly Func<(string, string, string, string, string, bool), Task<UpdateOperationResult>> _getResult;
+
+    public TestUpdaterWorker(Func<(string, string, string, string, string, bool), Task<UpdateOperationResult>> getResult)
+    {
+        _getResult = getResult;
+    }
+
+    public Task<UpdateOperationResult> RunAsync(string repoRootPath, string workspacePath, string dependencyName, string previousDependencyVersion, string newDependencyVersion, bool isTransitive)
+    {
+        return _getResult((repoRootPath, workspacePath, dependencyName, previousDependencyVersion, newDependencyVersion, isTransitive));
+    }
+
+    public static TestUpdaterWorker FromResults(params (string RepoRootPath, string WorkspacePath, string DependencyName, string PreviousDependencyVersion, string NewDependencyVersion, bool IsTransitive, UpdateOperationResult Result)[] results)
+    {
+        return new TestUpdaterWorker(((string RepoRootPath, string WorkspacePath, string DependencyName, string PreviousDependencyVersion, string NewDependencyVersion, bool IsTransitive) input) =>
+        {
+            foreach (var set in results)
+            {
+                if (set.RepoRootPath == input.RepoRootPath &&
+                    set.WorkspacePath == input.WorkspacePath &&
+                    set.DependencyName == input.DependencyName &&
+                    set.PreviousDependencyVersion == input.PreviousDependencyVersion &&
+                    set.NewDependencyVersion == input.NewDependencyVersion &&
+                    set.IsTransitive == input.IsTransitive)
+                {
+                    return Task.FromResult(set.Result);
+                }
+            }
+
+            throw new NotImplementedException($"No saved response for {input}");
+        });
+    }
+}

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

@@ -6,18 +6,27 @@ public class PackagesConfigUpdaterTests : TestBase
 {
     [Theory]
     [MemberData(nameof(PackagesDirectoryPathTestData))]
-    public void PathToPackagesDirectoryCanBeDetermined(string projectContents, string dependencyName, string dependencyVersion, string expectedPackagesDirectoryPath)
+    public async Task PathToPackagesDirectoryCanBeDetermined(string projectContents, string? packagesConfigContents, string dependencyName, string dependencyVersion, string expectedPackagesDirectoryPath)
     {
+        using var tempDir = new TemporaryDirectory();
+        string? packagesConfigPath = null;
+        if (packagesConfigContents is not null)
+        {
+            packagesConfigPath = Path.Join(tempDir.DirectoryPath, "packages.config");
+            await File.WriteAllTextAsync(packagesConfigPath, packagesConfigContents);
+        }
+
         var projectBuildFile = ProjectBuildFile.Parse("/", "project.csproj", projectContents);
-        var actualPackagesDirectorypath = PackagesConfigUpdater.GetPathToPackagesDirectory(projectBuildFile, dependencyName, dependencyVersion, "packages.config");
+        var actualPackagesDirectorypath = PackagesConfigUpdater.GetPathToPackagesDirectory(projectBuildFile, dependencyName, dependencyVersion, packagesConfigPath);
         Assert.Equal(expectedPackagesDirectoryPath, actualPackagesDirectorypath);
     }
 
-    public static IEnumerable<object[]> PackagesDirectoryPathTestData()
+    public static IEnumerable<object?[]> PackagesDirectoryPathTestData()
     {
         // project with namespace
         yield return
         [
+            // project contents
             """
             <Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
               <ItemGroup>
@@ -28,14 +37,20 @@ public class PackagesConfigUpdaterTests : TestBase
               </ItemGroup>
             </Project>
             """,
+            // packages.config contents
+            null,
+            // dependency name
             "Newtonsoft.Json",
+            // dependency version
             "7.0.1",
+            // expected packages directory path
             "../packages"
         ];
 
         // project without namespace
         yield return
         [
+            // project contents
             """
             <Project>
               <ItemGroup>
@@ -46,14 +61,20 @@ public class PackagesConfigUpdaterTests : TestBase
               </ItemGroup>
             </Project>
             """,
+            // packages.config contents
+            null,
+            // dependency name
             "Newtonsoft.Json",
+            // dependency version
             "7.0.1",
+            // expected packages directory path
             "../packages"
         ];
 
         // project with non-standard packages path
         yield return
         [
+            // project contents
             """
             <Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
               <ItemGroup>
@@ -64,9 +85,89 @@ public class PackagesConfigUpdaterTests : TestBase
               </ItemGroup>
             </Project>
             """,
+            // packages.config contents
+            null,
+            // dependency name
             "Newtonsoft.Json",
+            // dependency version
             "7.0.1",
+            // expected packages directory path
             "../not-a-path-you-would-expect"
         ];
+
+        // project without expected packages path, but has others
+        yield return
+        [
+            // project contents
+            """
+            <Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
+              <ItemGroup>
+                <Reference Include="Some.Other.Package, Version=1.2.3.4, Culture=neutral, PublicKeyToken=30ad4fe6b2a6aeed">
+                  <HintPath>..\..\..\still-a-usable-path\Some.Other.Package.1.2.3\lib\net45\Some.Other.Package.dll</HintPath>
+                  <Private>True</Private>
+                </Reference>
+              </ItemGroup>
+            </Project>
+            """,
+            // packages.config contents
+            """
+            <packages>
+              <package id="Newtonsoft.Json" version="7.0.1" targetFramework="net45" />
+            </packages>
+            """,
+            // dependency name
+            "Newtonsoft.Json",
+            // dependency version
+            "7.0.1",
+            // expected packages directory path
+            "../../../still-a-usable-path"
+        ];
+
+        // project without expected package, but exists in packages.config, default is returned
+        yield return
+        [
+            // project contents
+            """
+            <Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
+              <ItemGroup>
+              </ItemGroup>
+            </Project>
+            """,
+            // packages.config contents
+            """
+            <packages>
+              <package id="Newtonsoft.Json" version="7.0.1" targetFramework="net45" />
+            </packages>
+            """,
+            // dependency name
+            "Newtonsoft.Json",
+            // dependency version
+            "7.0.1",
+            // expected packages directory path
+            "../packages"
+        ];
+
+        // project without expected package and not in packages.config
+        yield return
+        [
+            // project contents
+            """
+            <Project xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
+              <ItemGroup>
+              </ItemGroup>
+            </Project>
+            """,
+            // packages.config contents
+            """
+            <packages>
+            </packages>
+            """,
+            // dependency name
+            "Newtonsoft.Json",
+            // dependency version
+            "7.0.1",
+            // expected packages directory path
+            null
+        ];
     }
 }

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

@@ -1,3 +1,7 @@
+using System.Text.Json;
+
+using NuGetUpdater.Core.Run;
+using NuGetUpdater.Core.Run.ApiModel;
 using NuGetUpdater.Core.Test.Updater;
 using NuGetUpdater.Core.Updater;
 
@@ -19,11 +23,12 @@ public abstract class UpdateWorkerTestBase : TestBase
         bool isTransitive = false,
         TestFile[]? additionalFiles = null,
         MockNuGetPackage[]? packages = null,
+        ExperimentsManager? experimentsManager = null,
         string projectFilePath = "test-project.csproj")
     {
         return useSolution
-            ? TestNoChangeforSolution(dependencyName, oldVersion, newVersion, projectFiles: [(projectFilePath, projectContents)], isTransitive, additionalFiles, packages)
-            : TestNoChangeforProject(dependencyName, oldVersion, newVersion, projectContents, isTransitive, additionalFiles, packages, projectFilePath);
+            ? TestNoChangeforSolution(dependencyName, oldVersion, newVersion, projectFiles: [(projectFilePath, projectContents)], isTransitive, additionalFiles, packages, experimentsManager)
+            : TestNoChangeforProject(dependencyName, oldVersion, newVersion, projectContents, isTransitive, additionalFiles, packages, experimentsManager, projectFilePath);
     }
 
     protected static Task TestUpdate(
@@ -37,11 +42,12 @@ public abstract class UpdateWorkerTestBase : TestBase
         TestFile[]? additionalFiles = null,
         TestFile[]? additionalFilesExpected = null,
         MockNuGetPackage[]? packages = null,
+        ExperimentsManager? experimentsManager = null,
         string projectFilePath = "test-project.csproj")
     {
         return useSolution
-            ? TestUpdateForSolution(dependencyName, oldVersion, newVersion, projectFiles: [(projectFilePath, projectContents)], projectFilesExpected: [(projectFilePath, expectedProjectContents)], isTransitive, additionalFiles, additionalFilesExpected, packages)
-            : TestUpdateForProject(dependencyName, oldVersion, newVersion, projectFile: (projectFilePath, projectContents), expectedProjectContents, isTransitive, additionalFiles, additionalFilesExpected, packages);
+            ? TestUpdateForSolution(dependencyName, oldVersion, newVersion, projectFiles: [(projectFilePath, projectContents)], projectFilesExpected: [(projectFilePath, expectedProjectContents)], isTransitive, additionalFiles, additionalFilesExpected, packages, experimentsManager)
+            : TestUpdateForProject(dependencyName, oldVersion, newVersion, projectFile: (projectFilePath, projectContents), expectedProjectContents, isTransitive, additionalFiles, additionalFilesExpected, packages, experimentsManager);
     }
 
     protected static Task TestUpdate(
@@ -54,11 +60,12 @@ public abstract class UpdateWorkerTestBase : TestBase
         bool isTransitive = false,
         TestFile[]? additionalFiles = null,
         TestFile[]? additionalFilesExpected = null,
-        MockNuGetPackage[]? packages = null)
+        MockNuGetPackage[]? packages = null,
+        ExperimentsManager? experimentsManager = null)
     {
         return useSolution
-            ? TestUpdateForSolution(dependencyName, oldVersion, newVersion, projectFiles: [projectFile], projectFilesExpected: [(projectFile.Path, expectedProjectContents)], isTransitive, additionalFiles, additionalFilesExpected, packages)
-            : TestUpdateForProject(dependencyName, oldVersion, newVersion, projectFile, expectedProjectContents, isTransitive, additionalFiles, additionalFilesExpected, packages);
+            ? TestUpdateForSolution(dependencyName, oldVersion, newVersion, projectFiles: [projectFile], projectFilesExpected: [(projectFile.Path, expectedProjectContents)], isTransitive, additionalFiles, additionalFilesExpected, packages, experimentsManager)
+            : TestUpdateForProject(dependencyName, oldVersion, newVersion, projectFile, expectedProjectContents, isTransitive, additionalFiles, additionalFilesExpected, packages, experimentsManager);
     }
 
     protected static Task TestNoChangeforProject(
@@ -69,6 +76,7 @@ public abstract class UpdateWorkerTestBase : TestBase
         bool isTransitive = false,
         TestFile[]? additionalFiles = null,
         MockNuGetPackage[]? packages = null,
+        ExperimentsManager? experimentsManager = null,
         string projectFilePath = "test-project.csproj")
         => TestUpdateForProject(
             dependencyName,
@@ -79,7 +87,8 @@ public abstract class UpdateWorkerTestBase : TestBase
             isTransitive,
             additionalFiles,
             additionalFilesExpected: additionalFiles,
-            packages: packages);
+            packages: packages,
+            experimentsManager: experimentsManager);
 
     protected static Task TestUpdateForProject(
         string dependencyName,
@@ -91,6 +100,7 @@ public abstract class UpdateWorkerTestBase : TestBase
         TestFile[]? additionalFiles = null,
         TestFile[]? additionalFilesExpected = null,
         MockNuGetPackage[]? packages = null,
+        ExperimentsManager? experimentsManager = null,
         string projectFilePath = "test-project.csproj",
         ExpectedUpdateOperationResult? expectedResult = null)
         => TestUpdateForProject(
@@ -103,6 +113,7 @@ public abstract class UpdateWorkerTestBase : TestBase
             additionalFiles,
             additionalFilesExpected,
             packages,
+            experimentsManager,
             expectedResult);
 
     protected static async Task TestUpdateForProject(
@@ -115,6 +126,7 @@ public abstract class UpdateWorkerTestBase : TestBase
         TestFile[]? additionalFiles = null,
         TestFile[]? additionalFilesExpected = null,
         MockNuGetPackage[]? packages = null,
+        ExperimentsManager? experimentsManager = null,
         ExpectedUpdateOperationResult? expectedResult = null)
     {
         additionalFiles ??= [];
@@ -134,7 +146,8 @@ public abstract class UpdateWorkerTestBase : TestBase
             await MockNuGetPackagesInDirectory(packages, temporaryDirectory);
 
             // run update
-            var worker = new UpdaterWorker(new TestLogger());
+            experimentsManager ??= new ExperimentsManager();
+            var worker = new UpdaterWorker(experimentsManager, new TestLogger());
             var projectPath = placeFilesInSrc ? $"src/{projectFilePath}" : projectFilePath;
             var actualResult = await worker.RunWithErrorHandlingAsync(temporaryDirectory, projectPath, dependencyName, oldVersion, newVersion, isTransitive);
             if (expectedResult is { })
@@ -172,7 +185,8 @@ public abstract class UpdateWorkerTestBase : TestBase
         TestFile[] projectFiles,
         bool isTransitive = false,
         TestFile[]? additionalFiles = null,
-        MockNuGetPackage[]? packages = null)
+        MockNuGetPackage[]? packages = null,
+        ExperimentsManager? experimentsManager = null)
         => TestUpdateForSolution(
             dependencyName,
             oldVersion,
@@ -182,7 +196,8 @@ public abstract class UpdateWorkerTestBase : TestBase
             isTransitive,
             additionalFiles,
             additionalFilesExpected: additionalFiles,
-            packages: packages);
+            packages: packages,
+            experimentsManager: experimentsManager);
 
     protected static async Task TestUpdateForSolution(
         string dependencyName,
@@ -193,7 +208,8 @@ public abstract class UpdateWorkerTestBase : TestBase
         bool isTransitive = false,
         TestFile[]? additionalFiles = null,
         TestFile[]? additionalFilesExpected = null,
-        MockNuGetPackage[]? packages = null)
+        MockNuGetPackage[]? packages = null,
+        ExperimentsManager? experimentsManager = null)
     {
         additionalFiles ??= [];
         additionalFilesExpected ??= [];
@@ -236,8 +252,9 @@ public abstract class UpdateWorkerTestBase : TestBase
         {
             await MockNuGetPackagesInDirectory(packages, temporaryDirectory);
 
+            experimentsManager ??= new ExperimentsManager();
             var slnPath = Path.Combine(temporaryDirectory, slnName);
-            var worker = new UpdaterWorker(new TestLogger());
+            var worker = new UpdaterWorker(experimentsManager, new TestLogger());
             await worker.RunAsync(temporaryDirectory, slnPath, dependencyName, oldVersion, newVersion, isTransitive);
         });
 
@@ -246,6 +263,27 @@ public abstract class UpdateWorkerTestBase : TestBase
         AssertContainsFiles(expectedResult, actualResult);
     }
 
+    public static async Task MockJobFileInDirectory(string temporaryDirectory)
+    {
+        var jobFile = new JobFile()
+        {
+            Job = new()
+            {
+                AllowedUpdates =
+                [
+                    new() { UpdateType = "all" }
+                ],
+                Source = new()
+                {
+                    Provider = "github",
+                    Repo = "test/repo",
+                    Directory = "/",
+                }
+            }
+        };
+        await File.WriteAllTextAsync(Path.Join(temporaryDirectory, "job.json"), JsonSerializer.Serialize(jobFile, RunWorker.SerializerOptions));
+    }
+
     public static async Task MockNuGetPackagesInDirectory(MockNuGetPackage[]? packages, string temporaryDirectory)
     {
         if (packages is not null)

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/Update/UpdateWorkerTests.DirsProj.cs

@@ -350,7 +350,8 @@ public partial class UpdateWorkerTests
             bool isTransitive = false,
             (string Path, string Content)[]? additionalFiles = null,
             (string Path, string Content)[]? additionalFilesExpected = null,
-            MockNuGetPackage[]? packages = null)
+            MockNuGetPackage[]? packages = null,
+            ExperimentsManager? experimentsManager = null)
         {
             additionalFiles ??= [];
             additionalFilesExpected ??= [];
@@ -363,8 +364,9 @@ public partial class UpdateWorkerTests
             {
                 await MockNuGetPackagesInDirectory(packages, temporaryDirectory);
 
+                experimentsManager ??= new ExperimentsManager();
                 var projectPath = Path.Combine(temporaryDirectory, projectFileName);
-                var worker = new UpdaterWorker(new TestLogger());
+                var worker = new UpdaterWorker(experimentsManager, new TestLogger());
                 await worker.RunAsync(temporaryDirectory, projectPath, dependencyName, oldVersion, newVersion, isTransitive);
             });