-
Notifications
You must be signed in to change notification settings - Fork 0
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
1080: Update Read and send errors #124
Conversation
Co-Authored-By: jherrflexion <118225331+jherrflexion@users.noreply.github.com> Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com> Co-Authored-By: Samuel Aquino <saquino@flexion.us> Co-Authored-By: Bella L. Quintero <96704946+pluckyswan@users.noreply.github.com>
… for Non Transient Errors Co-authored-by: Sylvie <sschuresko@flexion.us> Co-authored-by: jcrichlake <145698165+jcrichlake@users.noreply.github.com> Co-authored-by: pluckyswan <96704946+pluckyswan@users.noreply.github.com> Co-authored-by: James Herr <jherr@flexion.us>
|
||
func (suite *SenderTestSuite) Test_SendMessage_StatusCodeIs400_ReturnsNonTransientError() { | ||
sender, err := NewSender() | ||
assert.NoError(suite.T(), err) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it necessary to check for errors after each step if it's not the target of the test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Asserts on test setup have been removed
Co-authored-by: Sylvie <sschuresko@flexion.us> Co-authored-by: pluckyswan <96704946+pluckyswan@users.noreply.github.com> Co-authored-by: James Herr <jherr@flexion.us> Co-authored-by: jbiskie <jbiskie@flexion.us>
Co-authored-by: Sylvie <sschuresko@flexion.us> Co-authored-by: jcrichlake <145698165+jcrichlake@users.noreply.github.com> Co-authored-by: pluckyswan <96704946+pluckyswan@users.noreply.github.com> Co-authored-by: James Herr <jherr@flexion.us> Co-authored-by: jbiskie <jbiskie@flexion.us>
Quality Gate passedIssues Measures |
Description
Updates Read and Send Errors to move all Non Transient errors(error codes between 400 and 499) to the failure folder.
Issue
Issue 1080
Checklist
Note: You may remove items that are not applicable