Fix UserSelectionsApiTest being too strict about the submittor

geonetwork · Nov 7, 2024 · ddb7766 · ddb7766
1 parent 66e9c38
commit ddb7766
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/index/src/main/java/org/fao/geonet/index/es/EsRestClient.java b/index/src/main/java/org/fao/geonet/index/es/EsRestClient.java
@@ -336,23 +336,21 @@ public SearchResponse query(String index, Query.Builder queryBuilder, Query.Buil
     }
 
 
-    public String deleteByQuery(String index, String query) throws Exception {
+    public void deleteByQuery(String index, String query) throws Exception {
         if (!activated) {
-            return "";
+            throw new IllegalStateException("Index not yet activated.");
         }
 
         DeleteByQueryRequest request = DeleteByQueryRequest.of(
-            b -> b.index(new ArrayList<>(Arrays.asList(index)))
+            b -> b.index(index)
                 .q(query)
                 .refresh(true));
 
         final DeleteByQueryResponse deleteByQueryResponse =
             client.deleteByQuery(request);
 
 
-        if (deleteByQueryResponse.deleted() >= 0) {
-            return String.format("Record removed. %s.", deleteByQueryResponse.deleted());
-        } else {
+        if (!deleteByQueryResponse.failures().isEmpty()) {
             StringBuilder stringBuilder = new StringBuilder();
 
             deleteByQueryResponse.failures().forEach(f -> stringBuilder.append(f.toString()));