Further improve resource cleanup in tests

ruby-amqp · Feb 23, 2025 · 5543c40 · 5543c40
1 parent 2bb1aaf
commit 5543c40
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/spec/higher_level_api/integration/connection_recovery_spec.rb b/spec/higher_level_api/integration/connection_recovery_spec.rb
@@ -121,7 +121,7 @@
   it "recovers client-named queues" do
     with_open do |c|
       ch = c.create_channel
-      q  = ch.queue("bunny.tests.recovery.client-named#{rand}", durable: false)
+      q  = ch.queue("bunny.tests.recovery.client-named#{rand}", durable: false, exclusive: true)
       close_all_connections!
       wait_for_recovery_with { connections.any? }
       expect(ch).to be_open

diff --git a/spec/higher_level_api/integration/queue_declare_spec.rb b/spec/higher_level_api/integration/queue_declare_spec.rb
@@ -378,7 +378,7 @@
     context "when a queue exists" do
       it "returns true" do
         ch = connection.create_channel
-        q  = ch.queue("bunny.tests.queues.exists")
+        q  = ch.queue("bunny.tests.queues.exists", durable: true, arguments: {"x-expires" => 5000})
 
         expect(connection.queue_exists?(q.name)).to eq true
       end

diff --git a/spec/higher_level_api/integration/read_only_consumer_spec.rb b/spec/higher_level_api/integration/read_only_consumer_spec.rb
@@ -54,7 +54,13 @@
 
       expect(ch.queue(queue_name, auto_delete: true, durable: false).message_count).to eq 0
 
+      c3 = Bunny::new
+      c3.start
+      ch3 = c3.create_channel
+      ch3.queue_delete(queue_name)
+
       ch.close
+      ch3.close
     end
   end
 end # describe