Minor Refactoring

lib/Ratchet/Application/Server/App.php

@@ -49,8 +49,6 @@ class App implements ApplicationInterface {
         $this->_connections[$host->getResource()] = new Connection($host);
         $this->_resources[] = $host->getResource();
 
-        $recv_bytes = 1024;
-
         set_time_limit(0);
         ob_implicit_flush();
 
@@ -61,13 +59,18 @@ class App implements ApplicationInterface {
         $host->listen();
 
         do {
+            $this->loop($host);
+        } while (true);
+    }
+
+    protected function loop(SocketInterface $host, $recv_bytes = 1024) {
         $changed = $this->_resources;
 
         try {
             $num_changed = $host->select($changed, $write = null, $except = null, null);
         } catch (Exception $e) {
             // master had a problem?...what to do?
-                continue;
+            return;
         }
 
         foreach($changed as $resource) {
@@ -87,6 +90,7 @@ class App implements ApplicationInterface {
                         // 2) What if the last message in the frame is equal to $recv_bytes?  Would loop until another msg is sent
                         // 3) This failed...an intermediary can set their buffer lower and this still propagates a fragment
                         // Need to 1) proc_open the recv() calls.  2) ???
+
                         /*
                         while ($bytes === $recv_bytes) {
                             $bytes = $conn->recv($buf, $recv_bytes, 0);
@@ -116,7 +120,6 @@ class App implements ApplicationInterface {
                 $res = $new_res;
             }
         }
-        } while (true);
     }
 
     public function onOpen(Connection $conn) {

lib/Ratchet/Exception.php

@@ -5,6 +5,9 @@ namespace Ratchet;
  * Uses internal php methods to fill an Exception class (no parameters required)
  */
 class Exception extends \Exception {
+    /**
+     * @var SocketInterface
+     */
     protected $_socket;
 
     public function __construct(SocketInterface $socket) {

lib/Ratchet/Resource/Connection.php

@@ -50,7 +50,7 @@ class Connection {
      * @throws \InvalidArgumentException
      */
     public function &__get($name) {
-        if (!isset($this->_data[$name])) {
+        if (!$this->__isset($name)) {
             throw new \InvalidArgumentException("Attribute '{$name}' not found in Connection {$this->getID()}");
         }
 
@@ -60,4 +60,19 @@ class Connection {
             return $this->_data[$name];
         }
     }
+
+    /**
+     * @param mixed
+     * @return bool
+     */
+    public function __isset($name) {
+        return isset($this->_data[$name]);
+    }
+
+    /**
+     * @param mixed
+     */
+    public function __unset($name) {
+        unset($this->_data[$name]);
+    }
 }

tests/Ratchet/Tests/Mock/Application.php

@@ -5,18 +5,38 @@ use Ratchet\Tests\Mock\Socket as MockSocket;
 use Ratchet\Resource\Connection;
 
 class Application implements ApplicationInterface {
+    public $_app;
+
+    public $_conn_open;
+
+    public $_conn_recv;
+    public $_msg_recv;
+
+    public $_conn_close;
+
+    public $_conn_error;
+    public $_excep_error;
+
     public function __construct(ApplicationInterface $app = null) {
+        // probably should make this null app
+        $this->_app = $app;
     }
 
     public function onOpen(Connection $conn) {
+        $this->_conn_open = $conn;
     }
 
     public function onRecv(Connection $from, $msg) {
+        $this->_conn_recv = $from;
+        $this->_msg_recv  = $msg;
     }
 
     public function onClose(Connection $conn) {
+        $this->_conn_close = $conn;
     }
 
     public function onError(Connection $conn, \Exception $e) {
+        $this->_conn_error  = $conn;
+        $this->_excep_error = $e;
     }
 }

tests/Ratchet/Tests/Resource/ConnectionTest.php

@@ -7,16 +7,28 @@ use Ratchet\Tests\Mock\FakeSocket;
  * @covers Ratchet\Resource\Connection
  */
 class ConnectionTest extends \PHPUnit_Framework_TestCase {
+    protected $_fs;
     protected $_c;
 
     public function setUp() {
-        $this->_c = new Connection(new FakeSocket);
+        $this->_fs = new FakeSocket;
+        $this->_c  = new Connection($this->_fs);
     }
 
-    public function testCanGetWhatIsSet() {
-        $key = 'hello';
-        $val = 'world';
+    public static function keyAndValProvider() {
+        return array(
+            array('hello', 'world')
+        );
+    }
 
+    public function testGetSocketReturnsWhatIsSetInConstruct() {
+        $this->assertSame($this->_fs, $this->_c->getSocket());
+    }
+
+    /**
+     * @dataProvider keyAndValProvider
+     */
+    public function testCanGetWhatIsSet($key, $val) {
         $this->_c->{$key} = $val;
         $this->assertEquals($val, $this->_c->{$key});
     }
@@ -29,4 +41,21 @@ class ConnectionTest extends \PHPUnit_Framework_TestCase {
     public function testLambdaReturnValueOnGet() {
         $this->markTestIncomplete();
     }
+
+    /**
+     * @dataProvider keyAndValProvider
+     */
+    public function testIssetWorksOnOverloadedVariables($key, $val) {
+        $this->_c->{$key} = $val;
+        $this->assertTrue(isset($this->_c->{$key}));
+    }
+
+    /**
+     * @dataProvider keyAndValProvider
+     */
+    public function testUnsetMakesIssetReturnFalse($key, $val) {
+        $this->_c->{$key} = $val;
+        unset($this->_c->{$key});
+        $this->assertFalse(isset($this->_c->{$key}));
+    }
 }