Update composer.json to support symfony 6.4

.gitignore

@@ -1,5 +1,6 @@
 composer.phar
 phpunit.xml
+.phpunit.result.cache
 vendor/
 data/schema.graphql
 composer.lock

Dockerfile

@@ -0,0 +1,11 @@
+FROM php:8.3
+COPY --from=composer:latest /usr/bin/composer /usr/local/bin/composer
+RUN apt update
+RUN apt install git -y
+RUN apt-get install -y \
+        libzip-dev \
+        zip \
+  && docker-php-ext-install zip
+RUN pecl install xdebug && docker-php-ext-enable xdebug
+WORKDIR /var/www/html
+ENTRYPOINT ["tail", "-f", "/dev/null"]

composer.json

@@ -13,12 +13,14 @@
     }
   ],
   "require": {
-    "php": ">=5.5",
+    "php": "^8.3",
     "ext-mbstring": "*",
-    "symfony/property-access": "^2.8 || ^3.4 || ^4.4 || ^5.4"
+    "symfony/property-access": "^2.8 || ^3.4 || ^4.4 || ^5.4 || ^6.4"
   },
   "require-dev": {
-    "phpunit/phpunit": "^4.8"
+    "phpunit/phpunit": "~9.6",
+    "rector/rector": "*",
+    "phpspec/prophecy": "^1.19"
   },
   "autoload": {
     "psr-4": {

docker-compose.yml

@@ -0,0 +1,7 @@
+services:
+  app:
+    hostname: app
+    container_name: app
+    build: .
+    volumes:
+      - .:/var/www/html:rw

rector.php

@@ -0,0 +1,21 @@
+<?php
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\PHPUnit\Set\PHPUnitSetList;
+
+return RectorConfig::configure()
+    ->withPaths([
+        // __DIR__ . '/src',
+        __DIR__ . '/tests',
+    ])
+    // uncomment to reach your current PHP version
+    ->withPhpSets(php83: true)
+    ->withSets([
+        PHPUnitSetList::PHPUNIT_50,
+        PHPUnitSetList::PHPUNIT_60,
+        PHPUnitSetList::PHPUNIT_70,
+        PHPUnitSetList::PHPUNIT_80,
+        PHPUnitSetList::PHPUNIT_90,
+        ])
+    ->withTypeCoverageLevel(0);

tests/DataProvider/TestConfig.php

@@ -14,6 +14,7 @@
 
 class TestConfig extends AbstractConfig
 {
+    #[\Override]
     public function getRules()
     {
         return [

tests/DataProvider/TestConfigExtraFields.php

@@ -17,6 +17,7 @@ class TestConfigExtraFields extends AbstractConfig
 
     protected $extraFieldsAllowed = true;
 
+    #[\Override]
     public function getRules()
     {
         return [

tests/DataProvider/TestConfigInvalidRule.php

@@ -8,6 +8,7 @@
 
 class TestConfigInvalidRule extends AbstractConfig
 {
+    #[\Override]
     public function getRules()
     {
         return [

tests/DataProvider/TestEmptySchema.php

@@ -14,11 +14,13 @@
 
 class TestEmptySchema extends AbstractSchema
 {
+    #[\Override]
     public function build(SchemaConfig $config)
     {
     }
 
 
+    #[\Override]
     public function getName($config)
     {
         return 'TestSchema';

tests/DataProvider/TestEnumType.php

@@ -13,6 +13,7 @@
 
 class TestEnumType extends AbstractEnumType
 {
+    #[\Override]
     public function getValues()
     {
         return [

tests/DataProvider/TestExtendedType.php

@@ -15,6 +15,7 @@
 
 class TestExtendedType extends AbstractObjectType
 {
+    #[\Override]
     public function build($config)
     {
         $config->applyInterface(new TestInterfaceType())

tests/DataProvider/TestField.php

@@ -18,16 +18,19 @@ class TestField extends AbstractField
     /**
      * @return AbstractObjectType
      */
+    #[\Override]
     public function getType()
     {
         return new IntType();
     }
 
+    #[\Override]
     public function resolve($value, array $args, ResolveInfo $info)
     {
         return $value;
     }
 
+    #[\Override]
     public function getDescription()
     {
         return 'description';

tests/DataProvider/TestInputField.php

@@ -18,16 +18,19 @@ class TestInputField extends AbstractInputField
     /**
      * @return InputTypeInterface
      */
+    #[\Override]
     public function getType()
     {
         return new IntType();
     }
 
+    #[\Override]
     public function getDescription()
     {
         return 'description';
     }
 
+    #[\Override]
     public function getDefaultValue()
     {
         return 'default';

tests/DataProvider/TestInputObjectType.php

@@ -15,6 +15,7 @@
 
 class TestInputObjectType extends AbstractInputObjectType
 {
+    #[\Override]
     public function build($config)
     {
         $config->addField('name', new NonNullType(new StringType()));

tests/DataProvider/TestInterfaceType.php

@@ -15,11 +15,13 @@
 class TestInterfaceType extends AbstractInterfaceType
 {
 
+    #[\Override]
     public function resolveType($object)
     {
         return is_object($object) ? $object : new TestObjectType();
     }
 
+    #[\Override]
     public function build($config)
     {
         $config->addField('name', new StringType());

tests/DataProvider/TestListType.php

@@ -14,6 +14,7 @@
 
 class TestListType extends AbstractListType
 {
+    #[\Override]
     public function getItemType()
     {
         return new StringType();

tests/DataProvider/TestMutationObjectType.php

@@ -15,11 +15,13 @@
 
 class TestMutationObjectType extends AbstractMutationObjectType
 {
+    #[\Override]
     public function getOutputType()
     {
         return new StringType();
     }
 
+    #[\Override]
     public function build($config)
     {
         $this->addArgument('increment', new IntType());

tests/DataProvider/TestObjectType.php

@@ -17,6 +17,7 @@
 class TestObjectType extends AbstractObjectType
 {
 
+    #[\Override]
     public function build($config)
     {
         $config
@@ -41,9 +42,7 @@ public function build($config)
                  'args'    => [
                      'noop' => new IntType()
                  ],
-                 'resolve' => function ($value, $args, $info) {
-                   return ['address' => '1234 Street'];
-                 }
+                 'resolve' => fn($value, $args, $info) => ['address' => '1234 Street']
              ]
             )
             ->addField(
@@ -52,13 +51,12 @@ public function build($config)
                     'args'    => [
                         'value' => new NonNullType(new StringType())
                     ],
-                    'resolve' => function ($value, $args, $info) {
-                        return $args['value'];
-                    }
+                    'resolve' => fn($value, $args, $info) => $args['value']
                 ]
             );
     }
 
+    #[\Override]
     public function getInterfaces()
     {
         return [new TestInterfaceType()];

tests/DataProvider/TestSchema.php

@@ -19,28 +19,23 @@ class TestSchema extends AbstractSchema
 {
     private $testStatusValue = 0;
 
+    #[\Override]
     public function build(SchemaConfig $config)
     {
         $config->getQuery()->addFields([
             'me'     => [
                 'type'    => new TestObjectType(),
-                'resolve' => function ($value, $args, ResolveInfo $info) {
-                    return $info->getReturnType()->getData();
-                }
+                'resolve' => fn($value, $args, ResolveInfo $info) => $info->getReturnType()->getData()
             ],
             'status' => [
                 'type'    => new TestEnumType(),
-                'resolve' => function () {
-                    return $this->testStatusValue;
-                }
+                'resolve' => fn() => $this->testStatusValue
             ],
         ]);
         $config->getMutation()->addFields([
             'updateStatus' => [
                 'type'    => new TestEnumType(),
-                'resolve' => function () {
-                    return $this->testStatusValue;
-                },
+                'resolve' => fn() => $this->testStatusValue,
                 'args'    => [
                     'newStatus' => new TestEnumType(),
                     'list' => new ListType(new IntType())

tests/DataProvider/TestTimeType.php

@@ -14,6 +14,7 @@
 class TestTimeType extends AbstractScalarType
 {
 
+    #[\Override]
     public function getName()
     {
         return 'TestTime';
@@ -23,6 +24,7 @@ public function getName()
      * @param $value \DateTime
      * @return null|string
      */
+    #[\Override]
     public function serialize($value)
     {
         if ($value === null) {
@@ -32,6 +34,7 @@ public function serialize($value)
         return $value instanceof \DateTime ? $value->format('H:i:s') : $value;
     }
 
+    #[\Override]
     public function isValidValue($value)
     {
         if (is_object($value)) {
@@ -43,6 +46,7 @@ public function isValidValue($value)
         return $d && $d->format('H:i:s') == $value;
     }
 
+    #[\Override]
     public function getDescription()
     {
         return 'Representation time in "H:i:s" format';

tests/DataProvider/TestUnionType.php

@@ -13,18 +13,21 @@
 class TestUnionType extends AbstractUnionType
 {
 
+    #[\Override]
     public function getTypes()
     {
         return [
             new TestObjectType()
         ];
     }
 
+    #[\Override]
     public function resolveType($object)
     {
         return $object;
     }
 
+    #[\Override]
     public function getDescription()
     {
         return 'Union collect cars types';

tests/Issues/Issue109/Issue109Schema.php

@@ -11,6 +11,7 @@
 class Issue109Schema extends AbstractSchema
 {
 
+    #[\Override]
     public function build(SchemaConfig $config)
     {
         $config->setQuery(

tests/Issues/Issue109/Issue109Test.php

@@ -4,7 +4,7 @@
 
 use Youshido\GraphQL\Execution\Processor;
 
-class Issue109Test extends \PHPUnit_Framework_TestCase
+class Issue109Test extends \PHPUnit\Framework\TestCase
 {
 
     public function testInternalVariableArgument()

tests/Issues/Issue116/Issue116Test.php

@@ -10,7 +10,7 @@
 use Youshido\GraphQL\Type\Scalar\IntType;
 use Youshido\GraphQL\Type\Scalar\StringType;
 
-class Issue116Test extends \PHPUnit_Framework_TestCase
+class Issue116Test extends \PHPUnit\Framework\TestCase
 {
 
     public function testInternalVariableArgument()
@@ -48,14 +48,12 @@ public function testInternalVariableArgument()
                         'args'    => [
                             'first' => new IntType(),
                         ],
-                        'resolve' => function () {
-                            return [
-                                'pageInfo' => [
-                                    'totalEdges' => 10,
-                                    'cursors'    => []
-                                ]
-                            ];
-                        }
+                        'resolve' => fn() => [
+                            'pageInfo' => [
+                                'totalEdges' => 10,
+                                'cursors'    => []
+                            ]
+                        ]
                     ]
                 ]
             ])

tests/Issues/Issue131/Issue131Test.php

@@ -12,7 +12,7 @@
 use Youshido\GraphQL\Type\Scalar\IntType;
 use Youshido\GraphQL\Type\Scalar\StringType;
 
-class Issue131Test extends \PHPUnit_Framework_TestCase
+class Issue131Test extends \PHPUnit\Framework\TestCase
 {
 
     public function testInternalVariableArgument()
@@ -47,12 +47,10 @@ public function testInternalVariableArgument()
                                 ]
                             ]))
                         ],
-                        'resolve' => function ($source, $args) {
-                            return [
-                                'id' => '1',
-                                'name' => sprintf('Meeting with %d beans', count($args['related_beans'])),
-                            ];
-                        }
+                        'resolve' => fn($source, $args) => [
+                            'id' => '1',
+                            'name' => sprintf('Meeting with %d beans', count($args['related_beans'])),
+                        ]
                     ]
                 ]
             ])