Merge pull request #191 from Deuchnord/ask-before-generate-whole-schema

Ask before adding all Schema.org's vocabulary

Author: soyuka

src/Command/GenerateTypesCommand.php

@@ -24,6 +24,7 @@
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Logger\ConsoleLogger;
 use Symfony\Component\Console\Output\OutputInterface;
+use Symfony\Component\Console\Question\ConfirmationQuestion;
 use Symfony\Component\Yaml\Parser;
 use Twig\Environment;
 use Twig\Extension\DebugExtension;
@@ -119,6 +120,13 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $config = $parser->parse(file_get_contents(self::DEFAULT_CONFIG_FILE));
             unset($parser);
         } else {
+            $helper = $this->getHelper('question');
+            $question = new ConfirmationQuestion('Your project has no config file. The entire Schema.org vocabulary will be built.'.PHP_EOL.'Continue? [yN]', false);
+
+            if (!$helper->ask($input, $output, $question)) {
+                return 0;
+            }
+
             $config = [];
         }
 

tests/Command/GenerateTypesCommandTest.php

@@ -15,6 +15,7 @@
 
 use ApiPlatform\SchemaGenerator\Command\GenerateTypesCommand;
 use PHPUnit\Framework\TestCase;
+use Symfony\Component\Console\Application;
 use Symfony\Component\Console\Tester\CommandTester;
 use Symfony\Component\Filesystem\Filesystem;
 
@@ -436,4 +437,17 @@ public function testSupersededProperties(): void
 PHP
             , $creativeWork);
     }
+
+    public function testGenerationWithoutConfigFileQuestion(): void
+    {
+        // No config file is given.
+        $application = new Application();
+        $application->add(new GenerateTypesCommand());
+
+        $command = $application->find('generate-types');
+        $commandTester = new CommandTester($command);
+        $commandTester->setInputs(['n']);
+        self::assertEquals(0, $commandTester->execute([]));
+        $this->assertRegExp('/The entire Schema\.org vocabulary will be built/', $commandTester->getDisplay());
+    }
 }