Refactoring

ehmicky · Sep 5, 2021 · de18598 · de18598
1 parent 568200c
commit de18598
Showing 1 changed file with 11 additions and 16 deletions.
diff --git a/src/report/reporters/boxplot/main.js b/src/report/reporters/boxplot/main.js
@@ -179,28 +179,23 @@ const getBottomNewlines = function (mini) {
 
 const LABELS_HEIGHT = 1
 
-// Retrieve the blocks that show the min|max on the left|right
-const getMinMaxBlock = function (quantiles, statName) {
-  return getStat(quantiles, statName)
-}
-
 // Retrieve the width of those blocks
 const getMinMaxBlockWidth = function (combinations, statName) {
-  const combinationsA = combinations.filter(isMeasuredCombination)
-  return combinationsA.length === 0
-    ? 0
-    : Math.max(
-        ...combinationsA.map((combination) =>
-          getCombinationWidth(combination, statName),
-        ),
-      )
+  return Math.max(
+    ...combinations.map((combination) =>
+      getSingleMinMaxWidth(combination, statName),
+    ),
+  )
 }
 
-const getCombinationWidth = function ({ quantiles }, statName) {
-  return stringWidth(getStat(quantiles, statName))
+const getSingleMinMaxWidth = function ({ quantiles }, statName) {
+  return quantiles === undefined
+    ? 0
+    : addPadding(quantiles[statName].prettyPadded).length
 }
 
-const getStat = function (quantiles, statName) {
+// Retrieve the blocks that show the min|max on the left|right
+const getMinMaxBlock = function (quantiles, statName) {
   return addPadding(quantiles[statName].prettyPaddedColor)
 }