BMDCachePeopleTest

src/main/java/scot/carricksoftware/grants/cache/BMDCache.java

@@ -11,9 +11,8 @@ import java.util.List;
 
 public interface BMDCache {
 
-    @SuppressWarnings("unused")
+    @SuppressWarnings("UnusedReturnValue")
     List<Person> getPeople();
-    @SuppressWarnings("unused")
     void invalidatePeople();
 
 }

src/test/java/scot/carricksoftware/grants/cache/BMDCacheTest.java

@@ -1,31 +0,0 @@
-/*
- * Copyright (c) 2025.  Andrew Grant Carrick Software. All rights reserved
- *
- */
-
-package scot.carricksoftware.grants.cache;
-
-import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Test;
-import org.mockito.Mock;
-import scot.carricksoftware.grants.services.people.PersonService;
-
-import static org.junit.jupiter.api.Assertions.assertNotNull;
-
-public class BMDCacheTest {
-
-    @Mock
-    PersonService personServiceMock;
-
-    private BMDCache cache;
-
-    @BeforeEach
-    void setUp() {
-        cache = new BMDCacheImpl(personServiceMock);
-    }
-
-    @Test
-    void constructorTest() {
-        assertNotNull(cache);
-    }
-}

src/test/java/scot/carricksoftware/grants/cache/BMDPeopleCacheTest.java

@@ -0,0 +1,51 @@
+/*
+ * Copyright (c) 2025.  Andrew Grant Carrick Software. All rights reserved
+ *
+ */
+
+package scot.carricksoftware.grants.cache;
+
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
+import org.mockito.Mock;
+import org.mockito.junit.jupiter.MockitoExtension;
+import scot.carricksoftware.grants.services.people.PersonService;
+
+import static org.mockito.Mockito.times;
+import static org.mockito.Mockito.verify;
+
+@ExtendWith(MockitoExtension.class)
+public class BMDPeopleCacheTest {
+
+    @Mock
+    PersonService personServiceMock;
+
+    private BMDCache cache;
+
+    @BeforeEach
+    void setUp() {
+        cache = new BMDCacheImpl(personServiceMock);
+    }
+
+    @Test
+    void nullPeopleTest() {
+        cache.getPeople();
+        verify(personServiceMock).findAll();
+    }
+
+    @Test
+    void alreadyExistsTest() {
+        cache.getPeople();
+        cache.getPeople();
+        verify(personServiceMock, times(1)).findAll();
+    }
+
+    @Test
+    void invalidateTest() {
+        cache.getPeople();
+        cache.invalidatePeople();
+        cache.getPeople();
+        verify(personServiceMock, times(2)).findAll();
+    }
+}