Skip to content

Commit e70373b

Browse files
* Add highlight to console log output to make transform tool easier to read.
* Catch validation errors.
1 parent e907b57 commit e70373b

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

service/src/main/java/uk/nhs/adaptors/gp2gp/transformjsontoxmltool/TransformJsonToXml.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
import lombok.RequiredArgsConstructor;
3131
import lombok.extern.slf4j.Slf4j;
3232
import uk.nhs.adaptors.gp2gp.common.service.FhirParseService;
33+
import uk.nhs.adaptors.gp2gp.ehr.exception.EhrValidationException;
3334
import uk.nhs.adaptors.gp2gp.ehr.mapper.EhrExtractMapper;
3435
import uk.nhs.adaptors.gp2gp.ehr.mapper.MessageContext;
3536
import uk.nhs.adaptors.gp2gp.ehr.mapper.OutputMessageWrapperMapper;
@@ -65,12 +66,12 @@ public void run(String... args) {
6566
getFiles().forEach(file -> {
6667
String xmlResult = mapJsonToXml(file.getJsonFileInput());
6768
writeToFile(xmlResult, file.getJsonFileName());
68-
LOGGER.info("Successfully validated XML for file: {}", file.getJsonFileName());
6969
});
7070
} catch (NHSNumberNotFound | UnreadableJsonFileException | NoJsonFileFound | Hl7TranslatedResponseError e) {
71-
LOGGER.error("error: {}", e.getMessage());
71+
LOGGER.error("Failed to parse the provided JSON: {}", e.getMessage());
72+
} catch (EhrValidationException e) {
73+
LOGGER.error("Failed to validate the produced XML");
7274
}
73-
LOGGER.info("end");
7475
}
7576

7677
private List<InputFile> getFiles() throws UnreadableJsonFileException, NoJsonFileFound {

service/src/main/resources/logback.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
<configuration>
22
<appender name="TEXT" class="ch.qos.logback.core.ConsoleAppender">
33
<encoder>
4-
<pattern>${GP2GP_LOGGING_FORMAT:-%d{yyyy-MM-dd HH:mm:ss.SSS} Level=%-5level Logger=%logger{36} ConversationId=%X{ConversationId} TaskId=%X{TaskId} Thread="%thread" Message="%msg"%n}
4+
<pattern>${GP2GP_LOGGING_FORMAT:-%d{yyyy-MM-dd HH:mm:ss.SSS} %highlight(%-5level) Level=%-5level Logger=%logger{36} ConversationId=%X{ConversationId} TaskId=%X{TaskId} Thread="%thread" Message="%msg"%n}
55
</pattern>
66
</encoder>
77
</appender>

0 commit comments

Comments
 (0)