Refine SurrealDB hosting test coverage

[aaronpowell]

Closes: N/A

Overview

Moves SurrealDB coverage into the hosting public API tests, removes the old functional test, and adds unit assertions for data mounts, health check registration, init-file wiring, and creation-script annotations.

PR Checklist

• Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
• Based off latest main branch of toolkit
• PR doesn't include merge commits (always rebase on top of our main, if needed)
• New integration
  • Docs are written
  • Added description of major feature to project description for NuGet package (4000 total character limit, so don't push entire description over that)
• Tests for the changes have been added (for bug fixes / features) (if applicable)
• Contains NO breaking changes
• Every new API (including internal ones) has full XML docs
• Code follows all style conventions

Other information

The init-file and creation-script checks are intentionally unit-level and assert the resource model rather than running the full container workflow.

[github-actions]

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.sh | bash -s -- 1441

Or

• Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/CommunityToolkit/Aspire/main/eng/scripts/dogfood-pr.ps1) } 1441"

[Copilot]

Pull request overview

This PR refines the SurrealDB hosting integration’s test suite by moving coverage from Docker-based functional tests to unit/public-API tests that validate the Aspire resource model (annotations, mounts, init-file wiring, and creation scripts).

Changes:

• Added public API tests that assert health check annotation creation, container mount annotations, init-file container file wiring + env var, and creation-script annotations.
• Removed the previous Docker-driven SurrealDbFunctionalTests suite.
• Updated test/project wiring by removing a direct compile-include and adding InternalsVisibleTo for the tests.

Show a summary per file

File	Description
tests/CommunityToolkit.Aspire.Hosting.SurrealDb.Tests/SurrealDbPublicApiTests.cs	Adds unit/public-API assertions for health checks, mounts, init-file wiring, and creation-script annotations.
tests/CommunityToolkit.Aspire.Hosting.SurrealDb.Tests/SurrealDbFunctionalTests.cs	Removes Docker-based functional tests in favor of model-level assertions.
tests/CommunityToolkit.Aspire.Hosting.SurrealDb.Tests/CommunityToolkit.Aspire.Hosting.SurrealDb.Tests.csproj	Removes direct source inclusion now that internals are exposed to the test assembly.
src/CommunityToolkit.Aspire.Hosting.SurrealDb/CommunityToolkit.Aspire.Hosting.SurrealDb.csproj	Adds InternalsVisibleTo for the test project to enable internal-model assertions.

Review details

• Files reviewed: 4/4 changed files
• Comments generated: 1
• Review effort level: Low

[github-actions]

Package	Line Rate	Branch Rate	Complexity	Health
CommunityToolkit.Aspire.Hosting.SurrealDb	89%	74%	208	✔
CommunityToolkit.Aspire.Hosting.SurrealDb.ApiService	85%	62%	236	✔
CommunityToolkit.Aspire.SurrealDb	56%	42%	40	❌
Summary	86% (946 / 1103)	66% (278 / 424)	484	✔

[aaronpowell]

@Odonno turns out that you're not actually a member to be assigned review, even though you're the codeowner here 😅

Anyway, invited you and want to have a look over this