Remove environment variables that were used by the old test harness
The following discussion from !287 (merged) should be addressed:
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_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.
I suspect that
WESTON_MODULE_MAP is already abused to load custom renderers or backends, but that we don't care about.