mock_redis 0.19.0 → 0.20.0

data/spec/commands/xtrim_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+
+describe '#xtrim("mystream", 1000, approximate: true)' do
+  before { @key = 'mock-redis-test:xtrim' }
+
+  before :each do
+    @redises.xadd(@key, { key1: 'value1' }, id: '1234567891234-0')
+    @redises.xadd(@key, { key2: 'value2' }, id: '1234567891245-0')
+    @redises.xadd(@key, { key3: 'value3' }, id: '1234567891245-1')
+    @redises.xadd(@key, { key4: 'value4' }, id: '1234567891278-0')
+    @redises.xadd(@key, { key5: 'value5' }, id: '1234567891278-1')
+    @redises.xadd(@key, { key6: 'value6' }, id: '1234567891299-0')
+  end
+
+  it 'returns the number of elements deleted' do
+    expect(@redises.xtrim(@key, 4)).to eq 2
+  end
+
+  it 'deletes the oldes elements' do
+    @redises.xtrim(@key, 4)
+    expect(@redises.xrange(@key, '-', '+')).to eq(
+      [
+        ['1234567891245-1', { 'key3' => 'value3' }],
+        ['1234567891278-0', { 'key4' => 'value4' }],
+        ['1234567891278-1', { 'key5' => 'value5' }],
+        ['1234567891299-0', { 'key6' => 'value6' }]
+      ]
+    )
+  end
+end

data/spec/spec_helper.rb

@@ -12,6 +12,7 @@ require 'rspec/its'
 require 'redis'
 $LOAD_PATH.unshift(File.expand_path(File.join(__FILE__, '..', '..', 'lib')))
 require 'mock_redis'
+require 'timecop'
 
 $LOAD_PATH.unshift(File.expand_path(File.join(File.dirname(__FILE__), '..')))
 Dir['spec/support/**/*.rb'].each { |x| require x }
@@ -59,5 +60,6 @@ RSpec.configure do |config|
         @redises.send_without_checking(:del, key)
       end
     end
+    @redises._gsub_clear
   end
 end

data/spec/support/redis_multiplexer.rb

@@ -10,6 +10,16 @@ class RedisMultiplexer < BlankSlate
   def initialize(*a)
     @mock_redis = MockRedis.new(*a)
     @real_redis = Redis.new(*a)
+    _gsub_clear
+  end
+
+  def _gsub(from, to)
+    @gsub_from = from
+    @gsub_to = to
+  end
+
+  def _gsub_clear
+    @gsub_from = @gsub_to = ''
   end
 
   def method_missing(method, *args, &blk)
@@ -71,8 +81,10 @@ class RedisMultiplexer < BlankSlate
   def equalish?(a, b)
     if a == b
       true
+    elsif a.is_a?(String) && b.is_a?(String)
+      masked(a) == masked(b)
     elsif a.is_a?(Array) && b.is_a?(Array)
-      a.collect(&:to_s).sort == b.collect(&:to_s).sort
+      a.collect { |c| masked(c.to_s) }.sort == b.collect { |c| masked(c.to_s) }.sort
     elsif a.is_a?(Exception) && b.is_a?(Exception)
       a.class == b.class && a.message == b.message
     elsif a.is_a?(Float) && b.is_a?(Float)
@@ -83,6 +95,10 @@ class RedisMultiplexer < BlankSlate
     end
   end
 
+  def masked(str)
+    str.gsub(@gsub_from, @gsub_to)
+  end
+
   def mock
     @mock_redis
   end

data/spec/support/shared_examples/does_not_cleanup_empty_strings.rb

@@ -0,0 +1,14 @@
+shared_examples_for 'does not remove empty strings on error' do
+  it 'does not remove empty strings on error' do |example|
+    key = 'mock-redis-test:not-a-string'
+
+    method = method_from_description(example)
+    args = args_for_method(method).unshift(key)
+
+    @redises.set(key, '')
+    lambda do
+      @redises.send(method, *args)
+    end.should raise_error(RuntimeError)
+    @redises.get(key).should == ''
+  end
+end

data/spec/support/shared_examples/only_operates_on_hashes.rb

@@ -10,4 +10,6 @@ shared_examples_for 'a hash-only command' do
       @redises.send(method, *args)
     end.should raise_error(RuntimeError)
   end
+
+  it_should_behave_like 'does not remove empty strings on error'
 end

data/spec/support/shared_examples/only_operates_on_lists.rb

@@ -10,4 +10,6 @@ shared_examples_for 'a list-only command' do
       @redises.send(method, *args)
     end.should raise_error(RuntimeError)
   end
+
+  it_should_behave_like 'does not remove empty strings on error'
 end

data/spec/support/shared_examples/only_operates_on_sets.rb

