Job equality was not properly implemented.

Author: defunkt

HISTORY.md

@@ -1,3 +1,7 @@
+## 1.2.2 (2009-12-08)
+
+* Bugfix: Job equality was not properly implemented.
+
 ## 1.2.1 (2009-12-07)
 
 * Added `rake resque:workers` task for starting multiple workers.

lib/resque/job.rb

@@ -96,7 +96,7 @@ def inspect
     # Equality
     def ==(other)
       queue == other.queue &&
-        payload_class = other.payload_class &&
+        payload_class == other.payload_class &&
         args == other.args
     end
   end

test/resque_test.rb

@@ -59,8 +59,15 @@
   test "jobs can test for equality" do
     assert Resque::Job.create(:jobs, 'SomeJob', 20, '/tmp')
     assert Resque::Job.create(:jobs, 'some-job', 20, '/tmp')
-    job = Resque.reserve(:jobs)
-    assert_equal job, Resque.reserve(:jobs)
+    assert_equal Resque.reserve(:jobs), Resque.reserve(:jobs)
+
+    assert Resque::Job.create(:jobs, 'SomeMethodJob', 20, '/tmp')
+    assert Resque::Job.create(:jobs, 'SomeJob', 20, '/tmp')
+    assert_not_equal Resque.reserve(:jobs), Resque.reserve(:jobs)
+
+    assert Resque::Job.create(:jobs, 'SomeJob', 20, '/tmp')
+    assert Resque::Job.create(:jobs, 'SomeJob', 30, '/tmp')
+    assert_not_equal Resque.reserve(:jobs), Resque.reserve(:jobs)
   end
 
   test "can put jobs on a queue by way of a method" do