remove dead conditions

Author: staabm

tests/rules/QueryPlanAnalyzerRuleTest.php

@@ -62,10 +62,6 @@ public function testNotUsingIndex(): void
             self::markTestSkipped('query plan analyzer requires a active database connection');
         }
 
-        if (\PHP_VERSION_ID < 70300) {
-            self::markTestSkipped('not yet supported on php < 7.3');
-        }
-
         $this->numberOfAllowedUnindexedReads = true;
         $this->numberOfRowsNotRequiringIndex = 2;
 
@@ -111,10 +107,6 @@ public function testNotUsingIndexInDebugMode(): void
             self::markTestSkipped('query plan analyzer requires a active database connection');
         }
 
-        if (\PHP_VERSION_ID < 70300) {
-            self::markTestSkipped('not yet supported on php < 7.3');
-        }
-
         $this->debugMode = true;
         $this->numberOfAllowedUnindexedReads = true;
         $this->numberOfRowsNotRequiringIndex = 2;

tests/rules/SyntaxErrorInDibiPreparedStatementMethodRuleTest.php

@@ -30,10 +30,6 @@ public static function getAdditionalConfigFiles(): array
 
     public function testSyntaxErrorInQueryRule(): void
     {
-        if (\PHP_VERSION_ID < 70400) {
-            self::markTestSkipped('Test requires PHP 7.4.');
-        }
-
         if (MysqliQueryReflector::NAME === getenv('DBA_REFLECTOR')) {
             $expectedErrors = [
                 [

tests/rules/SyntaxErrorInPreparedStatementMethodRuleTest.php

@@ -30,10 +30,6 @@ public static function getAdditionalConfigFiles(): array
 
     public function testSyntaxErrorInQueryRule(): void
     {
-        if (\PHP_VERSION_ID < 70400) {
-            self::markTestSkipped('Test requires PHP 7.4.');
-        }
-
         if (MysqliQueryReflector::NAME === getenv('DBA_REFLECTOR')) {
             $expectedErrors = [
                 [
@@ -232,10 +228,6 @@ public function testSyntaxErrorInQueryRule(): void
 
     public function testBug94()
     {
-        if (\PHP_VERSION_ID < 70400) {
-            self::markTestSkipped('Test requires PHP 7.4.');
-        }
-
         if (MysqliQueryReflector::NAME === getenv('DBA_REFLECTOR')) {
             self::markTestSkipped('Error message different depending on version of the database.');
         } elseif (PdoPgSqlQueryReflector::NAME === getenv('DBA_REFLECTOR')) {
@@ -263,10 +255,6 @@ public function testBug94()
 
     public function testSyntaxErrorWithInferencePlaceholder()
     {
-        if (\PHP_VERSION_ID < 70400) {
-            self::markTestSkipped('Test requires PHP 7.4.');
-        }
-
         if (MysqliQueryReflector::NAME === getenv('DBA_REFLECTOR')) {
             $expectedErrors = [
                 [

tests/rules/SyntaxErrorInPreparedStatementMethodSubclassedRuleTest.php

@@ -30,10 +30,6 @@ public static function getAdditionalConfigFiles(): array
 
     public function testSyntaxErrorInQueryRule(): void
     {
-        if (\PHP_VERSION_ID < 70400) {
-            self::markTestSkipped('Test requires PHP 7.4.');
-        }
-
         $this->analyse([__DIR__ . '/data/syntax-error-in-method-subclassed.php'], $this->getExpectedErrors());
     }
 

tests/rules/SyntaxErrorInQueryFunctionRuleTest.php

@@ -30,10 +30,6 @@ public static function getAdditionalConfigFiles(): array
 
     public function testSyntaxErrorInQueryRule(): void
     {
-        if (\PHP_VERSION_ID < 70400) {
-            self::markTestSkipped('Test requires PHP 7.4.');
-        }
-
         $this->analyse([__DIR__ . '/data/syntax-error-in-query-function.php'], $this->getExpectedErrors());
     }
 

tests/rules/SyntaxErrorInQueryMethodRuleTest.php

@@ -30,10 +30,6 @@ public static function getAdditionalConfigFiles(): array
 
     public function testSyntaxErrorInQueryRule(): void
     {
-        if (\PHP_VERSION_ID < 70400) {
-            self::markTestSkipped('Test requires PHP 7.4.');
-        }
-
         if (MysqliQueryReflector::NAME === getenv('DBA_REFLECTOR')) {
             $expected = [
                 [

tests/rules/SyntaxErrorInQueryMethodSubclassedRuleTest.php

@@ -30,10 +30,6 @@ public static function getAdditionalConfigFiles(): array
 
     public function testSyntaxErrorInQueryRule(): void
     {
-        if (\PHP_VERSION_ID < 70400) {
-            self::markTestSkipped('Test requires PHP 7.4.');
-        }
-
         $this->analyse([__DIR__ . '/data/syntax-error-in-method-subclassed.php'], $this->getExpectedErrors());
     }
 

tests/sqlAst/SqlAstInferenceTest.php

@@ -11,7 +11,7 @@ class SqlAstInferenceTest extends TypeInferenceTestCase
 {
     public function dataFileAsserts(): iterable
     {
-        if (\PHP_VERSION_ID >= 70400 && 'pdo-pgsql' !== getenv('DBA_REFLECTOR')) {
+        if ('pdo-pgsql' !== getenv('DBA_REFLECTOR')) {
             yield from $this->gatherAssertTypes(__DIR__ . '/data/sql-ast-narrowing.php');
         }
     }

tests/sqlAst/config/bootstrap.php

@@ -17,9 +17,7 @@
 $config = RuntimeConfiguration::create();
 $config->errorMode(RuntimeConfiguration::ERROR_MODE_EXCEPTION);
 $config->analyzeWriteQueries(true);
-if (\PHP_VERSION_ID >= 70400) {
-    $config->utilizeSqlAst(true);
-}
+$config->utilizeSqlAst(true);
 
 // $config->debugMode(true);
 $reflector = ReflectorFactory::create(__DIR__);

tests/stringify/InferenceStringifyTest.php

@@ -12,7 +12,7 @@ public function dataFileAsserts(): iterable
     {
         yield from $this->gatherAssertTypes(__DIR__ . '/data/stringify.php');
 
-        if (\PHP_VERSION_ID >= 70400 && 'pdo-pgsql' !== getenv('DBA_REFLECTOR')) {
+        if ('pdo-pgsql' !== getenv('DBA_REFLECTOR')) {
             yield from $this->gatherAssertTypes(__DIR__ . '/data/ast-narrowed-stringify.php');
         }
     }