fix(ServiceBus): Workaround health API timeout

[HofmeisterAn]

What does this PR do?

-

Why is it important?

-

Related issues

• Relates fix(EventHubs): Workaround health API timeout #1624

Summary by CodeRabbit

• Bug Fixes

  • Enhanced Service Bus emulator startup health verification by adding log-based validation.
  • Simplified Event Hubs emulator initialization by removing legacy workarounds.

• Chores

  • Removed unused global imports.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[netlify]

✅ Deploy Preview for testcontainers-dotnet ready!

Name	Link
🔨 Latest commit	dde205f
🔍 Latest deploy log	https://app.netlify.com/projects/testcontainers-dotnet/deploys/696b7a42e1d425000817b4df
😎 Deploy Preview	https://deploy-preview-1625--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]

Walkthrough

The PR enhances Azure Emulator container wait strategies by replacing a custom workaround with a log-based readiness check before HTTP health verification. The workaround class is removed from EventHubs, and an unused global using statement is also cleaned up.

Changes

Cohort / File(s)	Summary
Emulator Wait Strategy Refinement src/Testcontainers.ServiceBus/ServiceBusBuilder.cs, src/Testcontainers.EventHubs/EventHubsBuilder.cs	Modified Init method wait strategies: prepended UntilMessageIsLogged("Emulator Service is Successfully Up!") before HTTP health checks with referencing comments. Removed custom WorkaroundGhIssue69 class from EventHubs, simplifying the wait strategy chain.
Dependency Cleanup src/Testcontainers.EventHubs/Usings.cs	Removed unused global using directive for System.Threading.Tasks.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• PR #1624: Modifies EventHubsBuilder wait strategy and directly relates to the WorkaroundGhIssue69 workaround being replaced in this PR.

Poem

🐰 With whiskers twitched and nose in the air,
We hopped through the wait strategies with care,
Removed the old workaround's twisted knot,
Log checks first, then HTTP on the dot!
Clean code hops faster, hip-hop hooray! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The pull request description is largely incomplete. While it includes the template structure with related issues (#1624), the mandatory sections 'What does this PR do?' and 'Why is it important?' contain only placeholder dashes (-) with no actual details.	Complete the mandatory sections by explaining what changes were made (patterns, design), why they are important (rationale/motivation), and how to test the PR. These details are essential for code review.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix(ServiceBus): Workaround health API timeout' is specific and directly related to the main change - addressing a health API timeout issue in ServiceBus by adding a log message check before the HTTP health check.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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 and usage tips.}