-
Notifications
You must be signed in to change notification settings - Fork 39
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
Add Mars OTC ELR scenarios to E2E UP Test #14360
Conversation
e02a325
to
6d526cd
Compare
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Manifest Files |
6d526cd
to
6aa898d
Compare
6aa898d
to
72beed7
Compare
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.
Nice stuff! I confirmed that the tests pass and I was able to find the action log about the invalid item getting filtered.
Also, hadn't quite processed how much faster the end2end_up tests are, wow!
Quality Gate passedIssues Measures |
This PR adds end 2 end testing scenarios for Mars OTC ELR reports moving through the UP.
These scenarios test when reports with the Mars OTC ELR topic
Test Steps:
./gradlew testend2endup
"Item 1 in the report was not valid. Reason: HL7 was not valid at OBX[1]-19[1].1"
Changes
Checklist
Testing
./prime test
or./gradlew testSmoke
against local Docker ReportStream container?Linked Issues