Omit span junctions when row separators are disabled

src/cell.js

@@ -164,6 +164,17 @@ class Cell {
    * @returns {String}
    */
   drawTop(drawRight) {
+    // Spanning cells can use top/bottom junctions for internal row separators.
+    // Do not let those junctions restore a separator that was explicitly hidden.
+    if (
+      this.y > 0 &&
+      this.chars.mid === '' &&
+      this.chars.leftMid === '' &&
+      this.chars.midMid === '' &&
+      this.chars.rightMid === ''
+    ) {
+      return '';
+    }
     let content = [];
     if (this.cells) {
       //TODO: cells should always exist - some tests don't fill it in though

test/issues/355-test.js

@@ -0,0 +1,23 @@
+const Table = require('../..');
+
+describe('empty row separators with column spans', () => {
+  const options = {
+    chars: { mid: '', 'left-mid': '', 'mid-mid': '', 'right-mid': '' },
+    colWidths: [5, 5],
+    style: { head: [], border: [] },
+  };
+
+  test('does not draw a junction below a spanning cell', () => {
+    const table = new Table(options);
+    table.push([{ content: 'wide', colSpan: 2 }], ['one', 'two']);
+
+    expect(table.toString()).toEqual(['┌───────────┐', '│ wide      │', '│ one │ two │', '└─────┴─────┘'].join('\n'));
+  });
+
+  test('does not draw a junction above a spanning cell', () => {
+    const table = new Table(options);
+    table.push(['one', 'two'], [{ content: 'wide', colSpan: 2 }]);
+
+    expect(table.toString()).toEqual(['┌─────┬─────┐', '│ one │ two │', '│ wide      │', '└───────────┘'].join('\n'));
+  });
+});