chore: update composer dependencies for PHP 8.2+ compatibility

dradisic · dradisic · commit 6a46c967b6bd · 2025-03-05T16:34:18.000+01:00
diff --git a/README.md b/README.md
@@ -2,7 +2,7 @@
 [![Actions Status](https://github.com/php-kafka/php-avro-schema-generator/workflows/CI/badge.svg)](https://github.com/php-kafka/php-avro-schema-generator/workflows/CI/badge.svg)
 [![Maintainability](https://api.codeclimate.com/v1/badges/41aecf21566d7e9bfb69/maintainability)](https://codeclimate.com/github/php-kafka/php-avro-schema-generator/maintainability)
 [![Test Coverage](https://api.codeclimate.com/v1/badges/41aecf21566d7e9bfb69/test_coverage)](https://codeclimate.com/github/php-kafka/php-avro-schema-generator/test_coverage)
-![Supported PHP versions: 7.4 .. 8.x](https://img.shields.io/badge/php-7.4%20..%208.x-blue.svg)
+![Supported PHP versions: ^8.2](https://img.shields.io/badge/php-%5E8.2-blue.svg)
 [![Latest Stable Version](https://poser.pugx.org/php-kafka/php-avro-schema-generator/v/stable)](https://packagist.org/packages/php-kafka/php-avro-schema-generator)
 
 ## Installation
@@ -53,7 +53,7 @@ How to enable optimizer:
 
 **Console example**
 ```bash
-./vendor/bin/avro-cli --optimizeFullNames --optimizeFieldOrder --optimizePrimitiveSchemas avro:subschema:merge ./example/schemaTemplates ./example/schema
+./bin/avro-cli  --optimizeFullNames --optimizeFieldOrder --optimizePrimitiveSchemas avro:subschema:merge ./example/schemaTemplates ./example/schema
 ```
 **PHP Example**
 ```php
@@ -87,7 +87,7 @@ After you have reviewed and adjusted your templates you will need to merge them
 
 **Console example**
 ```bash
-./vendor/bin/avro-cli avro:schema:generate ./example/classes ./example/schemaTemplates
+./bin/avro-cli  avro:schema:generate ./example/classes ./example/schemaTemplates
 ```
 
 **PHP Example**
diff --git a/composer.json b/composer.json
@@ -8,16 +8,17 @@
   ],
   "license": "MIT",
   "require": {
+    "php": ">=8.2",
     "ext-json": "*",
-    "flix-tech/avro-php": "^3.0|^4.0|^5.0",
-    "symfony/console": "^4.3|^5.1|^6.0",
-    "nikic/php-parser": "^4.13",
+    "flix-tech/avro-php": "^5.1",
+    "symfony/console": "^7.2",
+    "nikic/php-parser": "^5.4",
     "pimple/pimple": "^3.5"
   },
   "require-dev": {
-    "friendsofphp/php-cs-fixer": "^2.19|^3.15",
-    "infection/infection": "^0.25|^0.27",
-    "composer/xdebug-handler": "^2.0|^3.0",
+    "friendsofphp/php-cs-fixer": "^3.70",
+    "infection/infection": "^0.29",
+    "composer/xdebug-handler": "^3.0",
     "phpstan/phpstan": "^1.2",
     "phpunit/phpunit": "^9.3",
     "rregeer/phpunit-coverage-check": "^0.3",
diff --git a/docker-compose.yml b/docker-compose.yml
@@ -9,3 +9,7 @@ services:
         HOST_USER_ID: ${USER_ID}
     volumes:
       - ./:/var/www/html
+      - type: tmpfs
+        target: /tmp/test
+        tmpfs:
+          size: 256M
diff --git a/docker/dev/php/Dockerfile b/docker/dev/php/Dockerfile
@@ -1,18 +1,30 @@
-FROM php:7.4-cli-alpine3.15
+FROM php:8.2-cli-alpine3.19
 
 ARG HOST_USER_ID
 ARG HOST_USER
 
+# Create sanitized username as an ENV variable so it persists throughout the build
+ENV CLEAN_USER=default
+RUN if [ ! -z "$HOST_USER" ]; then \
+    export CLEAN_USER=$(echo "$HOST_USER" | sed 's/[^a-zA-Z0-9._-]/_/g') && \
+    echo "export CLEAN_USER=$CLEAN_USER" >> /etc/profile; \
+fi
+
 COPY php/files/bin/ /usr/local/bin/
 
 # SYS: Install required packages
 RUN apk --no-cache upgrade && \
     apk --no-cache add bash git sudo make autoconf gcc g++
 
 RUN if [ ! -z "$HOST_USER_ID" ]; then \
-        adduser -u $HOST_USER_ID -D -H $HOST_USER && \
-        echo "ALL ALL=NOPASSWD: ALL" >> /etc/sudoers; \
-    fi
+    adduser -u $HOST_USER_ID -D -H $CLEAN_USER && \
+    echo "ALL ALL=NOPASSWD: ALL" >> /etc/sudoers; \
+fi
+
+# Create a directory with proper permissions for test files
+RUN mkdir -p /tmp/test && \
+    chmod 777 /tmp/test && \
+    chown $CLEAN_USER:$CLEAN_USER /tmp/test
 
 # PHP: Install php extensions
 RUN mkdir /phpIni && \
@@ -23,9 +35,8 @@ RUN mkdir /phpIni && \
 # COMPOSER: install binary
 RUN curl -sS https://getcomposer.org/installer | php -- --install-dir=/usr/bin --filename=composer
 
-USER $HOST_USER
+USER $CLEAN_USER
 
 WORKDIR /var/www/html
 
 CMD tail -f /dev/null
-
diff --git a/example/generate.php b/example/generate.php
@@ -12,7 +12,7 @@
 use PhpKafka\PhpAvroSchemaGenerator\Generator\SchemaGenerator;
 use PhpParser\ParserFactory;
 
-$parser = (new ParserFactory())->create(ParserFactory::PREFER_PHP7);
+$parser = (new ParserFactory())->createForNewestSupportedVersion();
 $classPropertyParser = new ClassPropertyParser(new DocCommentParser());
 $classParser = new ClassParser($parser, $classPropertyParser);
 
diff --git a/src/Parser/ClassParser.php b/src/Parser/ClassParser.php
@@ -142,7 +142,7 @@ public function getNamespace(): ?string
         foreach ($this->statements as $statement) {
             if ($statement instanceof Namespace_) {
                 if ($statement->name instanceof Name) {
-                    return implode('\\', $statement->name->parts);
+                    return $statement->name->toString();
                 }
             }
         }
diff --git a/src/ServiceProvider/ParserServiceProvider.php b/src/ServiceProvider/ParserServiceProvider.php
@@ -24,7 +24,7 @@ public function register(Container $container): void
         };
 
         $container[Parser::class] = static function (Container $container): Parser {
-            return $container[ParserFactory::class]->create(ParserFactory::PREFER_PHP7);
+            return $container[ParserFactory::class]->createForNewestSupportedVersion();
         };
 
         $container[DocCommentParserInterface::class] = static function (): DocCommentParserInterface {
diff --git a/tests/Integration/Parser/ClassParserTest.php b/tests/Integration/Parser/ClassParserTest.php
@@ -20,7 +20,7 @@ public function testGetClassName(): void
     {
         $filePath = __DIR__ . '/../../../example/classes/SomeTestClass.php';
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode((string) file_get_contents($filePath));
         self::assertEquals('SomeTestClass', $parser->getClassName());
         self::assertEquals('SomeTestClass', $parser->getClassName());
@@ -30,7 +30,7 @@ public function testGetClassNameForInterface(): void
     {
         $filePath = __DIR__ . '/../../../example/classes/SomeTestInterface.php';
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode((string) file_get_contents($filePath));
         self::assertNull($parser->getClassName());
     }
@@ -39,7 +39,7 @@ public function testGetNamespace(): void
     {
         $filePath = __DIR__ . '/../../../example/classes/SomeTestClass.php';
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode((string) file_get_contents($filePath));
         self::assertEquals('PhpKafka\\PhpAvroSchemaGenerator\\Example', $parser->getNamespace());
         self::assertEquals('PhpKafka\\PhpAvroSchemaGenerator\\Example', $parser->getNamespace());
@@ -49,7 +49,7 @@ public function testGetProperties(): void
     {
         $filePath = __DIR__ . '/../../../example/classes/SomeTestClass.php';
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode((string) file_get_contents($filePath));
         $properties = $parser->getProperties();
         self::assertCount(16, $properties);
@@ -62,7 +62,7 @@ public function testGetProperties(): void
     public function testClassAndNamespaceAreNullWithNoCode(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $refObject = new \ReflectionObject($parser);
         $refProperty = $refObject->getProperty('statements');
         $refProperty->setAccessible(true);
@@ -76,7 +76,7 @@ public function testClassAndNamespaceAreNullWithNoCode(): void
     public function testClassWithNoParent(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode('<?php class foo {}');
         self::assertNull($parser->getNamespace());
         self::assertNull($parser->getParentClassName());
@@ -87,7 +87,7 @@ public function testClassWithNoParent(): void
     public function testClassWithNullableType(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode('
             <?php
                 class foo {
@@ -102,7 +102,7 @@ class foo {
     public function testClassWithUnionType(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode('
             <?php
                 class foo {
@@ -117,7 +117,7 @@ class foo {
     public function testClassWithDocUnionType(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode('
             <?php
                 class foo {
@@ -135,7 +135,7 @@ class foo {
     public function testClassWithAnnotations(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode('
             <?php
                 class foo {
@@ -159,7 +159,7 @@ class foo {
     public function testClassWithNoParentFile(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode('<?php class foo extends \RuntimeException {private $x;}');
         $properties = $parser->getProperties();
         self::assertEquals(1, count($properties));
diff --git a/tests/Integration/Parser/ClassPropertyParserTest.php b/tests/Integration/Parser/ClassPropertyParserTest.php
@@ -7,7 +7,6 @@
 use PhpKafka\PhpAvroSchemaGenerator\Parser\ClassParser;
 use PhpKafka\PhpAvroSchemaGenerator\Parser\ClassPropertyParser;
 use PhpKafka\PhpAvroSchemaGenerator\Parser\DocCommentParser;
-use PhpKafka\PhpAvroSchemaGenerator\PhpClass\PhpClassPropertyInterface;
 use PhpParser\ParserFactory;
 use PHPUnit\Framework\TestCase;
 
@@ -19,7 +18,7 @@ class ClassPropertyParserTest extends TestCase
     public function testNullDefaultProperty(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode('
             <?php
                 class foo {
@@ -37,7 +36,7 @@ class foo {
     public function testIntDefaultProperty(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode('
             <?php
                 class foo {
@@ -55,7 +54,7 @@ class foo {
     public function testFloatDefaultProperty(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode('
             <?php
                 class foo {
@@ -73,7 +72,7 @@ class foo {
     public function testEmptyStringDefaultProperty(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $parser->setCode('
             <?php
                 class foo {
diff --git a/tests/Integration/Registry/ClassRegistryTest.php b/tests/Integration/Registry/ClassRegistryTest.php
@@ -25,7 +25,7 @@ class ClassRegistryTest extends TestCase
     public function testClassDirectory(): void
     {
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $converter = new PhpClassConverter($parser);
         $registry = new ClassRegistry($converter);
         $result = $registry->addClassDirectory('/tmp');
@@ -39,7 +39,7 @@ public function testLoad(): void
         $classDir = __DIR__ . '/../../../example/classes';
 
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $converter = new PhpClassConverter($parser);
         $registry = (new ClassRegistry($converter))->addClassDirectory($classDir)->load();
 
@@ -58,7 +58,7 @@ public function testRegisterSchemaFileThatDoesntExist(): void
     {
         $fileInfo = new SplFileInfo('somenonexistingfile');
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $converter = new PhpClassConverter($parser);
         $registry = new ClassRegistry($converter);
 
@@ -73,13 +73,14 @@ public function testRegisterSchemaFileThatDoesntExist(): void
 
     public function testRegisterSchemaFileThatIsNotReadable(): void
     {
-        touch('testfile');
-        chmod('testfile', 222);
+        $filePath = '/tmp/test/testfile';
+        touch($filePath);
+        chmod($filePath, 222);
 
-        $fileInfo = new SplFileInfo('testfile');
+        $fileInfo = new SplFileInfo($filePath);
 
         $propertyParser = new ClassPropertyParser(new DocCommentParser());
-        $parser = new ClassParser((new ParserFactory())->create(ParserFactory::PREFER_PHP7), $propertyParser);
+        $parser = new ClassParser((new ParserFactory())->createForNewestSupportedVersion(), $propertyParser);
         $converter = new PhpClassConverter($parser);
         $registry = new ClassRegistry($converter);
 
@@ -94,7 +95,7 @@ public function testRegisterSchemaFileThatIsNotReadable(): void
         try {
             $method->invokeArgs($registry, [$fileInfo]);
         } finally {
-            unlink('testfile');
+            @unlink($filePath);
         }
     }
 }
diff --git a/tests/Integration/Registry/SchemaRegistryTest.php b/tests/Integration/Registry/SchemaRegistryTest.php
@@ -105,11 +105,11 @@ public function testRegisterSchemaFileThatDoesntExist(): void
 
     public function testRegisterSchemaFileThatIsNotReadable(): void
     {
-        touch('testfile');
-        chmod('testfile', 222);
-
-        $fileInfo = new SplFileInfo('testfile');
+        $filePath = '/tmp/test/testfile';
+        touch($filePath);
+        chmod($filePath, 222);
 
+        $fileInfo = new SplFileInfo($filePath);
         $registry = new SchemaRegistry();
 
         self::expectException(SchemaRegistryException::class);
@@ -123,15 +123,16 @@ public function testRegisterSchemaFileThatIsNotReadable(): void
         try {
             $method->invokeArgs($registry, [$fileInfo]);
         } finally {
-            unlink('testfile');
+            @unlink($filePath);
         }
     }
 
     public function testRegisterSchemaWithInvalidContent(): void
     {
-        touch('testfile');
+        $filePath = '/tmp/test/testfile';
+        touch($filePath);
 
-        $fileInfo = new SplFileInfo('testfile');
+        $fileInfo = new SplFileInfo($filePath);
 
         $registry = new SchemaRegistry();
 
@@ -146,7 +147,7 @@ public function testRegisterSchemaWithInvalidContent(): void
         try {
             $method->invokeArgs($registry, [$fileInfo]);
         } finally {
-            unlink('testfile');
+            @unlink($filePath);
         }
     }
 }
diff --git a/tests/Unit/Merger/SchemaMergerTest.php b/tests/Unit/Merger/SchemaMergerTest.php

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ public function getNamespace(): ?string`
`142`	`142`	`foreach ($this->statements as $statement) {`
`143`	`143`	`if ($statement instanceof Namespace_) {`
`144`	`144`	`if ($statement->name instanceof Name) {`
`145`		`- return implode('\\', $statement->name->parts);`
	`145`	`+ return $statement->name->toString();`
`146`	`146`	`}`
`147`	`147`	`}`
`148`	`148`	`}`
Original file line number	Diff line number	Diff line change
`@@ -105,11 +105,11 @@ public function testRegisterSchemaFileThatDoesntExist(): void`
`105`	`105`
`106`	`106`	`public function testRegisterSchemaFileThatIsNotReadable(): void`
`107`	`107`	`{`
`108`		`- touch('testfile');`
`109`		`- chmod('testfile', 222);`
`110`		`-`
`111`		`- $fileInfo = new SplFileInfo('testfile');`
	`108`	`+ $filePath = '/tmp/test/testfile';`
	`109`	`+ touch($filePath);`
	`110`	`+ chmod($filePath, 222);`
`112`	`111`
	`112`	`+ $fileInfo = new SplFileInfo($filePath);`
`113`	`113`	`$registry = new SchemaRegistry();`
`114`	`114`
`115`	`115`	`self::expectException(SchemaRegistryException::class);`
`@@ -123,15 +123,16 @@ public function testRegisterSchemaFileThatIsNotReadable(): void`
`123`	`123`	`try {`
`124`	`124`	`$method->invokeArgs($registry, [$fileInfo]);`
`125`	`125`	`} finally {`
`126`		`- unlink('testfile');`
	`126`	`+ @unlink($filePath);`
`127`	`127`	`}`
`128`	`128`	`}`
`129`	`129`
`130`	`130`	`public function testRegisterSchemaWithInvalidContent(): void`
`131`	`131`	`{`
`132`		`- touch('testfile');`
	`132`	`+ $filePath = '/tmp/test/testfile';`
	`133`	`+ touch($filePath);`
`133`	`134`
`134`		`- $fileInfo = new SplFileInfo('testfile');`
	`135`	`+ $fileInfo = new SplFileInfo($filePath);`
`135`	`136`
`136`	`137`	`$registry = new SchemaRegistry();`
`137`	`138`
`@@ -146,7 +147,7 @@ public function testRegisterSchemaWithInvalidContent(): void`
`146`	`147`	`try {`
`147`	`148`	`$method->invokeArgs($registry, [$fileInfo]);`
`148`	`149`	`} finally {`
`149`		`- unlink('testfile');`
	`150`	`+ @unlink($filePath);`
`150`	`151`	`}`
`151`	`152`	`}`
`152`	`153`	`}`