From 7d6d1279f25e80ab99514311731f5659809b8407 Mon Sep 17 00:00:00 2001 From: Paul-Christian Volkmer Date: Tue, 2 Apr 2024 18:28:36 +0200 Subject: [PATCH] fix: apply changes in API version 2.12.x OS API version 2.12.1 or newer fixes the issue of not using patient ID when adding patient object to procedure. Since this version, a NPE will occur of not providing a patient ID when adding a patient object to a procedure. --- .../DNPM/security/PersonPoolBasedPermissionEvaluatorTest.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/test/java/DNPM/security/PersonPoolBasedPermissionEvaluatorTest.java b/src/test/java/DNPM/security/PersonPoolBasedPermissionEvaluatorTest.java index a05f83a..8e8204a 100644 --- a/src/test/java/DNPM/security/PersonPoolBasedPermissionEvaluatorTest.java +++ b/src/test/java/DNPM/security/PersonPoolBasedPermissionEvaluatorTest.java @@ -92,6 +92,7 @@ class PersonPoolBasedPermissionEvaluatorTest { @Test void testShouldGrantPermissionByProcedureObject() { var patient = new Patient(onkostarApi); + patient.setId(1); patient.setPersonPoolCode("Pool2"); var object = new Procedure(onkostarApi); @@ -107,6 +108,7 @@ class PersonPoolBasedPermissionEvaluatorTest { void testShouldGrantPermissionByProcedureIdAndType() { doAnswer(invocationOnMock -> { var patient = new Patient(onkostarApi); + patient.setId(1); patient.setPersonPoolCode("Pool2"); var object = new Procedure(onkostarApi); @@ -124,6 +126,7 @@ class PersonPoolBasedPermissionEvaluatorTest { @Test void testShouldDenyPermissionByProcedureObject() { var patient = new Patient(onkostarApi); + patient.setId(1); patient.setPersonPoolCode("Pool1"); var object = new Procedure(onkostarApi); @@ -139,6 +142,7 @@ class PersonPoolBasedPermissionEvaluatorTest { void testShouldDenyPermissionByProcedureIdAndType() { doAnswer(invocationOnMock -> { var patient = new Patient(onkostarApi); + patient.setId(1); patient.setPersonPoolCode("Pool1"); var object = new Procedure(onkostarApi);