I think it's good to use these assertions in the test but we shouldn't duplicate the code. Maybe it makes sense to put `server_running?` in `Spring::Env` rather than in `Spring::Client::Run` where it is at present?