fake parameters are only created when feature is enabled (#30)

Parameters not provided in a variation will be provided as NULL

Author: davidhoelzel

docs/BundleConfiguration.md

@@ -7,6 +7,7 @@ The bundle provides following defaults:
 ```yaml
 twig_doc:
   doc_identifier: TWIG_DOC
+  use_fake_parameter: false
   directories:
     - '%twig.default_path%/components'
   categories:
@@ -51,6 +52,12 @@ MY_DOC_IDENTIFIER#}
 <div class="fancy-component"></div>
 ```
 
+### Fake Parameters
+
+By default, the creation of fake parameters is disabled!
+
+When enabled, the bundle fakes parameters based on parameter-config of the component
+
 ### Categories
 
 The bundle groups components into categories and optionally into sub-categories.

docs/ComponentConfiguration.md

@@ -85,8 +85,46 @@ components:
 
 ### Parameter Provision
 
-You must provide the types of your template parameters in the configuration. 
+You must provide the types of your template parameters in the configuration.
 As twig templates are not aware of types, there is no other possibility at the moment.
+
+Unless you enable use_fake_parameter in the bundle configuration, parameter-values for the components 
+must be configured in each variation for this component! 
+
+`use_fake_parameter: false`
+
+When a parameter is not provided in a variation, it will be set to NULL.
+
+Objects will not be provided as objects, only as arrays based on the variation configuration.
+
+e.g:
+```yaml
+...
+parameters:
+  car: App\Entity\Car
+  owner: String
+variation:
+  blue:
+    car: 
+      color: blue
+      manufacturer:
+        name: Mitsubishi
+```
+will result that car is provided as an array when the component is rendered:
+```php
+[
+  'color' => 'blue',
+  'manufacturer' => [
+    'name' => 'Mitsubishi'
+  ],
+  'owner' => null
+]
+```
+
+When you do not call fancy object-methods in your component-templates, this should suffice for most cases.
+
+`use_fake_paramter: true`
+
 As this bundle makes use of [Nelmio/Alice](https://github.com/nelmio/alice) and [FakerPhp](https://fakerphp.github.io), all you need to do is
 define the types of your parameters in the component configuration.
 The bundle will take care of creating a set of parameters for every component.

phpunit.xml.dist

@@ -1,11 +1,9 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
          xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
-         bootstrap="tests/bootstrap.php"
          cacheDirectory=".phpunit.cache"
          executionOrder="depends,defects"
          requireCoverageMetadata="true"
-         beStrictAboutCoverageMetadata="true"
          beStrictAboutOutputDuringTests="true"
          failOnRisky="true"
          failOnWarning="true"

src/Component/ComponentItemFactory.php

@@ -15,7 +15,8 @@
     public function __construct(
         private ValidatorInterface $validator,
         private CategoryService $categoryService,
-        private Faker $faker
+        private Faker $faker,
+        private bool $useFakeParams
     ) {
     }
 
@@ -105,7 +106,7 @@ private function parseVariations(?array $variations, ?array $parameters): array
 
         if (!$variations) {
             return [
-                'default' => $this->faker->getFakeData($parameters),
+                'default' => $this->createVariationParameters($parameters, []),
             ];
         }
 
@@ -125,11 +126,34 @@ private function createVariationParameters(array $parameters, array $variation):
     {
         $params = [];
 
+        if ($this->useFakeParams) {
+            return $this->createFakeParamValues($parameters, $variation);
+        }
+
+        foreach ($parameters as $name => $type) {
+            if (\is_array($type)) {
+                $params[$name] = $this->createVariationParameters($type, $variation[$name] ?? []);
+            } else {
+                $params[$name] = $variation[$name] ?? null;
+            }
+        }
+
+        return $params;
+    }
+
+    private function createFakeParamValues(array $parameters, array $variation): array
+    {
+        $params = [];
+
         foreach ($parameters as $name => $type) {
             if (\is_array($type)) {
                 $paramValue[$name] = $this->createVariationParameters($type, $variation[$name] ?? []);
             } else {
-                $paramValue = $this->faker->getFakeData([$name => $type], $variation[$name] ?? null);
+                if (\array_key_exists($name, $variation) && null === $variation[$name]) {
+                    $paramValue = [$name => null];
+                } else {
+                    $paramValue = $this->faker->getFakeData([$name => $type], $variation[$name] ?? null);
+                }
             }
             $params += $paramValue;
         }

src/Component/Data/Generator/NullGenerator.php

@@ -10,7 +10,7 @@ class NullGenerator implements GeneratorInterface
 {
     public function supports(string $type, mixed $context = null): bool
     {
-        return null === $context || '' === $context;
+        return true;
     }
 
     public function generate(string $type, mixed $context = null): null

src/DependencyInjection/Configuration.php

@@ -21,6 +21,7 @@ public function getConfigTreeBuilder(): TreeBuilder
                         ->thenInvalid('The twig_doc documentation identifier must match \w (regex)')
                     ->end()
                 ->end()
+                ->booleanNode('use_fake_parameter')->defaultFalse()->end()
                 ->arrayNode('breakpoints')
                     ->defaultValue([
                         'small' => 240,

src/DependencyInjection/TwigDocExtension.php

@@ -36,5 +36,8 @@ public function load(array $configs, ContainerBuilder $container): void
 
         $definition = $container->getDefinition('twig_doc.service.faker');
         $definition->setArgument('$generators', tagged_iterator('twig_doc.data_generator'));
+
+        $definition = $container->getDefinition('twig_doc.service.component_factory');
+        $definition->setArgument('$useFakeParams', $config['use_fake_parameter']);
     }
 }

templates/component/_invalid_component.html.twig

@@ -35,7 +35,7 @@
         {% if loop.first %}
         <ul>
         {% endif %}
-            <li>{{ key }}: {{ value }}</li>
+            <li>{{ key }}</li>
         {% if loop.last %}
         </ul>
         {% endif %}

tests/Functional/Service/ComponentItemFactoryTest.php

@@ -18,6 +18,7 @@
 use Qossmic\TwigDocBundle\Exception\InvalidComponentConfigurationException;
 use Qossmic\TwigDocBundle\Service\CategoryService;
 use Qossmic\TwigDocBundle\Tests\TestApp\Entity\Car;
+use Qossmic\TwigDocBundle\Tests\TestApp\Test\ConfigurableContainerTrait;
 use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
 
 #[CoversClass(ComponentItemFactory::class)]
@@ -29,6 +30,8 @@
 #[UsesClass(NullGenerator::class)]
 class ComponentItemFactoryTest extends KernelTestCase
 {
+    use ConfigurableContainerTrait;
+
     #[DataProvider('getValidComponents')]
     public function testValidComponent(array $componentData): void
     {
@@ -115,8 +118,10 @@ public function testFactoryCreatesDefaultVariationWithParameterTypes(): void
             ],
         ];
 
+        $container = $this->createContainer(['twig_doc' => ['use_fake_parameter' => true]]);
+
         /** @var ComponentItemFactory $factory */
-        $factory = self::getContainer()->get('twig_doc.service.component_factory');
+        $factory = $container->get('twig_doc.service.component_factory');
 
         $component = $factory->create($data);
 
@@ -135,6 +140,40 @@ public function testFactoryCreatesDefaultVariationWithParameterTypes(): void
         self::assertIsFloat($component->getVariations()['default']['complex']['amount']);
     }
 
+    public function testCreateObjectParameter(): void
+    {
+        $data = [
+            'name' => 'component',
+            'title' => 'title',
+            'description' => 'description',
+            'category' => 'MainCategory',
+            'path' => 'path',
+            'renderPath' => 'renderPath',
+            'parameters' => [
+                'car' => Car::class,
+            ],
+            'variations' => [
+                'variation1' => [
+                    'car' => [
+                        'color' => 'blue',
+                    ],
+                ],
+                'variation2' => [
+                    'car' => null,
+                ],
+            ],
+        ];
+
+        $container = $this->createContainer(['twig_doc' => ['use_fake_parameter' => true]]);
+
+        /** @var ComponentItemFactory $factory */
+        $factory = $container->get('twig_doc.service.component_factory');
+
+        $component = $factory->create($data);
+
+        self::assertInstanceOf(ComponentItem::class, $component);
+    }
+
     public function testCreateKeepsParamValueFromVariation(): void
     {
         $data = [
@@ -200,11 +239,14 @@ public function testCreateForObjectParameter(): void
         static::assertIsArray($variations['fuchsia']);
         static::assertArrayHasKey('car', $variations['fuchsia']);
 
-        $car = $variations['fuchsia']['car'];
+        $carData = $variations['fuchsia']['car'];
 
-        static::assertInstanceOf(Car::class, $car);
-        static::assertEquals('fuchsia', $car->getColor());
-        static::assertEquals('Mitsubishi', $car->getManufacturer()->getName());
+        static::assertEquals([
+            'color' => 'fuchsia',
+            'manufacturer' => [
+                'name' => 'Mitsubishi',
+            ],
+        ], $carData);
     }
 
     public function testCreateForParamWithOptionalVariationValue(): void
@@ -224,7 +266,7 @@ public function testCreateForParamWithOptionalVariationValue(): void
             'variations' => [
                 'variation1' => [
                     'stringParam' => 'Some cool hipster text',
-                    'optionalEmpty' => '',
+                    'optionalEmpty' => null,
                 ],
             ],
         ];
@@ -238,7 +280,7 @@ public function testCreateForParamWithOptionalVariationValue(): void
         self::assertIsArray($variations);
         self::assertArrayHasKey('variation1', $variations);
         self::assertArrayHasKey('secondParam', $variations['variation1']);
-        self::assertIsString($variations['variation1']['secondParam']);
+        self::assertNull($variations['variation1']['secondParam']);
         self::assertNull($variations['variation1']['optionalEmpty']);
     }
 

tests/TestApp/Kernel.php

@@ -17,9 +17,9 @@ class Kernel extends SymfonyKernel
 {
     use MicroKernelTrait;
 
-    public function __construct()
+    public function __construct(string $environment = 'test', bool $debug = false, private readonly array $extraConfigs = [])
     {
-        parent::__construct('test', false);
+        parent::__construct($environment, $debug);
     }
 
     public function registerBundles(): iterable
@@ -42,6 +42,10 @@ protected function configureContainer(ContainerBuilder $container, LoaderInterfa
         $loader->load(__DIR__.'/config/services.php');
         $loader->load(__DIR__.'/config/packages/*.php', 'glob');
         $loader->load(__DIR__.'/config/packages/*.yaml', 'glob');
+
+        foreach ($this->extraConfigs as $name => $config) {
+            $container->prependExtensionConfig($name, $config);
+        }
     }
 
     public function getCacheDir(): string