Merge pull request #411 from OpenKMIP/bug/fix-client-crypto-params

Fix a client bug for cryptographic parameter handling
This commit is contained in:
Peter Hamilton 2018-04-02 13:47:27 -04:00 committed by GitHub
commit fdfbba8a0e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 1 deletions

View File

@ -1243,12 +1243,15 @@ class ProxyKmipClient(object):
CryptographicParameters struct.
Returns:
None: if value is None
CryptographicParameters: a CryptographicParameters struct
Raises:
TypeError: if the input argument is invalid
"""
if not isinstance(value, dict):
if value is None:
return None
elif not isinstance(value, dict):
raise TypeError("Cryptographic parameters must be a dictionary.")
cryptographic_parameters = CryptographicParameters(

View File

@ -2600,6 +2600,15 @@ class TestProxyKmipClient(testtools.TestCase):
self.assertRaises(
ClientConnectionNotOpen, client.locate, *args)
def test_build_cryptographic_parameters_with_none(self):
"""
Test that an empty set of cryptographic parameters is processed
correctly.
"""
client = ProxyKmipClient()
result = client._build_cryptographic_parameters(None)
self.assertEqual(None, result)
def test_build_cryptographic_parameters_invalid(self):
"""
Test that the right error is raised when attempting to build