Closed bashir2 closed 1 month ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 69.63%. Comparing base (
383ae75
) to head (7e2056b
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Description of what I changed
Fixes #155; in particular see this comment for more details. The main motivation was to upgrade the Spring version to address vulnerabilities but it also enabled us to upgrade HAPI and some other library versions.
E2E test
TESTED:
Ran the gateway locally and tested both
list
andpatient
access-checker with a local Keycloak.Checklist: I completed these to help reviewers :)
[x] I have read and will follow the review process.
[x] I am familiar with Google Style Guides for the language I have coded in.
No? Please take some time and review Java and Python style guides.
[x] My IDE is configured to follow the Google code styles.
No? Unsure? -> configure your IDE.
[ ] I have added tests to cover my changes. (If you refactored existing code that was well tested you do not have to add tests)
[x] I ran
mvn clean package
right before creating this pull request and added all formatting changes to my commit.[x] All new and existing tests passed.
[x] My pull request is based on the latest changes of the master branch.
No? Unsure? -> execute command
git pull --rebase upstream master