Fix wait and wait_until ignored in bulk enqueue (#960)

Co-authored-by: alokhmutov <alokhmutov@gmail.com>

.rubocop.yml

@@ -1,6 +1,7 @@
 inherit_from: .rubocop_todo.yml
 
 require:
+  - rubocop-capybara
   - rubocop-performance
   - rubocop-rails
   - rubocop-rspec

app/models/good_job/execution.rb

@@ -224,9 +224,8 @@ def self.enqueue_args(active_job, overrides = {})
         queue_name: active_job.queue_name.presence || DEFAULT_QUEUE_NAME,
         priority: active_job.priority || DEFAULT_PRIORITY,
         serialized_params: active_job.serialize,
-        scheduled_at: active_job.scheduled_at,
       }
-
+      execution_args[:scheduled_at] = Time.zone.at(active_job.scheduled_at) if active_job.scheduled_at
       execution_args[:concurrency_key] = active_job.good_job_concurrency_key if active_job.respond_to?(:good_job_concurrency_key)
 
       reenqueued_current_execution = CurrentThread.active_job_id && CurrentThread.active_job_id == active_job.job_id

spec/lib/good_job/adapter_spec.rb

@@ -117,6 +117,21 @@ def perform(succeed: true)
       expect(provider_job_ids).to all be_present
     end
 
+    it 'enqueues queue_name, scheduled_at, priority' do
+      active_job = ExampleJob.new
+      active_job.queue_name = 'elephant'
+      active_job.priority = -55
+      active_job.scheduled_at = 10.minutes.from_now
+
+      adapter.enqueue_all([active_job])
+
+      expect(GoodJob::Job.last).to have_attributes(
+        queue_name: 'elephant',
+        priority: -55,
+        scheduled_at: be_within(0.1).of(10.minutes.from_now)
+      )
+    end
+
     context 'when a job fails to enqueue' do
       it 'does not set a provider_job_id' do
         allow(GoodJob::Execution).to receive(:insert_all).and_wrap_original do |original_method, *args|

spec/lib/good_job/bulk_spec.rb

@@ -114,5 +114,17 @@ def perform
       expect(job_1.provider_job_id).to be_present
       expect(job_2.provider_job_id).to be_nil
     end
+
+    it 'sets queue, scheduled_at, and priority' do
+      described_class.enqueue do
+        TestJob.set(queue: 'elephant', wait: 10.minutes, priority: 50).perform_later
+      end
+
+      expect(GoodJob::Job.last).to have_attributes(
+        queue_name: 'elephant',
+        scheduled_at: be_within(1.second).of(10.minutes.from_now),
+        priority: 50
+      )
+    end
   end
 end

spec/test_app/app/jobs/example_job.rb

@@ -4,7 +4,7 @@ class ExampleJob < ApplicationJob
 
   TYPES = [
     SUCCESS_TYPE = 'success',
-      ERROR_ONCE_TYPE = 'error_once',
+    ERROR_ONCE_TYPE = 'error_once',
     ERROR_FIVE_TIMES_TYPE = 'error_five_times',
     DEAD_TYPE = 'dead',
     SLOW_TYPE = 'slow',