SourabhBadhya commented on PR #4821: URL: https://github.com/apache/hive/pull/4821#issuecomment-1781342936
@hzxiongyinke , I would say it would be great to have integration test but having unit tests around whether the HMS token is added to the UGI would also be fine. You can simulate a lot of these logic by first constructing `conf` objects and adding the desired properties to it as needed and then expecting that in the UGI. You can also mock objects and make it pass through the logic and assert the expected method calls / values. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
