TODO changes and mostly theoretical minor skiplist change

[redis.git] / client-libraries / ruby / spec / redis_spec.rb

diff --git a/client-libraries/ruby/spec/redis_spec.rb b/client-libraries/ruby/spec/redis_spec.rb
index d4dde96417b8e486c4047dcb649d32194bcabec7..0da249e115ae567141115ce7bd00e34c59f52ec9 100644 (file)
--- a/client-libraries/ruby/spec/redis_spec.rb
+++ b/client-libraries/ruby/spec/redis_spec.rb
@@ -1,15 +1,16 @@
 require File.dirname(__FILE__) + '/spec_helper'
 require File.dirname(__FILE__) + '/spec_helper'

+require 'logger'

 
 class Foo
   attr_accessor :bar
   def initialize(bar)
     @bar = bar
   end
 
 class Foo
   attr_accessor :bar
   def initialize(bar)
     @bar = bar
   end

-  
+

   def ==(other)
     @bar == other.bar
   end
   def ==(other)
     @bar == other.bar
   end

-end  
+end

 
 describe "redis" do
   before(:all) do
 
 describe "redis" do
   before(:all) do


@@ -23,30 +24,41 @@ describe "redis" do
 
   after(:each) do
     @r.keys('*').each {|k| @r.del k}
 
   after(:each) do
     @r.keys('*').each {|k| @r.del k}

-  end  
+  end

 
   after(:all) do
     @r.quit
 
   after(:all) do
     @r.quit

-  end  
+  end
+
+  it "should be able connect without a timeout" do
+    lambda { Redis.new :timeout => 0 }.should_not raise_error
+  end
+
+  it "should be able to provide a logger" do
+    log = StringIO.new
+    r = Redis.new :db => 15, :logger => Logger.new(log)
+    r.ping
+    log.string.should include("ping")
+  end

 
 

-  it 'should be able to PING' do
-    @r.ping.should == 'PONG' 
+  it "should be able to PING" do
+    @r.ping.should == 'PONG'

   end
 
   it "should be able to GET a key" do
     @r['foo'].should == 'bar'
   end
   end
 
   it "should be able to GET a key" do
     @r['foo'].should == 'bar'
   end

-  
+

   it "should be able to SET a key" do
     @r['foo'] = 'nik'
     @r['foo'].should == 'nik'
   end
   it "should be able to SET a key" do
     @r['foo'] = 'nik'
     @r['foo'].should == 'nik'
   end

-  
+

   it "should properly handle trailing newline characters" do
     @r['foo'] = "bar\n"
     @r['foo'].should == "bar\n"
   end
   it "should properly handle trailing newline characters" do
     @r['foo'] = "bar\n"
     @r['foo'].should == "bar\n"
   end

-  
+

   it "should store and retrieve all possible characters at the beginning and the end of a string" do
     (0..255).each do |char_idx|
       string = "#{char_idx.chr}---#{char_idx.chr}"
   it "should store and retrieve all possible characters at the beginning and the end of a string" do
     (0..255).each do |char_idx|
       string = "#{char_idx.chr}---#{char_idx.chr}"


@@ -54,28 +66,45 @@ describe "redis" do
       @r['foo'].should == string
     end
   end
       @r['foo'].should == string
     end
   end

-  
+

   it "should be able to SET a key with an expiry" do
     @r.set('foo', 'bar', 1)
     @r['foo'].should == 'bar'
     sleep 2
     @r['foo'].should == nil
   end
   it "should be able to SET a key with an expiry" do
     @r.set('foo', 'bar', 1)
     @r['foo'].should == 'bar'
     sleep 2
     @r['foo'].should == nil
   end

