Fix discard job with nonexistent job class

app/models/good_job/base_execution.rb

@@ -92,14 +92,14 @@ def discrete?
     # Build an ActiveJob instance and deserialize the arguments, using `#active_job_data`.
     #
     # @param ignore_deserialization_errors [Boolean]
-    #   Whether to ignore ActiveJob::DeserializationError when deserializing the arguments.
+    #   Whether to ignore ActiveJob::DeserializationError and NameError when deserializing the arguments.
     #   This is most useful if you aren't planning to use the arguments directly.
     def active_job(ignore_deserialization_errors: false)
       ActiveJob::Base.deserialize(active_job_data).tap do |aj|
         aj.send(:deserialize_arguments_if_needed)
-      rescue ActiveJob::DeserializationError
-        raise unless ignore_deserialization_errors
       end
+    rescue ActiveJob::DeserializationError, NameError
+      raise unless ignore_deserialization_errors
     end
 
     private

spec/app/models/good_job/job_spec.rb

@@ -324,6 +324,18 @@ def perform
         end.to change { job.reload.status }.from(:scheduled).to(:discarded)
       end
     end
+
+    context 'when job class does not exist' do
+      before do
+        job.update!(serialized_params: { 'job_class' => 'NonexistentJob' })
+      end
+
+      it 'ignores the error and discards the job' do
+        expect do
+          job.discard_job("Discarded in test")
+        end.to change { job.reload.status }.from(:scheduled).to(:discarded)
+      end
+    end
   end
 
   describe '#force_discard_job' do