Skip to content

Decouple plugin fixtures from test assembly size - #5144

Merged
Widthdom merged 2 commits into
mainfrom
fix-issue5121
Aug 22, 2026
Merged

Decouple plugin fixtures from test assembly size#5144
Widthdom merged 2 commits into
mainfrom
fix-issue5121

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Make executable extractor-plugin tests use the dedicated, bounded CodeIndex.PluginIsolationFixture assembly.
  • Remove the plugin marker and the remaining executable-plugin copy dependency from CodeIndex.Tests.dll.
  • Add a regression contract that keeps the fixture within the production 4,096-type inspection limit.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Debug -f net9.0 --filter "FullyQualifiedName~ExtractorPluginRegistryTests|FullyQualifiedName~PluginFixture_IsDedicatedAndBounded_Issue5121" --no-restore (58 passed)
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Debug -f net8.0 --filter "FullyQualifiedName~ExtractorPluginRegistryTests|FullyQualifiedName~PluginFixture_IsDedicatedAndBounded_Issue5121" --no-restore (58 passed)
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false --no-restore (0 warnings, 0 errors)
  • dotnet format CodeIndex.sln --verify-no-changes --no-restore
  • Changelog fragment validation (38 fragments passed)
  • Full Release suite: net9.0 passed (11,269 passed, 423 skipped); the concurrent net8.0 run reported three unrelated process-sensitive failures, and each failed test passed immediately in isolation (11,745 passed, 7 skipped before isolation reruns).
  • Codex adversarial review: no blocking/actionable issues found.

Documentation and changelog

  • Added bilingual changelog.d/unreleased/5121.fixed.md.
  • The existing bilingual testing guide already documents use of the compact plugin isolation fixture, so no guide update was needed.

Follow-up candidates

  • None.

Fixes #5121

@Widthdom
Widthdom marked this pull request as ready for review August 22, 2026 02:50
@Widthdom
Widthdom merged commit 2943d87 into main Aug 22, 2026
11 checks passed
@Widthdom
Widthdom deleted the fix-issue5121 branch August 22, 2026 02:51
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.

Debug net9 plugin fixture tests exceed extension type inspection limit

1 participant