feat(api): update lastLoggedAt if external user authenticates with Pix Authentication Methods

Author: EmmanuelleBonnemay

api/lib/domain/usecases/authenticate-external-user.js

@@ -57,7 +57,10 @@ async function authenticateExternalUser({
     const token = tokenService.createAccessTokenForSaml({ userId: userFromCredentials.id, audience });
 
     await userLoginRepository.updateLastLoggedAt({ userId: userFromCredentials.id });
-
+    await authenticationMethodRepository.updateLastLoggedAtByIdentityProvider({
+      userId: userFromCredentials.id,
+      identityProvider: NON_OIDC_IDENTITY_PROVIDERS.PIX.code,
+    });
     return token;
   } catch (error) {
     if (error instanceof UserNotFoundError || error instanceof PasswordNotMatching) {

api/tests/unit/domain/usecases/authenticate-external-user_test.js

@@ -36,6 +36,7 @@ describe('Unit | Application | UseCase | authenticate-external-user', function (
     };
     authenticationMethodRepository = {
       create: sinon.stub(),
+      updateLastLoggedAtByIdentityProvider: sinon.stub(),
     };
     userRepository = {
       getBySamlId: sinon.stub(),
@@ -107,7 +108,6 @@ describe('Unit | Application | UseCase | authenticate-external-user', function (
 
       const expectedToken = 'expected returned token';
       tokenService.createAccessTokenForSaml.withArgs({ userId: user.id, audience }).resolves(expectedToken);
-
       // when
       await authenticateExternalUser({
         username: user.email,
@@ -125,6 +125,10 @@ describe('Unit | Application | UseCase | authenticate-external-user', function (
 
       // then
       expect(userLoginRepository.updateLastLoggedAt).to.have.been.calledWithExactly({ userId: user.id });
+      expect(authenticationMethodRepository.updateLastLoggedAtByIdentityProvider).to.have.been.calledWithExactly({
+        userId: user.id,
+        identityProvider: NON_OIDC_IDENTITY_PROVIDERS.PIX.code,
+      });
     });
 
     it("should throw an UnexpectedUserAccountError (with expected user's username or email) when the authenticated user does not match the expected one", async function () {