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.

Sign up for GitHub

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

Jump to bottom

increased test coverage #71

Merged
merged 3 commits into from
May 8, 2019
Merged

increased test coverage #71

merged 3 commits into from
May 8, 2019

Conversation

rhinof
Copy link
Contributor

@rhinof rhinof commented May 7, 2019 •
edited
Loading

#40

@rhinof rhinof requested a review from vladshub May 7, 2019 16:12
@coveralls
Copy link

coveralls commented May 7, 2019 •
edited
Loading

Coverage Status

Coverage increased (+1.9%) to 72.31% when pulling a3069de on coverage into bbbeac7 on v1.x.

improved deadlettering test
a3069de 
the test now validates that messages get rejected when handlers return
an error and all rety attempts did not sccueed
@rhinof rhinof changed the title added a test for health prob and fixed a minor race condition in worker increased test coverage May 7, 2019
@rhinof rhinof merged commit 665d2d1 into v1.x May 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vladshub vladshub vladshub approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@rhinof @coveralls @vladshub