-  
+
+  it "should be able to return a TTL for a key" do
+    @r.set('foo', 'bar', 1)
+    @r.ttl('foo').should == 1
+  end
+

   it "should be able to SETNX" do
     @r['foo'] = 'nik'
     @r['foo'].should == 'nik'
     @r.setnx 'foo', 'bar'
     @r['foo'].should == 'nik'
   end
   it "should be able to SETNX" do
     @r['foo'] = 'nik'
     @r['foo'].should == 'nik'
     @r.setnx 'foo', 'bar'
     @r['foo'].should == 'nik'
   end

-  # 
+  #
+  it "should be able to GETSET" do
+   @r.getset('foo', 'baz').should == 'bar'
+   @r['foo'].should == 'baz'
+  end
+  #

   it "should be able to INCR a key" do
     @r.del('counter')
     @r.incr('counter').should == 1
     @r.incr('counter').should == 2
     @r.incr('counter').should == 3
   end
   it "should be able to INCR a key" do
     @r.del('counter')
     @r.incr('counter').should == 1
     @r.incr('counter').should == 2
     @r.incr('counter').should == 3
   end

-  # 
+  #
+  it "should be able to INCRBY a key" do
+    @r.del('counter')
+    @r.incrby('counter', 1).should == 1
+    @r.incrby('counter', 2).should == 3
+    @r.incrby('counter', 3).should == 6
+  end
+  #

   it "should be able to DECR a key" do
     @r.del('counter')
     @r.incr('counter').should == 1
   it "should be able to DECR a key" do
     @r.del('counter')
     @r.incr('counter').should == 1


@@ -84,11 +113,11 @@ describe "redis" do
     @r.decr('counter').should == 2
     @r.decr('counter', 2).should == 0
   end
     @r.decr('counter').should == 2
     @r.decr('counter', 2).should == 0
   end

-  # 
+  #

   it "should be able to RANDKEY" do
     @r.randkey.should_not be_nil
   end
   it "should be able to RANDKEY" do
     @r.randkey.should_not be_nil
   end

-  # 
+  #

   it "should be able to RENAME a key" do
     @r.del 'foo'
     @r.del'bar'
   it "should be able to RENAME a key" do
     @r.del 'foo'
     @r.del'bar'


@@ -96,7 +125,7 @@ describe "redis" do
     @r.rename 'foo', 'bar'
     @r['bar'].should == 'hi'
   end
     @r.rename 'foo', 'bar'
     @r['bar'].should == 'hi'
   end

-  # 
+  #

   it "should be able to RENAMENX a key" do
     @r.del 'foo'
     @r.del 'bar'
   it "should be able to RENAMENX a key" do
     @r.del 'foo'
     @r.del 'bar'


@@ -129,7 +158,7 @@ describe "redis" do
     @r.del 'foo'
     @r.exists('foo').should be_false
   end
     @r.del 'foo'
     @r.exists('foo').should be_false
   end

-  # 
+  #

   it "should be able to KEYS" do
     @r.keys("f*").each { |key| @r.del key }
     @r['f'] = 'nik'
   it "should be able to KEYS" do
     @r.keys("f*").each { |key| @r.del key }
     @r['f'] = 'nik'


@@ -137,14 +166,18 @@ describe "redis" do
     @r['foo'] = 'qux'
     @r.keys("f*").sort.should == ['f','fo', 'foo'].sort
   end
     @r['foo'] = 'qux'
     @r.keys("f*").sort.should == ['f','fo', 'foo'].sort
   end

-  #BTM - TODO 
+  #
+  it "should be able to return a random key (RANDOMKEY)" do
+    3.times { @r.exists(@r.randomkey).should be_true }
+  end
+  #

   it "should be able to check the TYPE of a key" do
     @r['foo'] = 'nik'
     @r.type('foo').should == "string"
     @r.del 'foo'
     @r.type('foo').should == "none"
   end
   it "should be able to check the TYPE of a key" do
     @r['foo'] = 'nik'
     @r.type('foo').should == "string"
     @r.del 'foo'
     @r.type('foo').should == "none"
   end

