KNOX-3158: Enhancing the maintainability of knox-shell#1056
KNOX-3158: Enhancing the maintainability of knox-shell#1056Raghav-Mah3shwari wants to merge 1 commit intoapache:masterfrom
Conversation
|
@Raghav-Mah3shwari - please show that you have tested the shell functionality related to the changes that you have made. We need to know that there has been some manual testing for the shell work since much of that is hard to unit test. Some example output exercising the the affected code paths would be a good way to do this. |
|
Hi @lmccay can you give me some suggestions on how to test it and verify it. Thanks in advance |
smolnar82
left a comment
There was a problem hiding this comment.
Please use Knox's MesageFactory feature and the relevant interface (in this case this would be org.apache.knox.gateway.shell.KnoxShellMessages) instead of the plain Logger approach.
|
@Raghav-Mah3shwari — This PR has been inactive for some time. |
Increasing the maintainability of the code by reducing code smells from gateway-shell
How was this patch tested?
The changes were made in local first then a maven build was triggered. The build was successful