Merge pull request #24 from spinen/feature/supportLaravel7

Feature/support laravel7

Author: ssfinney

.travis.yml

@@ -13,6 +13,9 @@ matrix:
     - php: 7.3
     - php: 7.3
       env: SETUP=lowest
+    - php: 7.4
+    - php: 7.4
+      env: SETUP=lowest
 
 sudo: false
 

composer.json

@@ -14,21 +14,25 @@
         {
             "name": "Jimmy Puckett",
             "email": "[email protected]"
+        },
+        {
+            "name": "Stephen Finney",
+            "email": "[email protected]"
         }
     ],
     "require": {
         "php": ">=7.2",
         "ext-json": "*",
         "cviebrock/discourse-php": "^0.9.3",
         "guzzlehttp/guzzle": "^6.4",
-        "illuminate/auth": "~5.5|~6",
-        "illuminate/routing": "~5.5|~6",
-        "illuminate/support": "~5.5|~6"
+        "illuminate/auth": "~5.5|~6|~7",
+        "illuminate/routing": "~5.5|~6|~7",
+        "illuminate/support": "~5.5|~6|~7"
     },
     "require-dev": {
-        "mockery/mockery": "^1",
-        "phpunit/phpunit": "~7.0.1|~8.0",
-        "psy/psysh": "^0.9.9",
+        "mockery/mockery": "^1.3.1",
+        "phpunit/phpunit": "^8.4|^9.0",
+        "psy/psysh": "^0.10",
         "symfony/thanks": "^1.1",
         "symfony/var-dumper": "~3.0|^4.2"
     },

phpunit.xml.dist

@@ -14,7 +14,7 @@
          convertErrorsToExceptions="true"
          convertNoticesToExceptions="true"
          convertWarningsToExceptions="true"
-         processIsolation="true"
+         processIsolation="false"
          stopOnFailure="false"
          verbose="true">
 
@@ -48,7 +48,7 @@
              showOnlySummary="true"
              showUncoveredFiles="false"/>
         <log type="coverage-clover" target="build/phpunit/logs/clover.xml"/>
-        <log type="json" target="./build/phpunit/logs/logfile.json"/>
+        <log type="plain" target="./build/phpunit/logs/logfile.log"/>
         <log type="junit" target="./build/phpunit/logs/junit.xml"/>
     </logging>
 </phpunit>

src/Listeners/LogoutDiscourseUser.php

@@ -70,12 +70,18 @@ public function handle($event)
         ];
 
         // Get Discourse user to match this one, and send a Logout request to Discourse and get the response
-        $user = json_decode(
-            $this->client->get("users/by-external/{$event->user->id}.json", $configs)
-                         ->getBody()
-        )->user;
+        $response = $this->client->get("users/by-external/{$event->user->id}.json", $configs);
 
