Fixed client creation tests

matchory · May 14, 2024 · a9eb07e · a9eb07e
1 parent 2edb310
commit a9eb07e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 24 deletions.
diff --git a/tests/ConnectionManagerTest.php b/tests/ConnectionManagerTest.php
@@ -102,27 +102,6 @@ public function testCreatesConnectionsWithCacheInstance(): void
         self::assertSame($cache, $connection->getCache());
     }
 
-    public function testCreatesConnectionsWithLoggerInstance(): void
-    {
-        /** @var LoggerInterface&Mock $logger */
-        $logger = $this->mock(LoggerInterface::class);
-        $clientFactory = new ClientFactory($logger);
-        $instance = new ConnectionManager(
-            [
-                'connections' => [
-                    'foo' => [
-                        'servers' => [
-                            '0.0.0.0',
-                        ],
-                    ],
-                ],
-            ],
-            $clientFactory,
-            null,
-        );
-        $instance->connection('foo');
-    }
-
     /**
      * @throws ExpectationFailedException
      * @throws InvalidArgumentException

diff --git a/tests/Factories/ClientFactoryTest.php b/tests/Factories/ClientFactoryTest.php
@@ -21,13 +21,15 @@ public function testCreateClient(): void
 
     public function testCreateClientWithHosts(): void
     {
-        $hosts = ['foo', 'bar', 'baz'];
+        $config = [
+            'hosts' => ['foo', 'bar', 'baz']
+        ];
         $factory = new ClientFactory();
-        $client = $factory->createClient($hosts);
+        $client = $factory->createClient($config);
 
         self::assertContains(
             $client->transport->getConnection()->getHost(),
-            $hosts
+            $config['hosts']
         );
     }
 }