Merge pull request #84 from MarkRedeman/add_remove_subscribers_method

Add a removeSubscriber method to the HttpClient

Author: wtfzdotnet

lib/Tmdb/HttpClient/HttpClient.php

@@ -279,6 +279,20 @@ public function addSubscriber(EventSubscriberInterface $subscriber)
         $this->eventDispatcher->addSubscriber($subscriber);
     }
 
+    /**
+     * Remove a subscriber
+     *
+     * @param EventSubscriberInterface $subscriber
+     */
+    public function removeSubscriber(EventSubscriberInterface $subscriber)
+    {
+        if ($subscriber instanceof HttpClientEventSubscriber) {
+            $subscriber->attachHttpClient($this);
+        }
+
+        $this->eventDispatcher->removeSubscriber($subscriber);
+    }
+
     /**
      * @return GuestSessionToken|SessionToken
      */

test/Tmdb/Tests/HttpClient/HttpClientTest.php

@@ -158,6 +158,33 @@ public function shouldRegisterSubscribers()
         $this->testApi->addSubscriber($subscriber);
     }
 
+    /**
+     * @test
+     */
+    public function shouldDeregisterSubscribers()
+    {
+        $this->eventDispatcher = $this->getMock('Symfony\Component\EventDispatcher\EventDispatcherInterface');
+        $this->client  = new Client(new ApiToken('abcdef'), [
+            'adapter'          => $this->adapter,
+            'event_dispatcher' => $this->eventDispatcher
+        ]);
+        $this->testApi = new TestApi($this->client);
+
+        $this->eventDispatcher
+            ->expects($this->once())
+            ->method('addSubscriber')
+        ;
+
+        $this->eventDispatcher
+            ->expects($this->once())
+            ->method('removeSubscriber')
+        ;
+
+        $subscriber = $this->getMock('Symfony\Component\EventDispatcher\EventSubscriberInterface');
+        $this->testApi->addSubscriber($subscriber);
+        $this->testApi->removeSubscriber($subscriber);
+    }
+
     /**
      * @test
      */
@@ -222,4 +249,9 @@ public function addSubscriber($event)
     {
         $this->client->getHttpClient()->addSubscriber($event);
     }
+
+    public function removeSubscriber($event)
+    {
+        $this->client->getHttpClient()->removeSubscriber($event);
+    }
 }