@@ -10,4 +10,6 @@ shared_examples_for 'a set-only command' do
       @redises.send(method, *args)
     end.should raise_error(RuntimeError)
   end
+
+  it_should_behave_like 'does not remove empty strings on error'
 end

data/spec/support/shared_examples/only_operates_on_zsets.rb

@@ -10,6 +10,8 @@ shared_examples_for 'a zset-only command' do
       @redises.send(method, *args)
     end.should raise_error(RuntimeError)
   end
+
+  it_should_behave_like 'does not remove empty strings on error'
 end
 
 shared_examples_for 'arg 1 is a score' do

data/spec/transactions_spec.rb

@@ -10,11 +10,11 @@ describe 'transactions (multi/exec/discard)' do
       @redises.multi.should == 'OK'
     end
 
-    it 'forbids nesting' do
+    it 'does not permit nesting' do
       @redises.multi
       lambda do
         @redises.multi
-      end.should raise_error(Redis::CommandError)
+      end.should raise_error(Redis::CommandError, 'ERR MULTI calls can not be nested')
     end
 
     it 'cleans state of transaction wrapper if exception occurs during transaction' do
@@ -41,18 +41,18 @@ describe 'transactions (multi/exec/discard)' do
         r.set('test', 1)
         r.incr('counter')
       end
-      @redises.get('counter').should == '6'
-      @redises.get('test').should == '1'
+      @redises.get('counter').should eq '6'
+      @redises.get('test').should eq '1'
     end
 
-    it 'forbids nesting via blocks' do
+    it 'permits nesting via blocks' do
       # Have to use only the mock here. redis-rb has a bug in it where
       # nested #multi calls raise NoMethodError because it gets a nil
       # where it's not expecting one.
       @redises.mock.multi do |r|
         lambda do
           r.multi {}
-        end.should raise_error(Redis::CommandError)
+        end.should_not raise_error
       end
     end
 
@@ -64,27 +64,15 @@ describe 'transactions (multi/exec/discard)' do
           pr.incr('counter')
         end
       end
-      @redises.get('counter').should == '6'
-      @redises.get('test').should == '1'
-    end
-
-    it 'allows multi blocks within pipelined blocks' do
-      @redises.set('counter', 5)
-      @redises.pipelined do |pr|
-        pr.multi do |r|
-          r.set('test', 1)
-          r.incr('counter')
-        end
-      end
-      @redises.get('counter').should == '6'
-      @redises.get('test').should == '1'
+      @redises.get('counter').should eq '6'
+      @redises.get('test').should eq '1'
     end
   end
 
   context '#discard' do
     it "responds with 'OK' after #multi" do
       @redises.multi
-      @redises.discard.should == 'OK'
+      @redises.discard.should eq 'OK'
     end
 
     it "can't be run outside of #multi/#exec" do
@@ -110,8 +98,8 @@ describe 'transactions (multi/exec/discard)' do
     end
 
     it "makes commands respond with 'QUEUED'" do
-      @redises.set(@string, 'string').should == 'QUEUED'
-      @redises.lpush(@list, 'list').should == 'QUEUED'
+      @redises.set(@string, 'string').should eq 'QUEUED'
+      @redises.lpush(@list, 'list').should eq 'QUEUED'
     end
 
     it "gives you the commands' responses when you call #exec" do
@@ -119,7 +107,7 @@ describe 'transactions (multi/exec/discard)' do
       @redises.lpush(@list, 'list')
       @redises.lpush(@list, 'list')
 
-      @redises.exec.should == ['OK', 1, 2]
+      @redises.exec.should eq ['OK', 1, 2]
     end
 
     it "does not raise exceptions, but rather puts them in #exec's response" do
@@ -128,9 +116,9 @@ describe 'transactions (multi/exec/discard)' do
       @redises.lpush(@list, 'list')
 
       responses = @redises.exec
-      responses[0].should == 'OK'
+      responses[0].should eq 'OK'
       responses[1].should be_a(Redis::CommandError)
-      responses[2].should == 1
+      responses[2].should eq 1
     end
   end
 
@@ -151,9 +139,9 @@ describe 'transactions (multi/exec/discard)' do
         second_lpush_response = mult.lpush(@list, 'list')
       end
 
-      set_response.value.should == 'OK'
-      lpush_response.value.should == 1
-      second_lpush_response.value.should == 2
+      set_response.value.should eq 'OK'
+      lpush_response.value.should eq 1
+      second_lpush_response.value.should eq 2
     end
   end
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: mock_redis
 version: !ruby/object:Gem::Version
-  version: 0.19.0
+  version: 0.20.0
 platform: ruby
 authors:
