sucker_punch 0.1 → 0.2

data/lib/sucker_punch.rb

@@ -1,5 +1,8 @@
 require 'celluloid'
 require 'sucker_punch/exceptions'
+require 'sucker_punch/queue'
+require 'sucker_punch/worker'
+require 'sucker_punch/version'
 
 module SuckerPunch
   extend self
@@ -9,21 +12,14 @@ module SuckerPunch
   end
 
   def queue(options = {})
-    raise SuckerPunch::MissingQueueName unless options[:name]
-    raise SuckerPunch::MissingWorkerName unless options[:worker]
+    raise MissingQueueName unless options[:name]
+    raise MissingWorkerName unless options[:worker]
 
     klass         = options.fetch(:worker)
     registry_name = options.fetch(:name)
     size          = options.fetch(:size, nil)
 
-    Celluloid::Actor[registry_name] = if size
-                                        klass.send(:pool, size: size)
-                                      else
-                                        klass.send(:pool)
-                                      end
+    q = Queue.new(registry_name)
+    q.register(klass, size)
   end
-end
-
-require 'sucker_punch/queue'
-require 'sucker_punch/worker'
-require 'sucker_punch/version'
+end

data/lib/sucker_punch/queue.rb

@@ -1,9 +1,21 @@
 module SuckerPunch
-  module Queue
-    extend self
+  class Queue
+    attr_reader :name
 
-    def [](name)
+    def initialize(name)
+      @name = name
+    end
+
+    def self.[](name)
       Celluloid::Actor[name]
     end
+
+    def register(klass, size)
+      Celluloid::Actor[name] = if size
+                                  klass.send(:pool, size: size)
+                                else
+                                  klass.send(:pool)
+                                end
+    end
   end
 end

data/lib/sucker_punch/version.rb

@@ -1,3 +1,3 @@
 module SuckerPunch
-  VERSION = "0.1"
+  VERSION = "0.2"
 end

data/spec/queue_spec.rb

@@ -3,6 +3,9 @@ require 'spec_helper'
 class FakeWorker
   include Celluloid
 end
+class FakeOtherWorker
+  include Celluloid
+end
 
 describe SuckerPunch::Queue do
   describe ".[]" do
@@ -12,4 +15,19 @@ describe SuckerPunch::Queue do
       SuckerPunch::Queue[:fake]
     end
   end
+
+  describe "#register" do
+    before(:each) do
+      SuckerPunch::Queue.new(:crazy_queue).register(FakeWorker, 7)
+    end
+
+    it "turns the class into an actor" do
+      Celluloid::Actor[:crazy_queue].should be_a(FakeWorker)
+      Celluloid::Actor[:crazy_queue].methods.should include(:async)
+    end
+
+    it "sets worker size" do
+      Celluloid::Actor[:crazy_queue].size.should == 7
+    end
+  end
 end

data/spec/sucker_punch_spec.rb

@@ -6,24 +6,15 @@ end
 
 describe SuckerPunch do
   context "config" do
+
     context "properly configured" do
-      before(:all) do
+      it "registers the queue" do
+        SuckerPunch::Queue.any_instance.should_receive(:register).with(FakeWorker, 7)
+
         SuckerPunch.config do
           queue name: :crazy_queue, worker: FakeWorker, size: 7
         end
       end
-
-      it "turns the class into an actor" do
-        Celluloid::Actor[:crazy_queue].should be_a(FakeWorker)
-        Celluloid::Actor[:crazy_queue].methods.should include(:async)
-      end
-
-      it "allow asynchrounous processing" do
-      end
-
-      it "sets worker size" do
-        Celluloid::Actor[:crazy_queue].size.should == 7
-      end
     end
 
     context "with no queue name" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sucker_punch
 version: !ruby/object:Gem::Version
-  version: '0.1'
+  version: '0.2'
   prerelease: 
 platform: ruby
 authors: