Skip to content

Fix CI test failures after DistributedLockManager upgrade to v10.0.2 - #9

Merged
Phelecks merged 3 commits into
mainfrom
copilot/upgrade-all-nuget-packages
Mar 22, 2026
Merged

Phelecks merged 3 commits into
mainfrom
copilot/upgrade-all-nuget-packages

Conversation

Copilot AI commented Mar 22, 2026 •

Copy link
Copy Markdown
Contributor

Tricksfor.DistributedLockManager v10.0.2 introduced an ILoggerFactory constructor dependency on its internal DistributedLockService, breaking all integration tests with InvalidOperationException: No service for type 'Microsoft.Extensions.Logging.ILoggerFactory' has been registered.

Changes

  • DistributedNonceIntegrationTests.cs — add services.AddLogging() to the test DI setup so ILoggerFactory is available when the lock manager resolves its dependencies:
var services = new ServiceCollection();
services.AddLogging();              // required by DistributedLockManager v10.0.2
services.AddDistributedLockManager();
services.AddDistributedNonce();
services.AddSingleton<IConnectionMultiplexer>(_mux);

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update all NuGet packages including Nethereum Upgrade all NuGet packages to latest stable versions Mar 22, 2026
Copilot AI requested a review from Phelecks March 22, 2026 18:40
@Phelecks
Phelecks marked this pull request as ready for review March 22, 2026 19:12
Copilot AI review requested due to automatic review settings March 22, 2026 19:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates centrally-managed NuGet package versions to newer stable releases, including a major Nethereum upgrade, to keep the DistributedNonce library’s dependencies current.

Changes:

  • Bump Nethereum packages from 5.8.0 → 6.0.4.
  • Patch/minor upgrades for test/runtime dependencies (NUnit, Testcontainers, Microsoft.NET.Test.Sdk, Aspire Redis caching, DistributedLockManager, DI abstractions).

Copilot AI changed the title Upgrade all NuGet packages to latest stable versions Fix CI test failures after DistributedLockManager upgrade to v10.0.2 Mar 22, 2026
@sonarqubecloud

Copy link
Copy Markdown

@Phelecks
Phelecks merged commit 09e9b8e into main Mar 22, 2026
7 checks passed
@Phelecks
Phelecks deleted the copilot/upgrade-all-nuget-packages branch March 22, 2026 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants