Refactored log capture and formatting code in org.openqa.selenium.tes… (#13322)

* Refactored log capture and formatting code in org.openqa.selenium.testing and org.openqa.selenium.grid.log packages.

* Format files

---------

Co-authored-by: Diego Molina <[email protected]>
Co-authored-by: Diego Molina <[email protected]>

Author: manuelsblanco

java/src/org/openqa/selenium/grid/log/TerseFormatter.java

@@ -41,7 +41,7 @@ public class TerseFormatter extends Formatter {
    * Line separator string. This is the value of the line.separator property at the moment that the
    * TerseFormatter was created.
    */
-  private final String lineSeparator = System.getProperty("line.separator");
+  private final String lineSeparator = System.lineSeparator();
 
   /*
    * DGF - These have to be compile time constants to be used with switch
@@ -78,7 +78,12 @@ public synchronized String format(final LogRecord record) {
     buffer.append(' ');
     buffer.append(levelNumberToCommonsLevelName(record.getLevel()));
     String[] parts = record.getSourceClassName().split("\\.");
-    buffer.append(" [" + parts[parts.length - 1] + "." + record.getSourceMethodName() + "]");
+    buffer
+        .append(" [")
+        .append(parts[parts.length - 1])
+        .append(".")
+        .append(record.getSourceMethodName())
+        .append("]");
     buffer.append(SUFFIX);
     buffer.append(formatMessage(record)).append(lineSeparator);
     if (record.getThrown() != null) {

java/test/org/openqa/selenium/testing/CaptureLoggingRule.java

@@ -135,7 +135,7 @@ public String format(LogRecord record) {
             .append("]");
       }
       buffer.append(" - ");
-      buffer.append(formatMessage(record)).append(System.getProperty("line.separator"));
+      buffer.append(formatMessage(record)).append(System.lineSeparator());
       if (record.getThrown() != null) {
         final StringWriter trace = new StringWriter();
         record.getThrown().printStackTrace(new PrintWriter(trace));