* add bashate requirement so that it can be actually used;
* use xargs instead of find -exec to get negative results out;
* add pipefail option to know if find failed.
Change-Id: I5d88496608f865335e2ca970de7d9ba19bdcc89b
They were failing because test_deploy.TestDeployCreateService wasn't
properly unmocking mocked create_object_from_definition. Anyhow tearDown
is evil, tearDown with testtools works badly, we should use addCleanup
instead and using fixtures is even better than that.
To reproduce bug and verify that this patch fixes it, add
test_run_concurrency=echo 1
to .testr.conf, it'll run all tests in one process sequentially.
Tests were flaky because testr randomly splits list of tests across
processes (yay, test reprocudibility!) and sometimes
TestDeployCreateService wasn't run in the same process as failing tests.
Change-Id: I60beb43d5ab540e232f2f664212f86036d473cc6