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

OZG-6867 Renamed method getAddedResourceIdFromResponse()

parent 89ff7b63
No related branches found
No related tags found
No related merge requests found
...@@ -21,11 +21,11 @@ class KeycloakApiService { ...@@ -21,11 +21,11 @@ class KeycloakApiService {
public String addGroup(GroupRepresentation groupRepresentation) { public String addGroup(GroupRepresentation groupRepresentation) {
try (var response = groupsResource.add(groupRepresentation)) { try (var response = groupsResource.add(groupRepresentation)) {
return getAddedResourceIdFromResponse(response); return getCreatedResourceIdFromResponse(response);
} }
} }
String getAddedResourceIdFromResponse(Response response) { String getCreatedResourceIdFromResponse(Response response) {
if (response.getStatus() == Response.Status.CREATED.getStatusCode()) { if (response.getStatus() == Response.Status.CREATED.getStatusCode()) {
return extractResourceIdFromLocationHeader(response.getHeaderString("Location")); return extractResourceIdFromLocationHeader(response.getHeaderString("Location"));
} }
......
...@@ -58,7 +58,7 @@ class KeycloakApiServiceTest { ...@@ -58,7 +58,7 @@ class KeycloakApiServiceTest {
@BeforeEach @BeforeEach
void init() { void init() {
when(groupsResource.add(groupRepresentation)).thenReturn(response); when(groupsResource.add(groupRepresentation)).thenReturn(response);
doReturn(RESOURCE_ID).when(service).getAddedResourceIdFromResponse(response); doReturn(RESOURCE_ID).when(service).getCreatedResourceIdFromResponse(response);
} }
@Test @Test
...@@ -72,7 +72,7 @@ class KeycloakApiServiceTest { ...@@ -72,7 +72,7 @@ class KeycloakApiServiceTest {
void shouldGetAddedResourceIdFromResponse() { void shouldGetAddedResourceIdFromResponse() {
callService(); callService();
verify(service).getAddedResourceIdFromResponse(response); verify(service).getCreatedResourceIdFromResponse(response);
} }
@Test @Test
...@@ -88,7 +88,7 @@ class KeycloakApiServiceTest { ...@@ -88,7 +88,7 @@ class KeycloakApiServiceTest {
} }
@Nested @Nested
class TestGetAddedResourceIdFromResponse { class TestGetCreatedResourceIdFromResponse {
private final String RESOURCE_ID = GroupRepresentationTestFactory.create().getId(); private final String RESOURCE_ID = GroupRepresentationTestFactory.create().getId();
private final String LOCATION = LoremIpsum.getInstance().getUrl(); private final String LOCATION = LoremIpsum.getInstance().getUrl();
...@@ -131,7 +131,7 @@ class KeycloakApiServiceTest { ...@@ -131,7 +131,7 @@ class KeycloakApiServiceTest {
} }
private String callService() { private String callService() {
return service.getAddedResourceIdFromResponse(response); return service.getCreatedResourceIdFromResponse(response);
} }
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment