Automatically run console commands without asking for confirmation in…

… non-interactive mode.
spiral · Jan 8, 2024 · a048c3e · a048c3e
1 parent 09cac59
commit a048c3e
Show file tree

Hide file tree

Showing 4 changed files with 25 additions and 9 deletions.
diff --git a/src/Console/Command/Migrate/AbstractCommand.php b/src/Console/Command/Migrate/AbstractCommand.php
@@ -45,7 +45,7 @@ protected function verifyEnvironment(string $message = 'Confirmation is required
 
         $this->error($message);
 
-        if (!$this->askConfirmation()) {
+        if (!$this->askConfirmation() || !$this->isInteractive()) {
             $this->comment('Cancelling operation...');
 
             return false;

diff --git a/tests/src/Console/Command/CycleOrm/MigrateCommandTest.php b/tests/src/Console/Command/CycleOrm/MigrateCommandTest.php
@@ -37,12 +37,25 @@ protected function setUp(): void
     public function testMigrate(): void
     {
         // Create migration
-        $this->assertConsoleCommandOutputContainsStrings('cycle:migrate', [], self::USER_MIGRATION);
+        $this->assertConsoleCommandOutputContainsStrings('cycle:migrate', [
+            '--no-interaction' => true,
+        ], self::USER_MIGRATION);
 
-        $this->assertConsoleCommandOutputContainsStrings('cycle:migrate', [], [
+        $this->assertConsoleCommandOutputContainsStrings('cycle:migrate', [
+            '--no-interaction' => true,
+        ], [
             'Outstanding migrations found',
-            'Detecting schema changes...',
+            'Please run `migrate` first.',
+        ]);
+
+        $this->assertConsoleCommandOutputContainsStrings('migrate', [
             'was successfully executed.',
+        ]);
+
+        $this->assertConsoleCommandOutputContainsStrings('cycle:migrate', [
+            '--no-interaction' => true,
+        ], [
+            'Detecting schema changes...',
             'no database changes has been detected',
         ]);
     }

diff --git a/tests/src/Console/Command/CycleOrm/SyncCommandTest.php b/tests/src/Console/Command/CycleOrm/SyncCommandTest.php
@@ -20,7 +20,10 @@ final class SyncCommandTest extends ConsoleTest
     #[Env('SAFE_MIGRATIONS', 'false')]
     public function testUnsafeSync(): void
     {
-        $output = $this->runCommand('cycle:sync');
+        $output = $this->runCommand('cycle:sync', [
+            '--no-interaction' => true,
+        ]);
+
         $this->assertStringContainsString('This operation is not recommended for production environment.', $output);
         $this->assertStringContainsString('Cancelling operation...', $output);
     }
@@ -58,8 +61,7 @@ public function testSchemaDefaultsShouldBePassedToCompiler(): void
     {
         $config['schema']['defaults'][SchemaInterface::TYPECAST_HANDLER][] = 'foo';
 
-        $memory = new class implements MemoryInterface
-        {
+        $memory = new class implements MemoryInterface {
             private mixed $data;
 
             public function loadData(string $section): mixed

diff --git a/tests/src/Console/Command/Migrate/MigrateCommandTest.php b/tests/src/Console/Command/Migrate/MigrateCommandTest.php
@@ -34,14 +34,15 @@ public function tesForceMigrate(): void
     public function testUnsafeMigrate(): void
     {
         $db = $this->initMigrations();
-        $output = $this->runCommand('migrate');
+        $output = $this->runCommand('migrate', [
+            '--no-interaction' => true,
+        ]);
         $this->assertStringContainsString('Confirmation is required to run migrations!', $output);
         $this->assertStringContainsString('Cancelling operation...', $output);
         $this->assertCount(1, $db->getTables());
     }
 
     /**
-     * @return void
      * @throws \Throwable
      */
     public function initMigrations(): DatabaseInterface