Sure, but imagine, for example, given a production class with a single method, I add 3 methods which are used only in tests, the test will look more succinct, but the class won't. The coder who needs to understand the logic of the whole component needs to check usages of all 4 methods and scan through them to confirm that those 3 used only in tests and are not called in production and shouldn't be followed to understand how the component operates - waste of time and concentration, I think. Meanwhile it's could be impossible to use a debugger in a production to understand the workflow, while you can run the test in debugger easily if you don't get some steps.
[ Full content available at: https://github.com/apache/incubator-druid/pull/6349 ] This message was relayed via gitbox.apache.org for [email protected]
