Implemented metrics function on Server Model (#10)

Author: paulus7

docs/files/Models/Servers/Server.php.txt

@@ -443,16 +443,22 @@ class Server extends Model
     /**
      * Get Metrics for specified server.
      *
-     * @see https://docs.hetzner.cloud/#resources-servers-get-2
+     * @see https://docs.hetzner.cloud/#servers-get-metrics-for-a-server
      * @param string $type
      * @param string $start
      * @param string $end
      * @param int|null $step
+     * @return APIResponse
+     * @throws \LKDev\HetznerCloud\APIException
      */
     public function metrics(string $type, string $start, string $end, int $step = null)
     {
-        // ToDo
-        $this->httpClient->get($this->replaceServerIdInUri('servers/{id}/metrics?') . http_build_query(compact('type', 'start', 'end', 'step')));
+        $response = $this->httpClient->get($this->replaceServerIdInUri('servers/{id}/metrics?') . http_build_query(compact('type', 'start', 'end', 'step')));
+        if (!HetznerAPIClient::hasError($response)) {
+            return APIResponse::create([
+                'metrics' => json_decode((string)$response->getBody())->metrics
+            ], $response->getHeaders());
+        }
     }
 
     /**

src/Models/Servers/Server.php

@@ -481,16 +481,22 @@ public function changeReverseDNS(string $ip, string $dnsPtr): APIResponse
     /**
      * Get Metrics for specified server.
      *
-     * @see https://docs.hetzner.cloud/#resources-servers-get-2
+     * @see https://docs.hetzner.cloud/#servers-get-metrics-for-a-server
      * @param string $type
      * @param string $start
      * @param string $end
      * @param int|null $step
+     * @return APIResponse
+     * @throws \LKDev\HetznerCloud\APIException
      */
     public function metrics(string $type, string $start, string $end, int $step = null)
     {
-        // ToDo
-        $this->httpClient->get($this->replaceServerIdInUri('servers/{id}/metrics?') . http_build_query(compact('type', 'start', 'end', 'step')));
+        $response = $this->httpClient->get($this->replaceServerIdInUri('servers/{id}/metrics?') . http_build_query(compact('type', 'start', 'end', 'step')));
+        if (!HetznerAPIClient::hasError($response)) {
+            return APIResponse::create([
+                'metrics' => json_decode((string)$response->getBody())->metrics
+            ], $response->getHeaders());
+        }
     }
 
     /**

tests/Unit/Servers/ServerTest.php

@@ -195,4 +195,11 @@ public function testGet()
         $this->assertEquals($server->name, 'my-server');
         $this->assertEquals($server->status, 'running');
     }
+
+    public function testMetrics()
+    {
+        $apiResponse = $this->server->metrics('cpu,disk,network', date("c"), date("c"), 60);
+        $metrics = $apiResponse->getResponsePart('metrics');
+        $this->assertEquals([["1435781470.622", "42"]], $metrics->time_series->name_of_timeseries->values ?? null);
+    }
 }