Merge pull request #1 from jeremycrosbie/master

Use schedule name as class name if unspecified

lib/active_scheduler/resque_wrapper.rb

@@ -23,7 +23,7 @@ def self.wrap(schedule)
           class:        'ActiveScheduler::ResqueWrapper',
           queue:        queue,
           args: [{
-            job_class:  opts[:class],
+            job_class:  opts[:class] || job,
             queue_name: queue,
             arguments:  opts[:arguments]
           }]

spec/active_scheduler/resque_wrapper_spec.rb

@@ -66,6 +66,25 @@
         expect(wrapped['no_queue_job']['queue']).to eq 'default'
       end
     end
+
+    context "when the schedule name is the class name" do
+      let(:schedule) { YAML.load_file 'spec/fixtures/schedule_name_is_class_name.yaml' }
+
+      it "queues up a job, using the schedule name for the class name" do
+        expect(wrapped['MyScheduleNameIsClassNameJob']).to eq(
+          "class"       => "ActiveScheduler::ResqueWrapper",
+          "queue"       => "myscheduledjobqueue",
+          "description" => "It's a self-named job.",
+          "cron"       => "* * * * *",
+          "args"        => [{
+            "job_class"  => "MyScheduleNameIsClassNameJob",
+            "queue_name" => "myscheduledjobqueue",
+            "arguments"  => nil
+          }]
+        )
+      end
+
+    end
   end
 
   describe ".perform" do

spec/fixtures/schedule_name_is_class_name.yaml

@@ -0,0 +1,6 @@
+MyScheduleNameIsClassNameJob:
+  cron: "* * * * *"
+  queue: "myscheduledjobqueue"
+  args:
+    -
+  description: "It's a self-named job."