CLI-1473: Pull default site if non-interactive. (#1839)

* Pull default site if non-interactive. If your site has multiple databases, acli pull:db --no-interaction will not pull the default database, which is the logical default. * reorder sites in question * array_unshift --------- Co-authored-by: Dane Powell <[email protected]>
acquia · Jan 9, 2025 · 428a6e2 · 428a6e2
1 parent 98ce828
commit 428a6e2
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/src/Command/CommandBase.php b/src/Command/CommandBase.php
@@ -1409,6 +1409,10 @@ private function getCloudSites(EnvironmentResponse $cloudEnvironment): array
         $process = $this->sshHelper->executeCommand($cloudEnvironment->sshUrl, $command, false);
         $sites = array_filter(explode("\n", trim($process->getOutput())));
         if ($process->isSuccessful() && $sites) {
+            if ($key = array_search('default', $sites, true)) {
+                unset($sites[$key]);
+                array_unshift($sites, 'default');
+            }
             return $sites;
         }
 

diff --git a/tests/phpunit/src/Commands/Pull/PullCommandTest.php b/tests/phpunit/src/Commands/Pull/PullCommandTest.php
@@ -55,7 +55,7 @@ public function testPull(): void
         $this->mockGetFilesystem($localMachineHelper);
         $parts = explode('.', $environment->ssh_url);
         $sitegroup = reset($parts);
-        $this->mockExecuteRsync($localMachineHelper, $environment, '/mnt/files/' . $sitegroup . '.' . $environment->name . '/sites/bar/files/', $this->projectDir . '/docroot/sites/bar/files');
+        $this->mockExecuteRsync($localMachineHelper, $environment, '/mnt/files/' . $sitegroup . '.' . $environment->name . '/sites/default/files/', $this->projectDir . '/docroot/sites/default/files');
         $this->command->sshHelper = $sshHelper->reveal();
 
         // Pull database.

diff --git a/tests/phpunit/src/Commands/Pull/PullFilesCommandTest.php b/tests/phpunit/src/Commands/Pull/PullFilesCommandTest.php
@@ -81,7 +81,7 @@ public function testRefreshCloudFiles(): void
         $this->mockGetFilesystem($localMachineHelper);
         $parts = explode('.', $selectedEnvironment->ssh_url);
         $sitegroup = reset($parts);
-        $this->mockExecuteRsync($localMachineHelper, $selectedEnvironment, '/mnt/files/' . $sitegroup . '.' . $selectedEnvironment->name . '/sites/bar/files/', $this->projectDir . '/docroot/sites/bar/files');
+        $this->mockExecuteRsync($localMachineHelper, $selectedEnvironment, '/mnt/files/' . $sitegroup . '.' . $selectedEnvironment->name . '/sites/default/files/', $this->projectDir . '/docroot/sites/default/files');
 
         $this->command->sshHelper = $sshHelper->reveal();
 

diff --git a/tests/phpunit/src/Commands/Push/PushFilesCommandTest.php b/tests/phpunit/src/Commands/Push/PushFilesCommandTest.php
@@ -145,8 +145,8 @@ protected function mockExecuteCloudRsync(
             'rsync',
             '-avPhze',
             'ssh -o StrictHostKeyChecking=no',
-            $this->projectDir . '/docroot/sites/bar/files/',
-            $environment->ssh_url . ':/mnt/files/' . $sitegroup . '.' . $environment->name . '/sites/bar/files',
+            $this->projectDir . '/docroot/sites/default/files/',
+            $environment->ssh_url . ':/mnt/files/' . $sitegroup . '.' . $environment->name . '/sites/default/files',
         ];
         $localMachineHelper->execute($command, Argument::type('callable'), null, true)
             ->willReturn($process->reveal())