Revert to --purge-with-delete

Author: GromNaN

src/Command/LoadDataFixturesDoctrineODMCommand.php

@@ -44,7 +44,7 @@ protected function configure(): void
             ->addOption('group', null, InputOption::VALUE_IS_ARRAY | InputOption::VALUE_REQUIRED, 'Only load fixtures that belong to this group (use with --services)')
             ->addOption('append', null, InputOption::VALUE_NONE, 'Append the data fixtures instead of flushing the database first.')
             ->addOption('dm', null, InputOption::VALUE_REQUIRED, 'The document manager to use for this command.')
-            ->addOption('purge', null, InputOption::VALUE_OPTIONAL, 'Purge the database before loading the fixtures. If set to "delete", collections will be kept and documents deleted instead of dropping the collections.', null, self::getPurgeModes(...))
+            ->addOption('purge-with-delete', null, InputOption::VALUE_NONE, 'Purge the database using deleteMany() instead of dropping and recreating the collections.')
             ->setHelp(<<<'EOT'
 The <info>doctrine:mongodb:fixtures:load</info> command loads data fixtures from your application:
 
@@ -58,9 +58,9 @@ protected function configure(): void
 
     <info>php %command.full_name%</info> <comment>--group=group1</comment>
 
-If the collection uses search indexes or encryption, you can use the <info>--purge=delete</info> option to keep the collections instead of dropping them when purging the database:
+If the collection uses search indexes or encryption, you can use the <info>--purge-with-delete</info> option to keep the collections instead of dropping them when purging the database:
 
-  <info>php %command.full_name%</info> --purge=delete
+  <info>php %command.full_name%</info> --purge-with-delete
 EOT
         );
     }
@@ -72,13 +72,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
 
         $ui = new SymfonyStyle($input, $output);
 
-        if ($input->hasOption('purge')) {
-            if ($input->getOption('append')) {
-                $ui->error('The --purge option cannot be used with the --append option.');
-
-                return self::INVALID;
-            }
-        } elseif ($input->isInteractive() && ! $input->getOption('append')) {
+        if ($input->isInteractive() && ! $input->getOption('append')) {
             $helper   = $this->getHelper('question');
             $question = new ConfirmationQuestion('Careful, database will be purged. Do you want to continue (y/N) ?', false);
 
@@ -102,19 +96,17 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         }
 
         $purger = new MongoDBPurger($dm);
-        $purge  = $input->getOption('purge');
-        if ($purge) {
+        if ($input->getOption('purge-with-delete')) {
             if (! class_exists(MongoDBPurgeMode::class)) {
                 $ui->error('The --purge-with-delete option requires doctrine/data-fixtures >= 2.1.0.');
 
                 return self::INVALID;
             }
 
-            $purger->setPurgeMode(MongoDBPurgeMode::tryFrom($purge) ?? throw new InvalidArgumentException('Invalid purge mode: ' . $purge));
+            $purger->setPurgeMode(MongoDBPurgeMode::Delete);
         }
 
         $executor = new MongoDBExecutor($dm, $purger);
-
         $executor->setLogger(new class ($output) extends AbstractLogger {
             public function __construct(private OutputInterface $output)
             {

tests/Command/LoadDataFixturesDoctrineODMCommandTest.php

@@ -60,6 +60,18 @@ public function testExecute(): void
         $commandTester->execute([], ['interactive' => false]);
 
         $output = $commandTester->getDisplay();
+        $this->assertStringContainsString('purging database', $output);
+        $this->assertStringContainsString('loading Doctrine\Bundle\MongoDBBundle\Tests\Fixtures\CommandBundle\DataFixtures\UserFixtures', $output);
+        $this->assertStringContainsString('loading Doctrine\Bundle\MongoDBBundle\Tests\Fixtures\CommandBundle\DataFixtures\OtherFixtures', $output);
+    }
+
+    public function testExecutePurgeWithDelete(): void
+    {
+        $commandTester = new CommandTester($this->command);
+        $commandTester->execute(['--purge-with-delete'], ['interactive' => false]);
+
+        $output = $commandTester->getDisplay();
+        $this->assertStringContainsString('purging database', $output);
         $this->assertStringContainsString('loading Doctrine\Bundle\MongoDBBundle\Tests\Fixtures\CommandBundle\DataFixtures\UserFixtures', $output);
         $this->assertStringContainsString('loading Doctrine\Bundle\MongoDBBundle\Tests\Fixtures\CommandBundle\DataFixtures\OtherFixtures', $output);
     }