update vitest assertions and prettier formatting for avg_response_time ms conversion

Signed-off-by: Shoumi <shoumimukherjee@gmail.com>

Author: shoummu1

mcpgateway/static/admin.js

@@ -2842,12 +2842,16 @@ function createTopPerformersTable(entityType, data, isActive) {
         );
         row.appendChild(execCell);
 
-        // Avg Response Time (API returns seconds; convert to ms for display)
+        // Avg Response Time
         const avgTimeCell = document.createElement("td");
         avgTimeCell.className =
             "px-6 py-4 whitespace-nowrap text-sm text-gray-500 dark:text-gray-300 sm:px-6 sm:py-4";
         const avgTime = item.avg_response_time ?? item.avgResponseTime;
-        avgTimeCell.textContent = avgTime != null ? `${Math.round(Number(avgTime) * 1000)}ms` : "N/A";
+        if (avgTime != null) {
+            avgTimeCell.textContent = `${Math.round(Number(avgTime) * 1000)}ms`;
+        } else {
+            avgTimeCell.textContent = "N/A";
+        }
         row.appendChild(avgTimeCell);
 
         // Success Rate
@@ -3192,12 +3196,16 @@ function updateTableRows(tbody, entityType, data, page, perPage) {
         );
         row.appendChild(execCell);
 
-        // Avg Response Time (API returns seconds; convert to ms for display)
+        // Avg Response Time
         const avgTimeCell = document.createElement("td");
         avgTimeCell.className =
             "px-6 py-4 whitespace-nowrap text-sm text-gray-500 dark:text-gray-300 sm:px-6 sm:py-4";
         const avgTime = item.avg_response_time ?? item.avgResponseTime;
-        avgTimeCell.textContent = avgTime != null ? `${Math.round(Number(avgTime) * 1000)}ms` : "N/A";
+        if (avgTime != null) {
+            avgTimeCell.textContent = `${Math.round(Number(avgTime) * 1000)}ms`;
+        } else {
+            avgTimeCell.textContent = "N/A";
+        }
         row.appendChild(avgTimeCell);
 
         // Success Rate
@@ -3250,6 +3258,12 @@ function exportMetricsToCSV(topData) {
     ["tools", "resources", "prompts", "gateways", "servers"].forEach((type) => {
         if (topData[type] && Array.isArray(topData[type])) {
             topData[type].forEach((item, index) => {
+                const avgRespRaw =
+                    item.avg_response_time ?? item.avgResponseTime;
+                const avgRespDisplay =
+                    avgRespRaw != null
+                        ? `${Math.round(Number(avgRespRaw) * 1000)}ms`
+                        : "N/A";
                 rows.push([
                     type,
                     index + 1,
@@ -3260,9 +3274,7 @@ function exportMetricsToCSV(topData) {
                             item.executions ||
                             0,
                     ),
-                    item.avg_response_time ?? item.avgResponseTime
-                        ? `${Math.round((item.avg_response_time ?? item.avgResponseTime) * 1000)}ms`
-                        : "N/A",
+                    avgRespDisplay,
                     `${calculateSuccessRate(item)}%`,
                     formatLastUsed(item.last_execution || item.lastExecution),
                 ]);
@@ -3482,20 +3494,16 @@ function createMetricsCard(title, metrics) {
 
         const valueSpan = document.createElement("span");
         valueSpan.className = "font-medium dark:text-gray-200";
-        let displayValue;
-        if (value === "N/A") {
-            displayValue = "N/A";
-        } else if (metric.key === "avgResponseTime") {
-            const ms = Number(value);
-            displayValue = isNaN(ms) ? "N/A" : `${(ms * 1000).toFixed(1)} ms`;
-        } else if (metric.key === "lastExecutionTime") {
-            displayValue = typeof value === "string" && value.includes("T")
-                ? value.slice(0, 16).replace("T", " ")
-                : String(value);
+        if (metric.key === "avgResponseTime" && value !== "N/A") {
+            valueSpan.textContent = `${(Number(value) * 1000).toFixed(1)}ms`;
+        } else if (metric.key === "lastExecutionTime" && value !== "N/A") {
+            valueSpan.textContent =
+                typeof value === "string"
+                    ? value.slice(0, 16).replace("T", " ")
+                    : String(value);
         } else {
-            displayValue = String(value);
+            valueSpan.textContent = value === "N/A" ? "N/A" : String(value);
         }
-        valueSpan.textContent = displayValue;
 
         metricRow.appendChild(label);
         metricRow.appendChild(valueSpan);

tests/js/admin-parsing.test.js

@@ -296,7 +296,7 @@ describe("extractKPIData", () => {
         expect(result.totalExecutions).toBe(100);
         expect(result.successRate).toBe(90);
         expect(result.errorRate).toBe(10);
-        expect(result.avgResponseTime).toBeCloseTo(1.5, 1);
+        expect(result.avgResponseTime).toBeCloseTo(1500, 1);
     });
 
     test("aggregates across multiple categories", () => {
@@ -386,8 +386,8 @@ describe("extractKPIData", () => {
             },
         };
         const result = f()(data);
-        // Weighted avg = (100*2.0 + 100*4.0) / 200 = 3.0
-        expect(result.avgResponseTime).toBeCloseTo(3.0, 1);
+        // Weighted avg = (100*2.0 + 100*4.0) / 200 = 3.0s = 3000ms
+        expect(result.avgResponseTime).toBeCloseTo(3000, 1);
     });
 
     test("ignores N/A response time values", () => {