-- Brigade Engineering
+- Shane da Silva
 - Samuel Merritt
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-08-03 00:00:00.000000000 Z
+date: 2019-05-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -37,14 +37,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.3.0
+        version: 4.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.3.0
+        version: 4.1.0
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -73,10 +73,24 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: timecop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.9.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.9.1
 description: Instantiate one with `redis = MockRedis.new` and treat it like you would
   a normal Redis object. It supports all the usual Redis operations.
 email:
-- eng@brigade.com
+- shane@dasilva.io
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -109,6 +123,9 @@ files:
 - lib/mock_redis/pipelined_wrapper.rb
 - lib/mock_redis/set_methods.rb
 - lib/mock_redis/sort_method.rb
+- lib/mock_redis/stream.rb
+- lib/mock_redis/stream/id.rb
+- lib/mock_redis/stream_methods.rb
 - lib/mock_redis/string_methods.rb
 - lib/mock_redis/transaction_wrapper.rb
 - lib/mock_redis/undef_redis_methods.rb
@@ -151,7 +168,6 @@ files:
 - spec/commands/getbit_spec.rb
 - spec/commands/getrange_spec.rb
 - spec/commands/getset_spec.rb
-- spec/commands/hash_operator_spec.rb
 - spec/commands/hdel_spec.rb
 - spec/commands/hexists_spec.rb
 - spec/commands/hget_spec.rb
@@ -240,6 +256,11 @@ files:
 - spec/commands/type_spec.rb
 - spec/commands/unwatch_spec.rb
 - spec/commands/watch_spec.rb
+- spec/commands/xadd_spec.rb
+- spec/commands/xlen_spec.rb
+- spec/commands/xrange_spec.rb
+- spec/commands/xrevrange_spec.rb
+- spec/commands/xtrim_spec.rb
 - spec/commands/zadd_spec.rb
 - spec/commands/zcard_spec.rb
 - spec/commands/zcount_spec.rb
@@ -261,6 +282,7 @@ files:
 - spec/mock_redis_spec.rb
 - spec/spec_helper.rb
 - spec/support/redis_multiplexer.rb
+- spec/support/shared_examples/does_not_cleanup_empty_strings.rb
 - spec/support/shared_examples/only_operates_on_hashes.rb
 - spec/support/shared_examples/only_operates_on_lists.rb
 - spec/support/shared_examples/only_operates_on_sets.rb
@@ -268,7 +290,7 @@ files:
 - spec/support/shared_examples/only_operates_on_zsets.rb
 - spec/support/shared_examples/sorts_enumerables.rb
 - spec/transactions_spec.rb
-homepage: https://github.com/brigade/mock_redis
+homepage: https://github.com/sds/mock_redis
 licenses:
 - MIT
 metadata: {}
@@ -280,15 +302,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.2'
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.3
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Redis mock that just lives in memory; useful for testing.
@@ -327,7 +348,6 @@ test_files:
 - spec/commands/getbit_spec.rb
 - spec/commands/getrange_spec.rb
 - spec/commands/getset_spec.rb
-- spec/commands/hash_operator_spec.rb
 - spec/commands/hdel_spec.rb
 - spec/commands/hexists_spec.rb
 - spec/commands/hget_spec.rb
@@ -416,6 +436,11 @@ test_files:
 - spec/commands/type_spec.rb
 - spec/commands/unwatch_spec.rb
 - spec/commands/watch_spec.rb
+- spec/commands/xadd_spec.rb
+- spec/commands/xlen_spec.rb
+- spec/commands/xrange_spec.rb
+- spec/commands/xrevrange_spec.rb
+- spec/commands/xtrim_spec.rb
 - spec/commands/zadd_spec.rb
 - spec/commands/zcard_spec.rb
 - spec/commands/zcount_spec.rb
@@ -437,6 +462,7 @@ test_files:
 - spec/mock_redis_spec.rb
 - spec/spec_helper.rb
 - spec/support/redis_multiplexer.rb
+- spec/support/shared_examples/does_not_cleanup_empty_strings.rb
 - spec/support/shared_examples/only_operates_on_hashes.rb
 - spec/support/shared_examples/only_operates_on_lists.rb
 - spec/support/shared_examples/only_operates_on_sets.rb

data/spec/commands/hash_operator_spec.rb

@@ -1,21 +0,0 @@
-require 'spec_helper'
-
-describe '#[](key)' do
-  before do
-    @key = 'mock-redis-test:hash_operator'
-  end
-
-  it 'returns nil for a nonexistent value' do
-    @redises['mock-redis-test:does-not-exist'].should be_nil
-  end
-
-  it 'returns a stored string value' do
-    @redises[@key] = 'forsooth'
-    @redises[@key].should == 'forsooth'
-  end
-
-  it 'treats integers as strings' do
-    @redises[@key] = 100
-    @redises[@key].should == '100'
-  end
-end