Updating tests and swagger OAPI doc.

Author: krulis-martin

docs/swagger.yaml

@@ -5548,6 +5548,33 @@ paths:
       responses:
         '200':
           description: 'Placeholder response'
+  '/v1/users/external-login/{service}/{externalId}':
+    get:
+      summary: 'Get details of a user identified via external login.'
+      description: 'Get details of a user identified via external login.'
+      operationId: usersPresenterActionFindByExternalLogin
+      parameters:
+        -
+          name: service
+          in: path
+          description: 'External authentication service name'
+          required: true
+          schema:
+            type: string
+            minLength: 1
+            nullable: false
+        -
+          name: externalId
+          in: path
+          description: 'External user identifier'
+          required: true
+          schema:
+            type: string
+            minLength: 1
+            nullable: false
+      responses:
+        '200':
+          description: 'Placeholder response'
   '/v1/users/{id}/invalidate-tokens':
     post:
       summary: 'Invalidate all existing tokens issued for given user'

fixtures/demo/20-groups.neon

@@ -155,6 +155,13 @@ App\Model\Entity\Login:
                 - 'demoGroupMember<current()>\@example.com'
                 - "password"
 
+App\Model\Entity\ExternalLogin:
+    demoGroupMemberExternalLogin:
+        __construct:
+            - @demoGroupMember1
+            - 'ext-service'
+            - 'external-login1'
+
 App\Model\Entity\GroupInvitation:
     validInviation:
         __construct:

tests/Presenters/UsersPresenter.phpt

@@ -5,7 +5,6 @@ $container = require_once __DIR__ . "/../bootstrap.php";
 use App\Exceptions\ForbiddenRequestException;
 use App\Helpers\EmailVerificationHelper;
 use App\Helpers\AnonymizationHelper;
-use App\Model\Entity\Exercise;
 use App\Model\Entity\Login;
 use App\Model\Entity\ExternalLogin;
 use App\Model\Entity\User;
@@ -225,6 +224,36 @@ class TestUsersPresenter extends Tester\TestCase
         Assert::same($user->getId(), $result["payload"]["id"]);
     }
 
+    public function testFindByExternalLogin()
+    {
+        PresenterTestHelper::loginDefaultAdmin($this->container);
+        $extLogins = $this->presenter->externalLogins->findAll();
+        Assert::count(1, $extLogins);
+        $el = $extLogins[0];
+
+        $payload = PresenterTestHelper::performPresenterRequest(
+            $this->presenter,
+            $this->presenterPath,
+            'GET',
+            ['action' => 'findByExternalLogin', 'service' => $el->getAuthService(), 'externalId' => $el->getExternalId()]
+        );
+
+        Assert::same($el->getUser()->getId(), $payload["id"]);
+    }
+
+    public function testFindByExternalLoginEmptyResult()
+    {
+        PresenterTestHelper::loginDefaultAdmin($this->container);
+        Assert::exception(function () {
+            PresenterTestHelper::performPresenterRequest(
+                $this->presenter,
+                $this->presenterPath,
+                'GET',
+                ['action' => 'findByExternalLogin', 'service' => 'ext-service', 'externalId' => 'blabla']
+            );
+        }, App\Exceptions\NotFoundException::class);
+    }
+
     public function testUpdateProfileWithoutEmailAndPassword()
     {
         PresenterTestHelper::loginDefaultAdmin($this->container);
@@ -857,7 +886,7 @@ class TestUsersPresenter extends Tester\TestCase
         Assert::true($payload['privateData']['isExternal']);
         Assert::equal(['test-cas' => 'abc'], $payload['privateData']['externalIds']);
 
-        $els = $this->presenter->externalLogins->findAll();
+        $els = $this->presenter->externalLogins->findBy(['authService' => 'test-cas']);
         Assert::count(1, $els);
         Assert::equal($user->getId(), $els[0]->getUser()->getId());
         Assert::equal('abc', $els[0]->getExternalId());
@@ -890,7 +919,7 @@ class TestUsersPresenter extends Tester\TestCase
         Assert::true($payload['privateData']['isExternal']);
         Assert::equal(['test-cas' => 'abc'], $payload['privateData']['externalIds']);
 
-        $els = $this->presenter->externalLogins->findAll();
+        $els = $this->presenter->externalLogins->findBy(['authService' => 'test-cas']);
         Assert::count(1, $els);
         Assert::equal($user->getId(), $els[0]->getUser()->getId());
         Assert::equal('abc', $els[0]->getExternalId());
@@ -922,7 +951,7 @@ class TestUsersPresenter extends Tester\TestCase
         Assert::false($payload['privateData']['isExternal']);
         Assert::equal([], $payload['privateData']['externalIds']);
 
-        $els = $this->presenter->externalLogins->findAll();
+        $els = $this->presenter->externalLogins->findBy(['authService' => 'test-cas']);
         Assert::count(0, $els);
 
         $this->users->refresh($user);