test: Increase SlowMessageTransport delay to reduce test flakiness

[samtrion]

The delay in ExecuteAsync_WithPerEventTypeProcessingTimeout_UsesOverride was raised from 200ms to 5s to ensure it exceeds the override timeout, minimizing timer-jitter and improving test reliability.

Summary by CodeRabbit

• Tests
  • Improved test reliability for processing timeout validation by adjusting timing parameters.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: aaf5fa17-ce00-4fb4-8edb-fe40c0058b86

📥 Commits

Reviewing files that changed from the base of the PR and between dade20a and e095ad1.

📒 Files selected for processing (1)

• tests/NetEvolve.Pulse.Tests.Unit/Outbox/OutboxProcessorHostedServiceTests.cs

---

Walkthrough

A test timeout value is increased in the outbox processor hosted service tests to ensure reliable triggering of per-event-type processing timeouts during message transport operations.

Changes

Cohort / File(s)	Summary
Test Timeout Configuration tests/NetEvolve.Pulse.Tests.Unit/Outbox/OutboxProcessorHostedServiceTests.cs	Increased SlowMessageTransport delay from 200ms to 5 seconds in ExecuteAsync_WithPerEventTypeProcessingTimeout_UsesOverride test to reliably trigger processing timeout behavior.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A hop, a skip, a timer's call,
Five seconds now, no jitter at all!
The test waits long, the timeout true,
No more surprises—just what we knew! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: increasing SlowMessageTransport delay to reduce test flakiness, which matches the actual modification in the test file.

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

---

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.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.78%. Comparing base (dade20a) to head (e095ad1).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #435      +/-   ##
==========================================
+ Coverage   92.36%   92.78%   +0.41%     
==========================================
  Files         159      159              
  Lines        5960     5960              
  Branches      539      539              
==========================================
+ Hits         5505     5530      +25     
+ Misses        302      277      -25     
  Partials      153      153              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.