enh at google.com
Tue Jun 16 13:33:42 PDT 2020
i'm not sure i understand the intent behind this change:
Author: Rob Landley <rob at landley.net>
Date: Tue Nov 20 17:50:05 2018 -0600
Skip tests that don't have the executable bit set unless $TEST_ALL set.
i assumed this was to avoid running tests for stuff in pending, but it
seems like there's quite a random mix of tests not being run
even basics like date, test, touch, and xargs aren't having their
tests run. oversight?
either way, i wonder whether it would make more sense to have `make
tests` run _all_ the tests and add something like `make quicktests` if
that was your intention? or have a tests/pending/ directory if _that_
was your intention?
and it wouldn't hurt to modify the `make help` output too, if not
running all the tests was actually intentional:
tests - Run test suite against all compiled commands.
export TEST_HOST=1 to test host command, VERBOSE=1
to show diff, VERBOSE=fail to stop after first failure.
(this doesn't affect Android directly because i just run all the tests
in my runner. but it may explain how i've managed to send you a few
patches that didn't pass their tests... i only learned this today
when, out of curiosity, i did a clang coverage run to see what the
test coverage looks like and was surprised to see apparently missing
coverage for tests i'd helped write myself!)
More information about the Toybox