[test] add phpunit tests

Author: connorhu

.github/workflows/continuous-integration.yml

@@ -58,4 +58,4 @@ jobs:
         run: php data/bin/check_configuration.php
 
       - name: Run Tests
-        run: php data/bin/symfony symfony:test --trace
+        run: php vendor/bin/phpunit

.gitignore

@@ -7,3 +7,6 @@ lib/plugins/sfDoctrinePlugin/test/functional/fixtures/log/
 /vendor
 /composer.lock
 .php-cs-fixer.cache
+.phpunit.result.cache
+phpunit.xml
+/tests/fixtures/symfony/log

.php-cs-fixer.dist.php

@@ -10,8 +10,8 @@
     ->exclude('task/generator/skeleton/')
     ->exclude('plugins/sfDoctrinePlugin/data/generator/')
     // Exclude generated files (single files)
-    ->notPath('unit/config/fixtures/sfDefineEnvironmentConfigHandler/prefix_result.php')
-    ->notPath('unit/config/fixtures/sfFilterConfigHandler/result.php')
+    ->notPath('fixtures/sfDefineEnvironmentConfigHandler/prefix_result.php')
+    ->notPath('fixtures/sfFilterConfigHandler/result.php')
 ;
 
 $config = new PhpCsFixer\Config();

composer.json

@@ -8,7 +8,10 @@
         "swiftmailer/swiftmailer": "~5.2 || ^6.0"
     },
     "require-dev": {
-        "psr/log": "*"
+        "ext-zlib": "*",
+        "ext-apcu": "*",
+        "psr/log": "*",
+        "phpunit/phpunit": "^9.6"
     },
     "autoload": {
         "files": ["autoload.php"]
@@ -26,5 +29,9 @@
     "replace": {
         "lexpress/symfony1": "^1.5"
     },
+    "scripts": {
+        "test": "phpunit",
+        "php-cs-fix": "PHP_CS_FIXER_IGNORE_ENV=1 php-cs-fixer fix"
+    },
     "bin": ["data/bin/symfony"]
 }

lib/autoload/sfCoreAutoload.class.php

@@ -301,6 +301,12 @@ class sfCoreAutoload
         'sftesterresponse' => 'test/sfTesterResponse.class.php',
         'sftesteruser' => 'test/sfTesterUser.class.php',
         'sftesterviewcache' => 'test/sfTesterViewCache.class.php',
+        'symfony1applicationtestcase' => 'test/Symfony1ApplicationTestCase.php',
+        'sfinternalserverbasedfixturetesttrait' => 'test/sfInternalServerBasedFixtureTestTrait.php',
+        'symfony1projecttestcase' => 'test/Symfony1ProjectTestCase.php',
+        'testcasedrivenapplicationconfiguration' => 'test/TestCaseDrivenApplicationConfiguration.php',
+        'testcasedrivenconfigurationinterface' => 'test/TestCaseDrivenConfigurationInterface.php',
+        'testcasedrivenprojectconfiguration' => 'test/TestCaseDrivenProjectConfiguration.php',
         'sfbasicsecurityuser' => 'user/sfBasicSecurityUser.class.php',
         'sfsecurityuser' => 'user/sfSecurityUser.class.php',
         'sfuser' => 'user/sfUser.class.php',

lib/cache/sfAPCCache.class.php

