On 23/11/2009, sebb <seb...@gmail.com> wrote:
> The JUnit tests produce a lot of output, even if the tests are successful.
>
>  Is there really any need to print stack traces in the following method?
>
>  TestSharedPoolDataSource.PoolTest.run()
>
>  I propose to comment them out.
>
>  Similarly, the test case TestManual.testLogWriter() generates a lot of 
> output.
>  It's not clear to me what the test is trying to achieve.
>  Any ideas?
>

Does anyone know what TestAbandonedBasicDataSource.testAbandoned() is
trying to test? It seems to be similar to testAbandonedClose() except
that it does not close the connections.

As far as I can tell it achieves nothing at present, except to print
some  useless messages on System.err. I propose to remove the
printouts (and inline the resulting one-line private methods)

OK?

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to