-  # 
+  #

   it "should be able to push to the head of a list (LPUSH)" do
     @r.lpush "list", 'hello'
     @r.lpush "list", 42
   it "should be able to push to the head of a list (LPUSH)" do
     @r.lpush "list", 'hello'
     @r.lpush "list", 42


@@ -152,13 +185,13 @@ describe "redis" do
     @r.llen('list').should == 2
     @r.lpop('list').should == '42'
   end
     @r.llen('list').should == 2
     @r.lpop('list').should == '42'
   end

-  # 
+  #

   it "should be able to push to the tail of a list (RPUSH)" do
     @r.rpush "list", 'hello'
     @r.type('list').should == "list"
     @r.llen('list').should == 1
   end
   it "should be able to push to the tail of a list (RPUSH)" do
     @r.rpush "list", 'hello'
     @r.type('list').should == "list"
     @r.llen('list').should == 1
   end

-  # 
+  #

   it "should be able to pop the tail of a list (RPOP)" do
     @r.rpush "list", 'hello'
     @r.rpush"list", 'goodbye'
   it "should be able to pop the tail of a list (RPOP)" do
     @r.rpush "list", 'hello'
     @r.rpush"list", 'goodbye'


@@ -166,7 +199,7 @@ describe "redis" do
     @r.llen('list').should == 2
     @r.rpop('list').should == 'goodbye'
   end
     @r.llen('list').should == 2
     @r.rpop('list').should == 'goodbye'
   end

-  # 
+  #

   it "should be able to pop the head of a list (LPOP)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'
   it "should be able to pop the head of a list (LPOP)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'


@@ -174,14 +207,14 @@ describe "redis" do
     @r.llen('list').should == 2
     @r.lpop('list').should == 'hello'
   end
     @r.llen('list').should == 2
     @r.lpop('list').should == 'hello'
   end

-  # 
+  #

   it "should be able to get the length of a list (LLEN)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'
     @r.type('list').should == "list"
     @r.llen('list').should == 2
   end
   it "should be able to get the length of a list (LLEN)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'
     @r.type('list').should == "list"
     @r.llen('list').should == 2
   end

-  # 
+  #

   it "should be able to get a range of values from a list (LRANGE)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'
   it "should be able to get a range of values from a list (LRANGE)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'


@@ -192,7 +225,7 @@ describe "redis" do
     @r.llen('list').should == 5
     @r.lrange('list', 2, -1).should == ['1', '2', '3']
   end
     @r.llen('list').should == 5
     @r.lrange('list', 2, -1).should == ['1', '2', '3']
   end

-  # 
+  #

   it "should be able to trim a list (LTRIM)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'
   it "should be able to trim a list (LTRIM)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'


@@ -205,7 +238,7 @@ describe "redis" do
     @r.llen('list').should == 2
     @r.lrange('list', 0, -1).should == ['hello', 'goodbye']
   end
     @r.llen('list').should == 2
     @r.lrange('list', 0, -1).should == ['hello', 'goodbye']
   end

-  # 
+  #

   it "should be able to get a value by indexing into a list (LINDEX)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'
   it "should be able to get a value by indexing into a list (LINDEX)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'


@@ -213,7 +246,7 @@ describe "redis" do
     @r.llen('list').should == 2
     @r.lindex('list', 1).should == 'goodbye'
   end
     @r.llen('list').should == 2
     @r.lindex('list', 1).should == 'goodbye'
   end

-  # 
+  #

   it "should be able to set a value by indexing into a list (LSET)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'hello'
   it "should be able to set a value by indexing into a list (LSET)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'hello'


@@ -222,7 +255,7 @@ describe "redis" do
     @r.lset('list', 1, 'goodbye').should == 'OK'
     @r.lindex('list', 1).should == 'goodbye'
   end
     @r.lset('list', 1, 'goodbye').should == 'OK'
     @r.lindex('list', 1).should == 'goodbye'
   end

-  # 
+  #

   it "should be able to remove values from a list (LREM)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'
   it "should be able to remove values from a list (LREM)" do
     @r.rpush "list", 'hello'
     @r.rpush "list", 'goodbye'


