dependabot-nuget 0.342.2 → 0.343.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 60fbe0fbaa5bb2494d76a9cbe53912c79557910c13a5c2672721ca7665696995
-  data.tar.gz: 027bc2de484a8724567e9901a88f3dac47acba9aca76fa1edec8a521a5ed1078
+  metadata.gz: 79344385eec91d92a3c7182eb639186693399069d4c3c761528fe5090633a7ca
+  data.tar.gz: 174ba03152133f0c03fe6da96cae0b110b6f7642482c325be3dfab2a397b261f
 SHA512:
-  metadata.gz: 58150b153196abc1033d83ed24852e66bb8579f48bff844fdd867036fea12280cc53a7038d7f918189d741aabf1e2bfb628b41e1da86cc065b4ade0f68492e3e
-  data.tar.gz: d6509ec9845d2540cdf3823b94587dd386adc72ddbcaaa12670bfa9a9624bdb9dbd7b5b7ae0fe176a98b9d140f818c6c776e3bd3125d4fe0f37726577853fc9a
+  metadata.gz: 38b2c7817daaca765ff4f7736da12e50fc71d8a22d2e680ac740e26d106d79f2fea17b8f68ab35748dbb15dae2b966937b0120a2a8138958b85cf2652982b6c2
+  data.tar.gz: 30e4eebd3db7f09e8b9c6af6bbbfdee7a1456e0c1a001ef774ed71f9bfbba85bd9680623489a5e9893c942b7d2490051465d66b92932492eb8ec39f7d5a4b432

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/ApiModel/JobErrorBase.cs

@@ -38,7 +38,7 @@ public abstract record JobErrorBase : MessageBase
             case BadRequirementException badRequirement:
                 return new BadRequirement(badRequirement.Message);
             case BadResponseException badResponse:
-                return new PrivateSourceBadResponse([badResponse.Uri]);
+                return new PrivateSourceBadResponse([badResponse.Uri], badResponse.Message);
             case DependencyNotFoundException dependencyNotFound:
                 return new DependencyNotFound(string.Join(", ", dependencyNotFound.Dependencies));
             case HttpRequestException httpRequest:
@@ -48,7 +48,7 @@ public abstract record JobErrorBase : MessageBase
                         ioException.HttpRequestError == HttpRequestError.ResponseEnded)
                     {
                         // server hung up on us
-                        return new PrivateSourceBadResponse(NuGetContext.GetPackageSourceUrls(currentDirectory));
+                        return new PrivateSourceBadResponse(NuGetContext.GetPackageSourceUrls(currentDirectory), ioException.Message);
                     }
 
                     return new UnknownError(ex, jobId);
@@ -61,17 +61,17 @@ public abstract record JobErrorBase : MessageBase
                         return new PrivateSourceAuthenticationFailure(NuGetContext.GetPackageSourceUrls(currentDirectory));
                     case HttpStatusCode.TooManyRequests:
                     case HttpStatusCode.ServiceUnavailable:
-                        return new PrivateSourceBadResponse(NuGetContext.GetPackageSourceUrls(currentDirectory));
+                        return new PrivateSourceBadResponse(NuGetContext.GetPackageSourceUrls(currentDirectory), httpRequest.Message);
                     default:
                         if ((int)httpRequest.StatusCode / 100 == 5)
                         {
-                            return new PrivateSourceBadResponse(NuGetContext.GetPackageSourceUrls(currentDirectory));
+                            return new PrivateSourceBadResponse(NuGetContext.GetPackageSourceUrls(currentDirectory), httpRequest.Message);
                         }
 
                         return new UnknownError(ex, jobId);
                 }
             case InvalidDataException invalidData when invalidData.Message == "Central Directory corrupt.":
-                return new PrivateSourceBadResponse(NuGetContext.GetPackageSourceUrls(currentDirectory));
+                return new PrivateSourceBadResponse(NuGetContext.GetPackageSourceUrls(currentDirectory), invalidData.Message);
             case InvalidProjectFileException invalidProjectFile:
                 return new DependencyFileNotParseable(invalidProjectFile.ProjectFile);
             case MissingFileException missingFile:

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/ApiModel/PrivateSourceBadResponse.cs

