fix(jsonshcema): fix tests

Author: Zowac

features/openapi/docs.feature

@@ -307,14 +307,14 @@ Feature: Documentation support
     And the response should be in JSON
     And the header "Content-Type" should be equal to "application/json; charset=utf-8"
     And the OpenAPI class "Resource" exists
-    And the OpenAPI class "RelatedResource" exists
+    And the OpenAPI class "ResourceRelated" exists
     And the "resourceRelated" property for the OpenAPI class "Resource" should be equal to:
     """
     {
       "readOnly":true,
       "anyOf":[
         {
-          "$ref":"#/components/schemas/RelatedResource"
+          "$ref":"#/components/schemas/ResourceRelated"
         }
       ],
       "nullable":true

src/JsonSchema/SchemaFactory.php

@@ -14,13 +14,15 @@
 namespace ApiPlatform\JsonSchema;
 
 use ApiPlatform\Api\ResourceClassResolverInterface;
+use ApiPlatform\Exception\OperationNotFoundException;
 use ApiPlatform\Metadata\ApiProperty;
 use ApiPlatform\Metadata\CollectionOperationInterface;
 use ApiPlatform\Metadata\HttpOperation;
 use ApiPlatform\Metadata\Operation;
 use ApiPlatform\Metadata\Property\Factory\PropertyMetadataFactoryInterface;
 use ApiPlatform\Metadata\Property\Factory\PropertyNameCollectionFactoryInterface;
 use ApiPlatform\Metadata\Resource\Factory\ResourceMetadataCollectionFactoryInterface;
+use ApiPlatform\Metadata\Resource\ResourceMetadataCollection;
 use ApiPlatform\OpenApi\Factory\OpenApiFactory;
 use ApiPlatform\Util\ResourceClassInfoTrait;
 use Symfony\Component\PropertyInfo\Type;
@@ -270,7 +272,14 @@ private function getMetadata(string $className, string $type = Schema::TYPE_OUTP
         }
 
         if (null === $operation) {
-            $operation = $this->resourceMetadataFactory->create($className)->getOperation();
+            $resourceMetadataCollection = $this->resourceMetadataFactory->create($className);
+            try {
+                $operation = $resourceMetadataCollection->getOperation();
+            } catch (OperationNotFoundException $e) {
+                $operation = new HttpOperation();
+            }
+
+            $operation = $this->findOperationForType($resourceMetadataCollection, $type, $operation);
         } else {
             // The best here is to use an Operation when calling `buildSchema`, we try to do a smart guess otherwise
             if (!$operation->getClass()) {
@@ -279,20 +288,7 @@ private function getMetadata(string $className, string $type = Schema::TYPE_OUTP
                 if ($operation->getName()) {
                     $operation = $resourceMetadataCollection->getOperation($operation->getName());
                 } else {
-                    // Guess the operation and use the first one that matches criterias
-                    foreach ($resourceMetadataCollection as $resourceMetadata) {
-                        foreach ($resourceMetadata->getOperations() ?? [] as $op) {
-                            if ($operation instanceof CollectionOperationInterface && $op instanceof CollectionOperationInterface) {
-                                $operation = $op;
-                                break 2;
-                            }
-
-                            if (Schema::TYPE_INPUT === $type && \in_array($op->getMethod(), ['POST', 'PATCH', 'PUT'], true)) {
-                                $operation = $op;
-                                break 2;
-                            }
-                        }
-                    }
+                    $operation = $this->findOperationForType($resourceMetadataCollection, $type, $operation);
                 }
             }
         }
@@ -320,6 +316,26 @@ private function getMetadata(string $className, string $type = Schema::TYPE_OUTP
         ];
     }
 
+    private function findOperationForType(ResourceMetadataCollection $resourceMetadataCollection, string $type, Operation $operation)
+    {
+        // Find the operation and use the first one that matches criterias
+        foreach ($resourceMetadataCollection as $resourceMetadata) {
+            foreach ($resourceMetadata->getOperations() ?? [] as $op) {
+                if ($operation instanceof CollectionOperationInterface && $op instanceof CollectionOperationInterface) {
+                    $operation = $op;
+                    break 2;
+                }
+
+                if (Schema::TYPE_INPUT === $type && \in_array($op->getMethod(), ['POST', 'PATCH', 'PUT'], true)) {
+                    $operation = $op;
+                    break 2;
+                }
+            }
+        }
+
+        return $operation;
+    }
+
     private function getSerializerContext(Operation $operation, string $type = Schema::TYPE_OUTPUT): array
     {
         return Schema::TYPE_OUTPUT === $type ? ($operation->getNormalizationContext() ?? []) : ($operation->getDenormalizationContext() ?? []);

src/Metadata/Resource/ResourceMetadataCollection.php

@@ -89,9 +89,9 @@ public function getOperation(?string $operationName = null, bool $forceCollectio
         }
 
         // Idea:
-        // if ($metadata) {
-        //     return (new class extends HttpOperation {})->withResource($metadata);
-        // }
+//         if ($metadata) {
+//             return (new class extends HttpOperation {})->withResource($metadata);
+//         }
 
         $this->handleNotFound($operationName, $metadata);
     }
@@ -102,7 +102,7 @@ public function getOperation(?string $operationName = null, bool $forceCollectio
     private function handleNotFound(string $operationName, ?ApiResource $metadata): void
     {
         // Hide the FQDN in the exception message if possible
-        $shortName = $metadata ? $metadata->getShortName() : $this->resourceClass;
+        $shortName = $metadata?->getShortName() ? $metadata->getShortName() : $this->resourceClass;
         if (!$metadata && false !== $pos = strrpos($shortName, '\\')) {
             $shortName = substr($shortName, $pos + 1);
         }

tests/Fixtures/TestBundle/Entity/JsonSchemaResourceRelated.php

@@ -17,7 +17,7 @@
 use ApiPlatform\Metadata\ApiResource;
 
 #[ApiResource(
-    shortName: 'RelatedResource',
+    shortName: 'ResourceRelated',
 )]
 class JsonSchemaResourceRelated
 {