php8.3 support (#15)

.github/workflows/composer.yml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.4', '8.0', '8.1', '8.2' ]
+        php: [ '7.4', '8.0', '8.1', '8.2', '8.3' ]
 
     name: Composer outdated PHP ${{ matrix.php }}
 
@@ -29,4 +29,4 @@ jobs:
         run: composer update --no-progress --no-interaction
 
       - name: Composer outdated
-        run: composer outdated -D --strict
+        run: composer outdated -D --strict --ignore=phpunit/phpunit

.github/workflows/extensions_finder.yml

@@ -24,7 +24,7 @@ jobs:
         run: composer update --no-progress --no-interaction
 
       - name: Install PHP extensions finder as dev dependency
-        run: composer require efabrica/php-extensions-finder --dev
+        run: composer require -W efabrica/php-extensions-finder --dev
 
       - name: PHP extensions finder
         run: vendor/bin/php-extensions-finder check src tests

.github/workflows/phpstan.yml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.4', '8.0', '8.1', '8.2' ]
+        php: [ '7.4', '8.0', '8.1', '8.2', '8.3' ]
 
     name: PHPStan PHP ${{ matrix.php }}
 

.github/workflows/phpunit_mysql_5_7.yml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.4', '8.0', '8.1', '8.2' ]
+        php: [ '7.4', '8.0', '8.1', '8.2', '8.3' ]
 
     name: PHPunit PHP ${{ matrix.php }}
 

.github/workflows/phpunit_mysql_8.yml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.4', '8.0', '8.1', '8.2' ]
+        php: [ '7.4', '8.0', '8.1', '8.2', '8.3' ]
 
     name: PHPunit PHP ${{ matrix.php }}
 

.github/workflows/phpunit_psql_10.yml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.4', '8.0', '8.1', '8.2' ]
+        php: [ '7.4', '8.0', '8.1', '8.2', '8.3' ]
 
     name: PHPunit PHP ${{ matrix.php }}
 

.github/workflows/phpunit_psql_14.yml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.4', '8.0', '8.1', '8.2' ]
+        php: [ '7.4', '8.0', '8.1', '8.2', '8.3' ]
 
     name: PHPunit PHP ${{ matrix.php }}
 

.github/workflows/syntax_checker.yml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '7.4', '8.0', '8.1', '8.2' ]
+        php: [ '7.4', '8.0', '8.1', '8.2', '8.3' ]
 
     name: PHP syntax checker PHP ${{ matrix.php }}
 

CHANGELOG.md

@@ -1,6 +1,9 @@
 ## Change Log
 
 ### [Unreleased][unreleased]
+#### Added
+- PHP8.3 support
+- Symfony 7 support
 
 ### [1.2.0] - 2023-08-29
 #### Changed

composer.json

