Record topology only for actual declarations

[eglitobias]

Fixes #751. Topology recovery recorded Bunny::Queue/Bunny::Exchange handles in addition to the actual declarations, so a no_declare: true handle got redeclared with default properties and could overwrite the real recorded declaration. Now only queue_declare/exchange_declare record, which also covers passive without separate guards.

Repro (run rabbitmqctl close_all_connections during the sleep):

require "bunny"

# The quorum queue already exists, declared by someone else
setup = Bunny.new.start
setup.create_channel.quorum_queue("repro.qq")
setup.close

conn = Bunny.new(network_recovery_interval: 1).start
ch = conn.create_channel
ch.queue("repro.qq", no_declare: true).subscribe { |*| }

puts "Now run: rabbitmqctl close_all_connections"
sleep 10

puts "Channel open after recovery: #{ch.open?}"

• main: PRECONDITION_FAILED - inequivalent arg 'x-queue-type', then false
• this branch: true

[michaelklishin]

Thank you.