diff --git a/qa/logging-config/src/test/java/org/elasticsearch/common/logging/JsonLoggerTests.java b/qa/logging-config/src/test/java/org/elasticsearch/common/logging/JsonLoggerTests.java
index b484ba90a4da3..1f7479dd3dee5 100644
--- a/qa/logging-config/src/test/java/org/elasticsearch/common/logging/JsonLoggerTests.java
+++ b/qa/logging-config/src/test/java/org/elasticsearch/common/logging/JsonLoggerTests.java
@@ -46,6 +46,7 @@
* It has to be in a org.elasticsearch.common.logging package to use PrefixLogger
*/
public class JsonLoggerTests extends ESTestCase {
+ private static final String LINE_SEPARATOR = System.lineSeparator();
@BeforeClass
public static void initNodeName() {
@@ -109,15 +110,15 @@ public void testPrefixLoggerInJson() throws IOException {
public void testJsonInMessage() throws IOException {
final Logger testLogger = LogManager.getLogger("test");
- String json = "{\n" +
- " \"terms\" : {\n" +
- " \"user\" : [\n" +
- " \"u1\",\n" +
- " \"u2\",\n" +
- " \"u3\"\n" +
- " ],\n" +
- " \"boost\" : 1.0\n" +
- " }\n" +
+ String json = "{" + LINE_SEPARATOR +
+ " \"terms\" : {" + LINE_SEPARATOR +
+ " \"user\" : [" + LINE_SEPARATOR +
+ " \"u1\"," + LINE_SEPARATOR +
+ " \"u2\"," + LINE_SEPARATOR +
+ " \"u3\"" + LINE_SEPARATOR +
+ " ]," + LINE_SEPARATOR +
+ " \"boost\" : 1.0" + LINE_SEPARATOR +
+ " }" + LINE_SEPARATOR +
"}";
testLogger.info(json);
@@ -151,15 +152,15 @@ public void testStacktrace() throws IOException {
public void testJsonInStacktraceMessageIsSplitted() throws IOException {
final Logger testLogger = LogManager.getLogger("test");
- String json = "{\n" +
- " \"terms\" : {\n" +
- " \"user\" : [\n" +
- " \"u1\",\n" +
- " \"u2\",\n" +
- " \"u3\"\n" +
- " ],\n" +
- " \"boost\" : 1.0\n" +
- " }\n" +
+ String json = "{" + LINE_SEPARATOR +
+ " \"terms\" : {" + LINE_SEPARATOR +
+ " \"user\" : [" + LINE_SEPARATOR +
+ " \"u1\"," + LINE_SEPARATOR +
+ " \"u2\"," + LINE_SEPARATOR +
+ " \"u3\"" + LINE_SEPARATOR +
+ " ]," + LINE_SEPARATOR +
+ " \"boost\" : 1.0" + LINE_SEPARATOR +
+ " }" + LINE_SEPARATOR +
"}";
testLogger.error("error message " + json, new Exception(json));
diff --git a/server/src/test/java/org/elasticsearch/common/logging/JsonThrowablePatternConverterTests.java b/server/src/test/java/org/elasticsearch/common/logging/JsonThrowablePatternConverterTests.java
index 2f1fee1208540..111e94eb2b050 100644
--- a/server/src/test/java/org/elasticsearch/common/logging/JsonThrowablePatternConverterTests.java
+++ b/server/src/test/java/org/elasticsearch/common/logging/JsonThrowablePatternConverterTests.java
@@ -32,7 +32,8 @@
import static org.hamcrest.Matchers.equalTo;
public class JsonThrowablePatternConverterTests extends ESTestCase {
- JsonThrowablePatternConverter converter = JsonThrowablePatternConverter.newInstance(null, null);
+ private static final String LINE_SEPARATOR = System.lineSeparator();
+ private JsonThrowablePatternConverter converter = JsonThrowablePatternConverter.newInstance(null, null);
public void testNoStacktrace() throws IOException {
LogEvent event = Log4jLogEvent.newBuilder()
@@ -48,15 +49,15 @@ public void testNoStacktrace() throws IOException {
public void testStacktraceWithJson() throws IOException {
- String json = "{\n" +
- " \"terms\" : {\n" +
- " \"user\" : [\n" +
- " \"u1\",\n" +
- " \"u2\",\n" +
- " \"u3\"\n" +
- " ],\n" +
- " \"boost\" : 1.0\n" +
- " }\n" +
+ String json = "{" + LINE_SEPARATOR +
+ " \"terms\" : {" + LINE_SEPARATOR +
+ " \"user\" : [" + LINE_SEPARATOR +
+ " \"u1\"," + LINE_SEPARATOR +
+ " \"u2\"," + LINE_SEPARATOR +
+ " \"u3\"" + LINE_SEPARATOR +
+ " ]," + LINE_SEPARATOR +
+ " \"boost\" : 1.0" + LINE_SEPARATOR +
+ " }" + LINE_SEPARATOR +
"}";
Exception thrown = new Exception(json);
LogEvent event = Log4jLogEvent.newBuilder()
@@ -73,7 +74,7 @@ public void testStacktraceWithJson() throws IOException {
.findFirst()
.orElseThrow(() -> new AssertionError("no logs parsed"));
- int jsonLength = json.split("\n").length;
+ int jsonLength = json.split(LINE_SEPARATOR).length;
int stacktraceLength = thrown.getStackTrace().length;
assertThat("stacktrace should formatted in multiple lines. JsonLogLine= " + jsonLogLine+" result= "+result,
jsonLogLine.stacktrace().size(), equalTo(jsonLength + stacktraceLength));