@@ -1,10 +1,25 @@
+using System.Text.Json.Serialization;
+
 namespace NuGetUpdater.Core.Run.ApiModel;
 
 public record PrivateSourceBadResponse : JobErrorBase
 {
-    public PrivateSourceBadResponse(string[] urls)
+    [JsonIgnore]
+    public string Message { get; }
+
+    public PrivateSourceBadResponse(string[] urls, string message)
         : base("private_source_bad_response")
     {
         Details["source"] = $"({string.Join("|", urls)})";
+        Message = message;
+    }
+
+    public override string GetReport()
+    {
+        var report = base.GetReport();
+
+        // this extra info isn't part of the reported shape but is useful to have in the log
+        var fullReport = string.Concat(report, "\n", $"- message: {Message}");
+        return fullReport;
     }
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/ApiModel/PullRequest.cs

@@ -1,8 +1,12 @@
 using System.Collections.Immutable;
+using System.Text.Json.Serialization;
 
 namespace NuGetUpdater.Core.Run.ApiModel;
 
 public record PullRequest
 {
+    [JsonPropertyName("pr-number")]
+    public int? PrNumber { get; init; } = null;
+    [JsonPropertyName("dependencies")]
     public ImmutableArray<PullRequestDependency> Dependencies { get; init; } = [];
 }

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/PullRequestConverter.cs

@@ -1,4 +1,4 @@
-using System.Collections.Immutable;
+using System.Collections.Immutable;
 using System.Text.Json;
 using System.Text.Json.Serialization;
 
@@ -10,16 +10,36 @@ public class PullRequestConverter : JsonConverter<PullRequest>
 {
     public override PullRequest? Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
     {
-        if (reader.TokenType != JsonTokenType.StartArray)
+        PullRequest? result;
+        switch (reader.TokenType)
         {
-            throw new JsonException("expected array of pull request dependencies");
+            case JsonTokenType.StartArray:
+                // old format, array of arrays of dependencies
+                var dependencies = JsonSerializer.Deserialize<ImmutableArray<PullRequestDependency>>(ref reader, options);
+                result = new PullRequest()
+                {
+                    Dependencies = dependencies
+                };
+                break;
+            case JsonTokenType.StartObject:
+                // new format, direct object
+                // use the same deserializer options but exclude this special converter
+                var optionsWithoutThisCustomConverter = new JsonSerializerOptions(options);
+                for (int i = optionsWithoutThisCustomConverter.Converters.Count - 1; i >= 0; i--)
+                {
+                    if (optionsWithoutThisCustomConverter.Converters[i].GetType() == typeof(PullRequestConverter))
+                    {
+                        optionsWithoutThisCustomConverter.Converters.RemoveAt(i);
+                    }
+                }
+
+                result = JsonSerializer.Deserialize<PullRequest>(ref reader, optionsWithoutThisCustomConverter);
+                break;
+            default:
+                throw new JsonException("expected pull request object or array of pull request dependencies");
         }
 
-        var dependencies = JsonSerializer.Deserialize<ImmutableArray<PullRequestDependency>>(ref reader, options);
-        return new PullRequest()
-        {
-            Dependencies = dependencies
-        };
+        return result;
     }
 
     public override void Write(Utf8JsonWriter writer, PullRequest value, JsonSerializerOptions options)

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core/Run/UpdateHandlers/GroupUpdateAllVersionsHandler.cs

@@ -42,14 +42,8 @@ internal class GroupUpdateAllVersionsHandler : IUpdateHandler
         // group update, do all directories and merge
         // ungrouped update, do each dir separate
         await this.ReportUpdaterStarted(apiHandler);
-        if (job.DependencyGroups.Length > 0)
-        {
-            await RunGroupedDependencyUpdates(job, originalRepoContentsPath, caseInsensitiveRepoContentsPath, baseCommitSha, discoveryWorker, analyzeWorker, updaterWorker, apiHandler, experimentsManager, logger);
-        }
-        else
-        {
-            await RunUngroupedDependencyUpdates(job, originalRepoContentsPath, caseInsensitiveRepoContentsPath, baseCommitSha, discoveryWorker, analyzeWorker, updaterWorker, apiHandler, experimentsManager, logger);
-        }
+        await RunGroupedDependencyUpdates(job, originalRepoContentsPath, caseInsensitiveRepoContentsPath, baseCommitSha, discoveryWorker, analyzeWorker, updaterWorker, apiHandler, experimentsManager, logger);
+        await RunUngroupedDependencyUpdates(job, originalRepoContentsPath, caseInsensitiveRepoContentsPath, baseCommitSha, discoveryWorker, analyzeWorker, updaterWorker, apiHandler, experimentsManager, logger);
     }
 
     private async Task RunGroupedDependencyUpdates(Job job, DirectoryInfo originalRepoContentsPath, DirectoryInfo? caseInsensitiveRepoContentsPath, string baseCommitSha, IDiscoveryWorker discoveryWorker, IAnalyzeWorker analyzeWorker, IUpdaterWorker updaterWorker, IApiHandler apiHandler, ExperimentsManager experimentsManager, ILogger logger)
@@ -146,7 +140,7 @@ internal class GroupUpdateAllVersionsHandler : IUpdateHandler
                     }
                 }
 
