simple_queues 1.0.0 → 1.0.1

data/lib/simple_queues/redis.rb

@@ -1,12 +1,23 @@
+autoload :Redis, "redis"
+require 'json'
+
 module SimpleQueues
   # The Redis version of SimpleQueues.
   #
   # Messages are enqueued to the right, dequeued from the left - thus the most recent messages are at the end of the list.
   class Redis
-    def initialize(redis=Redis.new)
+    def initialize(redis = ::Redis.new)
       @redis = redis
     end
 
+    def serialize(message)
+      message.to_json
+    end
+
+    def unserialize(message)
+      JSON.parse(message) if message
+    end
+
     # Enqueues a new message to the Redis backend.
     #
     # @param queue_name [String, Symbol] The queue name, which must not be nil or the empty String.
@@ -16,8 +27,7 @@ module SimpleQueues
     def enqueue(queue_name, message)
       raise ArgumentError, "Queue name argument was nil - must not be" if queue_name.nil? || queue_name.to_s.empty?
       raise ArgumentError, "Message argument was nil - must not be" if message.nil?
-
-      @redis.rpush(queue_name.to_s, message.to_s)
+      @redis.rpush(queue_name.to_s, serialize(message))
     end
 
     # Dequeues a message, and waits forever for one to arrive.
@@ -38,7 +48,7 @@ module SimpleQueues
     def dequeue_with_timeout(queue_name, timeout)
       raise ArgumentError, "Queue name argument was nil - must not be" if queue_name.nil? || queue_name.to_s.empty?
 
-      @redis.blpop(queue_name.to_s, timeout)
+      unserialize(@redis.blpop(queue_name.to_s, timeout))
     end
   end
 end

data/lib/simple_queues/version.rb

@@ -1,3 +1,3 @@
 module SimpleQueues
-  VERSION = "1.0.0"
+  VERSION = "1.0.1"
 end

data/simple_queues.gemspec

@@ -18,6 +18,7 @@ Gem::Specification.new do |s|
   s.require_paths = ["lib"]
 
   s.add_dependency 'redis'
+  s.add_dependency 'json'
   s.add_development_dependency 'rspec'
   s.add_development_dependency 'yard'
   s.add_development_dependency 'RedCloth'

data/spec/enqueueing_spec.rb

@@ -10,7 +10,7 @@ describe SimpleQueues::Redis, "enqueue" do
   end
 
   it "should enqueue to the end of the list" do
-    redis.should_receive(:rpush).with("q", "message")
+    redis.should_receive(:rpush).with("q", '"message"')
     queue.enqueue("q", "message")
   end
 
@@ -29,12 +29,15 @@ describe SimpleQueues::Redis, "enqueue" do
   end
 
   it "accepts symbols as queue names, translating to a string" do
-    redis.should_receive(:rpush).with("pages_to_crawl", "http://blog.teksol.info/")
+    redis.should_receive(:rpush).with("pages_to_crawl", '"http://blog.teksol.info/"')
     queue.enqueue :pages_to_crawl, "http://blog.teksol.info/"
   end
 
-  it "translates the message to a String before enqueueing" do
-    redis.should_receive(:rpush).with("ops", "shutdown_and_destroy")
+  it "translates the message using JSON before enqueueing" do
+    redis.should_receive(:rpush).with("ops", '"shutdown_and_destroy"')
     queue.enqueue :ops, :shutdown_and_destroy
+
+    redis.should_receive(:rpush).with("ops", '[1,2,3]')
+    queue.enqueue :ops, [1, 2, 3]
   end
 end

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 1
   - 0
-  - 0
-  version: 1.0.0
+  - 1
+  version: 1.0.1
 platform: ruby
 authors: 
 - "Fran\xC3\xA7ois Beausoleil"
@@ -31,7 +31,7 @@ dependencies:
   type: :runtime
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
-  name: rspec
+  name: json
   prerelease: false
   requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
@@ -41,10 +41,10 @@ dependencies:
         segments: 
         - 0
         version: "0"
-  type: :development
+  type: :runtime
   version_requirements: *id002
 - !ruby/object:Gem::Dependency 
-  name: yard
+  name: rspec
   prerelease: false
   requirement: &id003 !ruby/object:Gem::Requirement 
     none: false
@@ -57,7 +57,7 @@ dependencies:
   type: :development
   version_requirements: *id003
 - !ruby/object:Gem::Dependency 
-  name: RedCloth
+  name: yard
   prerelease: false
   requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
@@ -69,6 +69,19 @@ dependencies:
         version: "0"
   type: :development
   version_requirements: *id004
+- !ruby/object:Gem::Dependency 
+  name: RedCloth
+  prerelease: false
+  requirement: &id005 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        version: "0"
+  type: :development
+  version_requirements: *id005
 description: Program to an interface, not an implementation - hides Redis (used as a queue) behind a simple interface
 email: 
 - francois@teksol.info