Added tests

Author: haphan

src/BlockStorage/v2/Models/Volume.php

@@ -160,17 +160,38 @@ public function parseMetadata(ResponseInterface $response): array
         return isset($json['metadata']) ? $json['metadata'] : [];
     }
 
+    /**
+     * Update the bootable status for a volume, mark it as a bootable volume.
+     *
+     * @param bool $bootable
+     */
     public function setBootable(bool $bootable)
     {
         $bootable = boolval($bootable);
         $this->execute($this->api->postVolumeBootable(), ['id' => $this->id, 'bootable' => $bootable]);
     }
 
+    /**
+     * Sets the image metadata for a volume.
+     *
+     * @param array $metadata
+     */
     public function setImageMetadata(array $metadata)
     {
         $this->execute($this->api->postImageMetadata(), ['id' => $this->id, 'metadata' => $metadata]);
     }
 
+    /**
+     * Administrator only. Resets the status, attach status, and migration status for a volume. Specify the os-reset_status action in the request body.
+     *
+     * @param array $options
+     *
+     * $options['status']             = (string)    The volume status.
+     * $options['migrationStatus']   = (string)    The volume migration status.
+     * $options['attachStatus']      = (string)    The volume attach status.      [OPTIONAL]
+     *
+     * @see https://developer.openstack.org/api-ref/block-storage/v2/index.html#volume-actions-volumes-action
+     */
     public function resetStatus(array $options)
     {
         $options += ['id' => $this->id];

src/BlockStorage/v2/Params.php

@@ -106,6 +106,7 @@ public function volumeMigrationStatus(): array
             'location'    => self::JSON,
             'required'    => false,
             'description' => 'The volume migration status.',
+            'sentAs'      => 'migration_status',
         ];
     }
 
@@ -116,6 +117,7 @@ public function volumeAttachStatus(): array
             'location'    => self::JSON,
             'required'    => false,
             'description' => 'The volume attach status.',
+            'sentAs'      => 'attach_status',
         ];
     }
 

tests/unit/BlockStorage/v2/Models/VolumeTest.php

@@ -88,4 +88,44 @@ public function test_it_resets_metadata()
 
         $this->volume->resetMetadata(['key1' => 'val1']);
     }
+
+    public function test_it_sets_volume_bootable()
+    {
+        $this->setupMock('POST', 'volumes/1/action', ['os-set_bootable' => ['bootable' => 'True']], [], new Response(200));
+
+        $this->volume->setBootable(true);
+    }
+
+    public function test_it_sets_image_meta_data()
+    {
+        $expectedJson = [
+            'os-set_image_metadata' => [
+                'metadata' => [
+                    'attr_foo' => 'foofoo',
+                    'attr_bar' => 'barbar',
+                ],
+            ],
+        ];
+
+        $this->setupMock('POST', 'volumes/1/action', $expectedJson, [], new Response(200));
+        $this->volume->setImageMetadata([
+            'attr_foo' => 'foofoo',
+            'attr_bar' => 'barbar',
+        ]);
+    }
+
+    public function test_it_resets_status()
+    {
+        $expectedJson = ['os-reset_status' => ['status' => 'available', 'attach_status' => 'detached', 'migration_status' => 'migrating']];
+
+        $this->setupMock('POST', 'volumes/1/action', $expectedJson, [], new Response(202));
+
+        $this->volume->resetStatus(
+            [
+                'status'          => 'available',
+                'attachStatus'    => 'detached',
+                'migrationStatus' => 'migrating',
+            ]
+        );
+    }
 }

tests/unit/Networking/v2/Extensions/Layer3/Models/FloatingIpTest.php

@@ -79,4 +79,11 @@ public function test_it_retrieves()
                             $this->floatingIp->id
         );
     }
+
+    public function test_it_associates_port()
+    {
+        $this->setupMock('PUT', 'v2.0/floatingips/id', ['floatingip' => ['port_id' => 'some-port-id']], [], 'FloatingIp');
+
+        $this->floatingIp->associatePort('some-port-id');
+    }
 }