bugfix: Tester completes as soon as there's a SubmissionResult

This is necessary for, e.g., handling compilation failures, which
should prevent attempts to execute the program.  I stupidly broke
this in an attempt to simplify the code (having forgotten why
this was necessary for correctness.)