[Tests] Add connection class tests

ruflin · Jan 17, 2024 · 4594e0b · 4594e0b
1 parent c24304d
commit 4594e0b
Show file tree

Hide file tree

Showing 3 changed files with 61 additions and 3 deletions.
diff --git a/tests/ClientConfigurationTest.php b/tests/ClientConfigurationTest.php
@@ -101,7 +101,7 @@ public function testFromEmptyArray(): void
             'port' => null,
             'path' => null,
             'url' => null,
-            'connections' => [], // host, port, path, timeout, transport, compression, persistent, timeout, username, password, config -> (curl, headers, url)
+            'connections' => [], // host, port, path, timeout, transport, compression, timeout, username, password, config -> (curl, headers, url)
             'roundRobin' => false,
             'retryOnConflict' => 0,
             'username' => null,
@@ -124,7 +124,7 @@ public function testFromArray(): void
             'port' => null,
             'path' => null,
             'url' => null,
-            'connections' => [], // host, port, path, timeout, transport, compression, persistent, timeout, username, password, config -> (curl, headers, url)
+            'connections' => [], // host, port, path, timeout, transport, compression, timeout, username, password, config -> (curl, headers, url)
             'roundRobin' => false,
             'retryOnConflict' => 0,
             'username' => 'Jdoe',

diff --git a/tests/ConnectionTest.php b/tests/ConnectionTest.php
@@ -75,6 +75,13 @@ public function testGetConfig(): void
         $connection = new Connection(['config' => ['url' => $url]]);
         $this->assertTrue($connection->hasConfig('url'));
         $this->assertEquals($url, $connection->getConfig('url'));
+
+        $connection->setConfig([]);
+        $this->assertFalse($connection->hasConfig('url'));
+
+        $connection->addConfig('url', $url);
+        $this->assertTrue($connection->hasConfig('url'));
+        $this->assertEquals($url, $connection->getConfig('url'));
     }
 
     /**
@@ -154,4 +161,55 @@ public function testPasswordFromClient(): void
 
         $this->assertEquals($password, $client->getConnection()->getPassword());
     }
+
+    /**
+     * @group unit
+     */
+    public function testPathFromClient(): void
+    {
+        $path = 'test';
+        $client = new Client(['path' => $path]);
+
+        $this->assertEquals($path, $client->getConnection()->getPath());
+
+        $changedPath = 'test2';
+
+        $client->getConnection()->setPath($changedPath);
+
+        $this->assertEquals($changedPath, $client->getConnection()->getPath());
+    }
+
+    /**
+     * @group unit
+     */
+    public function testPortFromClient(): void
+    {
+        $port = 9000;
+        $client = new Client(['port' => $port]);
+
+        $this->assertEquals($port, $client->getConnection()->getPort());
+
+        $changedPort = 9001;
+
+        $client->getConnection()->setPort($changedPort);
+
+        $this->assertEquals($changedPort, $client->getConnection()->getPort());
+    }
+
+    /**
+     * @group unit
+     */
+    public function testHostFromClient(): void
+    {
+        $host = 'localhost';
+        $client = new Client(['host' => $host]);
+
+        $this->assertEquals($host, $client->getConnection()->getHost());
+
+        $changedHost = 'localhostChanged';
+
+        $client->getConnection()->setHost($changedHost);
+
+        $this->assertEquals($changedHost, $client->getConnection()->getHost());
+    }
 }
diff --git a/tests/Query/MoreLikeThisTest.php b/tests/Query/MoreLikeThisTest.php
@@ -65,7 +65,7 @@ public function testSearch(): void
      */
     public function testSearchByDocument(): void
     {
-        $client = $this->_getClient(['persistent' => false]);
+        $client = $this->_getClient();
         $index = $client->getIndex('elastica_test');
         $index->create(
             [