UpdateRecordedYearOfBirthImpl tests updated (3)

src/main/java/scot/carricksoftware/grants/services/census/censusentry/UpdateRecordedYearOfBirthImpl.java

@@ -63,13 +63,8 @@ public class UpdateRecordedYearOfBirthImpl implements UpdateRecordedYearOfBirth
             personCommand.setRecordedYearOfBirth(dateString);
             personService.savePersonCommand(personCommand);
         } else {
-            logNoCommandError();
+            logger.info(" -- Person Command is null.");
         }
     }
 
-    private void logNoCommandError() {
-        throw new NullPointerException("Person Command is null.");
-    }
-
-
 }

src/test/java/scot/carricksoftware/grants/services/census/censusentry/UpdateRecordedYearNullCommandTest.java

@@ -0,0 +1,64 @@
+/*
+ * Copyright (c) 2025.  Andrew Grant Carrick Software. All rights reserved
+ *
+ */
+
+package scot.carricksoftware.grants.services.census.censusentry;
+
+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 org.springframework.boot.test.system.CapturedOutput;
+import org.springframework.boot.test.system.OutputCaptureExtension;
+import scot.carricksoftware.grants.commands.census.CensusEntryCommand;
+import scot.carricksoftware.grants.converters.people.PersonConverter;
+import scot.carricksoftware.grants.domains.census.Census;
+import scot.carricksoftware.grants.domains.people.Person;
+import scot.carricksoftware.grants.services.people.PersonService;
+
+import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.when;
+import static scot.carricksoftware.grants.enums.census.CensusDate.CENSUS_1881;
+
+@ExtendWith(OutputCaptureExtension.class)
+@ExtendWith(MockitoExtension.class)
+class UpdateRecordedYearNullCommandTest {
+
+    private UpdateRecordedYearOfBirth updateRecordedYearOfBirth;
+
+    @Mock
+    private Person personMock;
+
+    @Mock
+    private Census censusMock;
+
+    @Mock
+    private PersonService personServiceMock;
+
+    @Mock
+    private CensusEntryCommand censusEntryCommandMock;
+
+    @Mock
+    private PersonConverter personConverterMock;
+
+    @BeforeEach
+    void setUp() {
+        updateRecordedYearOfBirth = new UpdateRecordedYearOfBirthImpl(personConverterMock, personServiceMock);
+        when(censusEntryCommandMock.getPerson()).thenReturn(personMock);
+        when(censusEntryCommandMock.getCensus()).thenReturn(censusMock);
+        when(censusEntryCommandMock.getAge()).thenReturn("80");
+        when(censusMock.getCensusDate()).thenReturn(CENSUS_1881);
+    }
+
+    @Test
+    public void nullCommandTest(CapturedOutput capturedOutput) {
+        when(personConverterMock.convert(any())).thenReturn(null);
+        when(censusEntryCommandMock.getBirthYear()).thenReturn("");
+        updateRecordedYearOfBirth.updateRecordedYearOfBirth(censusEntryCommandMock);
+        assertTrue(capturedOutput.getOut().contains("Person Command is null"));
+    }
+
+}