Skip to content
Snippets Groups Projects
Commit 89d0fb60 authored by Lukasz Lopatowski's avatar Lukasz Lopatowski
Browse files

Merge branch 'itest-fixes' into 'release/1.7.1'

Itest fixes

See merge request !140
parents ff92436e 3baddfbc
No related branches found
No related tags found
2 merge requests!140Itest fixes,!137Draft: Release/1.7.1
Pipeline #93274 passed
...@@ -66,27 +66,15 @@ class OidcUserServiceImplTest { ...@@ -66,27 +66,15 @@ class OidcUserServiceImplTest {
//given //given
User existingUser = new User("testuser"); User existingUser = new User("testuser");
existingUser.setEmail("test@example.com"); existingUser.setEmail("test@example.com");
existingUser.setSamlToken("test@example.com");
//when //when
when(userService.existsBySamlToken("test-sub")).thenReturn(false); when(userService.existsByEmail("test@example.com")).thenReturn(true);
when(userService.existsBySamlToken("testuser")).thenReturn(true); when(userService.findByEmail("test@example.com")).thenReturn(existingUser);
when(userService.findBySamlToken("testuser")).thenReturn(Optional.of(existingUser));
User result = oidcUserService.checkUser(oidcUser); User result = oidcUserService.checkUser(oidcUser);
//then //then
assertEquals(existingUser, result); assertEquals(existingUser, result);
} }
@Test
void shouldThrowExceptionWhenPreferredUsernameDoesNotMatchEmail() {
//given
User existingUser = new User("testuser");
existingUser.setEmail("diffrent@example.com");
//when
when(userService.existsBySamlToken("test-sub")).thenReturn(false);
when(userService.existsBySamlToken("testuser")).thenReturn(true);
when(userService.findBySamlToken("testuser")).thenReturn(Optional.of(existingUser));
//then
assertThrows(ExternalUserMatchException.class, () -> oidcUserService.checkUser(oidcUser));
}
} }
\ No newline at end of file
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment