mock_redis 0.35.0 → 0.36.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ac517fcc81e569da0e5e0512027dd381ebd8c0553aaa9aa373ffb7be3afe983e
-  data.tar.gz: 5e64b3ac11ab9c403165c082b8c2db641858ffd7430a6f6fd1c3c88c1d7b7090
+  metadata.gz: dac18dd7d43fb11953bf9988a89d64d8f82a0d6b909836d215880b5fc6c75e98
+  data.tar.gz: e34af1ec074a43fbdeefa25e27543c57a146e0e22900842875adfca1fed3a698
 SHA512:
-  metadata.gz: c4ab0e03ac161be256d4920837fe1cc70da6ae57f29b4e75c1f2eec404b46ad2efa5679552b089eddd3234d49eb85a8ec0f1a51cbfa118654da31a0c43f4c7cb
-  data.tar.gz: 8c97fae25f874bdbbaedd52e0ac10bd793fae3cb711e7e6a368147301fbc653a35d0630e9c0ae172cc625b2b9c4f266d9d7dd08fe92441cfc5e04156d98a1ebd
+  metadata.gz: a47e3c68ad32983cd4017ba6541dbfe3004d197468179713d5102bbe094a9978256831f9ee3d3c9a7abf8ec0c07b18ee8c04b911e9ca326e8d04da8288b8131e
+  data.tar.gz: 119d337effa67690de4754bfa9bfe37bcceade74089b73d8328fd7818877b14afb708ba432ea7b52c6299186d7dad736b14de95acde7bb51909ad6096eb1a143

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # MockRedis Changelog
 
+### 0.36.0
+
+* Add support for `smismember`
+* Add support for `lmove` and `blmove`
+* Fix `zrem` to support passing array of integers
+
 ### 0.35.0
 
 * Add support for `getdel`

data/README.md

@@ -55,8 +55,8 @@ since it's an in-memory object confined to a single process. MockRedis
 makes every attempt to be Redis-compatible, but there are some
 necessary exceptions.
 
