Skip to content

Commit 41f5337

Browse files
committed
style: improve formatting in SetDefaultContributorRole use case and tests
1 parent 68e4ce0 commit 41f5337

4 files changed

Lines changed: 14 additions & 5 deletions

File tree

‎src/collections/domain/useCases/SetDefaultContributorRole.ts‎

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,9 @@ export class SetDefaultContributorRole implements UseCase<void> {
2222
collectionIdOrAlias: number | string = ROOT_COLLECTION_ID,
2323
roleAlias: RoleAlias | string
2424
): Promise<void> {
25-
return await this.collectionsRepository.setDefaultContributorRole(collectionIdOrAlias, roleAlias)
25+
return await this.collectionsRepository.setDefaultContributorRole(
26+
collectionIdOrAlias,
27+
roleAlias
28+
)
2629
}
2730
}

‎src/roles/index.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,4 +8,4 @@ const getUserSelectableRoles = new GetUserSelectableRoles(rolesRepository)
88
export { getUserSelectableRoles }
99

1010
export { Role } from './domain/models/Role'
11-
export { RoleAlias } from './domain/models/RoleAlias'
11+
export { RoleAlias } from './domain/models/RoleAlias'

‎test/integration/collections/SetDefaultContributorRole.test.ts‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,8 @@ describe('SetDefaultContributorRole', () => {
5050
test('should throw an error when the role alias does not exist', async () => {
5151
const nonExistentRoleAlias = 'invalidRoleAlias'
5252

53-
await expect(useCase.execute(testCollectionAlias, nonExistentRoleAlias)).rejects.toThrow(WriteError)
53+
await expect(useCase.execute(testCollectionAlias, nonExistentRoleAlias)).rejects.toThrow(
54+
WriteError
55+
)
5456
})
5557
})

‎test/unit/collections/SetDefaultContributorRole.test.ts‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,9 @@ describe('execute', () => {
99
collectionRepositoryStub.setDefaultContributorRole = jest.fn().mockResolvedValue(undefined)
1010
const testSetDefaultContributorRole = new SetDefaultContributorRole(collectionRepositoryStub)
1111

12-
await expect(testSetDefaultContributorRole.execute(1, RoleAlias.CURATOR)).resolves.toBeUndefined()
12+
await expect(
13+
testSetDefaultContributorRole.execute(1, RoleAlias.CURATOR)
14+
).resolves.toBeUndefined()
1315
expect(collectionRepositoryStub.setDefaultContributorRole).toHaveBeenCalledWith(1, 'curator')
1416
})
1517

@@ -20,7 +22,9 @@ describe('execute', () => {
2022
.mockRejectedValue(new WriteError())
2123
const testSetDefaultContributorRole = new SetDefaultContributorRole(collectionRepositoryStub)
2224

23-
await expect(testSetDefaultContributorRole.execute(1, RoleAlias.CURATOR)).rejects.toThrow(WriteError)
25+
await expect(testSetDefaultContributorRole.execute(1, RoleAlias.CURATOR)).rejects.toThrow(
26+
WriteError
27+
)
2428
expect(collectionRepositoryStub.setDefaultContributorRole).toHaveBeenCalledWith(1, 'curator')
2529
})
2630
})

0 commit comments

Comments
 (0)