#52 Don't interpret "\n" as newline, but escape it with '\'

Author: Bytekeeper

src/main/java/org/stt/persistence/stt/STTItemConverter.java

@@ -29,7 +29,7 @@ private String unescape(String activity) {
                 if (chars[i] == 'n') {
                     b.append('\n');
                 } else {
-                    b.append('\\').append(chars[i]);
+                    b.append(chars[i]);
                 }
             } else {
                 b.append(next);
@@ -99,6 +99,8 @@ private void escape(StringBuilder b, String activity) {
                 }
             } else if (next == '\n') {
                 b.append("\\n");
+            } else if (next == '\\') {
+                b.append("\\\\");
             } else {
                 b.append(next);
             }

src/test/java/org/stt/persistence/stt/STTItemConverterTest.java

@@ -11,6 +11,7 @@
 /**
  */
 public class STTItemConverterTest {
+    private static final LocalDateTime LDT_FOR_TEST = LocalDateTime.of(2019, 1, 10, 10, 11, 12);
     private STTItemConverter sut = new STTItemConverter();
 
     @Test(timeout = 10000)
@@ -63,8 +64,7 @@ public void shouldParseStartWithOpenEnd() {
         TimeTrackingItem item = sut.lineToTimeTrackingItem(lineToTest);
 
         // THEN
-        assertThat(item, is(new TimeTrackingItem("Some Activity",
-                LocalDateTime.of(2019, 1, 10, 10, 11, 12))));
+        assertThat(item, is(new TimeTrackingItem("Some Activity", LDT_FOR_TEST)));
     }
 
     @Test
@@ -76,8 +76,32 @@ public void shouldParseMultilineActivity() {
         TimeTrackingItem item = sut.lineToTimeTrackingItem(lineToTest);
 
         // THEN
-        assertThat(item, is(new TimeTrackingItem("Some\nActivity",
-                LocalDateTime.of(2019, 1, 10, 10, 11, 12))));
+        assertThat(item, is(new TimeTrackingItem("Some\nActivity", LDT_FOR_TEST)));
+    }
+
+    @Test
+    public void shouldIgnoreEscapedBackslash() {
+        // GIVEN
+        String lineToTest = "2019-01-10_10:11:12 Some\\\\nActivity";
+
+        // WHEN
+        TimeTrackingItem item = sut.lineToTimeTrackingItem(lineToTest);
+
+        // THEN
+        assertThat(item, is(new TimeTrackingItem("Some\\nActivity", LDT_FOR_TEST)));
     }
 
+    @Test
+    public void shouldEscapeBackslash() {
+        // GIVEN
+        TimeTrackingItem activity = new TimeTrackingItem("\\n", LDT_FOR_TEST);
+
+        // WHEN
+        String line = sut.timeTrackingItemToLine(activity);
+
+        // THEN
+        assertThat(line, is("2019-01-10_10:11:12 \\\\n"));
+    }
+
+
 }