From a08237124db56d129d820853e0a670580de080bf Mon Sep 17 00:00:00 2001
From: peterpaulo-azion <peterson.silva@azion.com>
Date: Thu, 13 Feb 2025 17:02:23 -0300
Subject: [PATCH] refactor: improve data volume formatting with new calculation
 method

---
 .../chart/format-c3-graph-props.js            | 51 +++++++++----------
 1 file changed, 25 insertions(+), 26 deletions(-)

diff --git a/src/modules/real-time-metrics/chart/format-c3-graph-props.js b/src/modules/real-time-metrics/chart/format-c3-graph-props.js
index a3d787a8f..3c4780d94 100644
--- a/src/modules/real-time-metrics/chart/format-c3-graph-props.js
+++ b/src/modules/real-time-metrics/chart/format-c3-graph-props.js
@@ -130,6 +130,22 @@ function formatPercentageDataUnit(data) {
   }).format(data / 100)
 }
 
+function calculateValueAndUnit(data, unit) {
+  if (data > CHART_RULES.DATA_VOLUME.tera) {
+    return { value: data / CHART_RULES.DATA_VOLUME.tera, formattedUnit: `tera${unit}` }
+  }
+  if (data > CHART_RULES.DATA_VOLUME.giga) {
+    return { value: data / CHART_RULES.DATA_VOLUME.giga, formattedUnit: `giga${unit}` }
+  }
+  if (data > CHART_RULES.DATA_VOLUME.mega) {
+    return { value: data / CHART_RULES.DATA_VOLUME.mega, formattedUnit: `mega${unit}` }
+  }
+  if (data > CHART_RULES.DATA_VOLUME.kilo) {
+    return { value: data / CHART_RULES.DATA_VOLUME.kilo, formattedUnit: `kilo${unit}` }
+  }
+  return { value: data, formattedUnit: unit }
+}
+
 /**
  * Format data for displaying byte unit
  * @param {number} data - The data to be formatted
@@ -137,39 +153,22 @@ function formatPercentageDataUnit(data) {
  * @returns {string} - Returns the formatted data for byte unit display
  */
 function formatBytesDataUnit(data, chartData) {
-  let value = data
-  let unit = 'byte'
+  const unit = chartData.dataUnit === 'bitsPerSecond' ? 'bit-per-second' : 'byte'
 
-  if (chartData.dataUnit === 'bitsPerSecond') {
-    unit = 'bit-per-second'
-  }
+  const { value, formattedUnit } = calculateValueAndUnit(Math.abs(data), unit)
 
-  if (data > CHART_RULES.DATA_VOLUME.tera) {
-    value = data / CHART_RULES.DATA_VOLUME.tera
-    unit = `tera${unit}`
-  } else if (data > CHART_RULES.DATA_VOLUME.giga) {
-    value = data / CHART_RULES.DATA_VOLUME.giga
-    unit = `giga${unit}`
-  } else if (data > CHART_RULES.DATA_VOLUME.mega) {
-    value = data / CHART_RULES.DATA_VOLUME.mega
-    unit = `mega${unit}`
-  } else if (data > CHART_RULES.DATA_VOLUME.kilo) {
-    value = data / CHART_RULES.DATA_VOLUME.kilo
-    unit = `kilo${unit}`
-  }
-
-  /**
-   * Formatter for byte display
-   */
-  const byteValueNumberFormatter = Intl.NumberFormat('en', {
+  const formattedData = Intl.NumberFormat('en', {
     notation: 'compact',
     style: 'unit',
-    unit,
+    unit: formattedUnit,
     unitDisplay: 'narrow',
     minimumFractionDigits: CHART_RULES.TO_FIXED_DATA_VOLUME,
     maximumFractionDigits: CHART_RULES.TO_FIXED_DATA_VOLUME
-  })
-  return byteValueNumberFormatter.format(value)
+  }).format(value)
+
+  const isNegativeData = data < 0
+
+  return isNegativeData ? `-${formattedData}` : formattedData
 }
 
 /**