Checkstyle fixes (#302)

* Update HAPI FHIR R4 to version `8.0.0`

* Update gradle resources to version `8.0.0`
* Breaking model changes now means that the `Parameter.getParameter(name)` now returns a `Type` which cannot now be cast to a `StringType`. We can work around this by finding the `ParametersParameterComponent` manually and then casting it to retrieve the value.

* Update HAPI FHIR R4 to version `8.0.0`

* Due to Parameter changes in the latest version, there is now a slightly different error phrasing in the error returned when a parameter is missing or empty.  Integration tests have been updated to reflect this.

* Update DeductionIntegrationTest.java

adjust modifiers to private

* * Address checkstyle [FinalClass] issue (Class <ClassName> should be declared as final).

* * Address checkstyle [MethodName] issue (Name `<TestName>` must match pattern `<Pattern>`.)

* * Address checkstyle [ModifierOrder] issue (`<Modifier>` modifier out of order with JLS suggestions.)

* * Address missed [MethodName] issue.

* * Address missed [ModifierOrder] issue.

* * Address Line is longer than 140 character issues.

* * Address [WhitespaceAfter / WhitespaceAround] checkstyle issue (`if` is not followed by whitespace.)

* * Address missed Line Length checkstyle issue.

* * Address [WhitespaceAround] checkstyle issue (`<character>` is not <preceded with / followed by> whitespace.)
* Address [IndentationLevel] checkstyle issues

* * Address [UnusedImports] checkstyle issues.

* * Address [RedundantModifier] CheckStyle issues (Redundant `<Modifier>` modifier.)

* * Address [LocalVariableName] Checkstyle issues (Name `<LocalVariableName>` must match pattern '^([a-z][a-zA-Z0-9]*|_)$'.)
* Address [MemberName] Checkstyle issues (Name `<LocalVariableName>` must match pattern '^([a-z][a-zA-Z0-9]*|_)$'.)

Author: MartinWheelerMT

src/intTest/java/uk/nhs/digital/nhsconnect/nhais/container/ActiveMqContainer.java

@@ -5,7 +5,7 @@
 
 import java.nio.file.Path;
 
-public class ActiveMqContainer extends GenericContainer<ActiveMqContainer> {
+public final class ActiveMqContainer extends GenericContainer<ActiveMqContainer> {
 
     public static final int ACTIVEMQ_PORT = 5672;
     private static ActiveMqContainer container;

src/intTest/java/uk/nhs/digital/nhsconnect/nhais/container/FakeMeshContainer.java

@@ -7,7 +7,7 @@
 import java.nio.file.Path;
 
 @Slf4j
-public class FakeMeshContainer extends GenericContainer<FakeMeshContainer> {
+public final class FakeMeshContainer extends GenericContainer<FakeMeshContainer> {
 
     public static final int FAKE_MESH_PORT = 8829;
     private static FakeMeshContainer container;

src/intTest/java/uk/nhs/digital/nhsconnect/nhais/container/MongoDbContainer.java

@@ -4,7 +4,7 @@
 import org.testcontainers.containers.GenericContainer;
 
 @Slf4j
-public class MongoDbContainer extends GenericContainer<MongoDbContainer> {
+public final class MongoDbContainer extends GenericContainer<MongoDbContainer> {
 
     public static final int MONGODB_PORT = 27017;
     public static final String DEFAULT_IMAGE_AND_TAG = "mongo:8.0";

src/intTest/java/uk/nhs/digital/nhsconnect/nhais/db/TimeToLiveTest.java

@@ -48,13 +48,13 @@ public class TimeToLiveTest {
     private NhaisMongoClientConfiguration mongoConfig;
 
     @Test
-    void when_ApplicationStarts_then_TtlIndexExistsForInboundStateWithValueFromConfiguration() {
+    void When_ApplicationStarts_Expect_TtlIndexExistsForInboundStateWithValueFromConfiguration() {
         var indexOperations = mongoTemplate.indexOps(InboundState.class);
         assertThat(timeToLiveIndexExists(indexOperations)).isTrue();
     }
 
     @Test
-    void when_ApplicationStarts_then_TtlIndexExistsForOutboundStateWithValueFromConfiguration() {
+    void When_ApplicationStarts_Expect_TtlIndexExistsForOutboundStateWithValueFromConfiguration() {
         var indexOperations = mongoTemplate.indexOps(OutboundState.class);
         assertThat(timeToLiveIndexExists(indexOperations)).isTrue();
     }
@@ -70,7 +70,7 @@ private boolean timeToLiveIndexExists(IndexOperations indexOperations) {
 
     @Test
     @Disabled("Long running test that depends on external TTL config, enable when needed")
-    void when_TimeToLiveHasPassedInInboundState_then_documentRemoved() {
+    void When_TimeToLiveHasPassedInInboundState_Expect_DocumentRemoved() {
         var inboundState = new InboundState()
             .setWorkflowId(WorkflowId.RECEP)
             .setSender("some_sender")
@@ -89,7 +89,7 @@ void when_TimeToLiveHasPassedInInboundState_then_documentRemoved() {
 
     @Test
     @Disabled("Long running test that depends on external TTL config, enable when needed")
-    void when_TimeToLiveHasPassedInOutboundState_then_documentRemoved() {
+    void When_TimeToLiveHasPassedInOutboundState_Expect_DocumentRemoved() {
         var inboundState = new OutboundState()
             .setWorkflowId(WorkflowId.RECEP)
             .setSender("some_sender")

src/intTest/java/uk/nhs/digital/nhsconnect/nhais/inbound/DeadLetterQueueTest.java

@@ -37,7 +37,7 @@ public class DeadLetterQueueTest extends IntegrationBaseTest {
     private ObjectMapper objectMapper;
 
     @Test
-    public void whenSendingInvalidMessage_toMeshInboundQueue_thenMessageIsSentToDeadLetterQueue() throws JMSException {
+    public void When_SendingInvalidMessageToMeshInboundQueue_Expect_MessageIsSentToDeadLetterQueue() throws JMSException {
         clearDeadLetterQueue(meshInboundQueueName);
         sendToMeshInboundQueue(MESSAGE_CONTENT);
 
@@ -48,10 +48,15 @@ public void whenSendingInvalidMessage_toMeshInboundQueue_thenMessageIsSentToDead
     }
 
     @Test
-    public void whenMeshOutboundQueueMessageCannotBeProcessed_thenMessageIsSentToDeadLetterQueue() throws Exception {
+    public void When_MeshOutboundQueueMessageCannotBeProcessed_Expect_MessageIsSentToDeadLetterQueue() throws Exception {
         String conversationId = Long.toString(System.currentTimeMillis());
         when(conversationIdService.getCurrentConversationId()).thenReturn(conversationId);
-        OutboundMeshMessage meshMessage = OutboundMeshMessage.create("XX11", WorkflowId.REGISTRATION, MESSAGE_CONTENT, "2020-01-01T00:00:00Z", "asdf");
+        OutboundMeshMessage meshMessage = OutboundMeshMessage.create(
+            "XX11",
+            WorkflowId.REGISTRATION, MESSAGE_CONTENT,
+            "2020-01-01T00:00:00Z",
+            "asdf"
+        );
         doThrow(RuntimeException.class).when(meshClient).authenticate();
 
         clearDeadLetterQueue(meshOutboundQueueName);
@@ -62,5 +67,4 @@ public void whenMeshOutboundQueueMessageCannotBeProcessed_thenMessageIsSentToDea
         assertThat(message.getStringProperty(JmsHeaders.CONVERSATION_ID)).isEqualTo(conversationId);
         assertThat(parseTextMessage(message)).isEqualTo(objectMapper.writeValueAsString(meshMessage));
     }
-
 }

src/intTest/java/uk/nhs/digital/nhsconnect/nhais/inbound/InboundMeshQueueMultiTransactionTest.java

@@ -65,10 +65,10 @@ public class InboundMeshQueueMultiTransactionTest extends IntegrationBaseTest {
     private static final String TRANSACTION_5_OPERATION_ID = OperationId.buildOperationId(RECIPIENT, TN_5);
     private static final String TRANSACTION_6_OPERATION_ID = OperationId.buildOperationId(RECIPIENT, TN_6);
     private static final Instant INTERCHANGE_TRANSLATION_TIMESTAMP = ZonedDateTime
-        .of(2020, 1, 25, 12, 35, 0, 0, TimestampService.UKZone)
+        .of(2020, 1, 25, 12, 35, 0, 0, TimestampService.UK_ZONE)
         .toInstant();
 
-    private static final Instant GENERATED_TIMESTAMP = ZonedDateTime.of(2020, 6, 10, 14, 38, 0, 0, TimestampService.UKZone)
+    private static final Instant GENERATED_TIMESTAMP = ZonedDateTime.of(2020, 6, 10, 14, 38, 0, 0, TimestampService.UK_ZONE)
         .toInstant();
     private static final String ISO_GENERATED_TIMESTAMP = new TimestampService().formatInISO(GENERATED_TIMESTAMP);
 
@@ -103,7 +103,9 @@ void setUp() {
     }
 
     @Test
-    void whenMeshInboundQueueRegistrationMessageIsReceived_thenMessageIsHandled(SoftAssertions softly) throws IOException, JMSException, JSONException {
+    void When_MeshInboundQueueRegistrationMessageIsReceived_Expect_MessageIsHandled(SoftAssertions softly)
+        throws IOException, JMSException, JSONException {
+
         var meshMessage = new MeshMessage()
             .setWorkflowId(WorkflowId.REGISTRATION)
             .setContent(new String(Files.readAllBytes(interchange.getFile().toPath())));
@@ -179,7 +181,7 @@ private void assertGpSystemInboundQueueMessages(
 
         // all transactions come from the same interchange and use the same conversation id
         String conversationId = message.getStringProperty("ConversationId");
-        if(previousConversationId == null) {
+        if (previousConversationId == null) {
             previousConversationId = conversationId;
         }
         softly.assertThat(conversationId).isEqualTo(previousConversationId);

src/intTest/java/uk/nhs/digital/nhsconnect/nhais/inbound/InboundMeshQueueRecepTest.java

@@ -6,7 +6,6 @@
 import org.junit.jupiter.api.Test;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.boot.test.mock.mockito.MockBean;
-import org.springframework.boot.test.mock.mockito.SpyBean;
 import org.springframework.core.io.Resource;
 import org.springframework.test.annotation.DirtiesContext;
 import uk.nhs.digital.nhsconnect.nhais.IntegrationBaseTest;
@@ -41,7 +40,7 @@ public class InboundMeshQueueRecepTest extends IntegrationBaseTest {
     private static final String SENDER = "FHS1";
     private static final String RECIPIENT = "GP05";
     private static final Instant TRANSLATION_TIMESTAMP = ZonedDateTime
-        .of(2020, 6, 20, 14, 0, 0, 0, TimestampService.UKZone)
+        .of(2020, 6, 20, 14, 0, 0, 0, TimestampService.UK_ZONE)
         .toInstant();
     // Mongo only supports millis precision
     private static final Instant PROCESSED_TIMESTAMP = Instant.now().truncatedTo(ChronoUnit.MILLIS);
@@ -65,7 +64,7 @@ void setUp() {
     }
 
     @Test
-    void whenMeshInboundQueueRecepMessageIsReceived_thenRecepHandled(SoftAssertions softly) throws IOException {
+    void When_MeshInboundQueueRecepMessageIsReceived_Expect_RecepHandled(SoftAssertions softly) throws IOException {
         createOutboundStateRecords();
 
         sendToMeshInboundQueue(new MeshMessage()

src/intTest/java/uk/nhs/digital/nhsconnect/nhais/inbound/InboundMeshQueueRegistrationTest.java

@@ -42,9 +42,9 @@ public class InboundMeshQueueRegistrationTest extends IntegrationBaseTest {
     private static final ReferenceTransactionType.Inbound TRANSACTION_TYPE = ReferenceTransactionType.Inbound.APPROVAL;
     private static final String OPERATION_ID = OperationId.buildOperationId(RECIPIENT, TN);
     private static final Instant TRANSLATION_TIMESTAMP = ZonedDateTime
-        .of(2020, 1, 25, 12, 35, 0, 0, TimestampService.UKZone)
+        .of(2020, 1, 25, 12, 35, 0, 0, TimestampService.UK_ZONE)
         .toInstant();
-    private static final Instant GENERATED_TIMESTAMP = ZonedDateTime.of(2020, 6, 10, 14, 38, 00, 0, TimestampService.UKZone)
+    private static final Instant GENERATED_TIMESTAMP = ZonedDateTime.of(2020, 6, 10, 14, 38, 00, 0, TimestampService.UK_ZONE)
         .toInstant();
     private static final String ISO_GENERATED_TIMESTAMP = new TimestampService().formatInISO(GENERATED_TIMESTAMP);
 
@@ -66,7 +66,9 @@ void setUp() {
     }
 
     @Test
-    void whenMeshInboundQueueRegistrationMessageIsReceived_thenMessageIsHandled(SoftAssertions softly) throws IOException, JMSException, JSONException {
+    void When_MeshInboundQueueRegistrationMessageIsReceived_Expect_MessageIsHandled(SoftAssertions softly)
+        throws IOException, JMSException, JSONException {
+
         var meshMessage = new MeshMessage()
             .setWorkflowId(WorkflowId.REGISTRATION)
             .setContent(new String(Files.readAllBytes(interchange.getFile().toPath())))

src/intTest/java/uk/nhs/digital/nhsconnect/nhais/inbound/InboundStateRepositoryTest.java

@@ -26,7 +26,7 @@ public class InboundStateRepositoryTest {
     private InboundStateRepository inboundStateRepository;
 
     @Test
-    void whenDuplicateInterchangeInboundStateInserted_thenThrowsException() {
+    void When_DuplicateInterchangeInboundStateInserted_Expect_ThrowsException() {
         var inboundState = new InboundState()
             .setWorkflowId(WorkflowId.REGISTRATION)
             .setSender("some_sender")
@@ -44,7 +44,7 @@ void whenDuplicateInterchangeInboundStateInserted_thenThrowsException() {
     }
 
     @Test
-    void whenDuplicateRecepInboundStateInserted_thenThrowsException() {
+    void When_DuplicateRecepInboundStateInserted_Expect_ThrowsException() {
         var inboundState = new InboundState()
             .setWorkflowId(WorkflowId.RECEP)
             .setSender("some_sender")

src/intTest/java/uk/nhs/digital/nhsconnect/nhais/inbound/InboundUserAcceptanceTest.java

@@ -103,7 +103,9 @@ private void verifyThatCloseQuarterNotificationIsNotPresentOnGpSystemInboundQueu
         assertThat(gpSystemInboundQueueMessage).isNull();
     }
 
-    private void verifyThatNonCloseQuarterNotificationMessageIsTranslated(TestData testData, Message gpSystemInboundQueueMessage) throws JMSException, JSONException {
+    private void verifyThatNonCloseQuarterNotificationMessageIsTranslated(TestData testData, Message gpSystemInboundQueueMessage)
+        throws JMSException, JSONException {
+
         assertThat(gpSystemInboundQueueMessage).isNotNull();
         // assert transaction type in JMS header is correct
         assertMessageHeaders(gpSystemInboundQueueMessage, "fp69_prior_notification");
@@ -130,19 +132,25 @@ private void assertOutboundRecepMessage(String recep) {
         List<String> messageIds = waitFor(() -> {
             List<String> inboxMessageIds = nhaisMeshClient.getInboxMessageIds();
             return inboxMessageIds.isEmpty() ? null : inboxMessageIds;
-        } );
+        });
         var meshMessage = nhaisMeshClient.getEdifactMessage(messageIds.get(0));
 
         Interchange expectedRecep = edifactParser.parse(recep);
         Interchange actualRecep = edifactParser.parse(meshMessage.getContent());
 
-        assertThat(meshMessage.getWorkflowId()).isEqualTo(WorkflowId.RECEP);
-        assertThat(actualRecep.getInterchangeHeader().getRecipient()).isEqualTo(expectedRecep.getInterchangeHeader().getRecipient());
-        assertThat(actualRecep.getInterchangeHeader().getSender()).isEqualTo(expectedRecep.getInterchangeHeader().getSender());
-        assertThat(actualRecep.getInterchangeHeader().getSequenceNumber()).isEqualTo(expectedRecep.getInterchangeHeader().getSequenceNumber());
+        assertThat(meshMessage.getWorkflowId())
+            .isEqualTo(WorkflowId.RECEP);
+        assertThat(actualRecep.getInterchangeHeader().getRecipient())
+            .isEqualTo(expectedRecep.getInterchangeHeader().getRecipient());
+        assertThat(actualRecep.getInterchangeHeader().getSender())
+            .isEqualTo(expectedRecep.getInterchangeHeader().getSender());
+        assertThat(actualRecep.getInterchangeHeader().getSequenceNumber())
+            .isEqualTo(expectedRecep.getInterchangeHeader().getSequenceNumber());
         assertThat(filterTimestampedSegments(actualRecep)).containsExactlyElementsOf(filterTimestampedSegments(expectedRecep));
-        assertThat(actualRecep.getInterchangeTrailer().getNumberOfMessages()).isEqualTo(expectedRecep.getInterchangeTrailer().getNumberOfMessages());
-        assertThat(actualRecep.getInterchangeTrailer().getSequenceNumber()).isEqualTo(expectedRecep.getInterchangeTrailer().getSequenceNumber());
+        assertThat(actualRecep.getInterchangeTrailer().getNumberOfMessages())
+            .isEqualTo(expectedRecep.getInterchangeTrailer().getNumberOfMessages());
+        assertThat(actualRecep.getInterchangeTrailer().getSequenceNumber())
+            .isEqualTo(expectedRecep.getInterchangeTrailer().getSequenceNumber());
     }
 
     private List<String> filterTimestampedSegments(Interchange recep) {