Skip to content

Implement UnsettledMessageCount in IConsumer #42

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

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

lukebakken
Copy link
Contributor

Follow-up to #37

  • Start adding to AmqpTests class from rabbitmq-amqp-java-client

@lukebakken lukebakken added this to the 0.1.0 milestone Aug 8, 2024
@lukebakken lukebakken requested a review from Gsantomaggio August 8, 2024 15:10
@lukebakken lukebakken self-assigned this Aug 8, 2024
Follow-up to #37

* Start adding to `AmqpTests` class from `rabbitmq-amqp-java-client`

* Add `IntegrationTest` base class that sets up a connection using the test name as the connection name.

* `IntegrationTest` also closes and disposes the connection correctly.

* Begin porting `QueueDeclareDeletePublishConsume` test from Java

* Finish `QueueDeclareDeletePublishConsume` test

* Delete queue after tests run

* Make `AmqpPublisher` truly async

* Remove publishing back-pressure since it is not present in the Java client
@lukebakken lukebakken force-pushed the rabbitmq-amqp-dotnet-client-37-followup branch from 1977f8b to bab4d9c Compare August 9, 2024 19:19
@lukebakken lukebakken marked this pull request as ready for review August 9, 2024 19:24
@lukebakken lukebakken merged commit 1075782 into main Aug 9, 2024
1 check passed
@lukebakken lukebakken deleted the rabbitmq-amqp-dotnet-client-37-followup branch August 9, 2024 19:24
lukebakken added a commit that referenced this pull request Aug 9, 2024
lukebakken added a commit that referenced this pull request Aug 22, 2024
…ollowup

Implement `UnsettledMessageCount` in `IConsumer`
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