@@ -5,16 +5,17 @@
     "keywords": ["PHP", "Database", "Fake data"],
     "license": "MIT",
     "require": {
-        "php": ">=7.4 <8.3",
+        "php": "^7.4 || ^8.0",
         "ext-json": "*",
         "ext-pdo": "*",
         "fakerphp/faker": "^1.17",
-        "symfony/console": "^4.2 | ^5.0 | ^6.0",
+        "symfony/console": "^4.2 | ^5.0 | ^6.0 | ^7.0",
         "nette/database": "^3.0"
     },
     "require-dev": {
         "phpunit/phpunit": "^9.5",
-        "lulco/phoenix": "^2.3"
+        "lulco/phoenix": "^2.3",
+        "phpstan/phpstan": "^1.12"
     },
     "autoload": {
         "psr-4": {

phpstan.neon

@@ -1,5 +1,7 @@
 parameters:
-    checkMissingIterableValueType: false
-    reportUnmatchedIgnoredErrors: true
+    level: max
+    reportUnmatchedIgnoredErrors: false
     ignoreErrors:
+        -
+            identifier: missingType.iterableValue
         - '#Parameter \#2 \$cacheStorage of class Nette\\Database\\Structure constructor expects Nette\\Caching\\IStorage, Nette\\Caching\\Storages\\MemoryStorage given\.#'

src/DataType/TextDataType.php

@@ -10,6 +10,6 @@ class TextDataType extends AbstractDataType
 
     public function populate(Column $column): string
     {
-        return $this->faker->realTextBetween(1, mt_rand(10, $this->max));
+        return $this->faker->realTextBetween(10, $this->max);
     }
 }

src/Database/Database.php

@@ -34,7 +34,7 @@ public function __construct(
     ) {
         $this->name = $name;
         try {
-            $connection = new Connection($dsn, $user, $password, $options);
+            $connection = new Connection($dsn, $user, $password, $options ?? []);
             $cacheStorage = new MemoryStorage();
             $structure = new Structure($connection, $cacheStorage);
             $database = new Explorer($connection, $structure);
@@ -141,7 +141,7 @@ private function addForeignKeysToTable(Table $table): void
             $table->addForeignKey(new ForeignKey(
                 [$columnName],
                 $referencedTableName,
-                [$this->database->getStructure()->getPrimaryKey($referencedTableName)]   // this is not always primary key, but nette database ignores this fact
+                [$this->database->getStructure()->getPrimaryKey($referencedTableName)] // this is not always primary key, but nette database ignores this fact
             ));
         }
     }
@@ -217,6 +217,7 @@ private function getExtendedColumnSettings(array $column): array
         $fullType = $column['vendor']['type'];
         $pattern = '/(.*?)\((.*?)\)(.*)/';
         preg_match($pattern, $fullType, $matches);
+        /** @var array<int, string> $matches */
         $type = trim($matches[1]);
         if (in_array($type, ['enum', 'set'])) {
             $values = str_replace("'", '', $matches[2]);

src/Populator/AutomaticPopulator.php

@@ -120,7 +120,7 @@ private function findDataTypeClassName(Column $column, Generator $faker)
 
         $class = new $className($faker);
         if (!$class instanceof DataTypeInterface) {
-            throw new Exception('Class "' . $className . '" is not type of '  . DataTypeInterface::class);
+            throw new Exception('Class "' . $className . '" is not type of ' . DataTypeInterface::class);
         }
         return $class;
     }

tests/Event/EventTest.php

@@ -17,89 +17,15 @@ abstract class EventTest extends TestCase
 {
     protected function createInput(): InputInterface
     {
-        return new class implements InputInterface
-        {
-            public function bind(InputDefinition $definition)
-            {
-
-            }
-
-            public function getArgument($name)
-            {
-
-            }
-
-            public function getArguments(): array
-            {
-
-            }
-
-            public function getFirstArgument(): string
-            {
-
-            }
-
-            public function getOption($name)
-            {
-
-            }
-
-            public function getOptions(): array
-            {
-
-            }
-
-            public function getParameterOption($values, $default = false, $onlyParams = false)
-            {
-
-            }
-
-            public function hasArgument($name): bool
-            {
-
-            }
-
-            public function hasOption($name): bool
-            {
-
-            }
-
-            public function hasParameterOption($values, $onlyParams = false): bool
-            {
-
-            }
-
-            public function isInteractive(): bool
-            {
-
-            }
-
-            public function setArgument($name, $value)
-            {
-
-            }
-
-            public function setInteractive($interactive)
-            {
-
-            }
-
-            public function setOption($name, $value)
-            {
-
-            }
-
-            public function validate()
-            {
-
-            }
-        };
+        if (PHP_VERSION_ID < 80200) {
+            return require __DIR__ . '/NullInputSymfony6.php';
+        }
+        return require __DIR__ . '/NullInputSymfony7.php';
     }
 
     protected function createOutput(): OutputInterface
     {
-        return new class implements OutputInterface
-        {
+        return new class implements OutputInterface {
             private array $messages = [];
 
             private OutputFormatterInterface $formatter;
@@ -194,8 +120,7 @@ public function isVeryVerbose(): bool
 
     protected function createPopulator(string $table, int $count = 10, ?string $databaseIdentifier = null): PopulatorInterface
     {
-        return new class($table, $count, $databaseIdentifier) extends AbstractPopulator
-        {
+        return new class($table, $count, $databaseIdentifier) extends AbstractPopulator {
             protected function generateData(Generator $faker): array
             {
                 return [];

tests/Event/NullInputSymfony6.php

@@ -0,0 +1,83 @@
+<?php
+
+declare(strict_types=1);
+
+use Symfony\Component\Console\Input\InputDefinition;
+use Symfony\Component\Console\Input\InputInterface;
+
+return new class implements InputInterface {
+    public function bind(InputDefinition $definition)
+    {
+
+    }
+
+    public function getArgument($name)
+    {
+
+    }
+
+    public function getArguments(): array
+    {
+
+    }
+
+    public function getFirstArgument(): string
+    {
+
+    }
+
+    public function getOption($name)
+    {
+
+    }
+
+    public function getOptions(): array
+    {
+
+    }
+
+    public function getParameterOption($values, $default = false, $onlyParams = false)
+    {
+
+    }
+
+    public function hasArgument($name): bool
+    {
+
+    }
+
+    public function hasOption($name): bool
+    {
+
+    }
+
+    public function hasParameterOption($values, $onlyParams = false): bool
+    {
+
+    }
+
+    public function isInteractive(): bool
+    {
+
+    }
+
+    public function setArgument($name, $value)
+    {
+
+    }
+
+    public function setInteractive($interactive)
+    {
+
+    }
+
+    public function setOption($name, $value)
+    {
+
+    }
+
+    public function validate()
+    {
+
+    }
+};