linux-test-project / lcov

LCOV
GNU General Public License v2.0
894 stars 240 forks source link

Make test harness more portable #178

Closed nega0 closed 1 year ago

nega0 commented 1 year ago

This PR supersedes #132 in the interest of a cleaner history.

The intent of this PR is to get the test harness to run on more platforms. It replaces or wraps platform (or age) specific bits with more generic alternatives. The patch is minor, and aside from .gitignore only touches 6 of the test harness files, and the common test Makefile. The tests themselves are all untouched. With these patches, the test harness successfully runs all tests on up-to-date macOS Ventura (and they all pass too).

 .gitignore                        |   53 ++++++++++++++++++++++++++++++++++++++
 tests/bin/test_run                |   11 ++++++-
 tests/bin/test_skip               |    3 +-
 tests/bin/testsuite_exit          |    5 ++-
 tests/bin/testsuite_init          |   31 +++++++++++++++++-----
 tests/common.mak                  |   10 +++----
 tests/gendiffcov/simple/script.sh |    2 -
 tests/lcov/extract/extract.sh     |    2 -
 8 files changed, 99 insertions(+), 18 deletions(-)
nega0 commented 1 year ago

Essentially the same as #132 with some minor changes and additions.

henry2cox commented 1 year ago

Excellent - thank you for your contribution. I'm traveling at the moment, but will squash and apply this as soon as I'm back in action. Henry