validate job before putting it on a queue.

This fixes resque#479

Author: steveklabnik

lib/resque.rb

@@ -274,6 +274,7 @@ def enqueue(klass, *args)
   #
   # This method is considered part of the `stable` API.
   def enqueue_to(queue, klass, *args)
+    validate(klass, queue)
     # Perform before_enqueue hooks. Don't perform enqueue if any hook returns false
     before_hooks = Plugin.before_enqueue_hooks(klass).collect do |hook|
       klass.send(hook, *args)

test/test_helper.rb

@@ -113,6 +113,12 @@ def save
   end
 end
 
+class JobWithNoQueue
+  def self.perform
+    "I don't have a queue."
+  end
+end
+
 def with_failure_backend(failure_backend, &block)
   previous_backend = Resque::Failure.backend
   Resque::Failure.backend = failure_backend

test/worker_test.rb

@@ -34,6 +34,12 @@
     assert_match('uninitialized constant', Resque::Failure.all['error'])
   end
 
+  it "validates jobs before enquing them." do
+    assert_raises Resque::NoQueueError do
+      Resque.enqueue(JobWithNoQueue)
+    end
+  end
+
   it "does not allow exceptions from failure backend to escape" do
     job = Resque::Job.new(:jobs, {})
     with_failure_backend BadFailureBackend do