Skip to content

Fix content root discovery with slnx#222

Merged
Hawxy merged 6 commits intoJasperFx:masterfrom
dmytro-pryvedeniuk:fix-content-root-discovery-with-slnx
Apr 2, 2026
Merged

Fix content root discovery with slnx#222
Hawxy merged 6 commits intoJasperFx:masterfrom
dmytro-pryvedeniuk:fix-content-root-discovery-with-slnx

Conversation

@dmytro-pryvedeniuk
Copy link
Copy Markdown
Contributor

@dmytro-pryvedeniuk dmytro-pryvedeniuk commented Mar 24, 2026

This PR fixes the issue with content root discovery for the tests using Alba + minimal api + slnx (like polecat).

image
  1. For net8 and net9 Microsoft.AspNetCore.Mvc.Testing packages are updated (see [release/8.0] Add .slnx fallback for TestHost content root discovery dotnet/aspnetcore#65000 and [release/9.0] Add .slnx fallback for TestHost content root discovery dotnet/aspnetcore#64953). For net10 the issue is fixed - polecat can just update to the recent version.
  2. Fixed all warnings. Including vulnerability warnings like warning NU1903: Package 'System.Text.Json' 8.0.4 has a known high severity vulnerability, https://github.com/advisories/GHSA-8g4q-xg66-9fp4 [/home/runner/work/alba/alba/src/Alba.sln] by pinning Microsoft.Extensions.Hosting and Microsoft.Identity.Web in the sample/test projects.
  3. Used net8 and net9 for Alba.Testing to be aligned with Alba and updated the existing tests.
  4. Updated the tests added recently in Avoid extra cmd arguments causing issues with Wolverine-based systems #220 to avoid changes in the documentation sample.

@dmytro-pryvedeniuk dmytro-pryvedeniuk marked this pull request as draft March 25, 2026 10:36
@dmytro-pryvedeniuk dmytro-pryvedeniuk marked this pull request as ready for review March 25, 2026 11:29
@dmytro-pryvedeniuk dmytro-pryvedeniuk marked this pull request as draft March 26, 2026 09:23
@dmytro-pryvedeniuk dmytro-pryvedeniuk marked this pull request as ready for review March 26, 2026 13:14
@Hawxy Hawxy merged commit 36fabd5 into JasperFx:master Apr 2, 2026
1 check passed
@dmytro-pryvedeniuk dmytro-pryvedeniuk deleted the fix-content-root-discovery-with-slnx branch April 2, 2026 14:34
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.

2 participants