fix tests

Author: Maxcastel

tests/Fixtures/app/AppKernel.php

@@ -286,7 +286,7 @@ class_exists(NativePasswordHasher::class) ? 'password_hashers' : 'encoders' => [
                 'parameters' => [
                     ApiPlatform\Metadata\HeaderParameter::class => [
                         'key' => 'X-API-Key',
-                        'required' => true,
+                        'required' => false,
                         'description' => 'API key for authentication',
                         'schema' => ['type' => 'string'],
                     ],
@@ -335,7 +335,7 @@ class_exists(NativePasswordHasher::class) ? 'password_hashers' : 'encoders' => [
                     'parameters' => [
                         ApiPlatform\Metadata\HeaderParameter::class => [
                             'key' => 'X-API-Key',
-                            'required' => true,
+                            'required' => false,
                             'description' => 'API key for authentication',
                             'schema' => ['type' => 'string'],
                         ],
@@ -354,7 +354,7 @@ class_exists(NativePasswordHasher::class) ? 'password_hashers' : 'encoders' => [
                 'parameters' => [
                     ApiPlatform\Metadata\HeaderParameter::class => [
                         'key' => 'X-API-Key',
-                        'required' => true,
+                        'required' => false,
                         'description' => 'API key for authentication',
                         'schema' => ['type' => 'string'],
                     ],

tests/Functional/DefaultParametersTest.php

@@ -46,7 +46,7 @@ public static function getResources(): array
      * api_platform.defaults.parameters with:
      *   HeaderParameter:
      *     key: 'X-API-Key'
-     *     required: true
+     *     required: false
      *     description: 'API key for authentication'
      *
      * The parameter appears in ALL resources and ALL their operations in the OpenAPI output.
@@ -84,7 +84,7 @@ public function testDefaultParameterAppearsInOpenApiForAllOperations(): void
                         $this->assertSame('X-API-Key', $param['name']);
                         $this->assertSame('header', $param['in']);
                         $this->assertSame('API key for authentication', $param['description']);
-                        $this->assertTrue($param['required']);
+                        $this->assertFalse($param['required']);
                         $this->assertFalse($param['deprecated']);
                         $this->assertArrayHasKey('schema', $param);
                         $this->assertSame('string', $param['schema']['type']);
@@ -148,4 +148,21 @@ public function testDefaultParameterAppearsInMultipleOperationTypes(): void
                 implode(', ', array_keys($operationMethodsWithParameter)))
         );
     }
+
+    public function testDefaultParametersDoNotBreakJsonLdDocumentation(): void
+    {
+        $response = self::createClient()->request('GET', '/docs.jsonld', [
+            'headers' => ['Accept' => 'application/ld+json'],
+        ]);
+
+        $this->assertResponseIsSuccessful();
+        $content = $response->toArray();
+
+        $this->assertArrayHasKey('@context', $content);
+
+        $this->assertTrue(
+            isset($content['entrypoint']) || isset($content['hydra:supportedClass']),
+            'JSON-LD response should have either "entrypoint" or "hydra:supportedClass" key'
+        );
+    }
 }

tests/Functional/JsonSchema/JsonSchemaTest.php

@@ -305,7 +305,7 @@ public function testJsonSchemaCanBeGeneratedWithDefaultParameters(): void
             foreach ($parameters as $parameter) {
                 if ('X-API-Key' === $parameter->getKey()) {
                     $hasDefaultParameters = true;
-                    $this->assertTrue($parameter->getRequired());
+                    $this->assertFalse($parameter->getRequired());
                     $this->assertSame('API key for authentication', $parameter->getDescription());
                     break;
                 }