Cleanup test setup methods

Call parent setUpBeforeClaass method instead of duplicating code. Use the VersionUtils class also in AsyncTest instead of hard-coded values.
MoonE · Nov 10, 2024 · 81a6085 · 81a6085
1 parent a8d2910
commit 81a6085
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 38 deletions.
diff --git a/tests/AsyncTestCase.php b/tests/AsyncTestCase.php
@@ -12,6 +12,7 @@
 use Psalm\Internal\RuntimeCaches;
 use Psalm\Internal\Type\TypeParser;
 use Psalm\Internal\Type\TypeTokenizer;
+use Psalm\Internal\VersionUtils;
 use Psalm\IssueBuffer;
 use Psalm\Tests\Internal\Provider\FakeParserCacheProvider;
 use Psalm\Type\Union;
@@ -40,11 +41,11 @@ public static function setUpBeforeClass(): void
         ini_set('memory_limit', '-1');
 
         if (!defined('PSALM_VERSION')) {
-            define('PSALM_VERSION', '5.0.0');
+            define('PSALM_VERSION', VersionUtils::getPsalmVersion());
         }
 
         if (!defined('PHP_PARSER_VERSION')) {
-            define('PHP_PARSER_VERSION', '5.0.0');
+            define('PHP_PARSER_VERSION', VersionUtils::getPhpParserVersion());
         }
 
         parent::setUpBeforeClass();

diff --git a/tests/Config/ConfigTest.php b/tests/Config/ConfigTest.php
@@ -56,15 +56,9 @@ class ConfigTest extends TestCase
 
     public static function setUpBeforeClass(): void
     {
-        self::$config = new TestConfig();
-
-        if (!defined('PSALM_VERSION')) {
-            define('PSALM_VERSION', VersionUtils::getPsalmVersion());
-        }
+        parent::setUpBeforeClass();
 
-        if (!defined('PHP_PARSER_VERSION')) {
-            define('PHP_PARSER_VERSION', VersionUtils::getPhpParserVersion());
-        }
+        self::$config = new TestConfig();
     }
 
     public function setUp(): void
@@ -1334,10 +1328,6 @@ function handleThrow(I $e) : void {
 
     public function testModularConfig(): void
     {
-        // hack to isolate Psalm from PHPUnit arguments
-        global $argv;
-        $argv = [];
-
         $root = __DIR__ . '/../fixtures/ModularConfig';
         $config = Config::loadFromXMLFile($root . '/psalm.xml', $root);
         $this->assertEquals(

diff --git a/tests/Config/PluginTest.php b/tests/Config/PluginTest.php
@@ -44,19 +44,13 @@ class PluginTest extends TestCase
 
     public static function setUpBeforeClass(): void
     {
+        parent::setUpBeforeClass();
+
         // hack to isolate Psalm from PHPUnit cli arguments
         global $argv;
         $argv = [];
 
         self::$config = new TestConfig();
-
-        if (!defined('PSALM_VERSION')) {
-            define('PSALM_VERSION', VersionUtils::getPsalmVersion());
-        }
-
-        if (!defined('PHP_PARSER_VERSION')) {
-            define('PHP_PARSER_VERSION', VersionUtils::getPhpParserVersion());
-        }
     }
 
     public function setUp(): void

diff --git a/tests/ProjectCheckerTest.php b/tests/ProjectCheckerTest.php
@@ -41,19 +41,13 @@ class ProjectCheckerTest extends TestCase
 
     public static function setUpBeforeClass(): void
     {
+        parent::setUpBeforeClass();
+
         // hack to stop Psalm seeing the phpunit arguments
         global $argv;
         $argv = [];
 
         self::$config = new TestConfig();
-
-        if (!defined('PSALM_VERSION')) {
-            define('PSALM_VERSION', VersionUtils::getPsalmVersion());
-        }
-
-        if (!defined('PHP_PARSER_VERSION')) {
-            define('PHP_PARSER_VERSION', VersionUtils::getPhpParserVersion());
-        }
     }
 
     public function setUp(): void

diff --git a/tests/StubTest.php b/tests/StubTest.php
@@ -35,15 +35,9 @@ class StubTest extends TestCase
 
     public static function setUpBeforeClass(): void
     {
-        self::$config = new TestConfig();
-
-        if (!defined('PSALM_VERSION')) {
-            define('PSALM_VERSION', VersionUtils::getPsalmVersion());
-        }
+        parent::setUpBeforeClass();
 
-        if (!defined('PHP_PARSER_VERSION')) {
-            define('PHP_PARSER_VERSION', VersionUtils::getPhpParserVersion());
-        }
+        self::$config = new TestConfig();
     }
 
     public function setUp(): void