Emysql icon indicating copy to clipboard operation
Emysql copied to clipboard

Published 20 hours ago • verified publisher icon Eonblast

Clean up tests

Open jlouis opened this issue 10 years ago • 3 comments

If we move all tests into one suite with adequate groups, we don't have to start/stop emysql pools all the time and we can speed up the test case. Also, it would enable us to have correct handling of coverage which would be very nice to have.

jlouis avatar Feb 27 '14 13:02 jlouis

+1 on coverage. Not sure about relevance of test run time?

hdiedrich avatar Feb 27 '14 14:02 hdiedrich

Currently, the tests run pretty fast, but one of the things we can do is to mark the suite as parallel and run it 2-3 times. This will probably be efficient at uncovering bugs pertaining to concurrent access in the code base. I can't do that as long as the suites are running each on their own.

jlouis avatar Feb 27 '14 14:02 jlouis

I have pushed some patches which addresses this partially. The code is covered somewhere between 40-80% depending on module. There are still more work to do, but it is a start.

jlouis avatar Feb 27 '14 15:02 jlouis