Skip to content
Snippets Groups Projects
Commit b4f40a2c authored by OZGCloud's avatar OZGCloud
Browse files

OZG-3539 OZG-6824 improve code after code review

parent a57802e0
No related branches found
No related tags found
No related merge requests found
...@@ -258,6 +258,7 @@ class UserSettingsResourceTest { ...@@ -258,6 +258,7 @@ class UserSettingsResourceTest {
@Test @Test
void shouldCheckAccess() { void shouldCheckAccess() {
when(resourceAssembler.toResource(any(), anyString(), anyString())).thenReturn(new HalEntityWrapper(null)); when(resourceAssembler.toResource(any(), anyString(), anyString())).thenReturn(new HalEntityWrapper(null));
resource.putUserSettings(USER_ID, userSettings); resource.putUserSettings(USER_ID, userSettings);
verify(resource).checkUserAccess(USER_ID); verify(resource).checkUserAccess(USER_ID);
...@@ -266,6 +267,7 @@ class UserSettingsResourceTest { ...@@ -266,6 +267,7 @@ class UserSettingsResourceTest {
@Test @Test
void shouldCallUserSettingsService() { void shouldCallUserSettingsService() {
when(resourceAssembler.toResource(any(), anyString(), anyString())).thenReturn(new HalEntityWrapper(null)); when(resourceAssembler.toResource(any(), anyString(), anyString())).thenReturn(new HalEntityWrapper(null));
resource.putUserSettings(USER_ID, userSettings); resource.putUserSettings(USER_ID, userSettings);
verify(userSettingsService).updateByUserId(USER_ID, userSettings); verify(userSettingsService).updateByUserId(USER_ID, userSettings);
...@@ -275,6 +277,7 @@ class UserSettingsResourceTest { ...@@ -275,6 +277,7 @@ class UserSettingsResourceTest {
void void
shouldCallResourceAssembler() { shouldCallResourceAssembler() {
when(resourceAssembler.toResource(any(), anyString(), anyString())).thenReturn(new HalEntityWrapper(null)); when(resourceAssembler.toResource(any(), anyString(), anyString())).thenReturn(new HalEntityWrapper(null));
resource.putUserSettings(USER_ID, userSettings); resource.putUserSettings(USER_ID, userSettings);
verify(resourceAssembler).toResource(updatedUserSettings, USER_ID, USER_MANAGER_URL); verify(resourceAssembler).toResource(updatedUserSettings, USER_ID, USER_MANAGER_URL);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment