-
Notifications
You must be signed in to change notification settings - Fork 1
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
fix: #7 fixed unit tests #8
base: main
Are you sure you want to change the base?
Conversation
Code coverage report is ready! 📈
|
Code coverage report is ready! 📈
|
unit tests are not executed in action. Reason could be, that the maven build in the GitHub action is using surefire plugin 2.x.x, and locally I am using 3.2.5. Todo: Further investigation needed, why there is a mismatch in the default versions of maven plugins, fix the root cause and get coverage displayed here |
Code coverage report is ready! 📈
|
Code coverage report is ready! 📈
|
testcoverage fixed, ready for review |
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.
To the best of my knowledge, I didn't find anything weird. However, it would be better to have someone else look into it, as Java and MQTT are beyond my scope.
private static final String USER_PROPERTIES_KEY_FOR_TOKEN = "10"; | ||
private static final String USER_PROPERTIES_KEY_FOR_TRACEPARENT = "11"; | ||
private static final String USER_PROPERTIES_KEY_FOR_PAYLOAD_FORMAT = "12"; | ||
public static final Logger LOG = LoggerFactory.getLogger(HiveMqMQTT5Client.class); |
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.
why do you need/want to make the logger public?
No description provided.