CLI-1365: Warn when logs need to be created

composer.json

@@ -47,7 +47,7 @@
         "symfony/yaml": "^6.3",
         "thecodingmachine/safe": "^2.4",
         "typhonius/acquia-logstream": "^0.0.14",
-        "typhonius/acquia-php-sdk-v2": "^3.3.2",
+        "typhonius/acquia-php-sdk-v2": "^3.4.0",
         "vlucas/phpdotenv": "^5.5",
         "zumba/amplitude-php": "^1.0.4"
     },

src/Command/Api/ApiBaseCommand.php

@@ -97,6 +97,7 @@ protected function interact(InputInterface $input, OutputInterface $output): voi
     /**
      * @throws \Acquia\Cli\Exception\AcquiaCliException
      * @throws \JsonException
+     * @throws \AcquiaCloudApi\Exception\ApiErrorException
      */
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
@@ -122,8 +123,9 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             $response = $acquiaCloudClient->request($this->method, $path);
             $exitCode = 0;
         } catch (ApiErrorException $exception) {
-            // Ignore PhpStorm warning here.
-            // @see https://youtrack.jetbrains.com/issue/WI-77190/Exception-is-never-thrown-when-thrown-from-submethod
+            if ($input->isInteractive()) {
+                throw $exception;
+            }
             $response = $exception->getResponseBody();
             $exitCode = 1;
         }

src/EventListener/ExceptionListener.php

@@ -10,6 +10,7 @@
 use Symfony\Component\Console\Event\ConsoleErrorEvent;
 use Symfony\Component\Console\Exception\CommandNotFoundException;
 use Symfony\Component\Console\Exception\RuntimeException;
+use TypeError;
 
 /**
  * Make exceptions warm and cuddly.
@@ -82,6 +83,9 @@ public function onConsoleError(ConsoleErrorEvent $event): void
         }
 
         if ($error instanceof ApiErrorException) {
+            if (($command = $event->getCommand()) && $error->getResponseBody()->error === 'not_found' && $command->getName() === 'api:environments:log-download') {
+                $this->helpMessages[] = "You must create logs (api:environments:log-create) prior to downloading them";
+            }
             switch ($errorMessage) {
                 case "There are no available Cloud IDEs for this application.\n":
                     $this->helpMessages[] = "Delete an existing IDE via <bg=$this->messagesBgColor;fg=$this->messagesFgColor;options=bold>acli ide:delete</> or contact your Account Manager or Acquia Sales to purchase additional IDEs.";
@@ -96,7 +100,7 @@ public function onConsoleError(ConsoleErrorEvent $event): void
             }
         }
 
-        if ($error instanceof \TypeError && str_contains($error->getMessage(), 'AcquiaCloudApi\Response')) {
+        if ($error instanceof TypeError && str_contains($error->getMessage(), 'AcquiaCloudApi\Response')) {
             $newErrorMessage = 'Cloud Platform API returned an unexpected data type. This could be due to missing permissions or a problem with your Cloud Platform application.';
         }
 

tests/phpunit/src/CommandTestBase.php

@@ -81,7 +81,7 @@ protected function setCommand(CommandBase $command): void
      *   An array of strings representing each input passed to the command input
      *   stream.
      */
-    protected function executeCommand(array $args = [], array $inputs = [], int $verbosity = Output::VERBOSITY_VERY_VERBOSE): void
+    protected function executeCommand(array $args = [], array $inputs = [], int $verbosity = OutputInterface::VERBOSITY_VERY_VERBOSE, ?bool $interactive = true): void
     {
         $cwd = $this->projectDir;
         $tester = $this->getCommandTester();
@@ -96,7 +96,7 @@ protected function executeCommand(array $args = [], array $inputs = [], int $ver
         }
 
         try {
-            $tester->execute($args, ['verbosity' => $verbosity]);
+            $tester->execute($args, ['verbosity' => $verbosity, 'interactive' => $interactive]);
         } catch (Exception $e) {
             if (getenv('ACLI_PRINT_COMMAND_OUTPUT')) {
                 print $this->getDisplay();

tests/phpunit/src/Commands/Api/ApiCommandTest.php

@@ -10,6 +10,7 @@
 use Acquia\Cli\Exception\AcquiaCliException;
 use Acquia\Cli\Tests\CommandTestBase;
 use AcquiaCloudApi\Exception\ApiErrorException;
+use Overtrue\PHPLint\Output\OutputInterface;
 use Symfony\Component\Console\Exception\MissingInputException;
 use Symfony\Component\Filesystem\Path;
 use Symfony\Component\Yaml\Yaml;
@@ -105,6 +106,10 @@ public function testArgumentsInteractionValidationFormat(): void
 
     /**
      * Tests invalid UUID.
+     *
+     * @throws \JsonException
+     * @throws \AcquiaCloudApi\Exception\ApiErrorException
+     * @throws \Exception
      */
     public function testApiCommandErrorResponse(): void
     {
@@ -117,7 +122,7 @@ public function testApiCommandErrorResponse(): void
             ->willThrow(new ApiErrorException($mockBody))
             ->shouldBeCalled();
 
-        // ApiCommandBase::convertApplicationAliastoUuid() will try to convert the invalid string to a uuid:
+        // ApiCommandBase::convertApplicationAliasToUuid() will try to convert the invalid string to a UUID:
         $this->clientProphecy->addQuery('filter', 'hosting=@*:' . $invalidUuid);
         $this->clientProphecy->request('get', '/applications')->willReturn([]);
 
@@ -128,7 +133,7 @@ public function testApiCommandErrorResponse(): void
             '0',
             // Would you like to link the Cloud application Sample application to this repository?
             'n',
-        ]);
+        ], \Symfony\Component\Console\Output\OutputInterface::VERBOSITY_VERY_VERBOSE, false);
 
         // Assert.
         $output = $this->getDisplay();

tests/phpunit/src/Commands/Remote/DrushCommandTest.php

@@ -27,13 +27,11 @@ public static function providerTestRemoteDrushCommand(): array
         return [
             [
                 [
-                    '-vvv' => '',
                     'drush_command' => 'status --fields=db-status',
                 ],
             ],
             [
                 [
-                    '-vvv' => '',
                     'drush_command' => 'status --fields=db-status',
                 ],
             ],
@@ -42,7 +40,6 @@ public static function providerTestRemoteDrushCommand(): array
 
     /**
      * @dataProvider providerTestRemoteDrushCommand
-     * @group serial
      */
     public function testRemoteDrushCommand(array $args): void
     {

tests/phpunit/src/Misc/LocalMachineHelperTest.php

@@ -40,7 +40,6 @@ public function testExecuteFromCmd(bool $interactive, bool|null $isTty, bool|nul
     {
         $localMachineHelper = $this->localMachineHelper;
         $localMachineHelper->setIsTty($isTty);
-        $this->input->setInteractive($interactive);
         $process = $localMachineHelper->executeFromCmd('echo "hello world"', null, null, $printOutput);
         $this->assertTrue($process->isSuccessful());
         assert(is_a($this->output, BufferedOutput::class));