address comments

Author: Frank Natividad

storage/src/delete_hmac_key.php

@@ -40,6 +40,7 @@ function delete_hmac_key($accessId, $projectId)
     $hmacKey = $storage->hmacKey($accessId, $projectId);
 
     $hmacKey->delete();
-    print("The key is deleted, though it may still appear in StorageClient.hmacKeys() results." . PHP_EOL);
+    print("The key is deleted, though it may still appear in StorageClient.hmacKeys(\$options=['showDeletedKeys'=>true])");
+    print("results when  is used." . PHP_EOL);
 }
 # [END storage_get_hmac_key]

storage/storage.php

@@ -476,7 +476,7 @@
     ->addArgument('accessId', InputArgument::REQUIRED, 'The Cloud Storage HMAC key access Id')
     ->addOption('activate', null, InputOption::VALUE_NONE, 'Activate an HMAC key')
     ->addOption('deactivate', null, InputOption::VALUE_NONE, 'Deactivate an HMAC key')
-    ->addOption('get', null, InputOption::VALUE_NONE, 'Get an HMAC key metadata')
+    ->addOption('get', null, InputOption::VALUE_NONE, 'Get an HMAC key\'s metadata')
     ->addOption('delete', null, InputOption::VALUE_NONE, 'Delete an HMAC key')
     ->setCode(function ($input, $output) {
         $projectId = $input->getArgument('projectId');

storage/test/HmacCommandTest.php

@@ -107,7 +107,7 @@ public function testHmacKeyGet()
     }
 
     /** @depends testHmacKeyGet */
-    public function testHmacKeyDeactivateActivate()
+    public function testHmacKeyActivate()
     {
         $this->commandTesterManage->execute(
         [
@@ -117,6 +117,11 @@ public function testHmacKeyDeactivateActivate()
         ],
         ['interactive' => false]);
         $this->assertContains("The HMAC key is now inactive", $this->getActualOutput());
+
+    }
+
+    /** @depends testHmacKeyActivate */
+    public function testHmacKeyDeactivate() {
         $this->commandTesterManage->execute(
           [
               'projectId' => self::$projectId,
@@ -126,7 +131,8 @@ public function testHmacKeyDeactivateActivate()
           ['interactive' => false]);
         $this->assertContains("The HMAC key is now active", $this->getActualOutput());
     }
-    /** @depends testHmacKeyDeactivateActivate */
+
+    /** @depends testHmacKeyDeactivate */
     public function testHmacKeyDelete()
     {
         $this->commandTesterManage->execute(