From a97b9ef3d25e4e914ee20e74d8e314364149d429 Mon Sep 17 00:00:00 2001
From: Rodrigo Duarte Sousa <rodrigods@lsd.ufcg.edu.br>
Date: Fri, 10 Apr 2015 15:35:20 -0300
Subject: [PATCH] Fix signed_saml2_assertion.xml tests fixture

This change updates the signed_saml2_assertion.xml fixture with
the current attributes/values used by keystone: openstack_user,
openstack_roles, openstack_project.

Change-Id: Ia296fe26434f43054e32a98ad4af25148aac5da2
---
 keystone/contrib/federation/idp.py                   | 6 ++----
 keystone/tests/unit/saml2/signed_saml2_assertion.xml | 6 +++---
 2 files changed, 5 insertions(+), 7 deletions(-)

diff --git a/keystone/contrib/federation/idp.py b/keystone/contrib/federation/idp.py
index 4bca858176..292abea3c8 100644
--- a/keystone/contrib/federation/idp.py
+++ b/keystone/contrib/federation/idp.py
@@ -156,9 +156,7 @@ class SAMLGenerator(object):
     def _create_attribute_statement(self, user, roles, project):
         """Create an object that represents a SAML AttributeStatement.
 
-        <ns0:AttributeStatement
-          xmlns:ns0="urn:oasis:names:tc:SAML:2.0:assertion"
-          xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
+        <ns0:AttributeStatement>
             <ns0:Attribute Name="openstack_user">
                 <ns0:AttributeValue
                   xsi:type="xs:string">test_user</ns0:AttributeValue>
@@ -169,7 +167,7 @@ class SAMLGenerator(object):
                 <ns0:AttributeValue
                   xsi:type="xs:string">member</ns0:AttributeValue>
             </ns0:Attribute>
-            <ns0:Attribute Name="openstack_projects">
+            <ns0:Attribute Name="openstack_project">
                 <ns0:AttributeValue
                   xsi:type="xs:string">development</ns0:AttributeValue>
             </ns0:Attribute>
diff --git a/keystone/tests/unit/saml2/signed_saml2_assertion.xml b/keystone/tests/unit/saml2/signed_saml2_assertion.xml
index 410f938889..f570642eef 100644
--- a/keystone/tests/unit/saml2/signed_saml2_assertion.xml
+++ b/keystone/tests/unit/saml2/signed_saml2_assertion.xml
@@ -49,14 +49,14 @@ UHeBXxQq/GmfBv3l+V5ObQ+EHKnyDodLHCk=</ns1:X509Certificate>
     </ns0:AuthnContext>
   </ns0:AuthnStatement>
   <ns0:AttributeStatement>
-    <ns0:Attribute FriendlyName="keystone_user" Name="user" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri">
+    <ns0:Attribute Name="openstack_user" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri">
       <ns0:AttributeValue xsi:type="xs:string">test_user</ns0:AttributeValue>
     </ns0:Attribute>
-    <ns0:Attribute FriendlyName="keystone_roles" Name="roles" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri">
+    <ns0:Attribute Name="openstack_roles" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri">
       <ns0:AttributeValue xsi:type="xs:string">admin</ns0:AttributeValue>
       <ns0:AttributeValue xsi:type="xs:string">member</ns0:AttributeValue>
     </ns0:Attribute>
-    <ns0:Attribute FriendlyName="keystone_project" Name="project" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri">
+    <ns0:Attribute Name="openstack_project" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri">
       <ns0:AttributeValue xsi:type="xs:string">development</ns0:AttributeValue>
     </ns0:Attribute>
   </ns0:AttributeStatement>