fix: round subscriber count

fixes #711
fent · Sep 25, 2020 · 4d30a2a · 4d30a2a
1 parent 0ecb0c4
commit 4d30a2a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lib/util.js b/lib/util.js
@@ -375,8 +375,8 @@ exports.parseAbbreviatedNumber = string => {
   if (match) {
     let [, num, multi] = match;
     num = parseFloat(num);
-    return multi === 'M' ? num * 1000000 :
-      multi === 'K' ? num * 1000 : num;
+    return Math.round(multi === 'M' ? num * 1000000 :
+      multi === 'K' ? num * 1000 : num);
   }
   return null;
 };

diff --git a/test/util-test.js b/test/util-test.js
@@ -575,10 +575,11 @@ describe('util.parseAbbreviatedNumber', () => {
   it('Parses abbreviated numbers', () => {
     assert.strictEqual(util.parseAbbreviatedNumber('41K'), 41000);
     assert.strictEqual(util.parseAbbreviatedNumber('1.5M'), 1500000);
+    assert.strictEqual(util.parseAbbreviatedNumber('8.19K '), 8190);
   });
   it('Parses non-abbreviated numbers', () => {
     assert.strictEqual(util.parseAbbreviatedNumber('1234'), 1234);
-    assert.strictEqual(util.parseAbbreviatedNumber('123.456'), 123.456);
+    assert.strictEqual(util.parseAbbreviatedNumber('123.456'), 123);
   });
   it('Returns `null` when given non-number', () => {
     assert.strictEqual(util.parseAbbreviatedNumber('abc'), null);