-                var updatedDependencyFiles = await tracker.StopTrackingAsync();
+                var updatedDependencyFiles = await tracker.StopTrackingAsync(restoreOriginalContents: true);
                 allUpdatedDependencyFiles = ModifiedFilesTracker.MergeUpdatedFileSet(allUpdatedDependencyFiles, updatedDependencyFiles);
             }
 
@@ -207,6 +201,15 @@ internal class GroupUpdateAllVersionsHandler : IUpdateHandler
                         continue;
                     }
 
+                    var matchingGroups = job.DependencyGroups
+                        .Where(group => group.GetGroupMatcher().IsMatch(dependency.Name))
+                        .ToImmutableArray();
+                    if (matchingGroups.Length > 0)
+                    {
+                        logger.Info($"Dependency {dependency.Name} skipped for ungrouped updates because it's a member of the following groups: {string.Join(", ", matchingGroups.Select(group => group.Name))}");
+                        continue;
+                    }
+
                     var dependencyInfo = RunWorker.GetDependencyInfo(job, dependency, allowCooldown: true);
                     var analysisResult = await analyzeWorker.RunAsync(repoContentsPath.FullName, discoveryResult, dependencyInfo);
                     if (analysisResult.Error is not null)

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/Analyze/VersionFinderTests.cs

@@ -293,7 +293,7 @@ public class VersionFinderTests : TestBase
         var error = JobErrorBase.ErrorFromException(exception, "TEST-JOB-ID", tempDir.DirectoryPath);
 
         // assert
-        var expected = new PrivateSourceBadResponse([feedUrl]);
+        var expected = new PrivateSourceBadResponse([feedUrl], "unused");
         var expectedJson = JsonSerializer.Serialize(expected, RunWorker.SerializerOptions);
         var actualJson = JsonSerializer.Serialize(error, RunWorker.SerializerOptions);
         Assert.Equal(expectedJson, actualJson);

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/Discover/DiscoveryWorkerTests.cs

@@ -1124,7 +1124,7 @@ public partial class DiscoveryWorkerTests : DiscoveryWorkerTestBase
             ],
             expectedResult: new()
             {
-                Error = new PrivateSourceBadResponse([$"{http.BaseUrl.TrimEnd('/')}/index.json"]),
+                Error = new PrivateSourceBadResponse([$"{http.BaseUrl.TrimEnd('/')}/index.json"], "unused"),
                 Path = "",
                 Projects = [],
             }
@@ -1199,7 +1199,7 @@ public partial class DiscoveryWorkerTests : DiscoveryWorkerTestBase
             ],
             expectedResult: new()
             {
-                Error = new PrivateSourceBadResponse([$"{http.BaseUrl.TrimEnd('/')}/index.json"]),
+                Error = new PrivateSourceBadResponse([$"{http.BaseUrl.TrimEnd('/')}/index.json"], "unused"),
                 Path = "",
                 Projects = [],
             }

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

@@ -148,7 +148,7 @@ public class HttpApiHandlerTests
         yield return [new DependencyNotFound("unused"), "record_update_job_error"];
         yield return [new JobRepoNotFound("unused"), "record_update_job_error"];
         yield return [new PrivateSourceAuthenticationFailure(["unused"]), "record_update_job_error"];
-        yield return [new PrivateSourceBadResponse(["unused"]), "record_update_job_error"];
+        yield return [new PrivateSourceBadResponse(["unused"], "unused"), "record_update_job_error"];
         yield return [new PrivateSourceTimedOut("unused"), "record_update_job_error"];
         yield return [new PullRequestExistsForLatestVersion("unused", "unused"), "record_update_job_error"];
         yield return [new PullRequestExistsForSecurityUpdate([]), "record_update_job_error"];

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