-* Blocking list commands (`#blpop`, `#brpop`, and `#brpoplpush`) work
-  as expected if there is data for them to retrieve. If you use one of
+* Blocking list commands (`#blmove`, `#blpop`, `#brpop`, and `#brpoplpush`)
+  work as expected if there is data for them to retrieve. If you use one of
   these commands with a nonzero timeout and there is no data for it to
   retrieve, then the command returns immediately. However, if you ask
   one of these commands for data with a 0 timeout (means "wait

data/lib/mock_redis/list_methods.rb

@@ -6,6 +6,20 @@ class MockRedis
     include Assertions
     include UtilityMethods
 
+    def blmove(source, destination, wherefrom, whereto, options = {})
+      options = { :timeout => options } if options.is_a?(Integer)
+      timeout = options.is_a?(Hash) && options[:timeout] || 0
+      assert_valid_timeout(timeout)
+
+      if llen(source) > 0
+        lmove(source, destination, wherefrom, whereto)
+      elsif timeout > 0
+        nil
+      else
+        raise MockRedis::WouldBlock, "Can't block forever"
+      end
+    end
+
     def blpop(*args)
       lists, timeout = extract_timeout(args)
       nonempty_list = first_nonempty_list(lists)
@@ -78,6 +92,22 @@ class MockRedis
       with_list_at(key, &:length)
     end
 
+    def lmove(source, destination, wherefrom, whereto)
+      assert_listy(source)
+      assert_listy(destination)
+
+      wherefrom = wherefrom.to_s.downcase
+      whereto = whereto.to_s.downcase
+
+      unless %w[left right].include?(wherefrom) && %w[left right].include?(whereto)
+        raise Redis::CommandError, 'ERR syntax error'
+      end
+
+      value = wherefrom == 'left' ? lpop(source) : rpop(source)
+      (whereto == 'left' ? lpush(destination, value) : rpush(destination, value)) unless value.nil?
+      value
+    end
+
     def lpop(key)
       with_list_at(key, &:shift)
     end

data/lib/mock_redis/set_methods.rb

@@ -64,6 +64,12 @@ class MockRedis
       with_set_at(key) { |s| s.include?(member.to_s) }
     end
 
+    def smismember(key, *members)
+      with_set_at(key) do |set|
+        members.flatten.map { |m| set.include?(m.to_s) }
+      end
+    end
+
     def smembers(key)
       with_set_at(key, &:to_a).map(&:dup).reverse
     end

data/lib/mock_redis/version.rb

@@ -2,5 +2,5 @@
 
 # Defines the gem version.
 class MockRedis
-  VERSION = '0.35.0'
+  VERSION = '0.36.0'
 end

data/lib/mock_redis/zset_methods.rb

@@ -147,7 +147,7 @@ class MockRedis
         else
           retval = args.map { |member| !!zscore(key, member.to_s) }.count(true)
           with_zset_at(key) do |z|
-            args.each { |member| z.delete?(member) }
+            args.each { |member| z.delete?(member.to_s) }
           end
         end
       end

data/spec/commands/blmove_spec.rb

@@ -0,0 +1,52 @@
+require 'spec_helper'
+
+describe '#blmove(source, destination, wherefrom, whereto, timeout)' do
+  before do
+    @list1 = 'mock-redis-test:blmove-list1'
+    @list2 = 'mock-redis-test:blmove-list2'
+
+    @redises.lpush(@list1, 'b')
+    @redises.lpush(@list1, 'a')
+
+    @redises.lpush(@list2, 'y')
+    @redises.lpush(@list2, 'x')
+  end
+
+  it 'returns the value moved' do
+    @redises.blmove(@list1, @list2, 'left', 'right').should == 'a'
+  end
+
+  it 'takes the first element of source and appends it to destination' do
+    @redises.blmove(@list1, @list2, 'left', 'right')
+
+    @redises.lrange(@list1, 0, -1).should == %w[b]
+    @redises.lrange(@list2, 0, -1).should == %w[x y a]
+  end
+
+  it 'raises an error on negative timeout' do
+    lambda do
+      @redises.blmove(@list1, @list2, 'left', 'right', :timeout => -1)
+    end.should raise_error(Redis::CommandError)
+  end
+
+  let(:default_error) { RedisMultiplexer::MismatchedResponse }
+  it_should_behave_like 'a list-only command'
+
+  context '[mock only]' do
+    it 'ignores positive timeouts and returns nil' do
+      @redises.mock.blmove('mock-redis-test:not-here', @list1, 'left', 'right', :timeout => 1).
+        should be_nil
+    end
+
+    it 'ignores positive legacy timeouts and returns nil' do
+      @redises.mock.blmove('mock-redis-test:not-here', @list1, 'left', 'right', 1).
+        should be_nil
+    end
+
+    it 'raises WouldBlock on zero timeout (no blocking in the mock)' do
+      lambda do
+        @redises.mock.blmove('mock-redis-test:not-here', @list1, 'left', 'right', :timeout => 0)
+      end.should raise_error(MockRedis::WouldBlock)
+    end
+  end
+end

data/spec/commands/lmove_spec.rb

@@ -0,0 +1,72 @@
+require 'spec_helper'
+
+describe '#lmove(source, destination, wherefrom, whereto)' do
+  before do
+    @list1 = 'mock-redis-test:lmove-list1'
+    @list2 = 'mock-redis-test:lmove-list2'
+
+    @redises.lpush(@list1, 'b')
+    @redises.lpush(@list1, 'a')
+
+    @redises.lpush(@list2, 'y')
+    @redises.lpush(@list2, 'x')
+  end
+
+  it 'returns the value moved' do
+    @redises.lmove(@list1, @list2, 'left', 'right').should == 'a'
+  end
+
+  it "returns nil and doesn't append if source empty" do
+    @redises.lmove('empty', @list1, 'left', 'right').should be_nil
+    @redises.lrange(@list1, 0, -1).should == %w[a b]
+  end
+
+  it 'takes the first element of source and prepends it to destination' do
+    @redises.lmove(@list1, @list2, 'left', 'left')
+
+    @redises.lrange(@list1, 0, -1).should == %w[b]
+    @redises.lrange(@list2, 0, -1).should == %w[a x y]
+  end
+
+  it 'takes the first element of source and appends it to destination' do
+    @redises.lmove(@list1, @list2, 'left', 'right')
+
+    @redises.lrange(@list1, 0, -1).should == %w[b]
+    @redises.lrange(@list2, 0, -1).should == %w[x y a]
+  end
+
+  it 'takes the last element of source and prepends it to destination' do
+    @redises.lmove(@list1, @list2, 'right', 'left')
+
+    @redises.lrange(@list1, 0, -1).should == %w[a]
+    @redises.lrange(@list2, 0, -1).should == %w[b x y]
+  end
+
+  it 'takes the last element of source and appends it to destination' do
+    @redises.lmove(@list1, @list2, 'right', 'right')
+
+    @redises.lrange(@list1, 0, -1).should == %w[a]
+    @redises.lrange(@list2, 0, -1).should == %w[x y b]
+  end
+
+  it 'rotates a list when source and destination are the same' do
+    @redises.lmove(@list1, @list1, 'left', 'right')
+    @redises.lrange(@list1, 0, -1).should == %w[b a]
+  end
+
+  it 'removes empty lists' do
+    @redises.llen(@list1).times { @redises.lmove(@list1, @list2, 'left', 'right') }
+    @redises.get(@list1).should be_nil
+  end
+
+  it 'raises an error for non-list source value' do
+    @redises.set(@list1, 'string value')
+
+    lambda do
+      @redises.lmove(@list1, @list2, 'left', 'right')
+    end.should raise_error(Redis::CommandError)
+  end
+
+  let(:default_error) { RedisMultiplexer::MismatchedResponse }
+  it_should_behave_like 'a list-only command'
+end

data/spec/commands/smismember_spec.rb

@@ -0,0 +1,34 @@
+require 'spec_helper'
+
+describe '#smismember(key, *members)' do
+  before do
+    @key = 'mock-redis-test:smismember'
+    @redises.sadd(@key, 'whiskey')
+    @redises.sadd(@key, 'beer')
+  end
+
+  it 'returns true if member is in set' do
+    @redises.smismember(@key, 'whiskey').should == [true]
+    @redises.smismember(@key, 'beer').should == [true]
+    @redises.smismember(@key, 'whiskey', 'beer').should == [true, true]
+    @redises.smismember(@key, %w[whiskey beer]).should == [true, true]
+  end
+
+  it 'returns false if member is not in set' do
+    @redises.smismember(@key, 'cola').should == [false]
+    @redises.smismember(@key, 'whiskey', 'cola').should == [true, false]
+    @redises.smismember(@key, %w[whiskey beer cola]).should == [true, true, false]
+  end
+
+  it 'stringifies member' do
+    @redises.sadd(@key, '1')
+    @redises.smismember(@key, 1).should == [true]
+    @redises.smismember(@key, [1]).should == [true]
+  end
+
+  it 'treats a nonexistent value as an empty set' do
+    @redises.smismember('mock-redis-test:nonesuch', 'beer').should == [false]
+  end
+
+  it_should_behave_like 'a set-only command'
+end

data/spec/commands/zrem_spec.rb

@@ -28,6 +28,13 @@ describe '#zrem(key, member)' do
     @redises.zrange(@key, 0, -1).should == %w[one two]
   end
 
+  it 'removes integer members inside an array from the set' do
+    member = 11
+    @redises.zadd(@key, 3, member)
+    @redises.zrem(@key, [member]).should == 1
+    @redises.zrange(@key, 0, -1).should == %w[one two]
+  end
+
   it 'supports a variable number of arguments' do
     @redises.zrem(@key, %w[one two])
     @redises.zrange(@key, 0, -1).should be_empty

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mock_redis
 version: !ruby/object:Gem::Version
-  version: 0.35.0
+  version: 0.36.0
 platform: ruby
 authors:
 - Shane da Silva
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-12-16 00:00:00.000000000 Z
+date: 2023-01-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby2_keywords
@@ -138,6 +138,7 @@ files:
 - spec/commands/bgsave_spec.rb
 - spec/commands/bitcount_spec.rb
 - spec/commands/bitfield_spec.rb
+- spec/commands/blmove_spec.rb
 - spec/commands/blpop_spec.rb
 - spec/commands/brpop_spec.rb
 - spec/commands/brpoplpush_spec.rb
@@ -191,6 +192,7 @@ files:
 - spec/commands/lindex_spec.rb
 - spec/commands/linsert_spec.rb
 - spec/commands/llen_spec.rb
+- spec/commands/lmove_spec.rb
 - spec/commands/lpop_spec.rb
 - spec/commands/lpush_spec.rb
 - spec/commands/lpushx_spec.rb
@@ -241,6 +243,7 @@ files:
 - spec/commands/sinterstore_spec.rb
 - spec/commands/sismember_spec.rb
 - spec/commands/smembers_spec.rb
+- spec/commands/smismember_spec.rb
 - spec/commands/smove_spec.rb
 - spec/commands/sort_list_spec.rb
 - spec/commands/sort_set_spec.rb
@@ -326,6 +329,7 @@ test_files:
 - spec/commands/bgsave_spec.rb
 - spec/commands/bitcount_spec.rb
 - spec/commands/bitfield_spec.rb
+- spec/commands/blmove_spec.rb
 - spec/commands/blpop_spec.rb
 - spec/commands/brpop_spec.rb
 - spec/commands/brpoplpush_spec.rb
@@ -379,6 +383,7 @@ test_files:
 - spec/commands/lindex_spec.rb
 - spec/commands/linsert_spec.rb
 - spec/commands/llen_spec.rb
+- spec/commands/lmove_spec.rb
 - spec/commands/lpop_spec.rb
 - spec/commands/lpush_spec.rb
 - spec/commands/lpushx_spec.rb
@@ -429,6 +434,7 @@ test_files:
 - spec/commands/sinterstore_spec.rb
 - spec/commands/sismember_spec.rb
 - spec/commands/smembers_spec.rb
+- spec/commands/smismember_spec.rb
 - spec/commands/smove_spec.rb
 - spec/commands/sort_list_spec.rb
 - spec/commands/sort_set_spec.rb