This is a cleanup of the test code that I had on my TODO. We should be using the 3-arg form of test_* instead of skip_all + test_done to skip tests. This doesn't cover all the cases that were using skip_all=*. The remaining ones were too hairy to trivially convert, so I've left them alone for now. This also includes a documentation patch to t/README describing how we'd like tests to be skipped. Ævar Arnfjörð Bjarmason (4): tests: implicitly skip SYMLINKS tests using <prereq> t/t5800-remote-helpers.sh: Skip with prereq on python <2.4 t/t7800-difftool.sh: Skip with prereq on no PERL t/README: Update "Skipping tests" to align with best practices t/README | 21 +++++++++++++-- t/t2007-checkout-symlink.sh | 14 +++------- t/t4004-diff-rename-symlink.sh | 18 +++++-------- t/t4011-diff-symlink.sh | 18 ++++--------- t/t4023-diff-rename-typechange.sh | 14 +++------- t/t4114-apply-typechange.sh | 28 ++++++++------------- t/t4115-apply-symlink.sh | 12 ++------- t/t4122-apply-symlink-inside.sh | 12 ++------- t/t5522-pull-symlink.sh | 14 +++------- t/t5800-remote-helpers.sh | 24 ++++++++---------- t/t6035-merge-dir-to-symlink.sh | 20 +++++---------- t/t7800-difftool.sh | 49 ++++++++++++++++-------------------- 12 files changed, 100 insertions(+), 144 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html