-
Notifications
You must be signed in to change notification settings - Fork 32
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
Support direct reply-to feature #82
Support direct reply-to feature #82
Conversation
Codecov Report
@@ Coverage Diff @@
## master #82 +/- ##
============================================
+ Coverage 93.56% 94.03% +0.47%
- Complexity 432 442 +10
============================================
Files 38 38
Lines 1025 1040 +15
Branches 43 45 +2
============================================
+ Hits 959 978 +19
+ Misses 48 47 -1
+ Partials 18 15 -3
Continue to review full report at Codecov.
|
Hi, thanks for the PR. I am not sure about creating a "real" queue for Please do the following changes before I can merge :
Thanks for your work |
src/main/java/com/github/fridujo/rabbitmq/mock/MockChannel.java
Outdated
Show resolved
Hide resolved
src/main/java/com/github/fridujo/rabbitmq/mock/MockChannel.java
Outdated
Show resolved
Hide resolved
src/test/java/com/github/fridujo/rabbitmq/mock/ChannelTest.java
Outdated
Show resolved
Hide resolved
src/test/java/com/github/fridujo/rabbitmq/mock/ChannelTest.java
Outdated
Show resolved
Hide resolved
src/test/java/com/github/fridujo/rabbitmq/mock/ChannelTest.java
Outdated
Show resolved
Hide resolved
c0064e9
to
c71b6fc
Compare
49a50a5
to
8b1ae79
Compare
144fdd3
to
c67d7b4
Compare
Seems solid ! |
Solves #78 |
No description provided.