Merge pull request #66 from solusio/bugfix-mkorsakov-update-fqdn-on-h…

…ostname-change-SIO-3769 BUGFIX SIO-3769 update fqdns on hostname change
solusio · Oct 25, 2021 · 4adf5b4 · 4adf5b4
2 parents 51b16a4 + 31e03d7
commit 4adf5b4
Show file tree

Hide file tree

Showing 3 changed files with 48 additions and 1 deletion.
diff --git a/modules/servers/solusiovps/lib/SolusAPI/Resources/ApiResource.php b/modules/servers/solusiovps/lib/SolusAPI/Resources/ApiResource.php
@@ -26,6 +26,11 @@ public function __construct(ClientInterface $connector)
         $this->connector = $connector;
     }
 
+    public function getConnector(): ClientInterface
+    {
+        return $this->connector;
+    }
+
     /**
      * @param $response
      * @return array

diff --git a/modules/servers/solusiovps/lib/SolusAPI/Resources/ServerResource.php b/modules/servers/solusiovps/lib/SolusAPI/Resources/ServerResource.php
@@ -98,9 +98,10 @@ public function resetPassword(int $id): array
 
     public function changeHostname(int $id, string $hostname): array
     {
-        return $this->processResponse($this->connector->patch("servers/{$id}", [
+        return $this->processResponse($this->getConnector()->patch("servers/{$id}", [
             'json' => [
                 'name' => $hostname,
+                'fqdns' => [ $hostname ],
             ],
         ]));
     }

diff --git a/tests/lib/SolusAPI/Resources/ServerResourceTest.php b/tests/lib/SolusAPI/Resources/ServerResourceTest.php
@@ -0,0 +1,41 @@
+<?php
+// Copyright 2021. Plesk International GmbH.
+
+namespace Tests\lib\SolusAPI\Resources;
+
+use Mockery;
+use PHPUnit\Framework\TestCase;
+use WHMCS\Module\Server\SolusIoVps\SolusAPI\Resources\ServerResource;
+
+/**
+ * @runTestsInSeparateProcesses
+ */
+class ServerResourceTest extends TestCase
+{
+    public function testChangeHostname(): void
+    {
+        $id = 1;
+        $hostname = 'test.domain.tld';
+
+        $serverResource = Mockery::mock(ServerResource::class)
+            ->shouldAllowMockingProtectedMethods()
+            ->makePartial();
+
+        $serverResource->shouldReceive('processResponse')->andReturn([]);
+        $serverResource->shouldReceive('getConnector->patch')->with(
+            "servers/{$id}",
+            [
+                'json' => [
+                    'name' => $hostname,
+                    'fqdns' => [ $hostname ],
+                ],
+            ],
+        );
+
+        $this->addToAssertionCount(
+            Mockery::getContainer()->mockery_getExpectationCount()
+        );
+
+        $serverResource->changeHostname($id, $hostname);
+    }
+}