Issue 137

src/features/account/repositories/account-repository/account-repository.ts

@@ -29,6 +29,16 @@ export class AccountRepository {
 
   findAccountsByUserId(id: string) {
     return database.account.findMany({
+      select: {
+        avatarUrl: true,
+        id: true,
+        socialMedia: {
+          select: {
+            id: true,
+            name: true,
+          },
+        },
+      },
       where: {
         userId: id,
       },

src/features/account/services/get-user-accounts-service.ts

@@ -1,4 +1,3 @@
-import type { AccountModel } from '@/features/account/models/account-model';
 import type { AccountRepository } from '@/features/account/repositories/account-repository/account-repository';
 import type { UserRepository } from '@/features/user/repositories/user-repository';
 import { UserNotFound } from '@/shared/errors/user-not-found-error';
@@ -9,7 +8,14 @@ type Input = {
 };
 
 type Output = {
-  accounts: AccountModel[];
+  accounts: {
+    avatarUrl: string;
+    id: string;
+    socialMedia: {
+      id: number;
+      name: string;
+    } | null;
+  }[];
 };
 
 export class GetUserAccountsService implements Service<Input, Output> {
@@ -28,7 +34,7 @@ export class GetUserAccountsService implements Service<Input, Output> {
     const accounts = await this.accountRepository.findAccountsByUserId(user.id);
 
     return {
-      accounts: accounts,
+      accounts,
     };
   }
 }

src/features/user/controllers/user-controller.test.ts

@@ -2,9 +2,13 @@ import { randomUUID } from 'crypto';
 import type { Request, Response } from 'express';
 import { mock, mockDeep } from 'vitest-mock-extended';
 
+import { GetUserAccountsService } from '@/features/account/services/get-user-accounts-service';
 import { UserCreateService } from '@/features/user/services/user-create-service';
 import { UserFindByIdService } from '@/features/user/services/user-find-by-id-service';
 import { HttpError } from '@/shared/errors/http-error';
+import { UserNotFound } from '@/shared/errors/user-not-found-error';
+import { HttpStatusCode } from '@/shared/protocols/http-client';
+import { accountRepositoryMock } from '@/shared/test-helpers/mocks/repositories/account-repository.mock';
 import { userRepositoryMock } from '@/shared/test-helpers/mocks/repositories/user-repository.mock';
 
 import { UserController } from './user-controller';
@@ -20,16 +24,25 @@ const makeSut = () => {
     new UserFindByIdService(userRepositoryMock)
   );
 
+  const mockGetUserAccountsService = mock<GetUserAccountsService>(
+    new GetUserAccountsService(userRepositoryMock, accountRepositoryMock)
+  );
+
   const userController = new UserController(
     mockUserCreateService,
-    mockUserFindByIdService
+    mockUserFindByIdService,
+    mockGetUserAccountsService
   );
 
   const req = mockDeep<Request>();
-  const res = mockDeep<Response>();
+  const res = {
+    json: vi.fn(),
+    status: vi.fn().mockReturnThis(),
+  } as unknown as Response;
   const next = vi.fn();
 
   return {
+    getUserAccountsService: mockGetUserAccountsService,
     next,
     req,
     res,
@@ -132,4 +145,55 @@ describe('[Controllers] UserController', () => {
       expect(next).toHaveBeenCalledWith(expect.any(Error));
     });
   });
+
+  describe('getAccounts', () => {
+    it('should call next with an error if user not found', async () => {
+      const { getUserAccountsService, next, req, res, userController } =
+        makeSut();
+
+      vi.spyOn(getUserAccountsService, 'execute').mockRejectedValueOnce(
+        new UserNotFound()
+      );
+
+      const uuid = randomUUID();
+      req.params = {
+        id: uuid,
+      };
+
+      await userController.getAccounts(req, res, next);
+
+      expect(next).toHaveBeenCalledWith(new Error('User not found'));
+    });
+
+    it('should return accounts from user', async () => {
+      const { getUserAccountsService, next, req, res, userController } =
+        makeSut();
+
+      const accounts = [
+        {
+          avatarUrl: 'avatar-url',
+          id: 'id1',
+          socialMedia: { id: 1, name: 'social-name' },
+        },
+        {
+          avatarUrl: 'avatar-url',
+          id: 'id2',
+          socialMedia: { id: 2, name: 'social-name' },
+        },
+      ];
+      vi.spyOn(getUserAccountsService, 'execute').mockResolvedValueOnce({
+        accounts,
+      });
+
+      const uuid = randomUUID();
+      req.params = {
+        id: uuid,
+      };
+      await userController.getAccounts(req, res, next);
+
+      expect(res.status).toHaveBeenCalledWith(HttpStatusCode.ok);
+      expect(res.json).toHaveBeenCalledWith(accounts);
+      expect(next).not.toHaveBeenCalled();
+    });
+  });
 });

src/features/user/controllers/user-controller.ts

@@ -1,5 +1,4 @@
-import type { RequestHandler } from 'express';
-
+import type { GetUserAccountsService } from '@/features/account/services/get-user-accounts-service';
 import type { UserCreateService } from '@/features/user/services/user-create-service';
 import type { UserFindByIdService } from '@/features/user/services/user-find-by-id-service';
 import {
@@ -30,11 +29,13 @@ export class UserController implements Controller {
     }
   };
 
-  getAccounts: RequestHandler = (req, res, next) => {
+  getAccounts: AsyncRequestHandler = async (req, res, next) => {
     try {
       const { id } = userFindByIdParamsSchema.parse(req.params);
 
-      return res.status(HttpStatusCode.ok).json({ id });
+      const { accounts } = await this.getUserAccountsService.execute({ id });
+
+      return res.status(HttpStatusCode.ok).json(accounts);
     } catch (error) {
       next(error);
     }
@@ -56,6 +57,7 @@ export class UserController implements Controller {
 
   constructor(
     private serviceCreate: UserCreateService,
-    private serviceFindById: UserFindByIdService
+    private serviceFindById: UserFindByIdService,
+    private getUserAccountsService: GetUserAccountsService
   ) {}
 }

src/features/user/routes/user-controller-factory.ts

@@ -1,4 +1,6 @@
 /* istanbul ignore file -- @preserve */
+import { AccountRepository } from '@/features/account/repositories/account-repository/account-repository';
+import { GetUserAccountsService } from '@/features/account/services/get-user-accounts-service';
 import { UserFindByIdService } from '@/features/user/services/user-find-by-id-service';
 import { BcryptAdapter } from '@/shared/infra/crypto/bcrypt-adapter';
 
@@ -8,15 +10,21 @@ import { UserCreateService } from '../services/user-create-service';
 
 export function userControllerFactory() {
   const userRepository = new UserRepository();
+  const accountRepository = new AccountRepository();
   const bcryptAdapter = new BcryptAdapter();
   const userServiceCreate = new UserCreateService(
     userRepository,
     bcryptAdapter
   );
   const userServiceFindById = new UserFindByIdService(userRepository);
+  const getUserAccountsService = new GetUserAccountsService(
+    userRepository,
+    accountRepository
+  );
   const userController = new UserController(
     userServiceCreate,
-    userServiceFindById
+    userServiceFindById,
+    getUserAccountsService
   );
 
   return { userController };