summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJenkins <jenkins@review.openstack.org>2015-03-24 16:37:38 +0000
committerGerrit Code Review <review@openstack.org>2015-03-24 16:37:38 +0000
commit2149d1b6fc2441ee2da6c6695af114f71b5f012b (patch)
treea0e6f008ea9e7006505fd165731d8ade28ce4018
parente86b11bbe1146533f189f2b56a1e0638b92684bb (diff)
parent52273ead1020d934f0fe82a13e8f2073e1954058 (diff)
downloadpython-barbicanclient-2149d1b6fc2441ee2da6c6695af114f71b5f012b.tar.gz
Merge "Container negative tests"
-rw-r--r--functionaltests/client/v1/functional/test_containers.py126
1 files changed, 126 insertions, 0 deletions
diff --git a/functionaltests/client/v1/functional/test_containers.py b/functionaltests/client/v1/functional/test_containers.py
index f373920..1b474b7 100644
--- a/functionaltests/client/v1/functional/test_containers.py
+++ b/functionaltests/client/v1/functional/test_containers.py
@@ -113,6 +113,79 @@ class GenericContainersTestCase(BaseContainersTestCase):
container_ref = self.behaviors.store_container(test_model)
self.assertIsNotNone(container_ref)
+ @testcase.attr('negative')
+ def test_create_defaults_duplicate_secret_refs(self):
+ """Covers creating a container with a duplicated secret ref."""
+ secrets = {'secret_1': self.secret_1,
+ 'secret_2': self.secret_1,
+ 'secret_3': self.secret_1}
+
+ test_model = self.behaviors.create_generic_container(
+ create_container_defaults_data, secrets=secrets)
+
+ e = self.assertRaises(Exception, self.behaviors.store_container,
+ test_model)
+
+ self.assertEqual(e.http_status, 400)
+
+ @testcase.attr('negative')
+ def test_get_non_existent_container(self):
+ """A get on a container that does not exist.
+
+ This should return a container incorrectly specified error since
+ the container does not have a correctly formatted UUID
+ """
+ base_url = self.behaviors.base_url
+ url = base_url + '/containers/notauuid'
+ e = self.assertRaises(ValueError, self.behaviors.get_container,
+ url)
+
+ self.assertEqual(e.message, 'Container incorrectly specified.')
+
+ @testcase.attr('negative')
+ def test_get_non_existent_container_valid_uuid(self):
+ """A get on a container that does not exist with valid UUID
+
+ This should return a 404.
+ """
+ base_url = self.behaviors.base_url
+ uuid = 'de305d54-75b4-431b-cccc-eb6b9e546013'
+ url = base_url + '/containers/' + uuid
+
+ e = self.assertRaises(Exception, self.behaviors.get_container,
+ url)
+
+ self.assertEqual(e.http_status, 404)
+
+ @testcase.attr('negative')
+ def test_delete_non_existent_container(self):
+ """A delete on a container that does not exist.
+
+ This should return a container incorrectly specified error since
+ the container does not have a correctly formatted UUID
+ """
+ base_url = self.behaviors.base_url
+ url = base_url + '/containers/notauuid'
+ e = self.assertRaises(ValueError, self.behaviors.get_container,
+ url)
+
+ self.assertEqual(e.message, 'Container incorrectly specified.')
+
+ @testcase.attr('negative')
+ def test_delete_non_existent_container_valid_uuid(self):
+ """A delete on a container that does not exist with valid UUID
+
+ This should return a 404.
+ """
+ uuid = 'de305d54-75b4-431b-cccc-eb6b9e546013'
+ base_url = self.barbicanclient.secrets._api._base_url
+ url = base_url + '/containers/' + uuid
+
+ e = self.assertRaises(Exception, self.behaviors.get_container,
+ url)
+
+ self.assertEqual(e.http_status, 404)
+
@utils.parameterized_dataset({'0': [0], '1': [1], '50': [50]})
@testcase.attr('positive')
def test_create_container_defaults_size(self, num_secrets):
@@ -187,3 +260,56 @@ class RSAContainersTestCase(BaseContainersTestCase):
get_resp = self.behaviors.get_container(container_ref)
self.assertEqual(get_resp.name, name)
+
+ @testcase.attr('negative')
+ def test_create_rsa_invalid_key_names(self):
+ """Covers creating an RSA container with incorrect names."""
+ incorrect_names_rsa_container = {
+ "name": "bad_container",
+ "secret1": self.secret_ref_1,
+ "secret2": self.secret_ref_2,
+ "secret3": self.secret_ref_3
+ }
+
+ e = self.assertRaises(TypeError, self.behaviors.create_rsa_container,
+ incorrect_names_rsa_container)
+
+ self.assertIn('got an unexpected keyword argument', e.message)
+
+ @testcase.attr('negative')
+ def test_create_rsa_no_public_key(self):
+ """Creating an rsa container without a public key should fail.
+
+ RSA containers must have at least a public key and private key.
+ """
+ no_public_key_rsa_container = {"name": "no_pub_key",
+ "private_key": self.secret_1,
+ "private_key_passphrase": self.secret_2,
+ }
+
+ test_model = self.behaviors.create_rsa_container(
+ no_public_key_rsa_container)
+
+ e = self.assertRaises(Exception, self.behaviors.store_container,
+ test_model)
+
+ self.assertEqual(e.http_status, 400)
+
+ @testcase.attr('negative')
+ def test_create_rsa_no_private_key(self):
+ """Creating an rsa container without a private key should fail.
+
+ RSA containers must have at least a public key and private key.
+ """
+ no_private_key_rsa_container = {
+ "name": "no_pub_key",
+ "public_key": self.secret_1,
+ "private_key_passphrase": self.secret_2}
+
+ test_model = self.behaviors.create_rsa_container(
+ no_private_key_rsa_container)
+
+ e = self.assertRaises(Exception, self.behaviors.store_container,
+ test_model)
+
+ self.assertEqual(e.http_status, 400)