keith-turner opened a new issue, #3486: URL: https://github.com/apache/accumulo/issues/3486
**Is your feature request related to a problem? Please describe.** Was looking into an issue and as part of that investigation I was wondering if AccumuloConfiguration.isPropertySet() was behaving properly. Looked through all of the isPropertySet() implementations and each seemed good. Looked at the tests and could only find that SystemConfiguration had a test for isPropertySet() and that it only tested the true case and not the false case. **Describe the solution you'd like** Adding unit test that verify the expected behavior of AccumuloConfiguration.isPropertySet for each impl would be useful to avoid regressions. Not sure if its workable, but wondering if it would make sense to create generic unit test suite for AccumuloConfiguration impls that can verify certain expected behaviors OR is it easier to have a unit test for each impl. -- 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]
