chore(AspireDashboard): Cover connection string provider

[HofmeisterAn]

What does this PR do?

This PR adds an assertion to test the Aspire dashboard connection string provider implementation and fixes an issue where the provider resolved the wrong method, causing it to enter an infinite loop.

Why is it important?

-

Related issues

-

Summary by CodeRabbit

• Bug Fixes
  • Improved dashboard connection handling so the application now uses the correct dashboard address when building the host connection string.
  • Added verification that the dashboard endpoint remains reachable and returns a successful response.

[netlify]

✅ Deploy Preview for testcontainers-dotnet ready!

Name	Link
🔨 Latest commit	0fe69c8
🔍 Latest deploy log	https://app.netlify.com/projects/testcontainers-dotnet/deploys/6a3c2a3ded6f3700086930b8
😎 Deploy Preview	https://deploy-preview-1713--testcontainers-dotnet.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 89b42730-3240-4cc3-8364-ae26d24cd9c0

📥 Commits

Reviewing files that changed from the base of the PR and between 9bbf81b and 0fe69c8.

📒 Files selected for processing (2)

• src/Testcontainers.AspireDashboard/AspireDashboardConnectionStringProvider.cs
• tests/Testcontainers.AspireDashboard.Tests/AspireDashboardContainerTest.cs

---

Walkthrough

The Aspire dashboard host connection string now comes from the dashboard address. The dashboard test still checks for HTTP 200 and now also verifies the dashboard address matches the container connection string.

Changes

Aspire dashboard host string

Layer / File(s)	Summary
Provider update and test assertion src/Testcontainers.AspireDashboard/AspireDashboardConnectionStringProvider.cs, tests/Testcontainers.AspireDashboard.Tests/AspireDashboardContainerTest.cs	GetHostConnectionString() now returns Container.GetDashboardAddress(), and the dashboard test adds an assertion that GetDashboardAddress() equals GetConnectionString() while keeping the HTTP 200 check.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• testcontainers/testcontainers-dotnet#1194: Touches the same AspireDashboardConnectionStringProvider.GetHostConnectionString() path and changes how the Aspire dashboard host string is derived.

Suggested labels

module

Poem

A rabbit hopped by, soft and keen,
To see the dashboard run clean.
One string took a turn,
And tests gave a purr—
Hoppy paths, and all pixels serene.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The PR description has the required sections, but 'Why is it important?' and 'How to test this PR' are effectively empty.	Add a real rationale, link any related issues, and include test steps or commands so the template is complete.
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: fixing the AspireDashboard connection string provider and its test coverage.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/cover-aspire-dashboard-connection-string-provider

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}