Closed apjanke closed 5 years ago
I'm deferring logging to a separate issue https://github.com/apjanke/octave-testify/issues/56 because it's a more complicated topic and doesn't block the refactoring itself.
I'm happy with the refactoring now. Moved test2_refactor
and runtests2_refactor
back to test2
and runtests2
in https://github.com/apjanke/octave-testify/commit/91d8fe76e71a11cbf1027283400f9b33a15b23ec.
The
test
function's code has some issues:eval
logic with generated temporary function strings, which is a little hard to follow.Same goes for
runtests
, to a lesser extent.Refactor the code to address this.
TODO
runtests2
Giveruntests2
log file handling (for the lower-leveltest
output)__run_test_suite__
use runtest2's summary output