Merge pull request #111 from rycius100/fix/awsProvider-topicExists

Fix for topicExists function

Author: k-k

src/Provider/AwsProvider.php

@@ -22,7 +22,7 @@
 
 namespace Uecode\Bundle\QPushBundle\Provider;
 
-use Aws\Sns\Exception\NotFoundException;
+use Aws\Sns\Exception\SnsException;
 use Aws\Sns\SnsClient;
 use Aws\Sqs\Exception\SqsException;
 use Aws\Sqs\SqsClient;
@@ -427,7 +427,7 @@ public function topicExists()
                 $this->sns->getTopicAttributes([
                     'TopicArn' => $topicArn
                 ]);
-            } catch (NotFoundException $e) {
+            } catch (SnsException $e) {
                 return false;
             }
 

tests/Message/NotificationTest.php

@@ -39,9 +39,6 @@ public function tearDown()
         $this->message = null;
     }
 
-    /**
-     * @expectedException PHPUnit_Framework_Error
-     */
     public function testConstructor()
     {
         $notification = new Notification(123, ['foo' => 'bar'], ['baz' => 'qux']);

tests/MockClient/SnsMockClient.php

@@ -22,7 +22,7 @@
 
 namespace Uecode\Bundle\QPushBundle\Tests\MockClient;
 
-use Aws\Sns\Exception\NotFoundException;
+use Aws\Sns\Exception\SnsException;
 use Doctrine\Common\Collections\ArrayCollection;
 
 /**
@@ -54,7 +54,7 @@ public function createTopic(array $args)
     public function getTopicAttributes(array $args)
     {
         if ($args['TopicArn'] == null) {
-            throw new NotFoundException;
+            throw new SnsException;
         }
 
         return new ArrayCollection([