mock_redis 0.17.1 → 0.17.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a55e7d747d39e4c40aedd4b13974ea1ba8fb78a3
-  data.tar.gz: b5efc0805cac7734fa78403f18b4ee4253866197
+  metadata.gz: a6964254237dcb85924a41b08802b43875d9285d
+  data.tar.gz: 96914898547b94bea6e7fcedd59275473f3bea2e
 SHA512:
-  metadata.gz: a0c1277f32ded504822a15d589c0c262d872593afe6a98e100cdc05e06ff5e04a39e5b8899089656de0f061dc197d632ce954550bae4fb61095df359f27bccef
-  data.tar.gz: 33183c7e84dc6d0ebc9990b44bb0ada35bbaca1a90a0ee801f94e8d240419cdcf108b9c1bf861a27f4efbc1e03a9f83052ddd3fe3c6da8a66849f71212d9e677
+  metadata.gz: 6c955b02cf5fd7c4b3b251a29fc6879d7156ea16875367041a24833f3812c11448ba011a9210e221fb89299ec50efa969913c0c9c774522fd31d3e21443f7380
+  data.tar.gz: 600fe03cfd32d108ae506e651026b5372d5a987511a92a8190b7b438d7360fd982645effbcac88668f11815fa5c072e111726b3d0529a340e0654cae37147259

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # MockRedis Changelog
 
+### master (unreleased)
+
+### 0.17.2
+
+* Allow negative out-of-bounds start and stop in `zrange`
+
 ### 0.17.1
 
 * Flatten args list passed to `hmset`

data/lib/mock_redis/version.rb

@@ -2,5 +2,5 @@
 
 # Defines the gem version.
 class MockRedis
-  VERSION = '0.17.1'.freeze
+  VERSION = '0.17.2'.freeze
 end

data/lib/mock_redis/zset_methods.rb

@@ -118,7 +118,9 @@ class MockRedis
 
     def zrange(key, start, stop, options = {})
       with_zset_at(key) do |z|
-        to_response(z.sorted[start.to_i..stop.to_i] || [], options)
+        start = [start.to_i, -z.sorted.size].max
+        stop = [stop.to_i, -z.sorted.size].max
+        to_response(z.sorted[start..stop] || [], options)
       end
     end
 

data/spec/commands/zrange_spec.rb

@@ -46,6 +46,22 @@ describe '#zrange(key, start, stop [, :with_scores => true])' do
     @redises.zrange(@key, 5, -1).should == []
   end
 
+  it 'returns entire list when start is out of bounds with negative end in bounds' do
+    @redises.zrange(@key, -5, -1).should == %w[Washington Adams Jefferson Madison]
+  end
+
+  it 'returns correct subset when start is out of bounds with positive end in bounds' do
+    @redises.zrange(@key, -5, 1).should == %w[Washington Adams]
+  end
+
+  it 'returns empty list when start is in bounds with negative end out of bounds' do
+    @redises.zrange(@key, 1, -5).should == []
+  end
+
+  it 'returns correct subset when start is in bounds with negative end in bounds' do
+    @redises.zrange(@key, 1, -1).should == %w[Adams Jefferson Madison]
+  end
+
   it 'returns the scores when :with_scores is specified' do
     @redises.zrange(@key, 0, 1, :with_scores => true).
       should == [['Washington', 1.0], ['Adams', 2.0]]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mock_redis
 version: !ruby/object:Gem::Version
-  version: 0.17.1
+  version: 0.17.2
 platform: ruby
 authors:
 - Brigade Engineering
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-11-16 00:00:00.000000000 Z
+date: 2017-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -281,7 +281,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: Redis mock that just lives in memory; useful for testing.
@@ -432,3 +432,4 @@ test_files:
 - spec/support/shared_examples/only_operates_on_zsets.rb
 - spec/support/shared_examples/sorts_enumerables.rb
 - spec/transactions_spec.rb
+has_rdoc: