fix message and add tests

acquia · Mar 14, 2024 · 6ef589d · 6ef589d
1 parent 807b79e
commit 6ef589d
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 2 deletions.
diff --git a/.phpstorm.meta.php b/.phpstorm.meta.php
@@ -4,6 +4,8 @@
   use AcquiaCloudApi\Response\AccountResponse;
   use AcquiaCloudApi\Response\ApplicationResponse;
   use AcquiaCloudApi\Response\ApplicationsResponse;
+  use AcquiaCloudApi\Response\CronResponse;
+  use AcquiaCloudApi\Response\CronsResponse;
   use AcquiaCloudApi\Response\DatabasesResponse;
   use AcquiaCloudApi\Response\EnvironmentResponse;
   use AcquiaCloudApi\Response\EnvironmentsResponse;
@@ -14,7 +16,9 @@
     'getApplicationByUuid' => ApplicationResponse::class,
     'getApplicationEnvironments' => EnvironmentsResponse::class,
     'getEnvironmentsDatabases' => DatabasesResponse::class,
-    'getEnvironment' => EnvironmentResponse::class
+    'getEnvironment' => EnvironmentResponse::class,
+    'getCron' => CronResponse::class,
+    'getCronJobsByEnvironmentId' => CronsResponse::class
   ]));
 
 }
diff --git a/src/EventListener/ExceptionListener.php b/src/EventListener/ExceptionListener.php
@@ -80,7 +80,7 @@ public function onConsoleError(ConsoleErrorEvent $event): void {
     }
 
     if ($error instanceof \TypeError && str_contains($error->getMessage(), 'AcquiaCloudApi\Response')) {
-      $newErrorMessage = 'Cloud API returned an unexpected data type; this could indicate a problem with your Cloud application.';
+      $newErrorMessage = 'Cloud Platform API returned an unexpected data type. This is not an issue with Acquia CLI but could indicate a problem with your Cloud Platform application.';
     }
 
     $this->helpMessages[] = "You can find Acquia CLI documentation at https://docs.acquia.com/acquia-cli/";

diff --git a/tests/phpunit/src/Application/ExceptionApplicationTest.php b/tests/phpunit/src/Application/ExceptionApplicationTest.php
@@ -93,4 +93,21 @@ public function testInvalidApplicationUuid(): void {
     self::assertStringContainsString('An alias consists of an application name', $buffer);
   }
 
+  /**
+   * @group serial
+   */
+  public function testApiTypeError(): void {
+    $tamper = function ($response): void {
+      $response[0]->server = [];
+    };
+    $this->mockRequest('getCronJobsByEnvironmentId', '24-a47ac10b-58cc-4372-a567-0e02b2c3d470', NULL, NULL, $tamper);
+    $this->setInput([
+      'command' => 'env:cron-copy',
+      'dest_env' => '24-a47ac10b-58cc-4372-a567-0e02b2c3d471',
+      'source_env' => '24-a47ac10b-58cc-4372-a567-0e02b2c3d470',
+    ]);
+    $buffer = $this->runApp();
+    self::assertStringContainsString('Cloud Platform API returned an unexpected data type.', $buffer);
+  }
+
 }