Set user agent string (#140)

* Set user agent string * this won't work * fix mock * Fixed test
acquia · Jun 10, 2020 · 71cff36 · 71cff36
1 parent 77426ad
commit 71cff36
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/src/Command/Api/ApiCommandBase.php b/src/Command/Api/ApiCommandBase.php
@@ -48,6 +48,7 @@ class ApiCommandBase extends CommandBase {
    * @param \Symfony\Component\Console\Output\OutputInterface $output
    *
    * @throws \Acquia\Cli\Exception\AcquiaCliException
+   * @throws \Exception
    */
   protected function initialize(InputInterface $input, OutputInterface $output) {
     parent::initialize($input, $output);
@@ -87,6 +88,8 @@ protected function execute(InputInterface $input, OutputInterface $output) {
     }
 
     $path = $this->getRequestPath($input);
+    $user_agent = sprintf("acli/%s", $this->getApplication()->getVersion());
+    $acquia_cloud_client->addOption('headers', ['User-Agent' => $user_agent]);
     $response = $acquia_cloud_client->request($this->method, $path);
     // @todo Add syntax highlighting to json output.
     $contents = json_encode($response, JSON_PRETTY_PRINT);

diff --git a/tests/phpunit/src/TestBase.php b/tests/phpunit/src/TestBase.php
@@ -182,6 +182,7 @@ protected function setUp($output = NULL): void {
     $this->cloudDatastore = new JsonFileStore($this->cloudConfigFilepath, 1);
     $this->amplitudeProphecy = $this->prophet->prophesize(Amplitude::class);
     $this->clientProphecy = $this->prophet->prophesize(Client::class);
+    $this->clientProphecy->addOption('headers', ['User-Agent' => 'acli/UNKNOWN'])->willReturn();
     $this->localMachineHelper = new LocalMachineHelper($this->input, $output, $logger);
     $this->clientServiceProphecy = $this->prophet->prophesize(ClientService::class);
     $this->clientServiceProphecy->getClient()->willReturn($this->clientProphecy->reveal());