Merge pull request #52 from weichenlin/master

Clear all listeners after "end" event

Author: WyriHaximus

src/Request.php

@@ -198,6 +198,7 @@ public function close(\Exception $error = null)
         }
 
         $this->emit('end', array($error, $this->response, $this));
+        $this->removeAllListeners();
     }
 
     protected function parseResponse($data)

tests/RequestTest.php

@@ -453,6 +453,19 @@ public function requestShouldRelayErrorEventsFromResponse()
         call_user_func($errorCallback, new \Exception('test'));
     }
 
+    /** @test */
+    public function requestShouldRemoveAllListenerAfterClosed()
+    {
+        $requestData = new RequestData('GET', 'http://www.example.com');
+        $request = new Request($this->connector, $requestData);
+
+        $request->on('end', function () {});
+        $this->assertCount(1, $request->listeners('end'));
+
+        $request->close();
+        $this->assertCount(0, $request->listeners('end'));
+    }
+
     private function successfulConnectionMock()
     {
         call_user_func($this->successfulAsyncConnectionMock());