Merge pull request #32 from packbackbooks/add-set-debugging-mode

PODB-266 Add the ability to set debugging mode

Author: zlayaAvocado

src/LtiServiceConnector.php

@@ -19,13 +19,19 @@ class LtiServiceConnector implements ILtiServiceConnector
 
     private $cache;
     private $client;
+    private $debuggingMode = false;
 
     public function __construct(ICache $cache, Client $client)
     {
         $this->cache = $cache;
         $this->client = $client;
     }
 
+    public function setDebuggingMode($enable)
+    {
+        $this->debuggingMode = $enable;
+    }
+
     public function getAccessToken(ILtiRegistration $registration, array $scopes)
     {
         // Get a unique cache key for the access token
@@ -108,14 +114,16 @@ public function makeServiceRequest(
         });
         $respBody = $response->getBody();
 
-        error_log('Syncing grade for this lti_user_id: '.
-            json_decode($request->getPayload()['body'])->userId.' '.print_r([
-                'request_method' => $request->getMethod(),
-                'request_url' => $request->getUrl(),
-                'request_body' => $request->getPayload()['body'],
-                'response_headers' => $respHeaders,
-                'response_body' => (string) $respBody,
-            ], true));
+        if ($this->debuggingMode) {
+            error_log('Syncing grade for this lti_user_id: '.
+                json_decode($request->getPayload()['body'])->userId.' '.print_r([
+                    'request_method' => $request->getMethod(),
+                    'request_url' => $request->getUrl(),
+                    'request_body' => $request->getPayload()['body'],
+                    'response_headers' => $respHeaders,
+                    'response_body' => (string) $respBody,
+                ], true));
+        }
 
         return [
             'headers' => $respHeaders,

tests/LtiServiceConnectorTest.php

@@ -287,11 +287,11 @@ public function testItGetsAll()
 
         // Makes two requests, but gets the method and URL once before making the request
         $this->request->shouldReceive('getMethod')
-            ->times(5)->andReturn($method);
+            ->times(3)->andReturn($method);
         $this->request->shouldReceive('getUrl')
-            ->times(5)->andReturn($this->url);
+            ->times(3)->andReturn($this->url);
         $this->request->shouldReceive('getPayload')
-            ->times(6)->andReturn($this->requestPayload);
+            ->times(2)->andReturn($this->requestPayload);
         // Doesn't find a matching link in on the second header, so only updates the URL once
         $this->request->shouldReceive('setUrl')
             ->once()->andReturn($this->request);