Escape newline characters in table cells

Resolve #51
ducminh-phan · Jan 29, 2022 · 241ba0d · 241ba0d
1 parent 63fd988
commit 241ba0d
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 1 deletion.
diff --git a/reformat_gherkin/ast_node/table_cell.py b/reformat_gherkin/ast_node/table_cell.py
@@ -10,7 +10,7 @@ def escape_table_cell_value(text: str) -> str:
     are used to separate cells in a row, we need to replace them by `\\|`.
     """
 
-    return text.replace("|", "\\|")
+    return text.replace("|", "\\|").replace("\n", "\\n")
 
 
 @prepare

diff --git a/tests/data/valid/special_characters_in_table_cell/expected_default.feature b/tests/data/valid/special_characters_in_table_cell/expected_default.feature
@@ -0,0 +1,10 @@
+Feature: Breaking gherkin reformatter
+
+  Scenario Outline:
+    Given I want to cause a bug
+    When I try to reformat with an example table with a message like <message>
+    Then gherkin reformatter explodes
+
+    Examples:
+      | message                 |
+      | a\| b\n c\r d\t e\r\n f |
diff --git a/tests/data/valid/special_characters_in_table_cell/input.feature b/tests/data/valid/special_characters_in_table_cell/input.feature
@@ -0,0 +1,10 @@
+Feature: Breaking gherkin reformatter
+
+  Scenario Outline:
+    Given I want to cause a bug
+    When I try to reformat with an example table with a message like <message>
+    Then gherkin reformatter explodes
+
+    Examples:
+      | message                 |
+      | a\| b\n c\r d\t e\r\n f\n |