From 614ab1f07d1cae49c06b8bcbd6132e0aa5827246 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/SumBusinessLogic.java | 6 ++++++ .../de/juplo/kafka/SumBusinessLogicTest.java | 17 +++++++++++++++++ 2 files changed, 23 insertions(+) diff --git a/src/main/java/de/juplo/kafka/SumBusinessLogic.java b/src/main/java/de/juplo/kafka/SumBusinessLogic.java index e662606..5cb906a 100644 --- a/src/main/java/de/juplo/kafka/SumBusinessLogic.java +++ b/src/main/java/de/juplo/kafka/SumBusinessLogic.java @@ -2,6 +2,7 @@ package de.juplo.kafka; import java.util.HashMap; import java.util.Map; +import java.util.Optional; public class SumBusinessLogic @@ -27,4 +28,9 @@ public class SumBusinessLogic state.put(user, 0l); } + + public synchronized Optional getSum(String user) + { + return Optional.empty(); + } } diff --git a/src/test/java/de/juplo/kafka/SumBusinessLogicTest.java b/src/test/java/de/juplo/kafka/SumBusinessLogicTest.java index 27c317a..ff9fdd0 100644 --- a/src/test/java/de/juplo/kafka/SumBusinessLogicTest.java +++ b/src/test/java/de/juplo/kafka/SumBusinessLogicTest.java @@ -24,4 +24,21 @@ public class SumBusinessLogicTest sumBusinessLogic.startSum("foo"); assertThatIllegalStateException().isThrownBy(() -> sumBusinessLogic.startSum("foo")); } + + @Test + @DisplayName("An empty Optional should be returned, for a non-existing sum") + public void testEmptyOptionalForNonExistingSum() + { + SumBusinessLogic sumBusinessLogic = new SumBusinessLogic(); + assertThat(sumBusinessLogic.getSum("foo")).isEmpty(); + } + + @Test + @DisplayName("A non-empty Optional should be returned, for an existing sum") + public void testNonEmptyOptionalForExistingSum() + { + SumBusinessLogic sumBusinessLogic = new SumBusinessLogic(); + sumBusinessLogic.startSum("foo"); + assertThat(sumBusinessLogic.getSum("foo")).isNotEmpty(); + } } -- 2.20.1