Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integration tests for RabbitMQ #38

Merged
merged 16 commits into from
Jan 13, 2019
Merged

Conversation

Kralizek
Copy link
Member

@Kralizek Kralizek commented Jan 12, 2019

  • RabbitMqBusEngine now uses new BufferSubject to handle the condition race between connection to RabbitMQ and subscription of the handlers
  • IBusHost does not expose the SubscribeToCommand and SubscribeToEvent methods anymore
  • IContext now expose Message for later uses
  • NybusDispatcher now accepts the whole message, not just the correlationId
  • Moved Integration test projects under tests/integration
  • RabbitMq integration project uses FakeRabbitMQ to fake a connection to a RabbitMQ node
  • RabbitMq integration project has a set of test pointing to a real RabbitMQ node, these tests are excluded during the normal CI pipeline

Fixes #24

@Kralizek Kralizek merged commit c639730 into Nybus-project:master Jan 13, 2019
@Kralizek Kralizek deleted the rabbitmq branch January 13, 2019 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant