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 0cfc23e68c7ab359d04f20194f660a5e7f5f3068..64a5e09354e0aac20407b0f7a57fd077d2d21995 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 @@ -98,14 +98,15 @@ public class AntragraumService { var rueckfragen = postfachNachrichtService.findRueckfragen(BAYERN_ID_SERVICE_KONTO_TYPE, getPostfachId(samlToken)) .map(rueckfrageMapper::fromPostfachNachricht) - .map(rueckfrage -> enrichByAnswers(rueckfrage, findAnswers(samlToken, rueckfrage.getId()))).toList(); + .map(rueckfrage -> enrichByAnswers(rueckfrage, samlToken)).toList(); return enrichRueckfragen(rueckfragen); } - Rueckfrage enrichByAnswers(Rueckfrage rueckfrage, Stream<PostfachNachricht> answers) { + Rueckfrage enrichByAnswers(Rueckfrage rueckfrage, String samlToken) { var rueckfrageBuilder = rueckfrage.toBuilder(); - getFirstSentAt(answers).ifPresent(firstSentAt -> rueckfrageBuilder.answeredAt(firstSentAt).status(RueckfrageStatus.ANSWERED)); + getFirstSentAt(findAnswers(samlToken, rueckfrage.getId())) + .ifPresent(firstSentAt -> rueckfrageBuilder.answeredAt(firstSentAt).status(RueckfrageStatus.ANSWERED)); return rueckfrageBuilder.build(); } 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 453dca3d1d2c1dd0d170d8ea90a019885ce9501e..8433d492e6c79d56cc689854aca408f5099ce80c 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 @@ -76,7 +76,6 @@ class AntragraumServiceTest { private final Rueckfrage rueckfrage = RueckfrageTestFactory.create(); private final PostfachNachricht postfachNachricht = PostfachNachrichtTestFactory.create(); - private final Stream<PostfachNachricht> answers = Stream.of(postfachNachricht); @BeforeEach void mock() { @@ -85,7 +84,6 @@ class AntragraumServiceTest { when(rueckfrageMapper.fromPostfachNachricht(any())).thenReturn(rueckfrage); doReturn(rueckfrage).when(service).enrichByAnswers(any(), any()); doReturn(Stream.of(rueckfrage)).when(service).enrichRueckfragen(any()); - doReturn(answers).when(service).findAnswers(any(), any()); doReturn(POSTFACH_ID).when(service).getPostfachId(any()); } @@ -124,18 +122,11 @@ class AntragraumServiceTest { verify(rueckfrageMapper).fromPostfachNachricht(postfachNachricht); } - @Test - void shouldCallFindAnswers() { - service.findRueckfragen(SAML_TOKEN).toList(); - - verify(service).findAnswers(SAML_TOKEN, RueckfrageTestFactory.ID); - } - @Test void shouldCallEnrichByAnswers() { service.findRueckfragen(SAML_TOKEN).toList(); - verify(service).enrichByAnswers(rueckfrage, answers); + verify(service).enrichByAnswers(rueckfrage, SAML_TOKEN); } @Test @@ -157,11 +148,22 @@ class AntragraumServiceTest { @Nested class TestEnrichByAnswers { + private static final String SAML_TOKEN = "TOKEN"; private final Rueckfrage rueckfrage = RueckfrageTestFactory.create(); + private final PostfachNachricht postfachNachricht = PostfachNachrichtTestFactory.create(); + private final Stream<PostfachNachricht> answers = Stream.of(postfachNachricht); @BeforeEach void mock() { doReturn(Optional.of(PostfachNachrichtTestFactory.SENT_AT)).when(service).getFirstSentAt(any()); + doReturn(answers).when(service).findAnswers(any(), any()); + } + + @Test + void shouldCallFindAnswers() { + enrichByAnswer(); + + verify(service).findAnswers(SAML_TOKEN, RueckfrageTestFactory.ID); } @Test @@ -187,7 +189,7 @@ class AntragraumServiceTest { } private Rueckfrage enrichByAnswer() { - return service.enrichByAnswers(rueckfrage, Stream.empty()); + return service.enrichByAnswers(rueckfrage, SAML_TOKEN); } }