fix: Jackson YAML mapper no longer add new lines (#117)

Author: brian-mulier

tzatziki-jackson/src/main/java/com/decathlon/tzatziki/utils/JacksonMapper.java

@@ -7,6 +7,7 @@
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.SerializationFeature;
 import com.fasterxml.jackson.dataformat.yaml.YAMLFactory;
+import com.fasterxml.jackson.dataformat.yaml.YAMLGenerator;
 import com.fasterxml.jackson.datatype.jdk8.Jdk8Module;
 import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule;
 import com.google.common.collect.Lists;
@@ -28,7 +29,7 @@ public class JacksonMapper implements MapperDelegate {
             .disable(SerializationFeature.FAIL_ON_EMPTY_BEANS)
             .enable(DeserializationFeature.ACCEPT_EMPTY_STRING_AS_NULL_OBJECT);
 
-    private static ObjectMapper yaml = configurator.apply(new ObjectMapper(new YAMLFactory()));
+    private static ObjectMapper yaml = configurator.apply(new ObjectMapper(YAMLFactory.builder().disable(YAMLGenerator.Feature.SPLIT_LINES).build()));
     private static ObjectMapper json = configurator.apply(new ObjectMapper());
     private static ObjectMapper nonDefaultJson = json.copy()
             .setSerializationInclusion(JsonInclude.Include.NON_DEFAULT);
@@ -113,8 +114,8 @@ private static String toJson(Object object, ObjectMapper objectMapper) throws Js
 
     @SneakyThrows
     public String toYaml(Object object) {
-        if (object instanceof String) {
-            return (String) object;
+        if (object instanceof String objectStr) {
+            return objectStr;
         }
         return yaml.writeValueAsString(object).replaceAll("^---\n?", "");
     }

tzatziki-logback/src/test/resources/com/decathlon/tzatziki/steps/logger.feature

@@ -56,7 +56,7 @@ Feature: to interact with the logger
       """
     Then if <level> == INFO => the logs contain:
       """
-      - ?e .*"message":"some [^ ]+ lines that should be there","logger_name":"com.decathlon.tzatziki.steps.LoggerSteps","thread_name":"main","level":"ERROR".*
+      - ?e  *.*"message":"some [^ ]+ lines that should be there","logger_name":"com.decathlon.tzatziki.steps.LoggerSteps","thread_name":"main","level":"ERROR".*
       """
     But if <level> == OFF => the logs are empty