fix: 🐛 remove findAllUsers

zhumeisongsong · Nov 14, 2024 · 29547f2 · 29547f2
1 parent 122ec12
commit 29547f2
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 29 deletions.
diff --git a/libs/user/domain/src/lib/user.repository.spec.ts b/libs/user/domain/src/lib/user.repository.spec.ts
@@ -10,10 +10,6 @@ class MockUserRepository implements UserRepository {
   async findById(id: string): Promise<User | null> {
     return this.users.find((user) => user.id === id) || null;
   }
-
-  async findAll(): Promise<User[]> {
-    return this.users;
-  }
 }
 
 describe('UserRepository', () => {
@@ -33,11 +29,4 @@ describe('UserRepository', () => {
     expect(user).toBeNull();
   });
 
-  test('findAll should return all users', async () => {
-    const users = await userRepository.findAll();
-    expect(users).toEqual([
-      { id: '1', name: 'John Doe' },
-      { id: '2', name: 'Jane Doe' },
-    ]);
-  });
 });
diff --git a/libs/user/domain/src/lib/user.repository.ts b/libs/user/domain/src/lib/user.repository.ts
@@ -2,5 +2,4 @@ import { User } from './user.entity';
 
 export interface UserRepository {
   findById(id: string): Promise<User | null>;
-  findAll(): Promise<User[]>;
 }
diff --git a/libs/user/usecase/src/lib/get-user.usecase.spec.ts b/libs/user/usecase/src/lib/get-user.usecase.spec.ts
@@ -8,7 +8,6 @@ describe('GetUserUsecase', () => {
   beforeEach(() => {
     userRepository = {
       findById: jest.fn(),
-      findAll: jest.fn(),
     } as unknown as jest.Mocked<UserRepository>;
 
     getUserUsecase = new GetUserUsecase(userRepository);
@@ -34,16 +33,4 @@ describe('GetUserUsecase', () => {
       expect(userRepository.findById).toHaveBeenCalledWith('1');
     });
   });
-
-  describe('executeAll', () => {
-    it('should return all users', async () => {
-      const users = [{ id: '1', name: 'John Doe' }, { id: '2', name: 'Jane Doe' }];
-      userRepository.findAll.mockResolvedValue(users);
-
-      const result = await getUserUsecase.executeAll();
-
-      expect(result).toEqual(users);
-      expect(userRepository.findAll).toHaveBeenCalled();
-    });
-  });
 });
diff --git a/libs/user/usecase/src/lib/get-user.usecase.ts b/libs/user/usecase/src/lib/get-user.usecase.ts
@@ -6,8 +6,4 @@ export class GetUserUsecase {
   async execute(id: string): Promise<User | null> {
     return this.userRepository.findById(id);
   }
-
-  async executeAll(): Promise<User[]> {
-    return this.userRepository.findAll();
-  }
 }