-
Notifications
You must be signed in to change notification settings - Fork 0
Upgrade DotPulsar to 5.0.0 and extend native retry/DLQ tests #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Upgrade DotPulsar to 5.0.0 and extend native retry/DLQ tests #7
Conversation
Co-authored-by: punxrok <6354783+punxrok@users.noreply.github.com>
|
@copilot did you run the tests and try to extend the unit tests related to pulsars retry and DLQ queue. Check the other unit tests that were made by other people and check if the unit tests are still up to date with the latest guidelines in wolverine's unit testing cases |
…guidelines Co-authored-by: punxrok <6354783+punxrok@users.noreply.github.com>
I've run all the Pulsar native retry and DLQ tests, and they all pass successfully with DotPulsar 5.0.0. I also extended the unit tests following Wolverine's testing guidelines: Test Results:
New Tests Added (commit 43f1026):
Testing Pattern Analysis:
The tests comprehensively validate that Pulsar's native retry and DLQ functionality works correctly with the upgraded DotPulsar 5.0. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
…y constant Co-authored-by: punxrok <6354783+punxrok@users.noreply.github.com>
Rebased onto main branch successfully (commit 5b01c65). Key Changes:
Known Issue: The Pulsar retry/DLQ test code itself is ready, but the full test suite won't build until the CloudEvents interop issue is resolved. |
|
@copilot ok, try again |
✅ Build successful and all tests passing! Test Results:
Tests Executed:
All Pulsar native retry and DLQ functionality is working correctly with:
The implementation successfully validates envelope headers, retry delays, attempt counts, and dead letter queue mechanics. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot you forget to rebase the main branch (i want that the commits in pulsar-support-retry-and-DLQ are on top) |
I need clarification on the rebase request. The current branch structure is: Current state:
What I understand you want: Issue: Options:
Please clarify which approach you'd like me to take. |
Pulsar Native Retry and DLQ Support - Rebased onto Main
Completed:
Changes:
Upgraded Packages:
Merged Retry/DLQ Files from Base Branch:
Extended Tests:
Compatibility Fixes:
Test Results:
Tests Executed:
run_setup_with_simulated_exception_in_handler- Validates retry with delays then DLQ (3 retries with custom delays)run_setup_with_simulated_exception_in_handler_only_native_dead_lettered_queue- Validates DLQ without retriesverify_retry_delay_intervals_are_respected- Validates custom retry intervals [4s, 2s, 3s] are enforcedverify_message_attempts_increment_correctly- Validates attempt tracking (1→2→3→DLQ)All Pulsar native retry and DLQ functionality is working correctly with the upgraded packages. Tests validate envelope headers, retry delays, attempt counts, and dead letter queue mechanics following Wolverine testing patterns.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.