Fix SecureServer to return null URI if server socket is already closed

Author: clue

src/SecureServer.php

@@ -121,7 +121,7 @@ public function __construct(ServerInterface $tcp, LoopInterface $loop, array $co
             throw new \BadMethodCallException('Encryption not supported on your platform (HHVM < 3.8?)'); // @codeCoverageIgnore
         }
 
-        // default to empty passphrase to surpress blocking passphrase prompt
+        // default to empty passphrase to suppress blocking passphrase prompt
         $context += array(
             'passphrase' => ''
         );
@@ -141,7 +141,12 @@ public function __construct(ServerInterface $tcp, LoopInterface $loop, array $co
 
     public function getAddress()
     {
-        return str_replace('tcp://' , 'tls://', $this->tcp->getAddress());
+        $address = $this->tcp->getAddress();
+        if ($address === null) {
+            return null;
+        }
+
+        return str_replace('tcp://' , 'tls://', $address);
     }
 
     public function pause()

tests/SecureServerTest.php

@@ -26,6 +26,18 @@ public function testGetAddressWillBePassedThroughToTcpServer()
         $this->assertEquals('127.0.0.1:1234', $server->getAddress());
     }
 
+    public function testGetAddressWillReturnNullIfTcpServerReturnsNull()
+    {
+        $tcp = $this->getMockBuilder('React\Socket\ServerInterface')->getMock();
+        $tcp->expects($this->once())->method('getAddress')->willReturn(null);
+
+        $loop = $this->getMockBuilder('React\EventLoop\LoopInterface')->getMock();
+
+        $server = new SecureServer($tcp, $loop, array());
+
+        $this->assertNull($server->getAddress());
+    }
+
     public function testPauseWillBePassedThroughToTcpServer()
     {
         $tcp = $this->getMockBuilder('React\Socket\ServerInterface')->getMock();