honeymoose
/
hapi-fhir
mirror of https://github.com/hapifhir/hapi-fhir.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Add comment to test method making clear its dubious value. (#5782)

This commit is contained in:
Luke deGruchy 2024-03-14 09:46:32 -04:00 committed by GitHub
parent fc4c31a779
commit ba5311d4cc
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
hapi-fhir-validation/src/test/java/ca/uhn/fhir/rest/server/interceptor/auth/AuthorizationInterceptorR4Test.java
View File

@ -2589,6 +2589,9 @@ public class AuthorizationInterceptorR4Test extends BaseValidationTestWithInline
assertEquals(false, ourHitMethod); assertEquals(false, ourHitMethod);
} }
// This test is of dubious value since it does NOT exercise DAO code. It simply exercises the AuthorizationInterceptor.
// In functional testing or with a more realistic integration test, this scenario, namely having ONLY a FHIR_PATCH
// role, will result in a failure to update the resource.
@Test @Test
public void testPatchAllowed() throws IOException { public void testPatchAllowed() throws IOException {
Observation obs = new Observation(); Observation obs = new Observation();