Fixed Socket bugs from Unit Testing

.gitignore

@@ -1,3 +1,4 @@
 phpunit.xml
 coverage
+docs
 sandbox

lib/Ratchet/Socket.php

@@ -41,9 +41,7 @@ class Socket {
      */
     public static function createFromConfig(ProtocolInterface $protocol) {
         $config = $protocol::getDefaultConfig();
-
+        $class  = get_called_class();
-// todo - this is wrong when class is extended, I don't have internet, can't look up the fn - unit test fails
-        $class  = __CLASS__;
 
         $socket = new $class($config['domain'] ?: null, $config['type'] ?: null, $config['protocol'] ?: null);
 
@@ -54,6 +52,8 @@ class Socket {
                 }
             }
         }
+
+        return $socket;
     }
 
     /**

tests/Ratchet/Tests/Mock/Socket.php

@@ -30,7 +30,7 @@ class Socket extends RealSocket {
     }
 
     public function set_option($level, $optname, $optval) {
-        if (!is_array($this->_options[$level])) {
+        if (!isset($this->_options[$level])) {
             $this->_options[$level] = Array();
         }
 

tests/Ratchet/Tests/SocketTest.php

@@ -54,8 +54,7 @@ class SocketTest extends \PHPUnit_Framework_TestCase {
         $protocol = new Protocol();
         $socket   = Socket::createFromConfig($protocol);
 
-        $constraint = $this->isInstanceOf('\\Ratchet\\Socket');
+        $this->assertInstanceOf('\\Ratchet\\Socket', $socket);
-        $this->assertThat($socket, $constraint);
     }
 
     public function testCreationFromConfigOutputMatchesInput() {