Ignore separators in code blocks

Author: darkriszty

src/modelFactory/selectionInterpreter.ts

@@ -16,7 +16,7 @@ export class SelectionInterpreter {
             index = -1,
             previousSplitIndex = -1;
         while ((index = line.indexOf("|", index + 1)) > -1) {
-            if (line[index - 1] != "\\") {
+            if (line[index - 1] != "\\" && !this.codeBlockOpenTill(line.substr(0, index))) {
                 result.push(line.substring(previousSplitIndex + 1, index));
                 previousSplitIndex = index;
             }
@@ -25,4 +25,8 @@ export class SelectionInterpreter {
 
         return result;
     }
+
+    private codeBlockOpenTill(text: string): boolean {
+        return (text.match(/`/g) || []).length % 2 != 0;
+    }
 }

test/systemTests/resources/escapeCodeBlock-input.md

@@ -1,6 +1,6 @@
 Type | Range|	Size
 -|-|-
-sbyte |	-128 `|`|to|`|` 127|	Signed 8-bit integer
+sbyte |	-128 `|to|` 127|	Signed 8-bit integer
 byte|	0 `|to|` 255|	Unsigned 8-bit integer
 char|	U+0000 `|to|` U+ffff|	Unicode 16-bit character
 short|	-32,768 `|to|` 32,767|	Signed 16-bit integer

test/unitTests/modelFactory/selectionInterpreter.test.ts

@@ -75,7 +75,7 @@ suite("SelectionInterpreter tests", () => {
         assert.equal(rows[1][1], "v2");
     });
 
-    test("allRows() doesn't consider separator that's in a code block 1", () => {
+    test("allRows() doesn't consider separator that's in a `code block` #1", () => {
         const text = "`h1|h2|h3`";
         const sut = createSut();
 
@@ -86,7 +86,7 @@ suite("SelectionInterpreter tests", () => {
         assert.equal(rows[0][0], "`h1|h2|h3`");
     });
 
-    test("allRows() doesn't consider separator that's in a code block 2", () => {
+    test("allRows() doesn't consider separator that's in a `code block` #2", () => {
         const text = "h1|h2`|still|h2`|h3";
         const sut = createSut();
 
@@ -99,15 +99,15 @@ suite("SelectionInterpreter tests", () => {
         assert.equal(rows[0][2], "h3");
     });
 
-    test("allRows() doesn't consider separator that's in a code block 3", () => {
+    test("allRows() doesn't consider separator that's in a `code block` #3", () => {
         const text = "`h1|h1`|h2|h3`|h3`";
         const sut = createSut();
 
         const rows = sut.allRows(text);
 
         assert.equal(rows.length, 1);
         assert.equal(rows[0].length, 3);
-        assert.equal(rows[0][0], "h1|h1");
+        assert.equal(rows[0][0], "`h1|h1`");
         assert.equal(rows[0][1], "h2");
         assert.equal(rows[0][2], "h3`|h3`");
     });