Skip to content
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

Bugfix - Stopped message processed mutliple times on shutdown #169

Merged
merged 5 commits into from
Nov 20, 2024

Conversation

troygilman0
Copy link
Contributor

@troygilman0 troygilman0 commented Nov 3, 2024

This is the fix for the bug described here #170

The source of this bug is the inbox.procStatus being incorrectly set to "idle" after the posionPill stops the inbox. The procStatus should only be set to "idle" if the procStatus is "running".

I added 2 tests:

  • TestInboxStop - unit test for the inbox
  • TestMultipleStops - integration test

@troygilman0 troygilman0 changed the title Bugfix - Process status incorrectly set to "idle" during shutdown procedure Bugfix - Stopped message processed mutliple times on shutdown Nov 3, 2024
@anthdm anthdm merged commit 9721d9c into anthdm:master Nov 20, 2024
1 check passed
@troygilman0 troygilman0 deleted the multi-stop-bug branch November 20, 2024 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants