Improve the OpenAPI normalizer (#2404)

* Improve the OpenAPI normalizer

* Fix tests

* Fix API Gateway normalizer

* Fix ApiGatewayNormalizerTest

Author: dunglas

.gitignore

@@ -7,3 +7,5 @@
 /tests/Fixtures/app/var/*
 /tests/Fixtures/app/cache/*
 /tests/Fixtures/app/logs/*
+/swagger.json
+/swagger.yaml

.travis.yml

@@ -52,6 +52,6 @@ script:
       vendor/bin/behat --suite=default --format=progress;
     fi
   - tests/Fixtures/app/console api:swagger:export --spec-version 2 > swagger.json && npx swagger-cli validate swagger.json && rm swagger.json
-  - tests/Fixtures/app/console api:swagger:export --spec-version 3 --yaml > swagger.yaml && npx swagger-cli validate swagger.yaml && rm swagger.yaml
+  - tests/Fixtures/app/console api:swagger:export --spec-version 2 --yaml > swagger.yaml && npx swagger-cli validate swagger.yaml && rm swagger.yaml
   - tests/Fixtures/app/console api:openapi:export > swagger.json && npx swagger-cli validate swagger.json && rm swagger.json
   - tests/Fixtures/app/console api:openapi:export --yaml > swagger.yaml && npx swagger-cli validate swagger.yaml && rm swagger.yaml

src/Bridge/Symfony/Bundle/Command/SwaggerCommand.php

@@ -15,7 +15,7 @@
 
 use ApiPlatform\Core\Documentation\Documentation;
 use ApiPlatform\Core\Metadata\Resource\Factory\ResourceNameCollectionFactoryInterface;
-use ApiPlatform\Core\OpenApi\Serializer\AbstractDocumentationNormalizer;
+use ApiPlatform\Core\Swagger\Serializer\DocumentationNormalizer;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Exception\InvalidOptionException;
 use Symfony\Component\Console\Input\InputInterface;
@@ -79,7 +79,7 @@ protected function execute(InputInterface $input, OutputInterface $output)
         }
 
         $documentation = new Documentation($this->resourceNameCollectionFactory->create(), $this->apiTitle, $this->apiDescription, $this->apiVersion, $this->apiFormats);
-        $data = $this->normalizer->normalize($documentation, AbstractDocumentationNormalizer::FORMAT, ['spec_version' => (int) $version]);
+        $data = $this->normalizer->normalize($documentation, DocumentationNormalizer::FORMAT, ['spec_version' => (int) $version]);
         $content = $input->getOption('yaml') ? Yaml::dump($data, 10, 2, Yaml::DUMP_OBJECT_AS_MAP | Yaml::DUMP_EMPTY_ARRAY_AS_SEQUENCE) : (json_encode($data, JSON_PRETTY_PRINT) ?: '');
 
         if (!empty($filename = $input->getOption('output')) && \is_string($filename)) {

src/Bridge/Symfony/Bundle/Resources/config/swagger.xml

@@ -39,34 +39,6 @@
             <tag name="serializer.normalizer" priority="17" />
         </service>
 
-        <service id="api_platform.openapi.normalizer.documentation" class="ApiPlatform\Core\OpenApi\Serializer\DocumentationNormalizer" public="false">
-            <argument type="service" id="api_platform.metadata.resource.metadata_factory" />
-            <argument type="service" id="api_platform.metadata.property.name_collection_factory" />
-            <argument type="service" id="api_platform.metadata.property.metadata_factory" />
-            <argument type="service" id="api_platform.resource_class_resolver" />
-            <argument type="service" id="api_platform.operation_method_resolver" />
-            <argument type="service" id="api_platform.operation_path_resolver" />
-            <argument type="service" id="api_platform.filter_locator" />
-            <argument type="service" id="api_platform.name_converter" on-invalid="ignore" />
-            <argument>%api_platform.oauth.enabled%</argument>
-            <argument>%api_platform.oauth.type%</argument>
-            <argument>%api_platform.oauth.flow%</argument>
-            <argument>%api_platform.oauth.tokenUrl%</argument>
-            <argument>%api_platform.oauth.authorizationUrl%</argument>
-            <argument>%api_platform.oauth.scopes%</argument>
-            <argument>%api_platform.swagger.api_keys%</argument>
-            <argument type="service" id="api_platform.subresource_operation_factory" />
-            <argument>%api_platform.collection.pagination.enabled%</argument>
-            <argument>%api_platform.collection.pagination.page_parameter_name%</argument>
-            <argument>%api_platform.collection.pagination.client_items_per_page%</argument>
-            <argument>%api_platform.collection.pagination.items_per_page_parameter_name%</argument>
-            <argument type="service" id="api_platform.formats_provider" />
-            <argument>%api_platform.collection.pagination.client_enabled%</argument>
-            <argument>%api_platform.collection.pagination.enabled_parameter_name%</argument>
-            <argument type="service" id="api_platform.swagger.normalizer.api_gateway" />
-            <tag name="serializer.normalizer" priority="18" />
-        </service>
-
         <service id="api_platform.swagger.command.swagger_command" class="ApiPlatform\Core\Bridge\Symfony\Bundle\Command\SwaggerCommand">
             <argument type="service" id="api_platform.serializer" />
             <argument type="service" id="api_platform.metadata.resource.name_collection_factory" />