feat(tests): adicionar teste para garantir que pool_sizes estão ordenados por capacidade

Author: CAFernandes

src/Json/Pool/JsonBufferPool.php

@@ -186,14 +186,29 @@ public static function getStatistics(): array
         // Sort pools by capacity for better readability
         ksort($poolsByCapacity);
 
+        // Sort pool_sizes by extracting numeric capacity for consistent ordering
+        uksort(
+            $poolSizes,
+            function ($a, $b) {
+            // Extract numeric capacity from formatted strings like "1.0KB (1024 bytes)"
+                preg_match('/\((\d+) bytes\)/', $a, $matchesA);
+                preg_match('/\((\d+) bytes\)/', $b, $matchesB);
+
+                $capacityA = isset($matchesA[1]) ? (int)$matchesA[1] : 0;
+                $capacityB = isset($matchesB[1]) ? (int)$matchesB[1] : 0;
+
+                return $capacityA <=> $capacityB;
+            }
+        );
+
         return [
             'reuse_rate' => round($reuseRate, 2),
             'total_operations' => $totalOperations,
             'current_usage' => self::$stats['current_usage'],
             'peak_usage' => self::$stats['peak_usage'],
             'total_buffers_pooled' => $totalBuffersInPools,
             'active_pool_count' => count(self::$pools),
-            'pool_sizes' => $poolSizes,  // Legacy format for backward compatibility
+            'pool_sizes' => $poolSizes,  // Legacy format sorted by capacity
             'pools_by_capacity' => array_values($poolsByCapacity),  // Enhanced format
             'detailed_stats' => self::$stats
         ];

tests/Json/Pool/JsonBufferPoolStatisticsTest.php

@@ -212,6 +212,47 @@ public function testStatisticsWithNoPools(): void
         $this->assertEquals(0.0, $stats['reuse_rate']);
     }
 
+    /**
+     * Test that pool_sizes are sorted by capacity for consistency
+     */
+    public function testPoolSizesSorting(): void
+    {
+        // Create buffers in reverse order to test sorting
+        $buffer16KB = JsonBufferPool::getBuffer(16384);
+        $buffer1KB = JsonBufferPool::getBuffer(1024);
+        $buffer8KB = JsonBufferPool::getBuffer(8192);
+        $buffer4KB = JsonBufferPool::getBuffer(4096);
+
+        JsonBufferPool::returnBuffer($buffer16KB);
+        JsonBufferPool::returnBuffer($buffer1KB);
+        JsonBufferPool::returnBuffer($buffer8KB);
+        JsonBufferPool::returnBuffer($buffer4KB);
+
+        $stats = JsonBufferPool::getStatistics();
+        $poolSizes = $stats['pool_sizes'];
+
+        // Get the keys (capacity strings) as an array
+        $capacityKeys = array_keys($poolSizes);
+
+        // Expected order: smallest to largest
+        $expectedOrder = [
+            '1.0KB (1024 bytes)',
+            '4.0KB (4096 bytes)',
+            '8.0KB (8192 bytes)',
+            '16.0KB (16384 bytes)'
+        ];
+
+        $this->assertEquals($expectedOrder, $capacityKeys, 'Pool sizes should be sorted by capacity');
+
+        // Also verify pools_by_capacity is sorted (already tested but good to be explicit)
+        $poolsByCapacity = $stats['pools_by_capacity'];
+        $capacities = array_column($poolsByCapacity, 'capacity_bytes');
+        $sortedCapacities = $capacities;
+        sort($sortedCapacities);
+
+        $this->assertEquals($sortedCapacities, $capacities, 'Pools by capacity should be sorted');
+    }
+
     /**
      * Test statistics consistency between formats
      */