jobs must be given a class

Author: defunkt

lib/resque/errors.rb

@@ -1,4 +1,7 @@
 module Resque
   # Raised whenever we need a queue but none is provided.
   class NoQueueError < RuntimeError; end
+
+  # Raised when trying to create a job without a class
+  class NoClassError < RuntimeError; end
 end

lib/resque/job.rb

@@ -40,6 +40,10 @@ def self.create(queue, klass, *args)
         raise NoQueueError.new("Jobs must be placed onto a queue.")
       end
 
+      if klass.to_s.empty?
+        raise NoClassError.new("Jobs must be given a class.")
+      end
+
       Resque.push(queue, :class => klass.to_s, :args => args)
     end
 

test/resque_test.rb

@@ -122,6 +122,12 @@
     assert_equal ["queue:people", "queues"], Resque.keys
   end
 
+  test "badly wants a class name, too" do
+    assert_raises Resque::NoClassError do
+      Resque::Job.create(:jobs, nil)
+    end
+  end
+
   test "keeps stats" do
     Resque::Job.create(:jobs, SomeJob, 20, '/tmp')
     Resque::Job.create(:jobs, BadJob)