[BUGFIX] Document 403 response of operations with security in OpenAPI

Classes/Exception/OperationNotAllowedException.php

@@ -14,8 +14,8 @@ class OperationNotAllowedException extends AbstractException implements OpenApiS
     public static function getOpenApiResponse(): OpenApiResponse
     {
         return parent::getOpenApiResponse()
-            ->statusCode(SymfonyResponse::HTTP_NOT_FOUND)
-            ->description(self::translate('exception.resource_not_found.title'));
+            ->statusCode(SymfonyResponse::HTTP_FORBIDDEN)
+            ->description(self::translate('exception.operation_not_allowed.title'));
     }
 
     public function __construct(OperationInterface $operation, int $code)

Documentation/Miscellaneous/Changelog/Index.rst

@@ -6,6 +6,7 @@ Changelog
 
 5.0.0
 =====
+- OpenAPI specification documents ``403 Forbidden`` (instead of ``404 Not Found``) for operations with ``security`` or ``security_post_denormalize``, the same status as returned by the API.
 - [!!!] A new file reference created by ``uidLocal`` requires ``fileToken`` returned for the file by the API, otherwise ``400 Bad Request`` is returned. The same applies to a new file of a property of type ``File`` or a class extending it (``{"uid": 101, "fileToken": "..."}``). The token is returned in the response of upload endpoints and of operations with ``"fileTokens"=true`` in ``normalizationContext`` (files offered for selection, ``file.fileToken`` for file references). Pass ``fileToken`` when saving a reference. Custom upload code has to grant tokens with ``FileTokenService::grant()``. See :ref:`handling_file_upload_file_token`.
 - Invalid values of file and file reference properties (e.g. a text or a boolean instead of an identifier or an object) return ``400 Bad Request`` instead of HTTP 500 or clearing the property.
 - Files (classes extending Extbase's ``File``) nested in a response (e.g. members of a collection) are serialized like other objects, the same as in an item response, instead of the file reference output.

Tests/Unit/Exception/OpenApiSupportingExceptionTest.php

@@ -0,0 +1,53 @@
+<?php
+
+declare(strict_types=1);
+
+namespace SourceBroker\T3api\Tests\Unit\Exception;
+
+use PHPUnit\Framework\Attributes\DataProvider;
+use PHPUnit\Framework\Attributes\Test;
+use SourceBroker\T3api\Exception\AbstractException;
+use SourceBroker\T3api\Exception\BadRequestException;
+use SourceBroker\T3api\Exception\InvalidPaginationParameterException;
+use SourceBroker\T3api\Exception\OpenApiSupportingExceptionInterface;
+use SourceBroker\T3api\Exception\OperationNotAllowedException;
+use SourceBroker\T3api\Exception\OriginNotAllowedException;
+use SourceBroker\T3api\Exception\ResourceNotFoundException;
+use SourceBroker\T3api\Exception\UnsupportedMediaTypeException;
+use SourceBroker\T3api\Exception\ValidationException;
+use TYPO3\TestingFramework\Core\Unit\UnitTestCase;
+
+class OpenApiSupportingExceptionTest extends UnitTestCase
+{
+    // translation of the response description creates `Locales` singleton
+    protected bool $resetSingletonInstances = true;
+
+    /**
+     * @return array<string, array{class-string<AbstractException&OpenApiSupportingExceptionInterface>}>
+     */
+    public static function exceptionClassDataProvider(): array
+    {
+        return [
+            'BadRequestException' => [BadRequestException::class],
+            'InvalidPaginationParameterException' => [InvalidPaginationParameterException::class],
+            'OperationNotAllowedException' => [OperationNotAllowedException::class],
+            'OriginNotAllowedException' => [OriginNotAllowedException::class],
+            'ResourceNotFoundException' => [ResourceNotFoundException::class],
+            'UnsupportedMediaTypeException' => [UnsupportedMediaTypeException::class],
+            'ValidationException' => [ValidationException::class],
+        ];
+    }
+
+    /**
+     * @param class-string<AbstractException&OpenApiSupportingExceptionInterface> $exceptionClass
+     */
+    #[Test]
+    #[DataProvider('exceptionClassDataProvider')]
+    public function openApiResponseHasStatusCodeOfException(string $exceptionClass): void
+    {
+        // `getStatusCode()` does not depend on constructor arguments
+        $exception = (new \ReflectionClass($exceptionClass))->newInstanceWithoutConstructor();
+
+        self::assertSame($exception->getStatusCode(), $exceptionClass::getOpenApiResponse()->statusCode);
+    }
+}