Refactor the RunPagesProcess function in the tests to take functional arguments
The following discussion from !40 (merged) should be addressed:
-
@nick.thomas started a discussion: Normally I'd prefer to pass the environment down so it only gets set for the subprocess, but
RunPagesProcess
already has far too many arguments :/Let's merge it like this and I'll refactor it to take functional args at some point.