fix: do not draw new line on the top (borderless) (#102)

gajus · Aug 15, 2019 · 117f902 · 117f902
1 parent 35f360d
commit 117f902
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/src/drawBorder.js b/src/drawBorder.js
@@ -35,12 +35,18 @@ const drawBorder = (columnSizeIndex, parts) => {
  * @returns {string}
  */
 const drawBorderTop = (columnSizeIndex, parts) => {
-  return drawBorder(columnSizeIndex, {
+  const border = drawBorder(columnSizeIndex, {
     body: parts.topBody,
     join: parts.topJoin,
     left: parts.topLeft,
     right: parts.topRight
   });
+
+  if (border === '\n') {
+    return '';
+  }
+
+  return border;
 };
 
 /**

diff --git a/test/drawBorder.js b/test/drawBorder.js
@@ -38,6 +38,19 @@ describe('drawBorderTop', () => {
     expect(drawBorderTop([1, 1], parts)).to.equal('╔═╤═╗\n');
     expect(drawBorderTop([5, 10], parts)).to.equal('╔═════╤══════════╗\n');
   });
+
+  it('no leading new line if borderless', () => {
+    const parts = {
+      topLeft: '',
+      topRight: '',
+      topBody: '',
+      topJoin: ''
+    };
+
+    expect(drawBorderTop([1], parts)).to.equal('');
+    expect(drawBorderTop([1, 1], parts)).to.equal('');
+    expect(drawBorderTop([5, 10], parts)).to.equal('');
+  });
 });
 
 describe('drawBorderJoin', () => {