Review and fixed issued raised by sonarqube 1.13.x#1890
Review and fixed issued raised by sonarqube 1.13.x#1890pratik-mahalle wants to merge 2 commits intomicrocks:1.13.xfrom
Conversation
Signed-off-by: Pratik Mahalle <[email protected]>
Signed-off-by: Pratik Mahalle <[email protected]>
| throw e; | ||
| } | ||
| @Testcontainers | ||
| class GooglePubSubProducerManagerIT { |
There was a problem hiding this comment.
Hum ... 🤔 It looks like you replaced the Managed implementation with its integration test. I think this is unintended. This should not even compile because the class name and file name do not match. Can you fix this?
lbroudoux
left a comment
There was a problem hiding this comment.
Unintended replacement of implementation by its IT
|
This pull request has been automatically marked as stale because it has not had recent activity 😴 It will be closed in 30 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation. There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. Microcks is a Cloud Native Computing Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model. Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here. Thank you for your patience ❤️ |
This PR addresses several code quality issues identified by SonarQube to improve code maintainability and test reliability.
Related
Fixes SonarQube code quality issues to improve maintainability and test reliability.