Merge pull request #1 from Syft-Application/FIX_srem_namespacing

FIX missing namespaced srem? method
Syft-Application · May 24, 2023 · d442c91 · d442c91
2 parents 7fd8ad2 + b724dfc
commit d442c91
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)