Skip to content

test: add first-party unit tests for Vulthil.xUnit and Vulthil.Extensions.Testing#345

Merged
Vulthil merged 1 commit into
mainfrom
test/testing-stack-first-party-tests
Jul 21, 2026
Merged

test: add first-party unit tests for Vulthil.xUnit and Vulthil.Extensions.Testing#345
Vulthil merged 1 commit into
mainfrom
test/testing-stack-first-party-tests

Conversation

@Vulthil

@Vulthil Vulthil commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add tests/Vulthil.Extensions.Testing.Tests covering Polling.WaitAsync (both the generic and
    non-generic cores: immediate success, success-on-Nth-attempt, timeout error aggregation,
    cancellation-vs-timeout distinction) and HttpResponseMessageExtensions.GetResponseAsync<T>.
  • Add tests/Vulthil.xUnit.Tests covering BaseUnitTestCase lifecycle (lazy Target timing,
    Use/UseReal/UseRealFor, Initialize/Dispose hook ordering, disposal ordering), HttpMock rule
    matching (URL, wildcard, headers, the predicate On(...) overload, unmatched requests, reset),
    SanitizeDatabaseName edge cases, and ContainerHost bookkeeping via fakes (no Docker).
  • Relocate the existing BaseUnitTestCaseDisposalTests from Vulthil.SharedKernel.Tests into the
    new project, since its subject is a Vulthil.xUnit type.
  • Wire both new projects into the solution under a new /tests/Testing/ folder, mirroring the
    existing /src/Testing/ grouping.

Verification

  • Full solution build: clean, 0 warnings/0 errors, both net9.0 and net10.0.
  • Vulthil.Extensions.Testing.Tests: 16/16 passing on both TFMs.
  • Vulthil.xUnit.Tests: 45/45 passing on both TFMs.
  • Vulthil.SharedKernel.Tests (post-relocation): 11/11 passing on both TFMs.
  • No public API changes: every PublicAPI.*.txt and CompatibilitySuppressions.xml is unchanged.

Backport to v1.0: no — test-only additions ship with the next release.

@claude

claude Bot commented Jul 21, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@Vulthil
Vulthil merged commit 1c937cd into main Jul 21, 2026
6 checks passed
@Vulthil
Vulthil deleted the test/testing-stack-first-party-tests branch July 21, 2026 15:29
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.

1 participant