]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Remove test_barbican from keymgr tests
authorJohn Griffith <john.griffith8@gmail.com>
Tue, 4 Nov 2014 22:34:28 +0000 (23:34 +0100)
committerThomas Goirand <thomas@goirand.fr>
Sun, 14 Dec 2014 09:18:31 +0000 (09:18 +0000)
commitbf02343c10cbcb09db21902c425d176819a61426
tree88a61308d3d3e2a387cbdae2002f6a38db7a0902
parent71e43a189636fe18a71880a85c01857906586685
Remove test_barbican from keymgr tests

Unfortunately it seems we have some very poorly
written keymgr unit tests, specifically
keymgr/test_barbican.py does this:
from barbicanclient.common import auth

First problem is that from a unit test perspective
that pretty much sucks, second problem is that
barbicanclient as of version 3.0.0.0 no longer
has an "auth" module, as a result Cinderunit tests
now fail.

The test_barbican.py unit tests need to be rewritten
to mock out the client components and actually test
Cinder components where needed without relying on
the barbicanclient.

For now in order to free up the gate, remove test_barbican.py

Change-Id: Id770c87c0d622ae4c1e0d71141ffa9aed29be2ec
Partial-Bug: #1389419
(cherry picked from commit 37e4a12652c803d60c18079b735d96e5c7890f07)
cinder/tests/keymgr/test_barbican.py [deleted file]