rediska 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2e4ac28eb56bf3a2cab47dc9b091b7182acc88b8
-  data.tar.gz: 5d6d48bc843232644fbfbf92063a7785a6db947e
+  metadata.gz: dd18585f05fa32a32467fc94d32a86721f605229
+  data.tar.gz: 33abc1203b205f9a67020ce205223c642c054d4a
 SHA512:
-  metadata.gz: 7b149f856ed62eb989464acdf464424d2c870a752b5e00014e0e83b7c3cb48c66a375d819a792641b7989f19868dc60d5f6b86d7f89a4f5ce335d5e0d8255b99
-  data.tar.gz: 5c5cfffc7e706fe27dee1ea447d36a9c58cff8f63e906060c516f1c9e4b4d190f618cc79a2b4746ae464091231e2a3aa521ae4192ab7eddd3307a8febb994971
+  metadata.gz: 9c6bcc3c79b86e2a5df2686e4b3a57e6c9093384f24bc6282aa18d8fffe842523f1b29609281e80b05ff2b37240d49e147b9e06dbddfd18b26e9e7063a56c585
+  data.tar.gz: fd8b78d827d350b5c63c0b6c49a5087fed940bd467e629870bf601ab8a49e428f44ce19f5d9902ce3514c512824454e3e264ca7dba24b73cb61265af8c549a34

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rediska (0.1.0)
+    rediska (0.1.1)
       redis (~> 3.0.0)
 
 GEM

data/lib/rediska/driver.rb

@@ -197,14 +197,14 @@ module Rediska
       data_type_check(key, Array)
       return unless data[key]
 
-      if start < 0 && data[key].count < start.abs
-        # Example: we have a list of 3 elements and we give it a ltrim list, -5, -1. This means it
-        # should trim to a max of 5. Since 3 < 5 we should not touch the list. This is consistent
-        # with behavior of real Redis's ltrim with a negative start argument.
-        data[key]
-      else
+      # Example: we have a list of 3 elements and we give it a ltrim list, -5, -1. This means it
+      # should trim to a max of 5. Since 3 < 5 we should not touch the list. This is consistent
+      # with behavior of real Redis's ltrim with a negative start argument.
+      unless start < 0 && data[key].count < start.abs
         data[key] = data[key][start..stop]
       end
+
+      'OK'
     end
 
     def lindex(key, index)
@@ -441,9 +441,9 @@ module Rediska
     end
 
     def expire(key, ttl)
-      return unless data[key]
+      return 0 unless data[key]
       data.expires[key] = Time.now + ttl
-      true
+      1
     end
 
     def ttl(key)

data/lib/rediska/version.rb

@@ -1,3 +1,3 @@
 module Rediska
-  VERSION = '0.1.0'.freeze
+  VERSION = '0.1.1'.freeze
 end

data/spec/support/keys.rb

@@ -26,6 +26,15 @@ shared_examples 'keys' do
     }.to raise_error(Redis::CommandError)
   end
 
+  it 'should return true when setting expires on keys that exist' do
+    subject.set('key1', '1')
+    expect(subject.expire('key1', 1)).to be_truthy
+  end
+
+  it 'should return false when attempting to set expires on a key that does not exist' do
+    expect(subject.expire('key1', 1)).to be_falsey
+  end
+
   it 'should determine if a key exists' do
     subject.set('key1', '1')
 

data/spec/support/lists.rb

@@ -117,7 +117,7 @@ shared_examples 'lists' do
     subject.rpush('key1', 'two')
     subject.rpush('key1', 'three')
 
-    subject.ltrim('key1', 1, -1)
+    expect(subject.ltrim('key1', 1, -1)).to eq('OK')
     expect(subject.lrange('key1', 0, -1)).to eq(['two', 'three'])
   end
 

data/spec/support/transactions.rb

@@ -13,10 +13,11 @@ shared_examples 'transactions' do
       transaction = subject.multi do |multi|
         multi.set('key1', '1')
         multi.set('key2', '2')
+        multi.expire('key1', 123)
         multi.mget('key1', 'key2')
       end
 
-      expect(transaction).to eq(['OK', 'OK', ['1', '2']])
+      expect(transaction).to eq(['OK', 'OK', true, ['1', '2']])
     end
   end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rediska
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - Leonid Beder
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-06-04 00:00:00.000000000 Z
+date: 2014-06-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis