David Knupp has posted comments on this change. Change subject: MPALA-4750: Rename test infra classes so they don't mimic test classes. ......................................................................
Patch Set 1: (3 comments) I did check that the same number of tests were collected (note -- that there is parallel change required for the tests in the aux directory that will be submitted as well.) These are the numbers, correcting for the problem with test_cancel_insert (IMPALA-4818). $ grep collected ~/tmp/pytest_collect_core* /home/dknupp/tmp/pytest_collect_core_IMPALA4750.txt:collecting ... collected 1830 items / 5 skipped /home/dknupp/tmp/pytest_collect_core_master.txt:collecting ... collected 1830 items / 5 skipped $ grep collected ~/tmp/pytest_collect_exhaustive* /home/dknupp/tmp/pytest_collect_exhaustive_IMPALA4750.txt:collecting ... collected 20881 items /home/dknupp/tmp/pytest_collect_exhaustive_master.txt:collecting ... collected 20881 items http://gerrit.cloudera.org:8080/#/c/5794/1//COMMIT_MSG Commit Message: PS1, Line 7: MPALA > Don't forget I. Done PS1, Line 9: that > extra word Done http://gerrit.cloudera.org:8080/#/c/5794/1/tests/experiments/test_targeted_perf.py File tests/experiments/test_targeted_perf.py: PS1, Line 33: TestMatrix > What's going on here? Weird. I don't know how that happened. For what it's worth, tests/experiments is not in the white list of directories defined in run-tests.py, so these tests never get run. Also, the name of the classmethod is even wrong -- add_test_dimensions() should be plural. So even if you try to run the tests in this module explicitly, this code still doesn't get executed. -- To view, visit http://gerrit.cloudera.org:8080/5794 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I06b7bc6fd99fbb637a47ba376bf9830705c1fce1 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: David Knupp <dkn...@cloudera.com> Gerrit-Reviewer: David Knupp <dkn...@cloudera.com> Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org> Gerrit-HasComments: Yes