From 7acccf19ab0f7b11aa5fcd70867ae3ea80de2dcb Mon Sep 17 00:00:00 2001 From: Kai Moritz Date: Sat, 13 Aug 2022 15:24:16 +0200 Subject: [PATCH] ROT: Erwartungen an SumBusinessLogic.getSum(String) --- .../java/de/juplo/kafka/AdderBusinessLogic.java | 6 ++++++ .../de/juplo/kafka/AdderBusinessLogicTest.java | 17 +++++++++++++++++ 2 files changed, 23 insertions(+) diff --git a/src/main/java/de/juplo/kafka/AdderBusinessLogic.java b/src/main/java/de/juplo/kafka/AdderBusinessLogic.java index a3848c4..df8c5f9 100644 --- a/src/main/java/de/juplo/kafka/AdderBusinessLogic.java +++ b/src/main/java/de/juplo/kafka/AdderBusinessLogic.java @@ -3,6 +3,7 @@ package de.juplo.kafka; import java.util.HashMap; import java.util.Map; +import java.util.Optional; public class AdderBusinessLogic @@ -28,4 +29,9 @@ public class AdderBusinessLogic state.put(user, 0l); } + + public synchronized Optional getSum(String user) + { + return Optional.empty(); + } } diff --git a/src/test/java/de/juplo/kafka/AdderBusinessLogicTest.java b/src/test/java/de/juplo/kafka/AdderBusinessLogicTest.java index 17fadb1..4c0ed4b 100644 --- a/src/test/java/de/juplo/kafka/AdderBusinessLogicTest.java +++ b/src/test/java/de/juplo/kafka/AdderBusinessLogicTest.java @@ -24,4 +24,21 @@ public class AdderBusinessLogicTest adder.startSum("foo"); assertThatIllegalStateException().isThrownBy(() -> adder.startSum("foo")); } + + @Test + @DisplayName("An empty Optional should be returned, for a non-existing sum") + public void testEmptyOptionalForNonExistingSum() + { + AdderBusinessLogic adder = new AdderBusinessLogic(); + assertThat(adder.getSum("foo")).isEmpty(); + } + + @Test + @DisplayName("A non-empty Optional should be returned, for an existing sum") + public void testNonEmptyOptionalForExistingSum() + { + AdderBusinessLogic adder = new AdderBusinessLogic(); + adder.startSum("foo"); + assertThat(adder.getSum("foo")).isNotEmpty(); + } } -- 2.20.1