Fixed wrong symbols count rounding.

Issue #5
theme-next · Jan 22, 2018 · eca327d · eca327d
1 parent f92312e
commit eca327d
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 9 deletions.
diff --git a/lib/helper.js b/lib/helper.js
@@ -19,9 +19,12 @@ var getFormatTime = function(minutes) {
 
 module.exports.symbolsCount = function(content) {
   var symbolsResult = getSymbols(content);
-  return symbolsResult < 1024
-    ? symbolsResult // < 999 => 100
-    : Math.round(((symbolsResult / 1000) * 10) / 10) + 'k'; // > 999 => 1.1k
+  if (symbolsResult > 9999) {
+    symbolsResult = Math.round(symbolsResult / 1000) + 'k'; // > 9999 => 11k
+  } else if (symbolsResult > 999) {
+    symbolsResult = Math.round(symbolsResult / 100) / 10 + 'k'; // > 999 => 1.1k
+  } // < 999 => 111
+  return symbolsResult;
 };
 
 module.exports.symbolsTime = function(content, awl, wpm) {
@@ -41,9 +44,9 @@ var getSymbolsTotal = function(site) {
 
 module.exports.symbolsCountTotal = function(site) {
   var symbolsResultTotal = getSymbolsTotal(site);
-  return symbolsResultTotal < 1024024
-    ? Math.round(symbolsResultTotal / 1000) + 'k' // < 999k => 100k
-    : Math.round(((symbolsResultTotal / 1000000 * 10)) / 10) + 'm'; // > 999k => 1.1m
+  return symbolsResultTotal < 1000000
+    ? Math.round(symbolsResultTotal / 1000) + 'k' // < 999k => 111k
+    : Math.round(symbolsResultTotal / 100000) / 10 + 'm'; // > 999k => 1.1m
 };
 
 module.exports.symbolsTimeTotal = function(site, awl, wpm) {

diff --git a/test/index.js b/test/index.js