Additional tests for 100% coverage of Factory

Author: clue

tests/FactoryTest.php

@@ -3,6 +3,7 @@
 use React\Datagram\Socket;
 use React\Datagram\Factory;
 use Clue\React\Block;
+use React\Promise;
 
 class FactoryTest extends TestCase
 {
@@ -69,6 +70,38 @@ public function testCreateServerRandomPort()
         $capturedServer->close();
     }
 
+    public function testCreateClientWithIpWillNotUseResolver()
+    {
+        $this->resolver->expects($this->never())->method('resolve');
+
+        $client = Block\await($this->factory->createClient('127.0.0.1:0'), $this->loop);
+        $client->close();
+    }
+
+    public function testCreateClientWithHostnameWillUseResolver()
+    {
+        $this->resolver->expects($this->once())->method('resolve')->with('example.com')->willReturn(Promise\resolve('127.0.0.1'));
+
+        $client = Block\await($this->factory->createClient('example.com:0'), $this->loop);
+        $client->close();
+    }
+
+    public function testCreateClientWithHostnameWillRejectIfResolverRejects()
+    {
+        $this->resolver->expects($this->once())->method('resolve')->with('example.com')->willReturn(Promise\reject(new \RuntimeException('test')));
+
+        $this->setExpectedException('RuntimeException');
+        Block\await($this->factory->createClient('example.com:0'), $this->loop);
+    }
+
+    public function testCreateClientWithHostnameWillRejectIfNoResolverIsGiven()
+    {
+        $this->factory = new Factory($this->loop);
+
+        $this->setExpectedException('Exception');
+        Block\await($this->factory->createClient('example.com:0'), $this->loop);
+    }
+
     /**
      * @expectedException Exception
      * @expectedExceptionMessage Unable to create client socket