Skip to content

Commit db29f84

Browse files
committed
test(account): corrige testes após refatoração da funcionalidade em commit anterior
1 parent db7cf28 commit db29f84

File tree

3 files changed

+7
-5
lines changed

3 files changed

+7
-5
lines changed

packages/account/domain/src/__tests__/mocks/dtos/account.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ export const account = {
1515
staff: false,
1616
fellow: false,
1717
manager: false,
18-
director: true,
18+
director: false,
1919
},
2020
gender: 'male',
2121
photo: '9c14863726af550ff0666c128c32286a',

packages/account/domain/src/__tests__/mocks/services/accounts.ts

+3-1
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,9 @@ export class AccountsServiceMock implements AccountsService {
2121
}
2222

2323
async findOne(id: string): Promise<Account | null> {
24-
throw new Error('Method not implemented.');
24+
const account = this.#accounts.find((a) => a.id === id);
25+
26+
return account ?? null;
2527
}
2628

2729
async findOneBy(filter: FindFilter<Account>): Promise<Account | null> {

packages/account/domain/src/__tests__/server/use-cases/change-roles.spec.ts

+3-3
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ describe('Change Roles UseCase', () => {
2828
});
2929

3030
it('member assign staff should throw access denied', async () => {
31-
const assign = { id: '', roles: merge(accountRole, { staff: true }) };
31+
const assign = { id: '66f706967d818c4004effb48', roles: merge(accountRole, { staff: true }) };
3232

3333
const assigner = merge(authUser, {
3434
roles: merge(accountRole, { member: true }),
@@ -40,7 +40,7 @@ describe('Change Roles UseCase', () => {
4040
});
4141

4242
it('staff assign manager should throw access denied', async () => {
43-
const assign = { id: '', roles: merge(accountRole, { manager: true }) };
43+
const assign = { id: '66f706967d818c4004effb48', roles: merge(accountRole, { manager: true }) };
4444

4545
const assigner = merge(authUser, {
4646
roles: merge(accountRole, { staff: true }),
@@ -52,7 +52,7 @@ describe('Change Roles UseCase', () => {
5252
});
5353

5454
it('manager assign staff should be ok', async () => {
55-
const assign = { id: '', roles: merge(accountRole, { staff: true }) };
55+
const assign = { id: '66f706967d818c4004effb48', roles: merge(accountRole, { staff: true }) };
5656

5757
const assigner = merge(authUser, {
5858
roles: merge(accountRole, { manager: true }),

0 commit comments

Comments
 (0)