openSUSE / qem-dashboard / 22347828447

95%

push

github

fix: reset backoff when amqp_watcher reconnects

The logic uses a backoff delay which will double when connection calls `close`
but this value is never reset after a reconnection. This commit ensures that
the value is reset after a new connection and every future disconnections will
restart the exponential backoff from the default value, set in disconnection
function.

The change is simple and it is backed by a new test file. The reason was
simply to keep it separate from dashboard.t which has more general test
cases. The test also implements test stubs, for the RabbitMQ and IOLoop in
order to control the behavior, which should be ideal for this case, as the
focus is in the behavior instead of the interaction.

related-issue: https://progress.opensuse.org/issues/195923
Co-authored-by: Tina Müller <tina.mueller@suse.com>
Signed-off-by: Ioannis Bonatakis <ybonatakis@suse.com>

116 of 116 new or added lines in 4 files covered. (100.0%)

1245 of 1307 relevant lines covered (95.26%)

7.9 hits per line