search

eclipse / kapua

Eclipse Public License 2.0
227 stars 160 forks source link

FIX: Logs of broken jms connection to event-broker on an integration test #4100

Closed Agnul97 closed 1 month ago

Agnul97 commented 3 months ago

A specific scenario in one of the "@ broker" ITs needed connection to the event-broken and therefore created an instance of jms-bus connection to it. The problem was that the bus nas not closed in the end, so when event-broker was destroyed at the end of the IT the client tried to re-connect to it as part of the logic of the JMS-bus still "open". I fixed the problem closing the jmb bus, at the end of the IT, before the destroy of docker containers

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 16.81%. Comparing base (c92b620) to head (9bebe29). Report is 7 commits behind head on develop.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/eclipse/kapua/pull/4100/graphs/tree.svg?width=650&height=150&src=pr&token=1P4N4CApH8&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse)](https://app.codecov.io/gh/eclipse/kapua/pull/4100?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=eclipse) ```diff @@ Coverage Diff @@ ## develop #4100 +/- ## ========================================== Coverage 16.81% 16.81% Complexity 22 22 ========================================== Files 2019 2019 Lines 52418 52418 Branches 4418 4418 ========================================== Hits 8813 8813 Misses 43208 43208 Partials 397 397 ```