Bugfix to prevent calling the API without an Authentication

src/MessageBird/Common/HttpClient.php

@@ -75,11 +75,11 @@ public function __construct($endpoint, $timeout = 10, $connectionTimeout = 2)
     }
 
     /**
-     * @param $string
+     * @param string $userAgent
      */
-    public function addUserAgentString($string)
+    public function addUserAgentString($userAgent)
     {
-        $this->userAgent[] = $string;
+        $this->userAgent[] = $userAgent;
     }
 
     /**
@@ -116,12 +116,18 @@ public function getRequestUrl($resourceName, $query)
      * @param string|null $body
      *
      * @return array
+     *
+     * @throws Exceptions\AuthenticateException
      * @throws Exceptions\HttpException
      */
     public function performHttpRequest($method, $resourceName, $query = null, $body = null)
     {
         $curl = curl_init();
 
+        if ($this->Authentication === null) {
+            throw new Exceptions\AuthenticateException('Can not perform API Request without Authentication');
+        }
+
         $headers = array (
             'User-agent: ' . implode(' ', $this->userAgent),
             'Accepts: application/json',

tests/integration/HttpClientTest.php

@@ -32,4 +32,17 @@ public function testHttpClientInvalidConnectionTimeout()
     {
         new HttpClient(Client::ENDPOINT, 10, new \stdClass());
     }
+
+    /**
+     * Test that requests can only be made when there is an Authentication set
+     *
+     * @test
+     * @expectedException \MessageBird\Exceptions\AuthenticateException
+     * @expectedExceptionMessageRegExp #Can not perform API Request without Authentication#
+     */
+    public function testHttpClientWithoutAuthenticationException()
+    {
+        $client = new HttpClient(Client::ENDPOINT);
+        $client->performHttpRequest('foo', 'bar');
+    }
 }