r4mercur commented on PR #4175: URL: https://github.com/apache/solr/pull/4175#issuecomment-4019799421
I added a new teststep which some bahavior test in the OverseerRolesTest class. Would you consider this combination of the two testcases now (unittestcase + integration test) sufficient for this feature to be tested right or would you prefer a stricter integration assertion arround the overseer election behaviour ? Then i would delete the unittest part and would try to rewrite the integration test with more stricter "feature" based assertion. -- 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]