@@ -40,32 +40,39 @@ public class JobErrorBaseTests : TestBase
 
     public static IEnumerable<object[]> GenerateErrorFromExceptionTestData()
     {
+        // something elevated to a bad response
+        yield return
+        [
+            new BadResponseException("nope", "http://nuget.example.com/v3/index.json"),
+            new PrivateSourceBadResponse(["http://nuget.example.com/v3/index.json"], "nope"),
+        ];
+
         // internal error from package feed
         yield return
         [
             new HttpRequestException("nope", null, HttpStatusCode.InternalServerError),
-            new PrivateSourceBadResponse(["http://nuget.example.com/v3/index.json"]),
+            new PrivateSourceBadResponse(["http://nuget.example.com/v3/index.json"], "nope"),
         ];
 
         // inner exception turns into private_source_bad_response; 500
         yield return
         [
-            new FatalProtocolException("nope", new HttpRequestException("nope", null, HttpStatusCode.InternalServerError)),
-            new PrivateSourceBadResponse(["http://nuget.example.com/v3/index.json"]),
+            new FatalProtocolException("nope", new HttpRequestException("inner nope", null, HttpStatusCode.InternalServerError)),
+            new PrivateSourceBadResponse(["http://nuget.example.com/v3/index.json"], "inner nope"),
         ];
 
         // inner exception turns into private_source_bad_response; ResponseEnded
         yield return
         [
-            new FatalProtocolException("nope", new HttpRequestException("nope", new HttpIOException(HttpRequestError.ResponseEnded))),
-            new PrivateSourceBadResponse(["http://nuget.example.com/v3/index.json"]),
+            new FatalProtocolException("nope", new HttpRequestException("inner nope", new HttpIOException(HttpRequestError.ResponseEnded))),
+            new PrivateSourceBadResponse(["http://nuget.example.com/v3/index.json"], "inner nope"),
         ];
 
         // service returned corrupt package
         yield return
         [
             new InvalidDataException("Central Directory corrupt."),
-            new PrivateSourceBadResponse(["http://nuget.example.com/v3/index.json"]),
+            new PrivateSourceBadResponse(["http://nuget.example.com/v3/index.json"], "Central Directory corrupt."),
         ];
 
         // top-level exception turns into private_source_authentication_failure

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

@@ -152,11 +152,12 @@ public class MessageReportTests
         yield return
         [
             // message
-            new PrivateSourceBadResponse(["url1", "url2"]),
+            new PrivateSourceBadResponse(["url1", "url2"], "some extra info"),
             // expected
             """
             Error type: private_source_bad_response
             - source: (url1|url2)
+            - message: some extra info
             """
         ];
 

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

@@ -407,7 +407,7 @@ public class SerializationTests : TestBase
     }
 
     [Fact]
