Merge pull request #26 from cheprasov/dev-with-versions

Added command GEODEL

Author: cheprasov

src/RedisClient/Command/Traits/Version3x2/GeoCommandsTrait.php

@@ -174,4 +174,17 @@ public function georadiusbymember($key, $member, $radius, $unit, $withcoord = fa
         return $this->returnCommand(['GEORADIUSBYMEMBER'], $params, $parse ? ResponseParser::PARSE_GEO_ARRAY : null);
     }
 
+    /**
+     * GEODEL key member [member ...]
+     * @link https://github.com/antirez/redis-doc/commit/0f0db1548bef3e005e775ead79b5b9c13be4baed
+     * @see \RedisClient\Command\Traits\Version2x6\SortedSetsCommandsTrait::zrem
+     *
+     * @param string $key
+     * @param string|string[] $members
+     * @return int The number of members removed from the sorted set, not including non existing members.
+     */
+    public function geodel($key, $members) {
+        return $this->zrem($key, $members);
+    }
+
 }

tests/Integration/Version3x2/GeoCommandsTest.php

@@ -266,4 +266,37 @@ public function test_georadiusbymember() {
         }
     }
 
+
+    /**
+     * @see GeoCommandsTrait::geodel
+     */
+    public function test_geodel() {
+        $Redis = static::$Redis;
+
+        $this->assertSame(3, $Redis->geoadd('Sicily', [
+            'Agrigento' => ['13.583333', '37.316667'],
+            'Palermo' => ['13.361389', '38.115556'],
+            'Catania' => ['15.087269', '37.502669']
+        ]));
+
+        $this->assertSame(['Agrigento', 'Palermo', 'Catania'], $Redis->georadiusbymember('Sicily', 'Agrigento', 300, 'km', 0, 0, 0, 0, true));
+
+        $this->assertSame(1, $Redis->geodel('Sicily', 'Palermo'));
+        $this->assertSame(['Agrigento', 'Catania'], $Redis->georadiusbymember('Sicily', 'Agrigento', 300, 'km', 0, 0, 0, 0, true));
+
+        $this->assertSame(0, $Redis->geodel('Sicily', 'Palermo'));
+        $this->assertSame(['Agrigento', 'Catania'], $Redis->georadiusbymember('Sicily', 'Agrigento', 300, 'km', 0, 0, 0, 0, true));
+
+        $this->assertSame(1, $Redis->geodel('Sicily', ['Catania', 'Catania', 'Palermo']));
+        $this->assertSame(['Agrigento'], $Redis->georadiusbymember('Sicily', 'Agrigento', 300, 'km', 0, 0, 0, 0, true));
+
+        $Redis->set('foo', 'bar');
+        try {
+            $Redis->geodel('foo', 'Agrigento');
+            $this->assertFalse('Expect Exception');
+        } catch (\Exception $Ex) {
+            $this->assertInstanceOf(ErrorResponseException::class, $Ex);
+        }
+    }
+
 }