fix: set request parameter to null when there is no variant

GenomicDataInfrastructure · Nov 29, 2024 · 6efe096 · 6efe096
1 parent 81f1a1c
commit 6efe096
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/...cture/discovery/datasets/infrastructure/beacon/persistence/BeaconDatasetIdsCollector.java b/...cture/discovery/datasets/infrastructure/beacon/persistence/BeaconDatasetIdsCollector.java
@@ -47,7 +47,8 @@ public Map<String, Integer> collect(DatasetSearchQuery query, String accessToken
 
         var beaconQuery = BeaconIndividualsRequestMapper.from(query);
 
-        if (beaconAuthorization == null || beaconQuery.getQuery().getFilters().isEmpty()) {
+        if (beaconAuthorization == null || (beaconQuery.getQuery().getFilters().isEmpty() &&
+                beaconQuery.getQuery().getRequestParameters().isEmpty())) {
             return null;
         }
 

diff --git a/.../discovery/datasets/infrastructure/beacon/persistence/BeaconIndividualsRequestMapper.java b/.../discovery/datasets/infrastructure/beacon/persistence/BeaconIndividualsRequestMapper.java
@@ -61,7 +61,7 @@ public BeaconIndividualsRequest from(
                         .testMode(false)
                         .pagination(new BeaconIndividualsRequestQueryPagination())
                         .filters(beaconFilters)
-                        .requestParameters(requestParameters)
+                        .requestParameters(requestParameters.isEmpty() ? null : requestParameters)
                         .build())
                 .build();
     }

diff --git a/...tructure/discovery/datasets/infrastructure/beacon/BeaconIndividualsRequestMapperTest.java b/...tructure/discovery/datasets/infrastructure/beacon/BeaconIndividualsRequestMapperTest.java
@@ -45,7 +45,7 @@ void shouldAcceptNullQuery() {
                                         .skip(0)
                                         .build())
                                 .filters(List.of())
-                                .requestParameters(List.of())
+                                .requestParameters(null)
                                 .build())
                         .build());
     }
@@ -71,7 +71,7 @@ void shouldAcceptEmptyQuery() {
                                         .skip(0)
                                         .build())
                                 .filters(List.of())
-                                .requestParameters(List.of())
+                                .requestParameters(null)
                                 .build())
                         .build());
     }
@@ -97,7 +97,7 @@ void shouldAcceptEmptyFacets() {
                                         .skip(0)
                                         .build())
                                 .filters(List.of())
-                                .requestParameters(List.of())
+                                .requestParameters(null)
                                 .build())
                         .build());
     }