fix tests

Author: soyuka

phpstan.neon.dist

@@ -56,7 +56,7 @@ parameters:
 		-
 			message: '#expects ApiPlatform\\Metadata\\GraphQl\\Operation\|null, ApiPlatform\\Metadata\\Operation given#'
 			paths:
-				- tests/GraphQl/Resolver/Factory/
+				- src/GraphQl/Tests/Resolver/Factory/
 		- '#Access to an undefined property Prophecy\\Prophecy\\ObjectProphecy<(\\?[a-zA-Z0-9_]+)+>::\$[a-zA-Z0-9_]+#'
 		# https://github.com/willdurand/Negotiation/issues/89#issuecomment-513283286
 		-
@@ -65,7 +65,7 @@ parameters:
 		# https://github.com/phpstan/phpstan-symfony/issues/76
 		-
 			message: '#Service "test" is not registered in the container\.#'
-			path: tests/GraphQl/Type/TypesContainerTest.php
+			path: src/GraphQl/Tests/Type/TypesContainerTest.php
 
 		# Expected, due to optional interfaces
 		- '#Method Symfony\\Component\\Serializer\\NameConverter\\NameConverterInterface::denormalize\(\) invoked with (2|3|4) parameters, 1 required\.#'

src/Api/CompositeIdentifierParser.php

@@ -17,6 +17,7 @@
  * Normalizes a composite identifier.
  *
  * @author Antoine Bluchet <[email protected]>
+ * @deprecated
  */
 final class CompositeIdentifierParser
 {

src/Api/ResourceClassResolverInterface.php

@@ -15,7 +15,7 @@
 
 class_exists(\ApiPlatform\Metadata\ResourceClassResolverInterface::class);
 
-if (false) {
+if (false) { // @phpstan-ignore-line
     interface ResourceClassResolverInterface extends \ApiPlatform\Metadata\ResourceClassResolverInterface
     {
     }

src/Exception/InvalidArgumentException.php

@@ -13,15 +13,11 @@
 
 namespace ApiPlatform\Exception;
 
-use ApiPlatform\Metadata\Exception\InvalidArgumentException as MetadataInvalidArgumentException;
-
 /**
  * Invalid argument exception.
  *
  * @author Kévin Dunglas <[email protected]>
- *
- * @deprecated use \ApiPlatform\Metadata\Exception\InvalidArgumentException
  */
-class InvalidArgumentException extends MetadataInvalidArgumentException
+class InvalidArgumentException extends \InvalidArgumentException implements ExceptionInterface
 {
 }

src/Exception/InvalidIdentifierException.php

@@ -18,6 +18,6 @@
  *
  * @author Antoine Bluchet <[email protected]>
  */
-final class InvalidIdentifierException extends \Exception implements ExceptionInterface
+class InvalidIdentifierException extends \Exception implements ExceptionInterface
 {
 }

src/Exception/InvalidUriVariableException.php

@@ -17,8 +17,6 @@
  * Identifier is not valid exception.
  *
  * @author Antoine Bluchet <[email protected]>
- *
- * @final
  */
 class InvalidUriVariableException extends \Exception implements ExceptionInterface
 {

src/GraphQl/.php-cs-fixer.dist.php

@@ -14,10 +14,6 @@
 namespace ApiPlatform\GraphQl\Tests\Fixtures\ApiResource;
 
 use ApiPlatform\Metadata\ApiProperty;
-use ApiPlatform\Tests\Fixtures\TestBundle\Entity\RelatedDummy;
-use ApiPlatform\Tests\Fixtures\TestBundle\Entity\RelatedOwnedDummy;
-use ApiPlatform\Tests\Fixtures\TestBundle\Entity\RelatedOwningDummy;
-use Doctrine\Common\Collections\Collection;
 
 /**
  * Dummy.
@@ -194,44 +190,6 @@ public function getArrayData()
         return $this->arrayData;
     }
 
-    public function getRelatedDummy(): ?RelatedDummy
-    {
-        return $this->relatedDummy;
-    }
-
-    public function setRelatedDummy(RelatedDummy $relatedDummy): void
-    {
-        $this->relatedDummy = $relatedDummy;
-    }
-
-    public function addRelatedDummy(RelatedDummy $relatedDummy): void
-    {
-        $this->relatedDummies->add($relatedDummy);
-    }
-
-    public function getRelatedOwnedDummy()
-    {
-        return $this->relatedOwnedDummy;
-    }
-
-    public function setRelatedOwnedDummy(RelatedOwnedDummy $relatedOwnedDummy): void
-    {
-        $this->relatedOwnedDummy = $relatedOwnedDummy;
-        if ($this !== $this->relatedOwnedDummy->getOwningDummy()) {
-            $this->relatedOwnedDummy->setOwningDummy($this);
-        }
-    }
-
-    public function getRelatedOwningDummy()
-    {
-        return $this->relatedOwningDummy;
-    }
-
-    public function setRelatedOwningDummy(RelatedOwningDummy $relatedOwningDummy): void
-    {
-        $this->relatedOwningDummy = $relatedOwningDummy;
-    }
-
     public function isDummyBoolean(): ?bool
     {
         return $this->dummyBoolean;
@@ -254,9 +212,4 @@ public function getDummy()
     {
         return $this->dummy;
     }
-
-    public function getRelatedDummies(): Collection|iterable
-    {
-        return $this->relatedDummies;
-    }
 }

src/GraphQl/Tests/Fixtures/ApiResource/Dummy.php

@@ -0,0 +1,31 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace ApiPlatform\Metadata\Exception;
+
+use ApiPlatform\Exception\InvalidIdentifierException as LegacyInvalidIdentifierException;
+
+if (class_exists(LegacyInvalidIdentifierException::class)) {
+    class InvalidIdentifierException extends LegacyInvalidIdentifierException
+    {
+    }
+} else {
+    /**
+     * Identifier is not valid exception.
+     *
+     * @author Antoine Bluchet <[email protected]>
+     */
+    final class InvalidIdentifierException extends \Exception implements ExceptionInterface
+    {
+    }
+}

src/Metadata/Exception/InvalidIdentifierException.php

@@ -13,11 +13,13 @@
 
 namespace ApiPlatform\Metadata;
 
+use ApiPlatform\Api\ResourceClassResolverInterface as LegacyResourceClassResolverInterface;
 use ApiPlatform\Metadata\Exception\RuntimeException;
 use ApiPlatform\Metadata\GraphQl\Operation as GraphQlOperation;
 use ApiPlatform\Metadata\Property\Factory\PropertyMetadataFactoryInterface;
 use ApiPlatform\Metadata\Property\Factory\PropertyNameCollectionFactoryInterface;
 use ApiPlatform\Metadata\Resource\Factory\ResourceMetadataCollectionFactoryInterface;
+use ApiPlatform\Metadata\Util\CompositeIdentifierParser;
 use ApiPlatform\Metadata\Util\ResourceClassInfoTrait;
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
 use Symfony\Component\PropertyAccess\PropertyAccess;