Skip to content

Conversation

@SolidProgramming
Copy link
Contributor

Updated

For NUnit3TestAdapter is 6.0.0 available but that completly breaks all of our tests. I investigated a bit and everytime we read the config from testhost it crashes. It may be because if the "criticalconfigvaluesloaded" function can't find a value it throws an error(in debug) and in release in does environment.exit(). Also i read that they fixed "Attempting to execute a native assembly crashes the runner.", maybe they haven't because i found that the ConfigFile test execute a function via reflection.

- RestSharp and RestSharp.Serializers.NewtonsoftJson to 113.0.0
- System.IdentityModel.Tokens.Jwt to 8.15.0
- System.Text.Encodings.Web to 10.0.1
- PuppeteerSharp to 20.2.5
- Swashbuckle.AspNetCore to 10.0.1
- Microsoft.IdentityModel.Tokens to 8.15.0

Also updated a using directive in Program.cs to reflect changes in the OpenAPI package structure. No functional code changes were made.
Upgraded bunit to 2.2.2 and Microsoft.NET.Test.Sdk to 18.0.1 in the test project. Refactored UiRsbLinkTest.cs to use new bunit APIs, including BunitContext, BunitNavigationManage.
Upgraded the NUnit3TestAdapter NuGet package in FWO.Test.csproj
@SolidProgramming SolidProgramming self-assigned this Dec 12, 2025
@SolidProgramming SolidProgramming added the dependencies Pull requests that update a dependency file label Dec 12, 2025
@SolidProgramming SolidProgramming marked this pull request as ready for review December 12, 2025 14:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates multiple NuGet packages across the codebase to their latest versions, addressing several open issues. The update includes both minor version bumps (PuppeteerSharp, RestSharp, JWT tokens) and major version updates (bunit 1.x → 2.x, Swashbuckle 8.x → 10.x, Microsoft.NET.Test.Sdk 17.x → 18.x).

Key changes:

  • Major framework updates: bunit upgraded to 2.2.2 with corresponding API migration in test code
  • Security updates: System.IdentityModel.Tokens.Jwt and Microsoft.IdentityModel.Tokens updated to 8.15.0
  • Breaking changes: Swashbuckle.AspNetCore updated to 10.0.1 with incomplete namespace migration

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
FWO.UI/FWO.Ui.csproj Updates PuppeteerSharp to 20.2.5
UiRsbLinkTest.cs Migrates test code to bunit 2.x API (TestContext → BunitContext, RenderComponent → Render, FakeNavigationManager → BunitNavigationManager); removes unused imports; adds explicit typing
FWO.Test/FWO.Test.csproj Updates bunit to 2.2.2, PuppeteerSharp to 20.2.5, NUnit3TestAdapter to 5.2.0, Microsoft.NET.Test.Sdk to 18.0.1, Microsoft.IdentityModel.Tokens to 8.15.0
FWO.Middleware.Server/Program.cs Changes using statement from Microsoft.OpenApi.Models to Microsoft.OpenApi (incomplete - causes compilation error)
FWO.Middleware.Server.csproj Updates PuppeteerSharp to 20.2.5 and Swashbuckle.AspNetCore to 10.0.1
FWO.Report/FWO.Report.csproj Updates PuppeteerSharp to 20.2.5
FWO.Mail/FWO.Mail.csproj Updates System.Text.Encodings.Web to 10.0.1 (version likely doesn't exist)
FWO.Config.File.csproj Updates System.IdentityModel.Tokens.Jwt to 8.15.0
FWO.Api.Client.csproj Updates RestSharp and RestSharp.Serializers.NewtonsoftJson to 113.0.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@SolidProgramming
Copy link
Contributor Author

@tpurschke Please make sure it installs/upgrades for you. I did an upgrade and the dotnet tests. The install test also works. Don't know why copilot thinks it's not compiling.

@sonarqubecloud
Copy link

@tpurschke tpurschke merged commit d454707 into CactuseSecurity:develop Dec 18, 2025
3 checks passed
@SolidProgramming SolidProgramming deleted the nuget_bumps branch December 19, 2025 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants