Bump the testing group with 4 updates#276
Closed
dependabot[bot] wants to merge 1 commit into
Closed
Conversation
Bumps NSubstitute from 5.3.0 to 6.0.0 Bumps Testcontainers from 4.12.0 to 4.13.0 Bumps Testcontainers.Kafka from 4.12.0 to 4.13.0 Bumps Testcontainers.PostgreSql from 4.12.0 to 4.13.0 --- updated-dependencies: - dependency-name: NSubstitute dependency-version: 6.0.0 dependency-type: direct:production update-type: version-update:semver-major dependency-group: testing - dependency-name: Testcontainers dependency-version: 4.13.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: testing - dependency-name: Testcontainers.Kafka dependency-version: 4.13.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: testing - dependency-name: Testcontainers.PostgreSql dependency-version: 4.13.0 dependency-type: direct:production update-type: version-update:semver-minor dependency-group: testing ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
added
.NET
vgmello
added a commit
that referenced
this pull request
Jul 15, 2026
Supersedes Dependabot #280, #279 (partial), #278, #273, #271, #276 (partial), #281 (Refit only), #263, #264: - Aspire 13.4.3 -> 13.4.6 - Microsoft.Orleans* 10.1.0 -> 10.2.1 (+ VersionOverride for Microsoft.Orleans.Analyzers in AppDomain.Api/AppDomain, matching how the source-generator reference is pinned independently of the main Orleans package group) - Google.Protobuf 3.35.0 -> 3.35.1, Grpc.Tools 2.80.0 -> 2.82.0 - OpenTelemetry.Instrumentation.AspNetCore 1.15.2 -> 1.16.0, OpenTelemetry.Instrumentation.GrpcCore beta.11 -> beta.13 - Scalar.AspNetCore 2.16.3 -> 2.16.11 - Testcontainers/.Kafka/.PostgreSql 4.12.0 -> 4.13.0 - Refit and Refit.HttpClientFactory 11.0.1 -> 13.1.0 (bumped together; HttpClientFactory 13.1.0 has an exact floor on Refit 13.1.0) - MessagePack 2.5.301 -> 2.5.302 (Aspire 13.4.6's StreamJsonRpc raised its own floor) - @types/node 25.x -> ^26.0.0 in both docs sites, lockfiles regenerated Intentionally NOT applied, each broke the build in isolation: - Microsoft.CodeAnalysis.Analyzers/CSharp/Common 5.0.0 -> 5.6.0 (PR #279, partial): JasperFx.RuntimeCompiler 4.5.0, pulled in by WolverineFx 5.39.2, hard-pins Microsoft.CodeAnalysis(.CSharp/.Scripting) to exactly 5.0.0. This bump is only safe alongside the WolverineFx major upgrade (PRs #282/#277), which is being deferred separately. - NSubstitute 5.3.0 -> 6.0.0 (PR #276, partial): 6.0.0 added proper nullable annotations to its public API (e.g. the CallInfo indexer), which surfaces CS8600 in ~38 existing call sites across the unit test suite that cast `x[0]` and read members into non-nullable properties. Needs a real test-code pass, not a drop-in version bump. - Refitter.MSBuild 2.0.0 -> 2.1.0 (PR #281, partial): fails code generation at build time with `MissingMethodException: System.Text.ValueStringBuilder.AsSpan()`. Unrelated to the Refit runtime library bump above, which is unaffected and kept. CentralPackageTransitivePinningEnabled (enabled in the prior commit) caught two additional floor violations these bumps triggered transitively, both fixed here: MessagePack needed 2.5.302 for Aspire 13.4.6's StreamJsonRpc, matching the earlier fix for a similar Refit floor issue. Verified: AppDomain.slnx and libs/Momentum/Momentum.slnx both build with 0 errors. Full test run: 175 passed / 34 failed (all Integration tests failing solely on missing local Docker, same pre-existing baseline as before this change) + 604 passed / 0 failed across all libs test projects. GitHub Actions bumps (#270 actions/cache, #262 actions/checkout) are gated behind a PreToolUse hook requiring separate confirmation before editing .github/workflows/*, not yet applied. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DqMbT47Qhz4ZjjccAEQL6B
Owner
|
Partially applied in #284: Testcontainers/.Kafka/.PostgreSql 4.12.0 -> 4.13.0 taken as-is. NSubstitute 5.3.0 -> 6.0.0 NOT applied: it added proper nullable annotations to its public API (e.g. the CallInfo indexer), which surfaces CS8600 in ~38 existing call sites across the unit test suite. Needs an actual test-code pass, not a drop-in bump. Leaving this open for the NSubstitute part. |
3 tasks
Owner
Contributor
Author
|
This pull request was built based on a group rule. Closing it will not ignore any of these versions in future pull requests. To ignore these dependencies, configure ignore rules in dependabot.yml |
vgmello
added a commit
that referenced
this pull request
Jul 17, 2026
## Summary - Bumps `NSubstitute` `5.3.0` → `6.0.0` (major). Fixes the `(T)x[0]` CallInfo indexer cast pattern in 3 test files that no longer compiles under 6.0's nullable-annotated public API, switching to the null-safe `x.ArgAt<T>(0)` extension. - Evaluated `Refitter.MSBuild` `2.0.0` → `2.1.0` but reverted: 2.1.0 refactored the MSBuild task itself and throws `MissingMethodException: System.Text.ValueStringBuilder.AsSpan()` during code generation. Left at 2.0.0 pending an upstream fix. Supersedes/consolidates: - #276 (testing group bump — Testcontainers part already on `main`; NSubstitute part applied here) - #286 (NSubstitute 5.3.0 → 6.0.0 — applied here) - #281 (api-client group — Refit.HttpClientFactory part already on `main`; Refitter.MSBuild part evaluated and skipped, see above) - #285 (Refitter.MSBuild 2.0.0 → 2.1.0 — evaluated and skipped, see above) ## Test plan - [x] `dotnet build AppDomain.slnx` — succeeds - [x] `dotnet test tests/AppDomain.Tests` (unit, excl. Integration) — 160 passed - [x] `dotnet test libs/Momentum/tests/Momentum.Extensions.Tests` — 341 passed 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Rebasing might not happen immediately, so don't worry if this takes some time.
Note: if you make any changes to this PR yourself, they will take precedence over the rebase.
Updated NSubstitute from 5.3.0 to 6.0.0.
Release notes
Sourced from NSubstitute's releases.
6.0.0
ℹ️ No changes from Release Candidate 1.
NSubstitute v6.0.0
From RC1 notes:
ArgMatchers.Matchingpredicate matcher as an alternative toIs(Expression<Predicate<T>>. (.NET6 and above.)Arg.Isnow accepts arg matchers.Full change list
... (truncated)
6.0.0-rc.1
NSubstitute v6.0.0 Release Candidate 1
Due to the large number of changes in this release, we wanted to start with a release candidate to ensure we've correctly captured breaking changes.
ArgMatchers.Matchingpredicate matcher as an alternative toIs(Expression<Predicate<T>>. (.NET6 and above.)Arg.Isnow accepts arg matchers.Full change list
... (truncated)
Commits viewable in compare view.
Updated Testcontainers from 4.12.0 to 4.13.0.
Release notes
Sourced from Testcontainers's releases.
4.13.0
What's Changed
Thank you to everyone who contributed and shared their feedback 🤜🤛.
The NuGet packages for this release have been attested for supply chain security using
actions/attest. This confirms the integrity and provenance of the artifacts and helps ensure they can be trusted: #33686956.🚀 Features
🐛 Bug Fixes
📖 Documentation
🧹 Housekeeping
📦 Dependency Updates
Commits viewable in compare view.
Updated Testcontainers.Kafka from 4.12.0 to 4.13.0.
Release notes
Sourced from Testcontainers.Kafka's releases.
4.13.0
What's Changed
Thank you to everyone who contributed and shared their feedback 🤜🤛.
The NuGet packages for this release have been attested for supply chain security using
actions/attest. This confirms the integrity and provenance of the artifacts and helps ensure they can be trusted: #33686956.🚀 Features
🐛 Bug Fixes
📖 Documentation
🧹 Housekeeping
📦 Dependency Updates
Commits viewable in compare view.
Updated Testcontainers.PostgreSql from 4.12.0 to 4.13.0.
Release notes
Sourced from Testcontainers.PostgreSql's releases.
4.13.0
What's Changed
Thank you to everyone who contributed and shared their feedback 🤜🤛.
The NuGet packages for this release have been attested for supply chain security using
actions/attest. This confirms the integrity and provenance of the artifacts and helps ensure they can be trusted: #33686956.🚀 Features
🐛 Bug Fixes
📖 Documentation
🧹 Housekeeping
📦 Dependency Updates
Commits viewable in compare view.
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions