Linting
Remove Unittestzero Asserts
services tests declassified
wait_for_request_status actually terminates in the failure case

- `minute_count` was never incremented, so a failed service request
  would effectively loop forever
- the main fix was to convert the method to use `wait_for`, with
  ancillary functionalization of utility functionality
- despite extreme temptation, I tried to stay focused on the broken `while` loop replacement
vm names are unique per-test

- I don't know if there's an expectation for each test to use the same
  vm_name, but I assume there isn't one since vm_name was remade in every test.
  I guess we'll find out.
A little more lint
Merge pull request #551 from seandst/srvcatalog-wait-for

service catalog test cleanup, switch to wait_for