-
Notifications
You must be signed in to change notification settings - Fork 669
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
MF-1199 - Add NATS messaging tests #1209
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test looks nice and shows how elegant are the abstractions (Publisher
and Subscriber
interfaces) that we introduced.
Approved from my side, but @dusanb94 needs to review.
Let's wait for tests to be finished and CI to pass. |
e1b60db
to
e408d49
Compare
Codecov Report
@@ Coverage Diff @@
## master #1209 +/- ##
==========================================
- Coverage 77.44% 76.96% -0.48%
==========================================
Files 104 106 +2
Lines 6875 6955 +80
==========================================
+ Hits 5324 5353 +29
- Misses 1164 1205 +41
- Partials 387 397 +10
Continue to review full report at Codecov.
|
c923b28
to
290e914
Compare
290e914
to
dc04f85
Compare
dc04f85
to
71868dd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
fb7d6dc
to
6f0d8e1
Compare
Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com>
Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com>
Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com>
Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com>
Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com>
Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com>
Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com>
6f0d8e1
to
3442afd
Compare
Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* Add test setup for mqtt Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com> * Add pubsub tests Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com> * Add message and payload helper funcs Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com> * Add time.Sleep after subscribe to topic Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com> * Add NATS tests setup Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com> * Add NATS test cases Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com> * Add message forwarded check to handler Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com> * Remove race affected mqtt tests Signed-off-by: Darko Draskovic <darko.draskovic@gmail.com>
Partially resolves #1199