Skip to content
Snippets Groups Projects
Verified Commit 64c29941 authored by Martin Weise's avatar Martin Weise
Browse files

Fixed tests

parent 58aa0dd0
No related branches found
No related tags found
1 merge request!379Fixed the sync
...@@ -55,6 +55,10 @@ ...@@ -55,6 +55,10 @@
| `dbrepo_tables_find` | Find table | | `dbrepo_tables_find` | Find table |
| `dbrepo_tables_findall` | List tables | | `dbrepo_tables_findall` | List tables |
| `dbrepo_tables_refresh` | Update database table schemas | | `dbrepo_tables_refresh` | Update database table schemas |
| `dbrepo_user_create` | Create user |
| `dbrepo_user_find` | Get user |
| `dbrepo_user_modify` | Update user |
| `dbrepo_users_list` | List users |
| `dbrepo_view_create` | Create view | | `dbrepo_view_create` | Create view |
| `dbrepo_view_delete` | Delete view | | `dbrepo_view_delete` | Delete view |
| `dbrepo_view_find` | Get view | | `dbrepo_view_find` | Get view |
......
...@@ -588,8 +588,7 @@ public class PrometheusEndpointMvcTest extends AbstractUnitTest { ...@@ -588,8 +588,7 @@ public class PrometheusEndpointMvcTest extends AbstractUnitTest {
} }
/* test */ /* test */
for (String metric : List.of("dbrepo_users_list", "dbrepo_user_find", "dbrepo_user_modify", for (String metric : List.of("dbrepo_users_list", "dbrepo_user_find", "dbrepo_user_modify")) {
"dbrepo_user_password_modify")) {
assertThat(registry) assertThat(registry)
.hasObservationWithNameEqualTo(metric); .hasObservationWithNameEqualTo(metric);
} }
......
package at.tuwien.service; package at.tuwien.service;
import at.tuwien.api.user.UserUpdateDto;
import at.tuwien.entities.user.User; import at.tuwien.entities.user.User;
import at.tuwien.exception.AuthServiceException; import at.tuwien.exception.AuthServiceException;
import at.tuwien.exception.UserNotFoundException; import at.tuwien.exception.UserNotFoundException;
import at.tuwien.gateway.KeycloakGateway;
import at.tuwien.repository.UserRepository; import at.tuwien.repository.UserRepository;
import at.tuwien.test.AbstractUnitTest; import at.tuwien.test.AbstractUnitTest;
import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.BeforeEach;
...@@ -15,11 +17,11 @@ import org.springframework.test.context.junit.jupiter.SpringExtension; ...@@ -15,11 +17,11 @@ import org.springframework.test.context.junit.jupiter.SpringExtension;
import java.util.List; import java.util.List;
import java.util.Optional; import java.util.Optional;
import java.util.UUID;
import static org.junit.jupiter.api.Assertions.assertEquals; import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertThrows; import static org.junit.jupiter.api.Assertions.assertThrows;
import static org.mockito.Mockito.any; import static org.mockito.Mockito.*;
import static org.mockito.Mockito.when;
@ExtendWith(SpringExtension.class) @ExtendWith(SpringExtension.class)
@SpringBootTest @SpringBootTest
...@@ -28,6 +30,9 @@ public class UserServiceUnitTest extends AbstractUnitTest { ...@@ -28,6 +30,9 @@ public class UserServiceUnitTest extends AbstractUnitTest {
@MockBean @MockBean
private UserRepository userRepository; private UserRepository userRepository;
@MockBean
private KeycloakGateway keycloakGateway;
@Autowired @Autowired
private UserService userService; private UserService userService;
...@@ -82,6 +87,9 @@ public class UserServiceUnitTest extends AbstractUnitTest { ...@@ -82,6 +87,9 @@ public class UserServiceUnitTest extends AbstractUnitTest {
.thenReturn(Optional.of(USER_1)); .thenReturn(Optional.of(USER_1));
when(userRepository.save(any(User.class))) when(userRepository.save(any(User.class)))
.thenReturn(USER_1); .thenReturn(USER_1);
doNothing()
.when(keycloakGateway)
.updateUser(any(UUID.class), any(UserUpdateDto.class));
/* test */ /* test */
final User response = userService.modify(USER_1, USER_1_UPDATE_DTO); final User response = userService.modify(USER_1, USER_1_UPDATE_DTO);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment