javax.ws.rs.ForbiddenException in alert if user does not have vocabulary data source role #2568#1902
Conversation
…ary data source role #2568
|
This PR broke some unit tests. @anton-abushkevich could you please fix the tests |
|
These tests passed in my local env and in Jenkins. The problem only happens in github check suite. It has something to do with temp directory on the server where build runs, assertion fails in this line: https://github.com/OHDSI/WebAPI/blob/master/src/test/java/org/ohdsi/webapi/cohortcharacterization/CohortCharacterizationServiceTest.java#L147. I do not sure why, but checks for other PRs (prior to this PR) were running with maven.test.skip=true, for example https://github.com/OHDSI/WebAPI/pull/1898/checks?check_run_id=2802600666. That is why checks in other PRs are green. |
|
Thanks for the followup! I've made a PR to disable the unit tests altogether. |
…ary data source role #2568 (OHDSI#1902)
Fixes OHDSI/Atlas#2568