-        $response = $this->client->post("admin/users/{$user->id}/log_out");
+        if ($response->getStatusCode() !== 200) {
+            $this->logger->warning(
+                "When getting user {$event->user->id} Discourse returned status code {$response->getStatusCode()}",
+                ['reason' => $response->getReasonPhrase()]
+            );
+            return;
+        }
+
+        $user = json_decode($response->getBody())->user;
+        $response = $this->client->post("admin/users/{$user->id}/log_out", $configs);
 
         if ($response->getStatusCode() !== 200) {
             $this->logger->notice(

tests/Listeners/LogoutDiscourseUserTest.php

@@ -129,7 +129,7 @@ public function it_logs_out_the_discourse_user_when_triggered()
                  ->andReturn(json_encode(['user' => $this->user_mock]));
 
         $this->response_mock->shouldReceive('getStatusCode')
-                 ->once()
+                 ->twice()
                  ->andReturn(200);
 
         $this->guzzle_mock->shouldReceive('get')
@@ -138,7 +138,7 @@ public function it_logs_out_the_discourse_user_when_triggered()
                           ->andReturn($this->response_mock);
 
         $this->guzzle_mock->shouldReceive('post')
-                          ->with('admin/users/1/log_out')
+                          ->with('admin/users/1/log_out', $configs)
                           ->andReturn($this->response_mock);
 
         $this->listener->handle($this->event_mock);
@@ -160,13 +160,12 @@ public function if_it_receives_no_user_it_does_nothing_and_returns()
     /**
      * @test
      */
-    public function if_discourse_response_code_is_not_200_log_a_notice_with_the_status_code()
+    public function on_getting_user_if_discourse_response_code_is_not_200_log_a_warning_with_the_status_code()
     {
         $this->user_mock->id = 1;
         $this->event_mock->user = $this->user_mock;
 
-        $this->logger_mock->shouldReceive('notice')->once();
-
+        $this->logger_mock->shouldReceive('warning')->once();
 
         $configs = [
             'base_uri' => 'http://discourse.example.com',
@@ -191,9 +190,53 @@ public function if_discourse_response_code_is_not_200_log_a_notice_with_the_stat
                           ->once()
                           ->andReturn($configs['headers']['Api-Username']);
 
-        $this->response_mock->shouldReceive('getBody')
+        $this->response_mock->shouldReceive('getStatusCode')
+                            ->andReturn(500);
+
+        $this->response_mock->shouldReceive('getReasonPhrase')
                             ->once()
-                            ->andReturn(json_encode(['user' => $this->user_mock]));
+                            ->andReturn('Server error');
+
+        $this->guzzle_mock->shouldReceive('get')
+                          ->with('users/by-external/1.json', $configs)
+                          ->once()
+                          ->andReturn($this->response_mock);
+
+        $this->listener->handle($this->event_mock);
+    }
+
+    /**
+     * @test
+     */
+    public function on_user_logout_if_discourse_response_code_is_not_200_log_a_notice_with_the_status_code()
+    {
+        $this->user_mock->id = 1;
+        $this->event_mock->user = $this->user_mock;
+
+        $this->logger_mock->shouldReceive('notice')->once();
+
+        $configs = [
+            'base_uri' => 'http://discourse.example.com',
+            'headers'  => [
+                'Api-Key'      => 'testkey',
+                'Api-Username' => 'testuser',
+            ],
+        ];
+
+        $this->config_mock->shouldReceive('get')
+                          ->with('services.discourse.url')
+                          ->once()
+                          ->andReturn($configs['base_uri']);
+
+        $this->config_mock->shouldReceive('get')
+                          ->with('services.discourse.api.key')
+                          ->once()
+                          ->andReturn($configs['headers']['Api-Key']);
+
+        $this->config_mock->shouldReceive('get')
+                          ->with('services.discourse.api.user')
+                          ->once()
+                          ->andReturn($configs['headers']['Api-Username']);
 
         $this->response_mock->shouldReceive('getStatusCode')
                             ->andReturn(500);
@@ -202,13 +245,23 @@ public function if_discourse_response_code_is_not_200_log_a_notice_with_the_stat
                             ->once()
                             ->andReturn('Server error');
 
+        $good_response = Mockery::mock(Response::class);
+
+        $good_response->shouldReceive('getStatusCode')
+                      ->once()
+                      ->andReturn(200);
+
+        $good_response->shouldReceive('getBody')
+                            ->once()
+                            ->andReturn(json_encode(['user' => $this->user_mock]));
+
         $this->guzzle_mock->shouldReceive('get')
                           ->with('users/by-external/1.json', $configs)
                           ->once()
-                          ->andReturn($this->response_mock);
+                          ->andReturn($good_response);
 
         $this->guzzle_mock->shouldReceive('post')
-                          ->with('admin/users/1/log_out')
+                          ->with('admin/users/1/log_out', $configs)
                           ->andReturn($this->response_mock);
 
         $this->listener->handle($this->event_mock);