Marmelator/tenant-user-management-service
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 202 Wiki Activity

RED-6625: Inconsistencies when creating a new user

Browse Source
This commit is contained in:
maverickstuder 2024-02-05 11:06:23 +01:00
parent dc2a11ac83
commit a20484e486
2 changed files with 22 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/main/java/com/knecon/fforesight/tenantusermanagement/service/UserService.java
View File

@ -73,6 +73,10 @@ public class UserService {
@CacheEvict(value = "${commons.keycloak.userCache}", allEntries = true, beforeInvocation = true)
public User createUser(CreateUserRequest user) {
if (StringUtils.isEmpty(user.getEmail())) {
throw new ResponseStatusException(HttpStatus.BAD_REQUEST, "Email address must be set");
}
String username = StringUtils.isEmpty(user.getUsername()) ? user.getEmail() : user.getUsername();
if (!this.getTenantUsersResource().search(username).isEmpty()) {
throw new ResponseStatusException(HttpStatus.BAD_REQUEST, "User with this username already exists");

18
src/test/java/com/knecon/fforesight/tests/UserTest.java
View File

@ -133,5 +133,23 @@ public class UserTest extends AbstractTenantUserManagementIntegrationTest {
}
}
@Test
public void testCreateUserWithoutMail() {
TenantContext.setTenantId(AbstractTenantUserManagementIntegrationTest.TEST_TENANT_ID);
CreateUserRequest createUserRequest = new CreateUserRequest();
createUserRequest.setFirstName("Test");
createUserRequest.setLastName("New User");
createUserRequest.setUsername(createUserRequest.getEmail());
createUserRequest.setRoles(tenantUserManagementProperties.getKcRoleMapping().getAllRoles());
try {
userClient.createUser(createUserRequest);
} catch (FeignException e) {
assertThat(e.status()).isEqualTo(400);
}
}
}