Skip to content
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.

Test amqp #52

Open
frederike-ramin opened this issue Jul 5, 2018 · 1 comment
Open

Test amqp #52

frederike-ramin opened this issue Jul 5, 2018 · 1 comment

Comments

@frederike-ramin
Copy link
Contributor

frederike-ramin commented Jul 5, 2018

Currently there are no tests for amqp.

Can be implemented via Dependency Injection.

@cmfcmf
Copy link
Contributor

cmfcmf commented Jul 5, 2018

Can be implemented via Dependency Injection.

To give some more context, the AmqpConnector should be refactored to receive the amqplib instance during construction. This allows a test to supply a fake amqplib instance which doesn't communicate to any real amqp service, but rather spies on the invoked methods and asserts that certain methods have been called. The Mocha docs have a list of supported spying libraries.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants