Merge pull request #27916 from cescoffier/fix-redis-fromMember-visibi…

…lity
quarkusio · Sep 14, 2022 · a5d62c2 · a5d62c2
2 parents 9402450 + 33c8625
commit a5d62c2
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/...ons/redis-client/runtime/src/main/java/io/quarkus/redis/datasource/geo/GeoSearchArgs.java b/...ons/redis-client/runtime/src/main/java/io/quarkus/redis/datasource/geo/GeoSearchArgs.java
@@ -56,7 +56,7 @@ public GeoSearchArgs<V> fromMember(V member) {
      * @param latitude the latitude
      * @return the current {@code GeoSearchArgs}
      */
-    private GeoSearchArgs<V> fromCoordinate(double longitude, double latitude) {
+    public GeoSearchArgs<V> fromCoordinate(double longitude, double latitude) {
         this.longitude = longitude;
         this.latitude = latitude;
         return this;

diff --git a/...sions/redis-client/runtime/src/test/java/io/quarkus/redis/datasource/GeoCommandsTest.java b/...sions/redis-client/runtime/src/test/java/io/quarkus/redis/datasource/GeoCommandsTest.java
@@ -494,6 +494,17 @@ void geosearchWithArgs() {
             assertThat(gv.geohash).isEmpty();
         });
 
+        args = new GeoSearchArgs<Place>().fromCoordinate(CRUSSOL_LONGITUDE, CRUSSOL_LATITUDE)
+                .byRadius(5, GeoUnit.KM).withCoordinates().withDistance().descending();
+        places = geo.geosearch(key, args);
+        assertThat(places).hasSize(1).allSatisfy(gv -> {
+            assertThat(gv.member).isEqualTo(Place.crussol);
+            assertThat(gv.longitude).isNotEmpty();
+            assertThat(gv.latitude).isNotEmpty();
+            assertThat(gv.distance).isNotEmpty();
+            assertThat(gv.geohash).isEmpty();
+        });
+
     }
 
     @Test