this should be a constant …
whoops
This changes how we run phase handlers, from now on each phase must complete successfully otherwise the subsequent phases will not be run.
some basic test cleanup