From 1409bda97acecd008db73a6571af8e01033024f8 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Simon=20Andr=C3=A9?= <smn.andre@gmail.com>
Date: Mon, 27 Nov 2023 21:30:48 +0100
Subject: [PATCH] Fix DataCollector return types

---
 .../src/DataCollector/TwigComponentDataCollector.php        | 6 +++---
 .../templates/Collector/twig_component.html.twig            | 4 ++--
 .../Unit/DataCollector/TwigComponentDataCollectorTest.php   | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/TwigComponent/src/DataCollector/TwigComponentDataCollector.php b/src/TwigComponent/src/DataCollector/TwigComponentDataCollector.php
index 75af72f9bd7..874bb7bf74e 100644
--- a/src/TwigComponent/src/DataCollector/TwigComponentDataCollector.php
+++ b/src/TwigComponent/src/DataCollector/TwigComponentDataCollector.php
@@ -88,9 +88,9 @@ public function getRenderCount(): int
         return $this->data['render_count'] ?? 0;
     }
 
-    public function getRenderTime(): int
+    public function getRenderTime(): float
     {
-        return $this->data['render_time'] ?? 0;
+        return (float) ($this->data['render_time'] ?? 0);
     }
 
     private function collectDataFromLogger(): void
@@ -149,7 +149,7 @@ private function collectDataFromLogger(): void
                 $renders[$renderId] += [
                     'render_end' => $profile[0],
                     'render_time' => $renderTime,
-                    'render_memory' => $profile[1],
+                    'render_memory' => (int) $profile[1],
                 ];
 
                 ++$components[$componentName]['render_count'];
diff --git a/src/TwigComponent/templates/Collector/twig_component.html.twig b/src/TwigComponent/templates/Collector/twig_component.html.twig
index cbe725d6ac0..fe6ded6a412 100644
--- a/src/TwigComponent/templates/Collector/twig_component.html.twig
+++ b/src/TwigComponent/templates/Collector/twig_component.html.twig
@@ -125,7 +125,7 @@
             <span class="sf-toolbar-value">{{ collector.renderCount }}</span>
             <span class="sf-toolbar-info-piece-additional-detail">
                 <span class="sf-toolbar-label">in</span>
-                <span class="sf-toolbar-value">{{ collector.renderTime }}</span>
+                <span class="sf-toolbar-value">{{ collector.renderTime|round }}</span>
                 <span class="sf-toolbar-label">ms</span>
             </span>
         {% endset %}
@@ -165,7 +165,7 @@
             <div class="metric-divider"></div>
             <div class="metric-group">
                 {{ _self.metric(collector.renderCount, "Render Count") }}
-                {{ _self.metric(collector.renderTime, "Render Time", "ms") }}
+                {{ _self.metric(collector.renderTime|round, "Render Time", "ms") }}
             </div>
             <div class="metric-divider"></div>
             <div class="metric-group">
diff --git a/src/TwigComponent/tests/Unit/DataCollector/TwigComponentDataCollectorTest.php b/src/TwigComponent/tests/Unit/DataCollector/TwigComponentDataCollectorTest.php
index 0c236508ffa..434ae91185d 100644
--- a/src/TwigComponent/tests/Unit/DataCollector/TwigComponentDataCollectorTest.php
+++ b/src/TwigComponent/tests/Unit/DataCollector/TwigComponentDataCollectorTest.php
@@ -51,7 +51,7 @@ public function testLateCollect(): void
         $this->assertIsIterable($dataCollector->getRenders());
         $this->assertEmpty($dataCollector->getRenders());
 
-        $this->assertSame(0, $dataCollector->getRenderTime());
+        $this->assertEquals(0.0, $dataCollector->getRenderTime());
     }
 
     public function testReset(): void