@@ -231,7 +264,7 @@ describe "redis" do
     @r.lrem('list', 1, 'hello').should == 1
     @r.lrange('list', 0, -1).should == ['goodbye']
   end
     @r.lrem('list', 1, 'hello').should == 1
     @r.lrange('list', 0, -1).should == ['goodbye']
   end

-  # 
+  #

   it "should be able add members to a set (SADD)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'
   it "should be able add members to a set (SADD)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'


@@ -239,7 +272,7 @@ describe "redis" do
     @r.scard('set').should == 2
     @r.smembers('set').sort.should == ['key1', 'key2'].sort
   end
     @r.scard('set').should == 2
     @r.smembers('set').sort.should == ['key1', 'key2'].sort
   end

-  # 
+  #

   it "should be able delete members to a set (SREM)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'
   it "should be able delete members to a set (SREM)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'


@@ -250,14 +283,14 @@ describe "redis" do
     @r.scard('set').should == 1
     @r.smembers('set').should == ['key2']
   end
     @r.scard('set').should == 1
     @r.smembers('set').should == ['key2']
   end

-  # 
+  #

   it "should be able count the members of a set (SCARD)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'
     @r.type('set').should == "set"
     @r.scard('set').should == 2
   end
   it "should be able count the members of a set (SCARD)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'
     @r.type('set').should == "set"
     @r.scard('set').should == 2
   end

-  # 
+  #

   it "should be able test for set membership (SISMEMBER)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'
   it "should be able test for set membership (SISMEMBER)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'


@@ -267,14 +300,14 @@ describe "redis" do
     @r.sismember('set', 'key2').should be_true
     @r.sismember('set', 'notthere').should be_false
   end
     @r.sismember('set', 'key2').should be_true
     @r.sismember('set', 'notthere').should be_false
   end

-  # 
+  #

   it "should be able to do set intersection (SINTER)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'
     @r.sadd "set2", 'key2'
     @r.sinter('set', 'set2').should == ['key2']
   end
   it "should be able to do set intersection (SINTER)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'
     @r.sadd "set2", 'key2'
     @r.sinter('set', 'set2').should == ['key2']
   end

-  # 
+  #

   it "should be able to do set intersection and store the results in a key (SINTERSTORE)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'
   it "should be able to do set intersection and store the results in a key (SINTERSTORE)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'


@@ -290,7 +323,7 @@ describe "redis" do
     @r.sadd "set2", 'key3'
     @r.sunion('set', 'set2').sort.should == ['key1','key2','key3'].sort
   end
     @r.sadd "set2", 'key3'
     @r.sunion('set', 'set2').sort.should == ['key1','key2','key3'].sort
   end

-  # 
+  #

   it "should be able to do set union and store the results in a key (SUNIONSTORE)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'
   it "should be able to do set union and store the results in a key (SUNIONSTORE)" do
     @r.sadd "set", 'key1'
     @r.sadd "set", 'key2'


@@ -299,7 +332,7 @@ describe "redis" do
     @r.sunionstore('newone', 'set', 'set2').should == 3
     @r.smembers('newone').sort.should == ['key1','key2','key3'].sort
   end
     @r.sunionstore('newone', 'set', 'set2').should == 3
     @r.smembers('newone').sort.should == ['key1','key2','key3'].sort
   end

-  # 
+  #

   it "should be able to do set difference (SDIFF)" do
      @r.sadd "set", 'a'
      @r.sadd "set", 'b'
   it "should be able to do set difference (SDIFF)" do
      @r.sadd "set", 'a'
      @r.sadd "set", 'b'


@@ -307,7 +340,7 @@ describe "redis" do
      @r.sadd "set2", 'c'
      @r.sdiff('set', 'set2').should == ['a']
    end
      @r.sadd "set2", 'c'
      @r.sdiff('set', 'set2').should == ['a']
    end

