]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Trim 5s+ from storwize unit tests
authorTom Barron <tpb@dyncloud.net>
Mon, 7 Mar 2016 20:05:21 +0000 (15:05 -0500)
committerTom Barron <tpb@dyncloud.net>
Mon, 7 Mar 2016 23:26:14 +0000 (18:26 -0500)
commitc10f5c2cd0d85e5ad496cbf59f7804f894b52c0c
tree99807488cdd7fb36807d37f98727e000f08e20d8
parenta63bfeac76c181f8a74f75658a9d14b1647a30e1
Trim 5s+ from storwize unit tests

The test test_run_ssh_fail_to_secondary_ip test case in the StorWize
Driver does a greenthread.sleep(random.randint()) inside a retry loop.
Its execution time often approaches 5s.

This commit mocks random.randint() in this test so that the
greenthread.sleep() duration is always zero, reducing execution time for
the test to under half a second.

The various test_storwize_consistency_group* tests trigger
FixedIntervalLoopingCalls.  Mocking them with ZeroIntervalLoopingCalls
reduces their total execution time from about 8s to under 3s.

We also fix the fake user and project ids in this file so that
it no longer emits FutureWarnings from oslo.versionedobjects about
invalid uuids as documented here [1].

[1] http://docs.openstack.org/developer/oslo.versionedobjects/api/fields.html#oslo_versionedobjects.fields.UUIDField

Change-Id: I05dc3c4a5a886ce323430055f34d6c578c0f4443
cinder/tests/unit/test_storwize_svc.py