izanami 0.19.0 → 0.20.0

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    Y2IxY2E3OWY5ZWE5N2UzNTNkN2JmMDRlNjVhZmE0NThkZTQ5ZmU0Ng==
+    ZmQ0ODM0MjA1MzFiNWRhNDI3MDA2MjUwZTE0ZGM5ZmYyN2EyZjFiMg==
   data.tar.gz: !binary |-
-    ZGE1MjM5NDI1ZjE1MDE1OGM2NzcyYjIzMWVmYTUxN2NhMmVlNDBlMA==
+    OWI2OGIyY2IyYmY1ZmYzMDU2ODgwY2I0OTJlYWUzYjJiN2RlZDA3Nw==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    ZWZlNDgwYTZhOGFiMWQwNzlkMTYyOTIxMjk0MzQ0ZWU4MzcxZWQ0YWE1MDAw
-    OTFiMTNjMGI3MDg0ZTZhMTA2NjkyMDZkODY0ZTQ2NjE5YzE0NTA5YmQyMDVk
-    ZDI2MjU2ZWNlODM0MjQxNDg0YzI3N2QwODc1Y2U2NTIyYmNkZWQ=
+    YmQwNTFmMjVmZjFkMWRiYTI5ZTg5M2I1MDNkM2VjZmI0YTA5OWUwZjJkYjFh
+    YTNlZGU5ZTBmYjZhMjZkZmJjZjI2Y2VjNjdjZTk1MzdlYTZhNGEyYTUwZDRl
+    MjQxMzc2NjA0MmFlMjdlMTVlOTU0Y2I0N2ZmZTAzMzA0OGQxYjY=
   data.tar.gz: !binary |-
-    ZDBjNzlmOTU4Y2FhOTRhZGQ3M2MwYWVkMjM3YmFkOTA2NzcxMjUyODEzYzky
-    YWE4MGM4ZTVjZmU4NWQ2ODIxNDkwNmRlYjA1ZTlhNzdmZTgzMTJiMDQwY2M2
-    MDYwYmE5MTYxN2NjMDIyNjA3ZDYyMWM4ZjBlN2VhNWY2ODFmZjg=
+    YThiMmJjOWQ3N2YxYzk2NDQ4ZTYzYjNhNTEzYTc4ZGJhNWU3OGYwMTMzNWY1
+    NjQzYjY3YzVmZTU2ZjAwOTM0MWZmNGU0YWQ3YTllYjA1ODcxYmYzYWExOTYx
+    YzY4ZTkwZjFkZWZhODI2YjczNDNmM2ViYzI0MDFlZmY4ZDgzNDA=

data/lib/izanami/mappers/command.rb

@@ -50,7 +50,7 @@ module Izanami
       # @return [Array] all the commands found.
       def take(number)
         options = { order: 'DESC', limit: [0, number] }
-        client.sort('ids', options).map { |id| find(id) }
+        client.sort('ids', options).map { |id| find(id) }.compact
       end
 
       # Delete the specified commands
@@ -58,14 +58,37 @@ module Izanami
       # @param [Array] ids the commands' ids.
       def delete(ids)
         client.multi do
-          client.del(ids)
-          client.srem('ids', ids)
+          delete_commands ids
+          delete_ids ids
         end
       end
 
+      # Delete the commands' attributes.
+      #
+      # @param [Array] ids the commands' ids.
+      def delete_commands(ids)
+        client.del(ids)
+      end
+
+      # Delete the commands' ids from the global IDs set.
+      #
+      # @param [Array] ids the commands' ids.
+      def delete_ids(ids)
+        client.srem('ids', ids)
+      end
+
       # Delete all commands
+      #
+      # @see {#delete}
       def delete_all
-        delete client.smembers('ids')
+        delete stored_ids
+      end
+
+      # Retrieve all stored IDs
+      #
+      # @return [Array] all the ids as Strings.
+      def stored_ids
+        client.smembers('ids')
       end
 
       # Save the hash as a command.

data/lib/izanami/version.rb

@@ -1,3 +1,3 @@
 module Izanami
-  VERSION = "0.19.0"
+  VERSION = "0.20.0"
 end

data/spec/izanami/mappers/command_spec.rb

@@ -84,6 +84,19 @@ describe Izanami::Mappers do
       it 'takes the elements sorted by their ids' do
         subject.take(3).should == commands.take(3)
       end
+
+      context 'when the IDs are inconsistent' do
+        before do
+          # remove the commands' data, but not their IDs,
+          # like they have been expired by redis
+          ids = commands.map { |c| c['id'] }
+          subject.send(:delete_commands, ids)
+        end
+
+        it 'takes zero commands' do
+          subject.take(3).should be_empty
+        end
+      end
     end
 
     describe '#count' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: izanami
 version: !ruby/object:Gem::Version
-  version: 0.19.0
+  version: 0.20.0
 platform: ruby
 authors:
 - Juan Hernández
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-28 00:00:00.000000000 Z
+date: 2013-09-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler