Refactor selectHost in Sched

Remove the unnecessary return error and add a test for selectHost.
I'm on the fence about this-- seems a bit odd to have a function that can't ever produce an error have an error return parameter.

On second thought, you should make it check len(priorityList) != 0, which is a legit error case for this function.