diff --git a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumService.java b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumService.java index 40aad53c4011c6cd6fac3c2e4f8117d07baff41a..35e0f7ba82970b6cbfa268eba848a6e1eef3f734 100644 --- a/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumService.java +++ b/nachrichten-manager-server/src/main/java/de/ozgcloud/nachrichten/antragraum/AntragraumService.java @@ -153,7 +153,7 @@ public class AntragraumService { public String sendRueckfrageAnswer(String samlToken, String rueckfrageId, PostfachNachricht nachricht) { verifyToken(samlToken); - verifyPostfachIdFromPostfachNachricht(samlToken, nachricht); + verifyAccessToPostfach(samlToken, nachricht); return postfachNachrichtService.persistAnswer(rueckfrageId, nachricht); } @@ -169,17 +169,11 @@ public class AntragraumService { var nachricht = nachrichtMapper.fromMapToPostfachMail(postfachNachrichtService.getById(id)); - verifyPostfachIdFromPostfachNachricht(samlToken, nachricht); + verifyAccessToPostfach(samlToken, nachricht); return nachricht; } - void verifyPostfachIdFromPostfachNachricht(String samlToken, PostfachNachricht nachricht) { - var vorgang = vorgangService.getVorgang(nachricht.getVorgangId()); - - verifyPostfachId(samlToken, nachricht, vorgang); - } - public void getAttachmentContent(AttachmentFileRequest request, OutputStream outputStream) { try { verifyAccessToFile(request); @@ -198,7 +192,7 @@ public class AntragraumService { void verifyAccessToFile(AttachmentFileRequest request) { verifyToken(request.getSamlToken()); var nachricht = attachedItemService.getPostfachNachricht(request.getNachrichtId()); - verifyAccessToPostfachNachricht(request.getSamlToken(), nachricht); + verifyAccessToPostfach(request.getSamlToken(), nachricht); verifyFileId(request.getFileId(), nachricht); } @@ -209,7 +203,7 @@ public class AntragraumService { } } - void verifyAccessToPostfachNachricht(String token, PostfachNachricht nachricht) { + void verifyAccessToPostfach(String token, PostfachNachricht nachricht) { var vorgang = vorgangService.getVorgang(nachricht.getVorgangId()); verifyPostfachId(token, nachricht, vorgang); verifyTrustLevel(token, nachricht, vorgang); diff --git a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumServiceTest.java b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumServiceTest.java index 8b6fb121f9dcfcb9dc8e206ae447d23b5c019e9c..aaeb23c758ac22f08d5f07a7d1b33f11698343a0 100644 --- a/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumServiceTest.java +++ b/nachrichten-manager-server/src/test/java/de/ozgcloud/nachrichten/antragraum/AntragraumServiceTest.java @@ -311,7 +311,7 @@ class AntragraumServiceTest { @DisplayName("Send rueckfragen answers") @Nested - class TestSendRueckfragenAnswers { + class TestSendRueckfrageAnswer { static final String SAML_TOKEN = "TOKEN"; static final String RUECKFRAGE_ID = UUID.randomUUID().toString(); static final PostfachNachricht NACHRICHT = PostfachNachrichtTestFactory.create(); @@ -319,7 +319,7 @@ class AntragraumServiceTest { @BeforeEach void mock() { doNothing().when(service).verifyToken(any()); - doNothing().when(service).verifyPostfachIdFromPostfachNachricht(any(), any()); + doNothing().when(service).verifyAccessToPostfach(any(), any()); } @Test @@ -337,10 +337,10 @@ class AntragraumServiceTest { } @Test - void shouldVerifyPostfachIdFromPostfachNachricht() { + void shouldVerifyAccessToPostfach() { service.sendRueckfrageAnswer(SAML_TOKEN, RUECKFRAGE_ID, NACHRICHT); - verify(service).verifyPostfachIdFromPostfachNachricht(SAML_TOKEN, NACHRICHT); + verify(service).verifyAccessToPostfach(SAML_TOKEN, NACHRICHT); } } @@ -416,7 +416,7 @@ class AntragraumServiceTest { when(nachrichtMapper.fromMapToPostfachMail(any())).thenReturn(nachricht); doNothing().when(service).verifyToken(any()); - doNothing().when(service).verifyPostfachIdFromPostfachNachricht(any(), any()); + doNothing().when(service).verifyAccessToPostfach(any(), any()); } @Test @@ -448,10 +448,10 @@ class AntragraumServiceTest { } @Test - void shouldCallVerifyPostfachIdFromPostfachNachricht() { + void shouldCallVerifyAccessToPostfach() { getRueckfrage(); - verify(service).verifyPostfachIdFromPostfachNachricht(samlToken, nachricht); + verify(service).verifyAccessToPostfach(samlToken, nachricht); } private PostfachNachricht getRueckfrage() { @@ -459,35 +459,6 @@ class AntragraumServiceTest { } } - @DisplayName("Verify postfachId from PostfachNachricht") - @Nested - class TestVerifyPostfachIdFromPostfachNachricht { - - private static final String SAML_TOKEN = "TOKEN"; - private static final PostfachNachricht POSTFACH_NACHRICHT = PostfachNachrichtTestFactory.create(); - private final Vorgang vorgang = VorgangTestFactory.create(); - - @BeforeEach - void mock() { - when(vorgangService.getVorgang(any())).thenReturn(vorgang); - doNothing().when(service).verifyPostfachId(SAML_TOKEN, POSTFACH_NACHRICHT, vorgang); - } - - @Test - void shouldCallVorgangService() { - service.verifyPostfachIdFromPostfachNachricht(SAML_TOKEN, POSTFACH_NACHRICHT); - - verify(vorgangService).getVorgang(MessageTestFactory.VORGANG_ID); - } - - @Test - void shouldCallVerifyPostfachId() { - service.verifyPostfachIdFromPostfachNachricht(SAML_TOKEN, POSTFACH_NACHRICHT); - - verify(service).verifyPostfachId(SAML_TOKEN, POSTFACH_NACHRICHT, vorgang); - } - } - @Nested class TestVerifyPostfachId { @@ -736,7 +707,7 @@ class AntragraumServiceTest { @BeforeEach void mock() { when(attachedItemService.getPostfachNachricht(AttachmentFileRequestTestFactory.NACHRICHT_ID)).thenReturn(nachricht); - doNothing().when(service).verifyAccessToPostfachNachricht(AttachmentFileRequestTestFactory.TOKEN, nachricht); + doNothing().when(service).verifyAccessToPostfach(AttachmentFileRequestTestFactory.TOKEN, nachricht); doNothing().when(service).verifyFileId(AttachmentFileRequestTestFactory.FILE_ID, nachricht); } @@ -755,10 +726,10 @@ class AntragraumServiceTest { } @Test - void shouldCallVerifyAccessToPostfachNachricht() { + void shouldCallVerifyAccessToPostfach() { callVerifyAccessToFile(); - verify(service).verifyAccessToPostfachNachricht(AttachmentFileRequestTestFactory.TOKEN, nachricht); + verify(service).verifyAccessToPostfach(AttachmentFileRequestTestFactory.TOKEN, nachricht); } @Test @@ -775,7 +746,7 @@ class AntragraumServiceTest { } @Nested - class TestVerifyAccessToPostfachNachricht { + class TestVerifyAccessToPostfach { private final PostfachNachricht nachricht = PostfachNachrichtTestFactory.create(); private final Vorgang vorgang = VorgangTestFactory.create(); @@ -789,27 +760,27 @@ class AntragraumServiceTest { @Test void shouldCallVorgangService() { - verifyAccessToPostfachNachricht(); + verifyAccessToPostfach(); verify(vorgangService).getVorgang(MessageTestFactory.VORGANG_ID); } @Test void shouldCallVerifyPostfachId() { - verifyAccessToPostfachNachricht(); + verifyAccessToPostfach(); verify(service).verifyPostfachId(AttachmentFileRequestTestFactory.TOKEN, nachricht, vorgang); } @Test void shouldVerifyTrustLevel() { - verifyAccessToPostfachNachricht(); + verifyAccessToPostfach(); verify(service).verifyTrustLevel(AttachmentFileRequestTestFactory.TOKEN, nachricht, vorgang); } - private void verifyAccessToPostfachNachricht() { - service.verifyAccessToPostfachNachricht(AttachmentFileRequestTestFactory.TOKEN, nachricht); + private void verifyAccessToPostfach() { + service.verifyAccessToPostfach(AttachmentFileRequestTestFactory.TOKEN, nachricht); } }