-    public void DeserializeExistingPullRequests()
+    public void DeserializeExistingPullRequestsOldFormat()
     {
         var jsonWrapperJson = """
             {
@@ -429,6 +429,41 @@ public class SerializationTests : TestBase
             """;
         var jobWrapper = RunWorker.Deserialize(jsonWrapperJson)!;
         Assert.Single(jobWrapper.Job.ExistingPullRequests);
+        Assert.Null(jobWrapper.Job.ExistingPullRequests[0].PrNumber);
+        Assert.Single(jobWrapper.Job.ExistingPullRequests[0].Dependencies);
+        Assert.Equal("Some.Package", jobWrapper.Job.ExistingPullRequests[0].Dependencies[0].DependencyName);
+        Assert.Equal(NuGetVersion.Parse("1.2.3"), jobWrapper.Job.ExistingPullRequests[0].Dependencies[0].DependencyVersion);
+        Assert.False(jobWrapper.Job.ExistingPullRequests[0].Dependencies[0].DependencyRemoved);
+        Assert.Null(jobWrapper.Job.ExistingPullRequests[0].Dependencies[0].Directory);
+    }
+
+    [Fact]
+    public void DeserializeExistingPullRequestsNewFormat()
+    {
+        var jsonWrapperJson = """
+            {
+                "job": {
+                    "source": {
+                        "provider": "github",
+                        "repo": "some/repo"
+                    },
+                    "existing-pull-requests": [
+                        {
+                            "pr-number": 123,
+                            "dependencies": [
+                                {
+                                    "dependency-name": "Some.Package",
+                                    "dependency-version": "1.2.3"
+                                }
+                            ]
+                        }
+                    ]
+                }
+            }
+            """;
+        var jobWrapper = RunWorker.Deserialize(jsonWrapperJson)!;
+        Assert.Single(jobWrapper.Job.ExistingPullRequests);
+        Assert.Equal(123, jobWrapper.Job.ExistingPullRequests[0].PrNumber);
         Assert.Single(jobWrapper.Job.ExistingPullRequests[0].Dependencies);
         Assert.Equal("Some.Package", jobWrapper.Job.ExistingPullRequests[0].Dependencies[0].DependencyName);
         Assert.Equal(NuGetVersion.Parse("1.2.3"), jobWrapper.Job.ExistingPullRequests[0].Dependencies[0].DependencyVersion);
@@ -766,7 +801,7 @@ public class SerializationTests : TestBase
 
         yield return
         [
-            new PrivateSourceBadResponse(["url1", "url2"]),
+            new PrivateSourceBadResponse(["url1", "url2"], "unused"),
             """
             {"data":{"error-type":"private_source_bad_response","error-details":{"source":"(url1|url2)"}}}
             """

data/helpers/lib/NuGetUpdater/NuGetUpdater.Core.Test/Run/UpdateHandlers/GroupUpdateAllVersionsHandlerTests.cs

@@ -270,7 +270,7 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
     }
 
     [Fact]
-    public async Task GeneratesCreatePullRequest_Grouped()
+    public async Task GeneratesCreatePullRequest_GroupedAndUngrouped()
     {
         // single groups specified; creates 1 PR for both directories
         await TestAsync(
@@ -284,7 +284,7 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                         Rules = new()
                         {
                             ["patterns"] = new[] { "*" },
-                            ["exclude-patterns"] = new[] { "Unrelated.*" },
+                            ["exclude-patterns"] = new[] { "Ungrouped.*" },
                         },
                     },
                 ],
@@ -304,7 +304,7 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                             Dependencies = [
                                 new("Some.Dependency", "1.0.0", DependencyType.PackageReference, TargetFrameworks: ["net9.0"]),
                                 new("Some.Other.Dependency", "3.0.0", DependencyType.PackageReference, TargetFrameworks: ["net9.0"]),
-                                new("Unrelated.Dependency", "5.0.0", DependencyType.PackageReference, TargetFrameworks: ["net9.0"]),
+                                new("Ungrouped.Dependency", "5.0.0", DependencyType.PackageReference, TargetFrameworks: ["net9.0"]),
                             ],
                             ImportedFiles = [],
                             AdditionalFiles = [],
@@ -321,7 +321,7 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                             Dependencies = [
                                 new("Some.Dependency", "1.0.0", DependencyType.PackageReference, TargetFrameworks: ["net9.0"]),
                                 new("Some.Other.Dependency", "3.0.0", DependencyType.PackageReference, TargetFrameworks: ["net9.0"]),
-                                new("Unrelated.Dependency", "5.0.0", DependencyType.PackageReference, TargetFrameworks: ["net9.0"]),
+                                new("Ungrouped.Dependency", "5.0.0", DependencyType.PackageReference, TargetFrameworks: ["net9.0"]),
                             ],
                             ImportedFiles = [],
                             AdditionalFiles = [],
@@ -338,6 +338,7 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                 {
                     "Some.Dependency" => "2.0.0",
                     "Some.Other.Dependency" => "4.0.0",
+                    "Ungrouped.Dependency" => "6.0.0",
                     _ => throw new NotImplementedException($"Test didn't expect to update dependency {dependencyInfo.Name}"),
                 };
                 return Task.FromResult(new AnalysisResult()
@@ -373,6 +374,7 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                         ["operation"] = "group_update_all_versions",
                     }
                 },
