Fix srem? not being namespaced

resque · May 25, 2023 · 0826717 · 0826717
1 parent 7fd8ad2
commit 0826717
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/redis/namespace.rb b/lib/redis/namespace.rb
@@ -138,7 +138,7 @@ class Namespace
       "rpush"            => [ :first ],
       "rpushx"           => [ :first ],
       "sadd"             => [ :first ],
-      "sadd?"             => [ :first ],
+      "sadd?"            => [ :first ],
       "scard"            => [ :first ],
       "scan"             => [ :scan_style, :second ],
       "scan_each"        => [ :scan_style, :all ],
@@ -159,6 +159,7 @@ class Namespace
       "spop"             => [ :first ],
       "srandmember"      => [ :first ],
       "srem"             => [ :first ],
+      "srem?"            => [ :first ],
       "sscan"            => [ :first ],
       "sscan_each"       => [ :first ],
       "strlen"           => [ :first ],

diff --git a/spec/redis_spec.rb b/spec/redis_spec.rb
@@ -363,6 +363,12 @@
     expect(@namespaced.sadd?('foo', 1)).to eq(false)
   end
 
+  it "should remove members" do
+    @namespaced.sadd('foo', 1)
+    expect(@namespaced.srem?('foo', 1)).to eq(true)
+    expect(@namespaced.srem?('foo', 1)).to eq(false)
+  end
+
   it "should add namespace to sort" do
     @namespaced.sadd('foo', 1)
     @namespaced.sadd('foo', 2)