Add default key for limits_concurrency

Author: ankane

lib/active_job/concurrency_controls.rb

@@ -5,6 +5,7 @@ module ConcurrencyControls
     extend ActiveSupport::Concern
 
     DEFAULT_CONCURRENCY_GROUP = ->(*) { self.class.name }
+    DEFAULT_CONCURRENCY_KEY = ->(*) { nil }
 
     included do
       class_attribute :concurrency_key, instance_accessor: false
@@ -15,7 +16,7 @@ module ConcurrencyControls
     end
 
     class_methods do
-      def limits_concurrency(key:, to: 1, group: DEFAULT_CONCURRENCY_GROUP, duration: SolidQueue.default_concurrency_control_period)
+      def limits_concurrency(key: DEFAULT_CONCURRENCY_KEY, to: 1, group: DEFAULT_CONCURRENCY_GROUP, duration: SolidQueue.default_concurrency_control_period)
         self.concurrency_key = key
         self.concurrency_limit = to
         self.concurrency_group = group

test/models/solid_queue/job_test.rb

@@ -10,6 +10,13 @@ def perform(job_result)
     end
   end
 
+  class NonOverlappingDefaultJob < ApplicationJob
+    limits_concurrency
+
+    def perform(job_result)
+    end
+  end
+
   class NonOverlappingGroupedJob1 < NonOverlappingJob
     limits_concurrency key: ->(job_result, **) { job_result }, group: "MyGroup"
   end
@@ -98,6 +105,16 @@ class NonOverlappingGroupedJob2 < NonOverlappingJob
     assert_equal active_job.concurrency_key, job.concurrency_key
   end
 
+  test "enqueue jobs with concurrency controls default options" do
+    active_job = NonOverlappingDefaultJob.perform_later(@result, name: "A")
+    assert_equal 1, active_job.concurrency_limit
+    assert_equal "SolidQueue::JobTest::NonOverlappingDefaultJob", active_job.concurrency_key
+
+    job = SolidQueue::Job.last
+    assert_equal active_job.concurrency_limit, job.concurrency_limit
+    assert_equal active_job.concurrency_key, job.concurrency_key
+  end
+
   test "enqueue jobs with concurrency controls in the same concurrency group" do
     assert_ready do
       active_job = NonOverlappingGroupedJob1.perform_later(@result, name: "A")