checkskip function for test generators

helps deal with parametrization that yields no params, warns in that case
Merge pull request #538 from seandst/testgen-checkskip

checkskip function for test generators
Added the delete method to providers

Added delete() method to Provider classes
Added test_provider_delete to tests
Merge pull request #545 from psav/cru_to_crud

Added the delete method to providers
Prevent profile assign failure

* when the provider does not exist, the profile is not assigned.
Merge pull request #548 from mfalesni/events_prevent_error_setup

Prevent profile assign failure
Linting
Remove Unittestzero Asserts
services tests declassified
Added syslogging
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
Only log warnings coming from cfme_tests
Merge pull request #551 from seandst/srvcatalog-wait-for

service catalog test cleanup, switch to wait_for
Merge pull request #549 from psav/syslog

Added syslogging
Merge pull request #552 from seandst/less-warnings

Only log warnings coming from cfme_tests