Remove test start output in pretty tracer

Sean does not want and its currently broken.

Change-Id: I7c3cf17bfd8d273823774cd162ae011161c36d3d
Remove redundant waiter in create_volume cleanup

If wait_on_delete is True we are already waiting
for the volume delete, so the addCleanup_with_wait
call is redundant.

This just changes the addCleanup_with_wait call
to a normal addCleanup on the volume delete in
the case that wait_on_delete is True.

If wait_on_delete is False we still do the delete
but the wait is moved to the end of the test run
so it's asynchronous.

Change-Id: I6c4cd1a00ddc79de4c59c512b2d794e7380591f2
Partial-Bug: #1367857
Properly print STDOUT and STDERR in test_wrappers

If the unit tests in test_wrappers.py fail while using subprocess to
run the wrapper scripts STDOUT and STDERR are incorrectly used in the
exception method. Currently str() on the stringIO object is being
used for the message, which will only print information about the open
file. This commit fixes this by calling the read() method on the pipe
objects to print their contents.

Change-Id: I8fbf560fe2c706f1de96bb089730ec3f92fe3327
Unskip test_stack_update_add_remove()

test_stack_update_add_remove was skipped because of bug 1308682 which
is now in a state which should allow us to unskip it.

Change-Id: I9b728abe3432f61f883d50b4a9d7d8d9c2b1f8b2
Related-Bug: #1308682
Update Heat CLI test to pass with random hash

The failing test assumes ordering of the returned JSON
blob and asserts that the given server resource precedes
another resource.  This updates it and allows the test
to pass when the resource is listed at the end of the
section, which may happen depending on hash + dict
ordering.

Change-Id: I279a1a1f3081214b0a2b0f712d311d148acd7b8e
Fail tempest if 0 tests are run

This commit adds a check to subunit-trace to fail if the test run
doesn't actually run anything. Despite technically passing without
running anything, if you're passing a regex to testr that runs 0 tests
that is probably an error, so treat it as such. This also uncovered a
bug in the test_wrappers unit test where the test was using a bad test
regex but still passsing. Since we start enforcing it here the unit
test is also updated to use the proper regex.

Change-Id: I109a8cd6eb731e5b8daf5cc2d6a76498ddd48c6d
Merge "Remove redundant waiter in create_volume cleanup"
Merge "Remove test start output in pretty tracer"
Merge "Update Heat CLI test to pass with random hash"
Merge "Properly print STDOUT and STDERR in test_wrappers"
Merge "Fail tempest if 0 tests are run"
Merge "Unskip test_stack_update_add_remove()"
Merge branch 'master' of https://github.com/openstack/tempest