minor #61323 [JsonStreamer] fix expected stream to native value transformers (xabbuh)

nicolas-grekas · nicolas-grekas · commit 1df3a3ded975 · 2025-08-04T15:37:32.000+02:00
This PR was merged into the 7.3 branch. Discussion ---------- [JsonStreamer] fix expected stream to native value transformers | Q | A | ------------- | --- | Branch? | 7.3 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | | License | MIT This was probably a copy-and-paste mistake from the corresponding write test. With PHPUnit < 12.0 this isn't an issue as closure are not considered when testing for equality. This changes with PHPUnit 12 where `sebastian/comparator` 7 is used which also compares closures. Commits ------- 4736edb fix expected stream to native value transformers
diff --git a/src/Symfony/Component/JsonStreamer/Tests/Mapping/Read/AttributePropertyMetadataLoaderTest.php b/src/Symfony/Component/JsonStreamer/Tests/Mapping/Read/AttributePropertyMetadataLoaderTest.php
@@ -44,8 +44,8 @@ public function testRetrieveValueTransformer()
         $this->assertEquals([
             'id' => new PropertyMetadata('id', Type::string(), [], [DivideStringAndCastToIntValueTransformer::class]),
             'active' => new PropertyMetadata('active', Type::string(), [], [StringToBooleanValueTransformer::class]),
-            'name' => new PropertyMetadata('name', Type::string(), [], [\Closure::fromCallable('strtolower')]),
-            'range' => new PropertyMetadata('range', Type::string(), [], [\Closure::fromCallable(DummyWithValueTransformerAttributes::concatRange(...))]),
+            'name' => new PropertyMetadata('name', Type::string(), [], [\Closure::fromCallable('strtoupper')]),
+            'range' => new PropertyMetadata('range', Type::string(), [], [\Closure::fromCallable(DummyWithValueTransformerAttributes::explodeRange(...))]),
         ], $loader->load(DummyWithValueTransformerAttributes::class));
     }
 
