Fix consider empty resource declaration as an actual resource

Author: abluchet

src/Metadata/Extractor/YamlExtractor.php

@@ -52,9 +52,7 @@ private function extractResources(array $resourcesYaml, string $path)
     {
         foreach ($resourcesYaml as $resourceName => $resourceYaml) {
             if (null === $resourceYaml) {
-                $this->resources[$resourceName] = null;
-
-                continue;
+                $resourceYaml = [];
             }
 
             if (!is_array($resourceYaml)) {

tests/Fixtures/FileConfigurations/bad_declaration.yml

@@ -0,0 +1,2 @@
+resources:
+    'ApiPlatform\Core\Tests\Fixtures\TestBundle\Entity\Dummy': ''

tests/Metadata/Resource/Factory/ExtractorResourceMetadataFactoryTest.php

@@ -17,6 +17,7 @@
 use ApiPlatform\Core\Metadata\Resource\Factory\ExtractorResourceNameCollectionFactory;
 use ApiPlatform\Core\Metadata\Resource\Factory\ResourceMetadataFactoryInterface;
 use ApiPlatform\Core\Metadata\Resource\ResourceMetadata;
+use ApiPlatform\Core\Tests\Fixtures\TestBundle\Entity\Dummy;
 use ApiPlatform\Core\Tests\Fixtures\TestBundle\Entity\FileConfigDummy;
 
 /**
@@ -35,6 +36,7 @@ public function testXmlCreateResourceMetadata($expectedResourceMetadata)
 
         $resourceMetadataFactory = new ExtractorResourceMetadataFactory(new XmlExtractor([$configPath]));
         $resourceMetadata = $resourceMetadataFactory->create(FileConfigDummy::class);
+        $resourceMetadataDummy = $resourceMetadataFactory->create(Dummy::class);
 
         $this->assertInstanceOf(ResourceMetadata::class, $resourceMetadata);
         $this->assertEquals($expectedResourceMetadata, $resourceMetadata);
@@ -125,6 +127,7 @@ public function testYamlCreateResourceMetadata(ResourceMetadata $expectedResourc
 
         $resourceMetadataFactory = new ExtractorResourceMetadataFactory(new YamlExtractor([$configPath]));
         $resourceMetadata = $resourceMetadataFactory->create(FileConfigDummy::class);
+        $resourceMetadataDummy = $resourceMetadataFactory->create(Dummy::class);
 
         $this->assertInstanceOf(ResourceMetadata::class, $resourceMetadata);
         $this->assertEquals($expectedResourceMetadata, $resourceMetadata);
@@ -217,4 +220,15 @@ public function testCreateWithMalformedYaml()
 
         (new ExtractorResourceMetadataFactory(new YamlExtractor([$configPath])))->create(FileConfigDummy::class);
     }
+
+    /**
+     * @expectedException \ApiPlatform\Core\Exception\InvalidArgumentException
+     * @expectedExceptionMessageRegExp /"ApiPlatform\\Core\\Tests\\Fixtures\\TestBundle\\Entity\\Dummy" setting is expected to be null or an array, string given in ".+\/Fixtures\/FileConfigurations\/bad_declaration\.yml"\./
+     */
+    public function testCreateWithBadDeclaration()
+    {
+        $configPath = __DIR__.'/../../../Fixtures/FileConfigurations/bad_declaration.yml';
+
+        (new ExtractorResourceMetadataFactory(new YamlExtractor([$configPath])))->create(FileConfigDummy::class);
+    }
 }