amqp / rhea-promise

A promisified layer over rhea AMQP client

Apache License 2.0

30 stars 19 forks source link

Tests for ensuring event listeners are removed when remove() or close() is called #63

Closed ramya-rao-a closed 4 years ago

ramya-rao-a commented 4 years ago

Description

Brief description of the changes made in the PR. This helps in making better changelog This PR adds the below tests

remove() and close() on senders and receivers should remove all event listeners. This refactors the code that was added in #34 to ensure that the removal happens after close() completes
refactored a session test to move the test on event listeners being removed to match the test files for sender & receiver

extra:

error when creating sender bubbles up via the promise returned by createSender(). The same for createReceiver() is added but commented due to unexpected behavior which needs further investigation

Reference to any github issues

58