Remove environment variables that were used by the old test harness
The following discussion from !287 (merged) should be addressed:
-
@daniels started a discussion: (+3 comments) Shouldn't this be removed in the commit which moves
tests: migrate internal-screenshot
? After that, the only users of the reference path have the C definition.
We should remove environment variables like WESTON_TEST_REFERENCE_PATH
and WESTON_MODULE_MAP
. These were only used by the test suite and are nowadays automatically set or have working defaults. There should be no need to pick them up from the environment anymore.
WESTON_TEST_OUTPUT_PATH
might be still useful, might not.
WESTON_DATA_DIR
is used by shared/file-util.c
, but it probably should go as well.
However, one thing to consider is if people want to run a cross-built test suite. If anyone does, they might already be overriding e.g. WESTON_TEST_REFERENCE_PATH
.
I suspect that WESTON_MODULE_MAP
is already abused to load custom renderers or backends, but that we don't care about.