]> 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)
committerJohn Griffith <john.griffith8@gmail.com>
Tue, 4 Nov 2014 22:38:40 +0000 (23:38 +0100)
commit37e4a12652c803d60c18079b735d96e5c7890f07
treedb8813b9cb28851a63367524400fd6efb12bc3cb
parent29cb45d2927d0799d3881b46e0690ba457ae57ca
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
cinder/tests/keymgr/test_barbican.py [deleted file]