feat: Adiciona script de teste para funcionalidade de limpeza de cache do EnvironmentHelper e melhora o gerenciamento de cache

Author: CAFernandes

scripts/test-cache-clearing.php

@@ -0,0 +1,50 @@
+<?php
+/**
+ * Test script to demonstrate EnvironmentHelper cache clearing functionality
+ */
+
+require_once __DIR__ . '/../vendor/autoload.php';
+
+use PivotPHP\CycleORM\Helpers\EnvironmentHelper;
+
+echo "Testing EnvironmentHelper cache clearing...\n\n";
+
+// First calls - populate cache
+echo "1. First calls (populating cache):\n";
+$result1 = EnvironmentHelper::isTesting();
+$result2 = EnvironmentHelper::isProduction();
+$result3 = EnvironmentHelper::getEnvironment();
+
+echo "   isTesting(): " . ($result1 ? 'true' : 'false') . "\n";
+echo "   isProduction(): " . ($result2 ? 'true' : 'false') . "\n";
+echo "   getEnvironment(): {$result3}\n\n";
+
+// Verify cache is populated by checking performance
+$start = microtime(true);
+for ($i = 0; $i < 1000; $i++) {
+    EnvironmentHelper::isTesting();
+}
+$cachedTime = microtime(true) - $start;
+echo "2. Cache performance (1000 calls): " . number_format($cachedTime * 1000, 2) . " ms\n\n";
+
+// Clear cache
+echo "3. Clearing cache...\n";
+EnvironmentHelper::clearCache();
+echo "   Cache cleared!\n\n";
+
+// Verify same results after cache clear
+echo "4. After cache clear (should rebuild cache):\n";
+$result4 = EnvironmentHelper::isTesting();
+$result5 = EnvironmentHelper::isProduction();
+$result6 = EnvironmentHelper::getEnvironment();
+
+echo "   isTesting(): " . ($result4 ? 'true' : 'false') . "\n";
+echo "   isProduction(): " . ($result5 ? 'true' : 'false') . "\n";
+echo "   getEnvironment(): {$result6}\n\n";
+
+// Verify results are consistent
+$consistent = ($result1 === $result4) && ($result2 === $result5) && ($result3 === $result6);
+echo "5. Consistency check: " . ($consistent ? "✅ PASS" : "❌ FAIL") . "\n";
+echo "   Results are " . ($consistent ? "identical" : "different") . " before and after cache clear\n\n";
+
+echo "✅ Cache clearing functionality works correctly!\n";

src/Helpers/EnvironmentHelper.php

@@ -4,20 +4,25 @@
 
 class EnvironmentHelper
 {
+    /**
+     * Cache centralizado para todos os métodos de ambiente.
+     *
+     * @var array<string, bool|string>
+     */
+    private static array $cache = [];
     /**
      * Verifica se o ambiente é produção.
      * Resultado é cached para melhor performance.
      */
     public static function isProduction(): bool
     {
-        static $cachedResult = null;
-        if ($cachedResult !== null) {
-            return $cachedResult;
+        if (isset(self::$cache['isProduction'])) {
+            return (bool) self::$cache['isProduction'];
         }
 
         $env = function_exists('env') ? env('APP_ENV', 'production') : ($_ENV['APP_ENV'] ?? 'production');
-        $cachedResult = in_array($env, ['production', 'prod'], true);
-        return $cachedResult;
+        self::$cache['isProduction'] = in_array($env, ['production', 'prod'], true);
+        return (bool) self::$cache['isProduction'];
     }
 
     /**
@@ -26,14 +31,13 @@ public static function isProduction(): bool
      */
     public static function isDevelopment(): bool
     {
-        static $cachedResult = null;
-        if ($cachedResult !== null) {
-            return $cachedResult;
+        if (isset(self::$cache['isDevelopment'])) {
+            return (bool) self::$cache['isDevelopment'];
         }
 
         $env = function_exists('env') ? env('APP_ENV', 'development') : ($_ENV['APP_ENV'] ?? 'development');
-        $cachedResult = in_array($env, ['development', 'dev', 'local'], true);
-        return $cachedResult;
+        self::$cache['isDevelopment'] = in_array($env, ['development', 'dev', 'local'], true);
+        return (bool) self::$cache['isDevelopment'];
     }
 
     /**
@@ -43,9 +47,8 @@ public static function isDevelopment(): bool
      */
     public static function isTesting(): bool
     {
-        static $cachedResult = null;
-        if ($cachedResult !== null) {
-            return $cachedResult;
+        if (isset(self::$cache['isTesting'])) {
+            return (bool) self::$cache['isTesting'];
         }
 
         // Check APP_ENV from multiple sources
@@ -61,17 +64,17 @@ public static function isTesting(): bool
 
         foreach ($envSources as $envValue) {
             if (in_array($envValue, ['testing', 'test'], true)) {
-                $cachedResult = true;
-                return $cachedResult;
+                self::$cache['isTesting'] = true;
+                return (bool) self::$cache['isTesting'];
             }
         }
 
         // Check if running under PHPUnit
-        $cachedResult = defined('PHPUNIT_RUNNING') ||
-                       (isset($_ENV['PHPUNIT_RUNNING']) && $_ENV['PHPUNIT_RUNNING']) ||
-                       (isset($_SERVER['PHPUNIT_RUNNING']) && $_SERVER['PHPUNIT_RUNNING']);
+        self::$cache['isTesting'] = defined('PHPUNIT_RUNNING') ||
+                                   (isset($_ENV['PHPUNIT_RUNNING']) && $_ENV['PHPUNIT_RUNNING']) ||
+                                   (isset($_SERVER['PHPUNIT_RUNNING']) && $_SERVER['PHPUNIT_RUNNING']);
 
-        return $cachedResult;
+        return (bool) self::$cache['isTesting'];
     }
 
     /**
@@ -80,14 +83,13 @@ public static function isTesting(): bool
      */
     public static function getEnvironment(): string
     {
-        static $cachedResult = null;
-        if ($cachedResult !== null) {
-            return $cachedResult;
+        if (isset(self::$cache['getEnvironment'])) {
+            return (string) self::$cache['getEnvironment'];
         }
 
         $env = function_exists('env') ? env('APP_ENV', 'production') : ($_ENV['APP_ENV'] ?? 'production');
-        $cachedResult = is_string($env) ? $env : 'production';
-        return $cachedResult;
+        self::$cache['getEnvironment'] = is_string($env) ? $env : 'production';
+        return (string) self::$cache['getEnvironment'];
     }
 
     /**
@@ -96,7 +98,6 @@ public static function getEnvironment(): string
      */
     public static function clearCache(): void
     {
-        // Reset all static caches by calling each method with a flag
-        // This is a simple approach - in production the cache should rarely need clearing
+        self::$cache = [];
     }
 }