Fix the output of the GET /activity to just contain the array of ws i…

…ds (#12271)

not the full page object.

Signed-off-by: Lukas Krejci <lkrejci@redhat.com>

wsmaster/che-core-api-workspace-activity/src/main/java/org/eclipse/che/api/workspace/activity/WorkspaceActivityService.java

@@ -130,6 +130,6 @@ public Response getWorkspacesByActivity(
     Page<String> data =
         workspaceActivityManager.findWorkspacesInStatus(status, limit, maxItems, skipCount);
 
-    return Response.ok(data).header("Link", createLinkHeader(data)).build();
+    return Response.ok(data.getItems()).header("Link", createLinkHeader(data)).build();
   }
 }

wsmaster/che-core-api-workspace-activity/src/test/java/org/eclipse/che/api/workspace/activity/WorkspaceActivityServiceTest.java

@@ -13,6 +13,7 @@
 
 import static com.jayway.restassured.RestAssured.given;
 import static java.util.Collections.emptyList;
+import static java.util.Collections.singletonList;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.anyInt;
 import static org.mockito.ArgumentMatchers.anyLong;
@@ -134,14 +135,15 @@ public void shouldBeAbleToQueryWithoutTimeConstraints() throws ServerException {
   public void shouldIgnoredMinDurationWhenThresholdSpecified() throws Exception {
     when(workspaceActivityManager.findWorkspacesInStatus(
             eq(WorkspaceStatus.STOPPED), anyLong(), anyInt(), anyLong()))
-        .thenReturn(new Page<>(emptyList(), 0, 1, 0));
+        .thenReturn(new Page<>(singletonList("ws-1"), 0, 1, 1));
 
     Response response =
         given()
             .when()
             .get(URI.create(SERVICE_PATH + "?status=STOPPED&threshold=15&minDuration=55"));
 
     assertEquals(response.getStatusCode(), 200);
+    assertEquals(response.getBody().print(), "[\"ws-1\"]");
     verify(workspaceActivityManager, times(1))
         .findWorkspacesInStatus(
             eq(WorkspaceStatus.STOPPED), eq(15L), eq(Pages.DEFAULT_PAGE_SIZE), eq(0L));