Resolve test failures and warnings

zendesk · Jan 10, 2025 · 9d8472d · 9d8472d
1 parent 0ce373e
commit 9d8472d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/test/lib/samson/secrets/vault_server_test.rb b/test/lib/samson/secrets/vault_server_test.rb
@@ -29,7 +29,9 @@
     it "is invalid with an invalid cert" do
       server.ca_cert = "nope"
       refute_valid server
-      server.errors.full_messages.must_equal ["Ca cert is invalid: PEM_read_bio_X509: no start line"]
+      server.errors.full_messages.must_equal(
+        ["Ca cert is invalid: PEM_read_bio_X509: no start line (Expecting: CERTIFICATE)"]
+      )
     end
 
     it "is invalid with duplicate name" do

diff --git a/test/models/job_queue_test.rb b/test/models/job_queue_test.rb
@@ -335,7 +335,7 @@ def with_staggering_enabled(stagger_interval: 1.second, &block)
         mock_timer_task = mock(execute: true)
         expected_task_params = {now: true, timeout_interval: 10, execution_interval: 1.second}
         instance.expects(:dequeue_staggered_job)
-        Concurrent::TimerTask.expects(:new).with(expected_task_params).yields.returns(mock_timer_task)
+        Concurrent::TimerTask.expects(:new).with(**expected_task_params).yields.returns(mock_timer_task)
 
         with_staggering_enabled do
           instance.send(:start_staggered_job_dequeuer)