Having it be set in the runalltests script means you cant easily run tests that use it individually, making debugging tests even harder. I would suggest either reverting to using full paths or have imppy.sh set a path to imp and have the test file paths be relative to that.