@@ -32,7 +32,7 @@ public function initialize($options = array())
     {
         parent::initialize($options);
 
-        $this->enabled = function_exists('apc_store') && ini_get('apc.enabled');
+        $this->enabled = function_exists('apcu_store') && apcu_enabled();
     }
 
     /**
@@ -69,14 +69,16 @@ public function has($key)
      * @see sfCache
      *
      * @param mixed|null $lifetime
+     *
+     * @return bool|array
      */
     public function set($key, $data, $lifetime = null)
     {
         if (!$this->enabled) {
             return true;
         }
 
-        return apc_store($this->getOption('prefix').$key, $data, $this->getLifetime($lifetime));
+        return apcu_store($this->getOption('prefix').$key, $data, $this->getLifetime($lifetime));
     }
 
     /**
@@ -88,7 +90,7 @@ public function remove($key)
             return true;
         }
 
-        return apc_delete($this->getOption('prefix').$key);
+        return apcu_delete($this->getOption('prefix').$key);
     }
 
     /**
@@ -101,7 +103,7 @@ public function clean($mode = sfCache::ALL)
         }
 
         if (sfCache::ALL === $mode) {
-            return apc_clear_cache('user');
+            return apcu_clear_cache();
         }
     }
 
@@ -138,7 +140,7 @@ public function removePattern($pattern)
             return true;
         }
 
-        $infos = apc_cache_info('user');
+        $infos = apcu_cache_info();
         if (!is_array($infos['cache_list'])) {
             return;
         }
@@ -147,7 +149,7 @@ public function removePattern($pattern)
 
         foreach ($infos['cache_list'] as $info) {
             if (preg_match($regexp, $info['info'])) {
-                apc_delete($info['info']);
+                apcu_delete($info['info']);
             }
         }
     }
@@ -165,7 +167,7 @@ protected function getCacheInfo($key)
             return false;
         }
 
-        $infos = apc_cache_info('user');
+        $infos = apcu_cache_info();
 
         if (is_array($infos['cache_list'])) {
             foreach ($infos['cache_list'] as $info) {
@@ -181,7 +183,7 @@ protected function getCacheInfo($key)
     private function fetch($key, &$success)
     {
         $has = null;
-        $value = apc_fetch($key, $has);
+        $value = apcu_fetch($key, $has);
         // the second argument was added in APC 3.0.17. If it is still null we fall back to the value returned
         if (null !== $has) {
             $success = $has;

lib/cache/sfEAcceleratorCache.class.php

@@ -14,6 +14,8 @@
  * @author     Fabien Potencier <fabien.potencier@symfony-project.com>
  *
  * @version    SVN: $Id$
+ *
+ * @deprecated
  */
 class sfEAcceleratorCache extends sfCache
 {

lib/cache/sfFileCache.class.php

@@ -63,7 +63,13 @@ public function get($key, $default = null)
             return $default;
         }
 
-        return $data[self::READ_DATA];
+        $rawData = $data[self::READ_DATA];
+
+        if ('FaLSe' === $rawData) {
+            $rawData = false;
+        }
+
+        return $rawData;
     }
 
     /**
@@ -87,6 +93,10 @@ public function set($key, $data, $lifetime = null)
             $this->clean(sfCache::OLD);
         }
 
+        if (false === $data) {
+            $data = 'FaLSe';
+        }
+
         return $this->write($this->getFilePath($key), $data, time() + $this->getLifetime($lifetime));
     }
 

lib/cache/sfSQLiteCache.class.php

@@ -72,6 +72,10 @@ public function get($key, $default = null)
             $data = $this->dbh->singleQuery(sprintf("SELECT data FROM cache WHERE key = '%s' AND timeout > %d", sqlite_escape_string($key), time()));
         }
 
+        if ('FaLSe' === $data) {
+            $data = false;
+        }
+
         return null === $data ? $default : $data;
     }
 
@@ -81,7 +85,7 @@ public function get($key, $default = null)
     public function has($key)
     {
         if ($this->isSqLite3()) {
-            return (int) $this->dbh->querySingle(sprintf("SELECT count(*) FROM cache WHERE key = '%s' AND timeout > %d", $this->dbh->escapeString($key), time()));
+            return $this->dbh->querySingle(sprintf("SELECT count(*) FROM cache WHERE key = '%s' AND timeout > %d", $this->dbh->escapeString($key), time())) > 0;
         }
 
         return (bool) $this->dbh->query(sprintf("SELECT key FROM cache WHERE key = '%s' AND timeout > %d", sqlite_escape_string($key), time()))->numRows();
@@ -98,6 +102,10 @@ public function set($key, $data, $lifetime = null)
             $this->clean(sfCache::OLD);
         }
 
+        if (false === $data) {
+            $data = 'FaLSe';
+        }
+
         if ($this->isSqLite3()) {
             return $this->dbh->exec(sprintf("INSERT OR REPLACE INTO cache (key, data, timeout, last_modified) VALUES ('%s', '%s', %d, %d)", $this->dbh->escapeString($key), $this->dbh->escapeString($data), time() + $this->getLifetime($lifetime), time()));
         }

lib/cache/sfXCacheCache.class.php

@@ -14,6 +14,8 @@
  * @author     Fabien Potencier <fabien.potencier@symfony-project.com>
  *
  * @version    SVN: $Id$
+ *
+ * @deprecated
  */
 class sfXCacheCache extends sfCache
 {