-  # 
+  #

   it "should be able to do set difference and store the results in a key (SDIFFSTORE)" do
      @r.sadd "set", 'a'
      @r.sadd "set", 'b'
   it "should be able to do set difference and store the results in a key (SDIFFSTORE)" do
      @r.sadd "set", 'a'
      @r.sadd "set", 'b'


@@ -316,27 +349,28 @@ describe "redis" do
      @r.sdiffstore('newone', 'set', 'set2')
      @r.smembers('newone').should == ['a']
    end
      @r.sdiffstore('newone', 'set', 'set2')
      @r.smembers('newone').should == ['a']
    end

-  # 
+  #

   it "should be able move elements from one set to another (SMOVE)" do
     @r.sadd 'set1', 'a'
     @r.sadd 'set1', 'b'
     @r.sadd 'set2', 'x'
   it "should be able move elements from one set to another (SMOVE)" do
     @r.sadd 'set1', 'a'
     @r.sadd 'set1', 'b'
     @r.sadd 'set2', 'x'

-    @r.smove('set1', 'set2', 'a').should be_true 
+    @r.smove('set1', 'set2', 'a').should be_true

     @r.sismember('set2', 'a').should be_true
     @r.delete('set1')
   end
   #
   it "should be able to do crazy SORT queries" do
     @r.sismember('set2', 'a').should be_true
     @r.delete('set1')
   end
   #
   it "should be able to do crazy SORT queries" do

+    # The 'Dogs' is capitialized on purpose

     @r['dog_1'] = 'louie'
     @r['dog_1'] = 'louie'

-    @r.rpush 'dogs', 1
+    @r.rpush 'Dogs', 1

     @r['dog_2'] = 'lucy'
     @r['dog_2'] = 'lucy'

-    @r.rpush 'dogs', 2
+    @r.rpush 'Dogs', 2

     @r['dog_3'] = 'max'
     @r['dog_3'] = 'max'

-    @r.rpush 'dogs', 3
+    @r.rpush 'Dogs', 3

     @r['dog_4'] = 'taj'
     @r['dog_4'] = 'taj'

-    @r.rpush 'dogs', 4
-    @r.sort('dogs', :get => 'dog_*', :limit => [0,1]).should == ['louie']
-    @r.sort('dogs', :get => 'dog_*', :limit => [0,1], :order => 'desc alpha').should == ['taj']
+    @r.rpush 'Dogs', 4
+    @r.sort('Dogs', :get => 'dog_*', :limit => [0,1]).should == ['louie']
+    @r.sort('Dogs', :get => 'dog_*', :limit => [0,1], :order => 'desc alpha').should == ['taj']

   end
 
   it "should be able to handle array of :get using SORT" do
   end
 
   it "should be able to handle array of :get using SORT" do


@@ -355,38 +389,61 @@ describe "redis" do
     @r.sort('dogs', :get => ['dog:*:name', 'dog:*:breed'], :limit => [0,1]).should == ['louie', 'mutt']
     @r.sort('dogs', :get => ['dog:*:name', 'dog:*:breed'], :limit => [0,1], :order => 'desc alpha').should == ['taj', 'terrier']
   end
     @r.sort('dogs', :get => ['dog:*:name', 'dog:*:breed'], :limit => [0,1]).should == ['louie', 'mutt']
     @r.sort('dogs', :get => ['dog:*:name', 'dog:*:breed'], :limit => [0,1], :order => 'desc alpha').should == ['taj', 'terrier']
   end

-  # 
-  it "should provide info" do
+  #
+  it "should provide info (INFO)" do

     [:last_save_time, :redis_version, :total_connections_received, :connected_clients, :total_commands_processed, :connected_slaves, :uptime_in_seconds, :used_memory, :uptime_in_days, :changes_since_last_save].each do |x|
     @r.info.keys.should include(x)
     end
   end
     [:last_save_time, :redis_version, :total_connections_received, :connected_clients, :total_commands_processed, :connected_slaves, :uptime_in_seconds, :used_memory, :uptime_in_days, :changes_since_last_save].each do |x|
     @r.info.keys.should include(x)
     end
   end

