Fixed a minor formatting issue for JBehave embedded tables.

serenity-bdd · Jun 2, 2015 · 08aba9b · 08aba9b
1 parent 2e74fdf
commit 08aba9b
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 1 deletion.
diff --git a/build.gradle b/build.gradle
@@ -102,7 +102,7 @@ subprojects {
         if (!JavaVersion.current().isJava8Compatible()) {
             jvmArgs '-XX:MaxPermSize=256m'
         }
-//        maxParallelForks = 2
+        maxParallelForks = 4
     }
 
     task integrationTests(type: Test) {

diff --git a/serenity-core/src/main/java/net/thucydides/core/reports/html/ExampleTable.java b/serenity-core/src/main/java/net/thucydides/core/reports/html/ExampleTable.java
@@ -39,7 +39,9 @@ public ExampleTable(String tableContents) {
 
     public static String stripBracketsFromOuterPipes(String text) {
         text = StringUtils.replace(text, "[|", "|");
+        text = StringUtils.replace(text, "［|", "|");
         text = StringUtils.replace(text,"|]","|");
+        text = StringUtils.replace(text,"|］","|");
         text = StringUtils.replace(text,LEFT_BRACKET + "|","|");
         text = StringUtils.replace(text,"|" + RIGHT_BRACKET,"|");
         return text;

diff --git a/serenity-core/src/test/java/net/thucydides/core/reports/html/WhenFormattingForHTML.java b/serenity-core/src/test/java/net/thucydides/core/reports/html/WhenFormattingForHTML.java
@@ -259,6 +259,16 @@ public void should_convert_embedded_text_tables__with_square_brackets_into_html_
         assertThat(formattedValue, is("A table:<br><table class='embedded'><thead><th>name</th><th>age</th></thead><tbody><tr><td>Bill</td><td>20</td></tr></tbody></table>"));
     }
 
+    @Test
+    public void should_convert_embedded_jbehave_style_tables__with_square_brackets_into_html_tables() {
+        when(issueTracking.getShortenedIssueTrackerUrl()).thenReturn(null);
+        Formatter formatter = new Formatter(issueTracking);
+
+        String formattedValue = formatter.convertAnyTables("Given I have the following pet:\n［|name | status |\n|Fido | available |］");
+
+        assertThat(formattedValue, is("Given I have the following pet:<br><table class='embedded'><thead><th>name</th><th>status</th></thead><tbody><tr><td>Fido</td><td>available</td></tr></tbody></table>"));
+    }
+
 
     @Test
     public void should_ignore_isolated_pipes() {