Merge pull request #1318 from s01ipsist/update_testing_matrix

Add security supported versions to testing matrix

Author: byroot

.github/workflows/test.yaml

@@ -148,7 +148,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        redis: ["7.0", "6.2", "6.0", "5.0"]
+        redis: ["8.2", "8.0", "7.4", "7.2", "7.0", "6.2", "6.0", "5.0"]
     runs-on: ubuntu-latest
     env:
       LOW_TIMEOUT: "0.14"

test/redis/commands_on_geo_test.rb

@@ -59,20 +59,36 @@ def test_georadiusbymember_with_options_count_sort
     assert_equal [["Palermo", "166.2742"]], city
   end
 
+  def coordinates_catania
+    if version >= "8.0"
+      ["15.087267458438873", "37.50266842333162"]
+    else
+      ["15.08726745843887329", "37.50266842333162032"]
+    end
+  end
+
+  def coordinates_palermo
+    if version >= "8.0"
+      ["13.361389338970184", "38.1155563954963"]
+    else
+      ["13.36138933897018433", "38.11555639549629859"]
+    end
+  end
+
   def test_geopos
     location = r.geopos("Sicily", "Catania")
-    assert_equal [["15.08726745843887329", "37.50266842333162032"]], location
+    assert_equal [coordinates_catania], location
 
     locations = r.geopos("Sicily", ["Palermo", "Catania"])
-    assert_equal [["13.36138933897018433", "38.11555639549629859"], ["15.08726745843887329", "37.50266842333162032"]], locations
+    assert_equal [coordinates_palermo, coordinates_catania], locations
   end
 
   def test_geopos_nonexistant_location
     location = r.geopos("Sicily", "Rome")
     assert_equal [nil], location
 
     locations = r.geopos("Sicily", ["Rome", "Catania"])
-    assert_equal [nil, ["15.08726745843887329", "37.50266842333162032"]], locations
+    assert_equal [nil, coordinates_catania], locations
   end
 
   def test_geodist

test/support/conf/redis-7.4.conf

@@ -0,0 +1,3 @@
+appendonly no
+save ""
+enable-debug-command yes

test/support/conf/redis-8.0.conf

@@ -0,0 +1,3 @@
+appendonly no
+save ""
+enable-debug-command yes

test/support/conf/redis-8.2.conf

@@ -0,0 +1,3 @@
+appendonly no
+save ""
+enable-debug-command yes