On Thursday, March 27, 2014 17:01:24 Dylan Baker wrote: > On Thursday, March 27, 2014 15:34:03 Dylan Baker wrote: > > Rather than have 2 base classes, let's just have one. > > > > v2: - remove test that no longer is valid > > > > Signed-off-by: Dylan Baker <[email protected]> > > --- > > > > framework/exectest.py | 26 +++++++++----------------- > > framework/gleantest.py | 12 ++++++------ > > framework/gtest.py | 4 ++-- > > framework/tests/exectest_test.py | 7 +------ > > tests/es3conform.py | 8 +++++--- > > tests/igt.py | 7 ++++--- > > tests/oglconform.py | 7 ++++--- > > 7 files changed, 31 insertions(+), 40 deletions(-) > > [snip]
Sorry for the empty email, my mail client got itself into a state. Anyway: there are three bugs in this patch; 1) igt still returns ExecTest instead of Test 2) piglit-print-command.py still checks for ExecTest instances instead of Test 3) Test has two implementations of run. I have fixed all of these already, and the changes are pushed to github already.
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
