Make HeartbeatProcess#tick! lease argument backward-compatible

The lease parameter was added to tick! in v0.85.0 but callers in the
monolith (MultiTick#heartbeat) still use the old 1-arg signature.
Make lease optional so existing callers work without changes.

Bump ci-queue to v0.94.0.

Author: acaron

ruby/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    ci-queue (0.93.0)
+    ci-queue (0.94.0)
       logger
 
 GEM

ruby/lib/ci/queue/redis/base.rb

@@ -318,8 +318,10 @@ def shutdown!
             end
           end
 
-          def tick!(id, lease)
-            send_message(TICK_COMMAND, id: id, lease: lease.to_s)
+          def tick!(id, lease = nil)
+            payload = { id: id }
+            payload[:lease] = lease.to_s if lease
+            send_message(TICK_COMMAND, **payload)
             @restart_attempts = 0
           rescue IOError, Errno::EPIPE => error
             @restart_attempts = (@restart_attempts || 0) + 1

ruby/lib/ci/queue/version.rb

@@ -2,7 +2,7 @@
 
 module CI
   module Queue
-    VERSION = '0.93.0'
+    VERSION = '0.94.0'
     DEV_SCRIPTS_ROOT = ::File.expand_path('../../../../../redis', __FILE__)
     RELEASE_SCRIPTS_ROOT = ::File.expand_path('../redis', __FILE__)
   end

ruby/test/ci/queue/redis/heartbeat_process_test.rb

@@ -108,4 +108,21 @@ def test_tick_sends_valid_tick_payload
 
     assert_equal ["tick!", { "id" => "test_id", "lease" => "lease_id" }], JSON.parse(payload)
   end
+
+  def test_tick_without_lease_omits_lease_from_payload
+    pipe = FakePipe.new
+    @hp.instance_variable_set(:@pipe, pipe)
+
+    @hp.tick!("test_id")
+
+    raw = pipe.buffer
+    header_size = [0].pack("L").bytesize
+    size = raw.byteslice(0, header_size).unpack1("L")
+    payload = raw.byteslice(header_size, size)
+
+    parsed = JSON.parse(payload)
+    assert_equal "tick!", parsed[0]
+    assert_equal "test_id", parsed[1]["id"]
+    refute parsed[1].key?("lease"), "lease should be omitted when not provided"
+  end
 end