+                // first the grouped updates
                 // for "/src"
                 new UpdatedDependencyList()
                 {
@@ -395,7 +397,7 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                         },
                         new()
                         {
-                            Name = "Unrelated.Dependency",
+                            Name = "Ungrouped.Dependency",
                             Version = "5.0.0",
                             Requirements = [
                                 new() { Requirement = "5.0.0", File = "/src/project.csproj", Groups = ["dependencies"] },
@@ -426,7 +428,7 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                         },
                         new()
                         {
-                            Name = "Unrelated.Dependency",
+                            Name = "Ungrouped.Dependency",
                             Version = "5.0.0",
                             Requirements = [
                                 new() { Requirement = "5.0.0", File = "/test/project.csproj", Groups = ["dependencies"] },
@@ -508,6 +510,129 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                     PrBody = EndToEndTests.TestPullRequestBody,
                     DependencyGroup = "test-group",
                 },
+                // now the ungrouped updates
+                // for "/src"
+                new UpdatedDependencyList()
+                {
+                    Dependencies = [
+                        new()
+                        {
+                            Name = "Some.Dependency",
+                            Version = "1.0.0",
+                            Requirements = [
+                                new() { Requirement = "1.0.0", File = "/src/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                        new()
+                        {
+                            Name = "Some.Other.Dependency",
+                            Version = "3.0.0",
+                            Requirements = [
+                                new() { Requirement = "3.0.0", File = "/src/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                        new()
+                        {
+                            Name = "Ungrouped.Dependency",
+                            Version = "5.0.0",
+                            Requirements = [
+                                new() { Requirement = "5.0.0", File = "/src/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                    ],
+                    DependencyFiles = ["/src/project.csproj"],
+                },
+                new CreatePullRequest()
+                {
+                    Dependencies = [
+                        new()
+                        {
+                            Name = "Ungrouped.Dependency",
+                            Version = "6.0.0",
+                            Requirements = [
+                                new() { Requirement = "6.0.0", File = "/src/project.csproj", Groups = ["dependencies"], Source = new() { SourceUrl = null } },
+                            ],
+                            PreviousVersion = "5.0.0",
+                            PreviousRequirements = [
+                                new() { Requirement = "5.0.0", File = "/src/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                    ],
+                    UpdatedDependencyFiles = [
+                        new()
+                        {
+                            Directory = "/src",
+                            Name = "project.csproj",
+                            Content = "updated contents",
+                        },
+                    ],
+                    BaseCommitSha = "TEST-COMMIT-SHA",
+                    CommitMessage = EndToEndTests.TestPullRequestCommitMessage,
+                    PrTitle = EndToEndTests.TestPullRequestTitle,
+                    PrBody = EndToEndTests.TestPullRequestBody,
+                    DependencyGroup = null,
+                },
+                // for "/test"
+                new UpdatedDependencyList()
+                {
+                    Dependencies = [
+                        new()
+                        {
+                            Name = "Some.Dependency",
+                            Version = "1.0.0",
+                            Requirements = [
+                                new() { Requirement = "1.0.0", File = "/test/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                        new()
+                        {
+                            Name = "Some.Other.Dependency",
+                            Version = "3.0.0",
+                            Requirements = [
+                                new() { Requirement = "3.0.0", File = "/test/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                        new()
+                        {
+                            Name = "Ungrouped.Dependency",
+                            Version = "5.0.0",
+                            Requirements = [
+                                new() { Requirement = "5.0.0", File = "/test/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                    ],
+                    DependencyFiles = ["/test/project.csproj"],
+                },
+                new CreatePullRequest()
+                {
+                    Dependencies = [
+                        new()
+                        {
+                            Name = "Ungrouped.Dependency",
+                            Version = "6.0.0",
+                            Requirements = [
+                                new() { Requirement = "6.0.0", File = "/test/project.csproj", Groups = ["dependencies"], Source = new() { SourceUrl = null } },
+                            ],
+                            PreviousVersion = "5.0.0",
+                            PreviousRequirements = [
+                                new() { Requirement = "5.0.0", File = "/test/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                    ],
+                    UpdatedDependencyFiles = [
+                        new()
+                        {
+                            Directory = "/test",
+                            Name = "project.csproj",
+                            Content = "updated contents",
+                        },
+                    ],
+                    BaseCommitSha = "TEST-COMMIT-SHA",
+                    CommitMessage = EndToEndTests.TestPullRequestCommitMessage,
+                    PrTitle = EndToEndTests.TestPullRequestTitle,
+                    PrBody = EndToEndTests.TestPullRequestBody,
+                    DependencyGroup = null,
+                },
                 new MarkAsProcessed("TEST-COMMIT-SHA"),
             ]
         );
@@ -608,6 +733,7 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                         ["operation"] = "group_update_all_versions",
                     }
                 },
+                // grouped check
                 new UpdatedDependencyList()
                 {
                     Dependencies = [
@@ -660,6 +786,29 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                     PrBody = EndToEndTests.TestPullRequestBody,
                     DependencyGroup = "test-group",
                 },
+                // ungrouped check
+                new UpdatedDependencyList()
+                {
+                    Dependencies = [
+                        new()
+                        {
+                            Name = "Some.Dependency",
+                            Version = "1.0.0",
+                            Requirements = [
+                                new() { Requirement = "1.0.0", File = "/src/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                        new()
+                        {
+                            Name = "Some.Other.Dependency",
+                            Version = "3.0.0",
+                            Requirements = [
+                                new() { Requirement = "3.0.0", File = "/src/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                    ],
+                    DependencyFiles = ["/src/project.csproj"],
+                },
                 new MarkAsProcessed("TEST-COMMIT-SHA"),
             ]
         );
@@ -770,6 +919,7 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                         ["operation"] = "group_update_all_versions",
                     }
                 },
+                // grouped check
                 new UpdatedDependencyList()
                 {
                     Dependencies = [
@@ -822,6 +972,29 @@ public class GroupUpdateAllVersionsHandlerTests : UpdateHandlersTestsBase
                     PrBody = EndToEndTests.TestPullRequestBody,
                     DependencyGroup = "test-group-1",
                 },
+                // ungrouped check
+                new UpdatedDependencyList()
+                {
+                    Dependencies = [
+                        new()
+                        {
+                            Name = "Package.For.Group.One",
+                            Version = "1.0.0",
+                            Requirements = [
+                                new() { Requirement = "1.0.0", File = "/src/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                        new()
+                        {
+                            Name = "Package.For.Group.Two",
+                            Version = "2.0.0",
+                            Requirements = [
+                                new() { Requirement = "2.0.0", File = "/src/project.csproj", Groups = ["dependencies"] },
+                            ],
+                        },
+                    ],
+                    DependencyFiles = ["/src/project.csproj"],
+                },
                 new MarkAsProcessed("TEST-COMMIT-SHA"),
             ]
         );

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

@@ -452,7 +452,7 @@ public class MSBuildHelperTests : TestBase
             // output
             "Response status code does not indicate success: 500 (Internal Server Error).",
             // expectedError
-            new PrivateSourceBadResponse(["http://localhost/test-feed"]),
+            new PrivateSourceBadResponse(["http://localhost/test-feed"], "unused"),
         ];
 
         yield return
@@ -460,7 +460,7 @@ public class MSBuildHelperTests : TestBase
             // output
             "The response ended prematurely. (ResponseEnded)",
             // expectedError
-            new PrivateSourceBadResponse(["http://localhost/test-feed"]),
+            new PrivateSourceBadResponse(["http://localhost/test-feed"], "unused"),
         ];
 
         yield return
@@ -468,7 +468,7 @@ public class MSBuildHelperTests : TestBase
             // output
             "The file is not a valid nupkg.",
             // expectedError
-            new PrivateSourceBadResponse(["http://localhost/test-feed"]),
+            new PrivateSourceBadResponse(["http://localhost/test-feed"], "unused"),
         ];
 
         yield return
@@ -476,7 +476,7 @@ public class MSBuildHelperTests : TestBase
             // output
             "The content at 'http://localhost/test-feed/Packages(Id='Some.Package',Version='1.2.3')' is not valid XML.",
             // expectedError
-            new PrivateSourceBadResponse(["http://localhost/test-feed"]),
+            new PrivateSourceBadResponse(["http://localhost/test-feed"], "unused"),
         ];
 
         yield return

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: dependabot-nuget
 version: !ruby/object:Gem::Version
-  version: 0.342.2
+  version: 0.343.1
 platform: ruby
 authors:
 - Dependabot
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.342.2
+        version: 0.343.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.342.2
+        version: 0.343.1
 - !ruby/object:Gem::Dependency
   name: debug
   requirement: !ruby/object:Gem::Requirement
@@ -551,7 +551,7 @@ licenses:
 - MIT
 metadata:
   bug_tracker_uri: https://github.com/dependabot/dependabot-core/issues
-  changelog_uri: https://github.com/dependabot/dependabot-core/releases/tag/v0.342.2
+  changelog_uri: https://github.com/dependabot/dependabot-core/releases/tag/v0.343.1
 rdoc_options: []
 require_paths:
 - lib