Validate mapping exists when creating/updating a protocol
This patch validates that a mapping exists when adding or updating
a federation protocol.
Change-Id: I996f94d26eb0f2c679542ba13a03bbaa4442486a
Closes-Bug: #1571878
diff --git a/keystone_tempest_plugin/tests/api/identity/v3/test_identity_providers.py b/keystone_tempest_plugin/tests/api/identity/v3/test_identity_providers.py
index ebd00d4..619fc65 100644
--- a/keystone_tempest_plugin/tests/api/identity/v3/test_identity_providers.py
+++ b/keystone_tempest_plugin/tests/api/identity/v3/test_identity_providers.py
@@ -14,6 +14,7 @@
from tempest.lib.common.utils import data_utils
from tempest.lib import decorators
+from tempest.lib import exceptions as lib_exc
from keystone_tempest_plugin.tests.api.identity import base
from keystone_tempest_plugin.tests.api.identity.v3 import fixtures
@@ -217,9 +218,12 @@
# a non existent mapping ID
mapping_id = data_utils.rand_uuid_hex()
protocol_id = data_utils.rand_uuid_hex()
- protocol = self._create_protocol(idp_id, protocol_id, mapping_id)
-
- self._assert_protocol_attributes(protocol, protocol_id, mapping_id)
+ self.assertRaises(
+ lib_exc.BadRequest,
+ self._create_protocol,
+ idp_id,
+ protocol_id,
+ mapping_id)
@decorators.idempotent_id('c73311e7-c207-4c11-998f-532a91f1b0d1')
def test_update_protocol_from_identity_provider_unknown_mapping(self):
@@ -232,7 +236,9 @@
# Update the identity provider protocol using a non existent
# mapping_id
new_mapping_id = data_utils.rand_uuid_hex()
- protocol = self.idps_client.update_protocol_mapping(
- idp_id, protocol_id, new_mapping_id)['protocol']
-
- self._assert_protocol_attributes(protocol, protocol_id, new_mapping_id)
+ self.assertRaises(
+ lib_exc.BadRequest,
+ self.idps_client.update_protocol_mapping,
+ idp_id,
+ protocol_id,
+ new_mapping_id)