-
Notifications
You must be signed in to change notification settings - Fork 41
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
Fix for accessToken from Message Testing API request #17081
Fix for accessToken from Message Testing API request #17081
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Manifest Files |
Test Results1 271 tests ±0 1 267 ✅ ±0 7m 57s ⏱️ +51s Results for commit 48d61da. ± Comparison against base commit 20905dd. This pull request removes 2 and adds 2 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
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
Integration Test Results 60 files 60 suites 43m 50s ⏱️ Results for commit 48d61da. ♻️ This comment has been updated with latest results. |
Quality Gate failedFailed conditions |
This PR uses the Bearer token from the API request and passes it over to the CLI functions and removes unnecessary flags for filter validation results.
Test Steps:
processFhirDataRequest
outside of the CLI context should work as long as we pass in a valid bearer token. For this we can use a unit test:Changes
Checklist
Testing
./prime test
or./gradlew testSmoke
against local Docker ReportStream container?npm run lint:write
?Process
Linked Issues
To Be Done
Create GitHub issues to track the work remaining, if any