Fix Tests

Author: LKaemmerling

CHANGELOG.md

@@ -1,7 +1,6 @@
 # Changelog
 ## master
  + Add `httpClient`-Method to `LKDev\HetznerCloud\HetznerAPIClient`
- + Remove the `backup_window`-Property from the  `enableBackups`-Method in the Class `LKDev\HetznerCloud\Models\Servers`
  + Add `labels`-Property to `LKDev\HetznerCloud\Models\FloatingIp\FloatingIP`
  + Add `labels`-Property to `LKDev\HetznerCloud\Models\Server\Server`
  + Add `labels`-Property to `LKDev\HetznerCloud\Models\Image\Image`

src/Models/ISOs/ISOs.php

@@ -27,7 +27,7 @@ class ISOs extends Model
      * @return array
      * @throws \LKDev\HetznerCloud\APIException
      */
-    public function all(RequestOpts $requestOpts): array
+    public function all(RequestOpts $requestOpts = null): array
     {
         if ($requestOpts == null) {
             $requestOpts = new RequestOpts();

src/Models/Images/Image.php

@@ -161,12 +161,20 @@ public function __construct(
      */
     public function update(string $description = null, string $type = null, array $labels = null): Image
     {
+        $params = [];
+        if ($description != null) {
+            $params['description'] = $description;
+        }
+
+        if ($type != null) {
+            $params['type'] = $type;
+        }
+
+        if ($labels != null) {
+            $params['labels'] = $labels;
+        }
         $response = $this->httpClient->put('images/' . $this->id, [
-            'json' => [
-                'description' => $description == null ? $this->description : $type,
-                'type' => $type == null ? $this->type : $type,
-                'labels' => $labels == null ? $this->labels : $labels,
-            ],
+            'json' => $params,
         ]);
         if (!HetznerAPIClient::hasError($response)) {
             return self::parse(json_decode((string)$response->getBody())->image);

src/Models/Prices/Prices.php

@@ -31,7 +31,7 @@ class Prices extends Model
      * @return \stdClass
      * @throws \LKDev\HetznerCloud\APIException
      */
-    public function all(RequestOpts $requestOpts): \stdClass
+    public function all(RequestOpts $requestOpts = null): \stdClass
     {
         if ($requestOpts == null) {
             $requestOpts = new RequestOpts();

src/Models/Servers/Server.php

@@ -379,12 +379,17 @@ public function changeType(ServerType $serverType, bool $upgradeDisk = false): A
      * Enables and configures the automatic daily backup option for the server. Enabling automatic backups will increase the price of the server by 20%
      *
      * @see https://docs.hetzner.cloud/#resources-server-actions-post-11
+     * @param string|null $backupWindow
      * @return APIResponse
      * @throws \LKDev\HetznerCloud\APIException
      */
-    public function enableBackups(): APIResponse
+    public function enableBackups(string $backupWindow = null): APIResponse
     {
-        $response = $this->httpClient->post($this->replaceServerIdInUri('servers/{id}/actions/enable_backup'));
+        $response = $this->httpClient->post($this->replaceServerIdInUri('servers/{id}/actions/enable_backup'), [
+            'json' => [
+                'backup_window' => $backupWindow,
+            ],
+        ]);
         if (!HetznerAPIClient::hasError($response)) {
             return APIResponse::create([
                 'action' => Action::parse(json_decode((string)$response->getBody())->action)

src/Models/Servers/Types/ServerTypes.php

@@ -27,7 +27,7 @@ class ServerTypes extends Model
      * @return array
      * @throws \LKDev\HetznerCloud\APIException
      */
-    public function all(RequestOpts $requestOpts): array
+    public function all(RequestOpts $requestOpts = null): array
     {
         if ($requestOpts == null) {
             $requestOpts = new RequestOpts();

tests/Unit/Servers/ServerTest.php

@@ -158,7 +158,7 @@ public function testPowerOn()
 
     public function testEnableBackups()
     {
-        $apiResponse = $this->server->enableBackups();
+        $apiResponse = $this->server->enableBackups('22-02');
         $this->assertEquals('enable_backup', $apiResponse->getResponsePart('action')->command);
         $this->assertEquals($this->server->id, $apiResponse->getResponsePart('action')->resources[0]->id);
         $this->assertEquals('server', $apiResponse->getResponsePart('action')->resources[0]->type);