-  # 
-  it "should be able to flush the database" do
+  #
+  it "should be able to flush the database (FLUSHDB)" do

     @r['key1'] = 'keyone'
     @r['key2'] = 'keytwo'
     @r.keys('*').sort.should == ['foo', 'key1', 'key2'].sort #foo from before
     @r.flushdb
     @r.keys('*').should == []
   end
     @r['key1'] = 'keyone'
     @r['key2'] = 'keytwo'
     @r.keys('*').sort.should == ['foo', 'key1', 'key2'].sort #foo from before
     @r.flushdb
     @r.keys('*').should == []
   end

-  # 
+  #
+  it "should raise exception when manually try to change the database" do
+    lambda { @r.select(0) }.should raise_error
+  end
+  #

   it "should be able to provide the last save time (LASTSAVE)" do
     savetime = @r.lastsave
     Time.at(savetime).class.should == Time
     Time.at(savetime).should <= Time.now
   end
   it "should be able to provide the last save time (LASTSAVE)" do
     savetime = @r.lastsave
     Time.at(savetime).class.should == Time
     Time.at(savetime).should <= Time.now
   end

-  
+

   it "should be able to MGET keys" do
     @r['foo'] = 1000
     @r['bar'] = 2000
     @r.mget('foo', 'bar').should == ['1000', '2000']
     @r.mget('foo', 'bar', 'baz').should == ['1000', '2000', nil]
   end
   it "should be able to MGET keys" do
     @r['foo'] = 1000
     @r['bar'] = 2000
     @r.mget('foo', 'bar').should == ['1000', '2000']
     @r.mget('foo', 'bar', 'baz').should == ['1000', '2000', nil]
   end

-  
+
+  it "should be able to mapped MGET keys" do
+    @r['foo'] = 1000
+    @r['bar'] = 2000
+    @r.mapped_mget('foo', 'bar').should == { 'foo' => '1000', 'bar' => '2000'}
+    @r.mapped_mget('foo', 'baz', 'bar').should == { 'foo' => '1000', 'bar' => '2000'}
+  end
+

   it "should bgsave" do
     @r.bgsave.should == 'OK'
   end
   it "should bgsave" do
     @r.bgsave.should == 'OK'
   end

-  
+
+  it "should be able to ECHO" do
+    @r.echo("message in a bottle\n").should == "message in a bottle\n"
+  end
+
+  it "should raise error when invoke MONITOR" do
+    lambda { @r.monitor }.should raise_error
+  end
+
+  it "should raise error when invoke SYNC" do
+    lambda { @r.sync }.should raise_error
+  end
+

   it "should handle multiple servers" do
     require 'dist_redis'
     @r = DistRedis.new(:hosts=> ['localhost:6379', '127.0.0.1:6379'], :db => 15)
   it "should handle multiple servers" do
     require 'dist_redis'
     @r = DistRedis.new(:hosts=> ['localhost:6379', '127.0.0.1:6379'], :db => 15)


@@ -405,10 +462,17 @@ describe "redis" do
       pipeline.lpush 'list', "hello"
       pipeline.lpush 'list', 42
     end
       pipeline.lpush 'list', "hello"
       pipeline.lpush 'list', 42
     end

-    
+

     @r.type('list').should == "list"
     @r.llen('list').should == 2
     @r.lpop('list').should == '42'
   end
     @r.type('list').should == "list"
     @r.llen('list').should == 2
     @r.lpop('list').should == '42'
   end

-  
+
+  it "should AUTH when connecting with a password" do
+    r = Redis.new(:password => 'secret')
+    r.stub!(:connect_to)
+    r.should_receive(:call_command).with(['auth', 'secret'])
+    r.connect_to_server
+  end
+

 end
 end