Skip to content

Conversation

@noname007
Copy link
Contributor

The host may crashed beforerpush was executed

Q A
Is bugfix? ✔️/❌
New feature? ✔️/❌
Breaks BC? ✔️/❌
Fixed issues

@samdark samdark added the pr:request for unit tests Unit tests are needed. label May 23, 2024
@samdark
Copy link
Member

samdark commented May 23, 2024

Is there a way to test that?

@noname007
Copy link
Contributor Author

use debug mode and add some code to emulate the crash scenario
image

@noname007 noname007 changed the title at leat once msg consume Enhancement: Ensure messages are consumed at least once. Mar 2, 2025
@noname007
Copy link
Contributor Author

hi @samdark ,I need your help to review unit test code.

@samdark samdark requested review from a team March 3, 2025 10:02
@samdark samdark removed the pr:request for unit tests Unit tests are needed. label Mar 5, 2025
@samdark samdark self-assigned this Mar 5, 2025
noname007 and others added 2 commits March 6, 2025 11:58
Copy link
Member

@samdark samdark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 The tests are good now. One last thing — a line for CHANGELOG.

Copy link
Member

@samdark samdark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 The tests are good now. One last thing — a line for CHANGELOG.

@samdark samdark merged commit d6372d8 into yiisoft:master Mar 11, 2025
7 checks passed
@samdark
Copy link
Member

samdark commented Mar 11, 2025

Awesome. Thank you!

@samdark samdark modified the milestone: 2.3.8 Mar 11, 2025
@noname007
Copy link
Contributor Author

@samdark thanks for your help

1 similar comment
@noname007
Copy link
Contributor Author

@samdark thanks for your help

@samdark
Copy link
Member

samdark commented Mar 12, 2025

You're welcome.

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.

2 participants