coordinator 0.0.13 → 0.0.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1a6f186704baaa5be4569f4b82a00a9d2bf46683
-  data.tar.gz: b54daff3d8f3354601e95503d99367dcf6748dbf
+  metadata.gz: 4a588480c336550aaca3f8e23ee7138363c9719f
+  data.tar.gz: 35e8f36438f042c03cf668b6ebdcfef9ed244012
 SHA512:
-  metadata.gz: 7cfbedf5fe4f2d2525e5dd00d52917f64da6c45f168ed282d80d1ceae995f42aaa037c4292de565564bdde00146416718a052805f45f8e67e1a23ead8c22ab9e
-  data.tar.gz: a7c9b7e0f295735e82ff4b316fda3c98efa0b1cbf8d73f62dc1e868aa50d069ce7c3ae3798ef3845a259dfa7f7fd1755ef81fdfb95b53cc6da391abab3e65c3a
+  metadata.gz: b1d3a81354fdda554f8b8bdb1b191f33e9a737405c2d7921bfb8f63a8841a6e20dedb3d7cda3b7d7dc5a19984d3ad8a978bfb04ffe346d05c7e3a1c762539b0c
+  data.tar.gz: f8dd4e04a3bc6f3ae23d945c5ec0571795d56325e142e3f0bcb530ef1b5727d4a89e69a7b4a976d482778e44e64e59c2a60c5865b1049941311a732aae900954

data/lib/coordinator/redis_queue.rb

@@ -12,12 +12,12 @@ module Coordinator
     def push(item)
       return false if full?
       data = serialize(item)
-      @redis.rpush(@queue_name, data) unless items.include?(data)
+      @redis.rpush(@queue_name, data) unless serialized_items.include?(data)
     end
 
     def left_push(item)
       data = serialize(item)
-      @redis.lpush(@queue_name, data) unless items.include?(data)
+      @redis.lpush(@queue_name, data) unless serialized_items.include?(data)
     end
 
     def pop
@@ -49,7 +49,7 @@ module Coordinator
     end
 
     def items
-      @redis.lrange(@queue_name, 0, length).map { |i| deserialize(i) }
+      serialized_items.map { |i| deserialize(i) }
     end
 
     def full?
@@ -58,6 +58,10 @@ module Coordinator
 
     private
 
+    def serialized_items
+      @redis.lrange(@queue_name, 0, length)
+    end
+
     def serialize(item)
       item.is_a?(String) ? item : item.to_json
     end

data/lib/coordinator/version.rb

@@ -1,3 +1,3 @@
 module Coordinator
-  VERSION = "0.0.13"
+  VERSION = "0.0.14"
 end

data/test/unit/redis_queue_test.rb

@@ -13,6 +13,17 @@ describe 'Coordinator::RedisQueue' do
       assert_equal 2, @queue.length
     end
 
+    it 'adds same hash to queue only once' do
+      el = {
+        'id' => 10,
+        'subject' => 'testing'
+      }
+
+      @queue.push(el)
+      @queue.push(el)
+      assert_equal 1, @queue.length
+    end
+
     it 'adds the same item only once' do
       @queue.push("a")
       @queue.push("a")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: coordinator
 version: !ruby/object:Gem::Version
-  version: 0.0.13
+  version: 0.0.14
 platform: ruby
 authors:
 - Tyler Mercier
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-30 00:00:00.000000000 Z
+date: 2014-11-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: http