From 9a344ce5d234225f94ca626d87175210d4fd1d28 Mon Sep 17 00:00:00 2001
From: nam-hle <hoangnamb1.288@gmail.com>
Date: Mon, 19 Apr 2021 09:35:16 +0700
Subject: [PATCH] feat: optimize text truncation (#148)

Co-authored-by: Nam Hoang Le <nam.hoang.le@mgm-tp.com>
---
 .README/usage/text_truncation.md     |  2 +-
 src/truncateTableData.js             |  1 +
 test/README/usage/text_truncating.js |  2 +-
 test/truncateTableData.js            | 47 ++++++++++++++++++++++++++--
 4 files changed, 47 insertions(+), 5 deletions(-)

diff --git a/.README/usage/text_truncation.md b/.README/usage/text_truncation.md
index 7b3a060..c50c26f 100644
--- a/.README/usage/text_truncation.md
+++ b/.README/usage/text_truncation.md
@@ -33,6 +33,6 @@ console.log(output);
 ║ t amet, consectetur  ║
 ║ adipiscing elit. Pha ║
 ║ sellus pulvinar nibh ║
-║ sed mauris conva...  ║
+║ sed mauris convall…  ║
 ╚══════════════════════╝
 ```
diff --git a/src/truncateTableData.js b/src/truncateTableData.js
index e78b7ec..1385443 100644
--- a/src/truncateTableData.js
+++ b/src/truncateTableData.js
@@ -11,6 +11,7 @@ export default (rows, config) => {
     return cells.map((content, index) => {
       return truncate(content, {
         length: config.columns[index].truncate,
+        omission: '…',
       });
     });
   });
diff --git a/test/README/usage/text_truncating.js b/test/README/usage/text_truncating.js
index 1223027..b1d534d 100644
--- a/test/README/usage/text_truncating.js
+++ b/test/README/usage/text_truncating.js
@@ -26,7 +26,7 @@ describe('README.md usage/', () => {
 ║ t amet, consectetur  ║
 ║ adipiscing elit. Pha ║
 ║ sellus pulvinar nibh ║
-║ sed mauris conva...  ║
+║ sed mauris convall…  ║
 ╚══════════════════════╝
         `);
   });
diff --git a/test/truncateTableData.js b/test/truncateTableData.js
index e244dfa..1dfa56c 100644
--- a/test/truncateTableData.js
+++ b/test/truncateTableData.js
@@ -22,7 +22,7 @@ describe('truncateTableData', () => {
 
         expect(truncateTableData(rows, makeConfig(rows, {columnDefault: {
           truncate: 20,
-        }}))).to.deep.equal([['a'.repeat(17) + '...']]);
+        }}))).to.deep.equal([['a'.repeat(19) + '…']]);
       });
     });
 
@@ -39,7 +39,7 @@ describe('truncateTableData', () => {
               truncate: 30,
             },
           },
-        }))).to.deep.equal([['a'.repeat(27) + '...']]);
+        }))).to.deep.equal([['a'.repeat(29) + '…']]);
       });
     });
   });
@@ -57,7 +57,48 @@ describe('truncateTableData', () => {
             truncate: 30,
           },
         },
-      }))).to.deep.equal([['a'.repeat(27) + '...', 'b'.repeat(17) + '...'], ['c'.repeat(27) + '...', 'd'.repeat(17) + '...']]);
+      }))).to.deep.equal([
+        ['a'.repeat(29) + '…', 'b'.repeat(19) + '…'],
+        ['c'.repeat(29) + '…', 'd'.repeat(19) + '…']]);
+    });
+  });
+
+  context('edge cases', () => {
+    context('truncate = 0', () => {
+      it('returns ellipsis only', () => {
+        const rows = [['a'.repeat(100)]];
+        expect(truncateTableData(rows, makeConfig(rows, {
+          columns: {0: {truncate: 0}},
+        }))).to.deep.equal([['…']]);
+      });
+    });
+
+    context('truncate = 1', () => {
+      it('returns ellipsis only', () => {
+        const rows = [['a'.repeat(100)]];
+        expect(truncateTableData(rows, makeConfig(rows, {
+          columns: {0: {truncate: 1}},
+
+        }))).to.deep.equal([['…']]);
+      });
+    });
+
+    context('truncate = 2', () => {
+      it('returns 2-length string with ellipsis', () => {
+        const rows = [['a'.repeat(100)]];
+        expect(truncateTableData(rows, makeConfig(rows, {
+          columns: {0: {truncate: 2}},
+        }))).to.deep.equal([['a…']]);
+      });
+    });
+
+    context('empty string', () => {
+      it('returns empty string', () => {
+        const rows = [['']];
+        expect(truncateTableData(rows, makeConfig(rows, {
+          columns: {0: {truncate: 100}},
+        }))).to.deep.equal([['']]);
+      });
     });
   });
 });