fix: Use normalisation context when none is provided in ApiTestAssertionsTrait

kevin-macquer-omnilog · kevin-macquer-omnilog · commit 7bf75e3ef9b8 · 2024-02-12T14:54:45.000+01:00
diff --git a/src/Symfony/Bundle/Test/ApiTestAssertionsTrait.php b/src/Symfony/Bundle/Test/ApiTestAssertionsTrait.php
@@ -119,6 +119,8 @@ public static function assertMatchesResourceCollectionJsonSchema(string $resourc
             $operation = $operationName ? (new GetCollection())->withName($operationName) : new GetCollection();
         }
 
+        $serializationContext = $serializationContext ?? $operation->getNormalizationContext();
+
         $schema = $schemaFactory->buildSchema($resourceClass, $format, Schema::TYPE_OUTPUT, $operation, null, ($serializationContext ?? []) + [BackwardCompatibleSchemaFactory::SCHEMA_DRAFT4_VERSION => true]);
 
         static::assertMatchesJsonSchema($schema->getArrayCopy());
@@ -134,6 +136,8 @@ public static function assertMatchesResourceItemJsonSchema(string $resourceClass
             $operation = $operationName ? (new Get())->withName($operationName) : new Get();
         }
 
+        $serializationContext = $serializationContext ?? $operation->getNormalizationContext();
+
         $schema = $schemaFactory->buildSchema($resourceClass, $format, Schema::TYPE_OUTPUT, $operation, null, ($serializationContext ?? []) + [BackwardCompatibleSchemaFactory::SCHEMA_DRAFT4_VERSION => true]);
 
         static::assertMatchesJsonSchema($schema->getArrayCopy());

Original file line number	Diff line number	Diff line change
`@@ -119,6 +119,8 @@ public static function assertMatchesResourceCollectionJsonSchema(string $resourc`
`119`	`119`	`$operation = $operationName ? (new GetCollection())->withName($operationName) : new GetCollection();`
`120`	`120`	`}`
`121`	`121`
	`122`	`+ $serializationContext = $serializationContext ?? $operation->getNormalizationContext();`
	`123`	`+`
`122`	`124`	`$schema = $schemaFactory->buildSchema($resourceClass, $format, Schema::TYPE_OUTPUT, $operation, null, ($serializationContext ?? []) + [BackwardCompatibleSchemaFactory::SCHEMA_DRAFT4_VERSION => true]);`
`123`	`125`
`124`	`126`	`static::assertMatchesJsonSchema($schema->getArrayCopy());`
`@@ -134,6 +136,8 @@ public static function assertMatchesResourceItemJsonSchema(string $resourceClass`
`134`	`136`	`$operation = $operationName ? (new Get())->withName($operationName) : new Get();`
`135`	`137`	`}`
`136`	`138`
	`139`	`+ $serializationContext = $serializationContext ?? $operation->getNormalizationContext();`
	`140`	`+`
`137`	`141`	`$schema = $schemaFactory->buildSchema($resourceClass, $format, Schema::TYPE_OUTPUT, $operation, null, ($serializationContext ?? []) + [BackwardCompatibleSchemaFactory::SCHEMA_DRAFT4_VERSION => true]);`
`138`	`142`
`139`	`143`	`static::assertMatchesJsonSchema($schema->getArrayCopy());`