[java-identity-provider] 02/04: IDP-2036 Mapped Attribute Definition does not implements all the common Attributes

[Rod Widdowson]

This is an automated email from the git hooks/post-receive script.

rdw pushed a commit to branch maint-4
in repository java-identity-provider.

View the commit online:
http://git.shibboleth.net/view/?p=java-identity-provider.git;a=commit;h=12ef69eeea1390f007907e1b310b3db30af33dec

commit 12ef69eeea1390f007907e1b310b3db30af33dec
Author: Rod Widdowson <rdw at steadingsoftware.com>
AuthorDate: Sat Nov 12 14:52:09 2022 +0000

    IDP-2036 Mapped Attribute Definition does not implements all the common Attributes
    
    https://shibboleth.atlassian.net/browse/IDP-2036
    
    Add test and fix.
---
 .../spring/ad/mapped/impl/MappedAttributeDefinitionParser.java        | 4 ++--
 .../spring/ad/mapped/MappedAttributeDefinitionParserTest.java         | 2 ++
 .../attribute/resolver/spring/ad/mapped/resolver/mappedNoDefault.xml  | 1 +
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/impl/MappedAttributeDefinitionParser.java b/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/impl/MappedAttributeDefinitionParser.java
index 1fd96bdb1..e66938f82 100644
--- a/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/impl/MappedAttributeDefinitionParser.java
+++ b/idp-attribute-resolver-spring/src/main/java/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/impl/MappedAttributeDefinitionParser.java
@@ -34,13 +34,13 @@ import org.w3c.dom.Element;
 
 import net.shibboleth.ext.spring.util.SpringSupport;
 import net.shibboleth.idp.attribute.resolver.ad.mapped.impl.MappedAttributeDefinition;
-import net.shibboleth.idp.attribute.resolver.spring.BaseResolverPluginParser;
+import net.shibboleth.idp.attribute.resolver.spring.ad.BaseAttributeDefinitionParser;
 import net.shibboleth.idp.attribute.resolver.spring.impl.AttributeResolverNamespaceHandler;
 import net.shibboleth.utilities.java.support.primitive.StringSupport;
 import net.shibboleth.utilities.java.support.xml.ElementSupport;
 
 /** Bean definition parser for a {@link MappedAttributeDefinition}. */
-public class MappedAttributeDefinitionParser extends BaseResolverPluginParser {
+public class MappedAttributeDefinitionParser extends BaseAttributeDefinitionParser {
 
     /** Schema type name. */
     @Nonnull public static final QName TYPE_NAME_RESOLVER =
diff --git a/idp-attribute-resolver-spring/src/test/java/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/MappedAttributeDefinitionParserTest.java b/idp-attribute-resolver-spring/src/test/java/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/MappedAttributeDefinitionParserTest.java
index 74c0e9a17..8d87b8cb6 100644
--- a/idp-attribute-resolver-spring/src/test/java/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/MappedAttributeDefinitionParserTest.java
+++ b/idp-attribute-resolver-spring/src/test/java/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/MappedAttributeDefinitionParserTest.java
@@ -69,6 +69,8 @@ public class MappedAttributeDefinitionParserTest extends BaseAttributeDefinition
     @Test public void noDefault() {
         final MappedAttributeDefinition defn = getDefinition("resolver/mappedNoDefault.xml");
 
+        assertTrue(defn.isDependencyOnly());
+        assertTrue(defn.isPreRequested());
         assertFalse(defn.isPassThru());
         assertEquals(defn.getValueMaps().size(), 1);
         assertNull(defn.getDefaultAttributeValue());
diff --git a/idp-attribute-resolver-spring/src/test/resources/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/resolver/mappedNoDefault.xml b/idp-attribute-resolver-spring/src/test/resources/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/resolver/mappedNoDefault.xml
index ef8e960aa..09d27e09e 100644
--- a/idp-attribute-resolver-spring/src/test/resources/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/resolver/mappedNoDefault.xml
+++ b/idp-attribute-resolver-spring/src/test/resources/net/shibboleth/idp/attribute/resolver/spring/ad/mapped/resolver/mappedNoDefault.xml
@@ -1,5 +1,6 @@
 <AttributeDefintion xsi:type="Mapped"
 	xmlns="urn:mace:shibboleth:2.0:resolver" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" id="Mapped"
+    dependencyOnly="true" preRequested="true"
     xsi:schemaLocation="urn:mace:shibboleth:2.0:resolver http://shibboleth.net/schema/idp/shibboleth-attribute-resolver.xsd">
     <InputAttributeDefinition ref="TheOrphan" />                    
 	<ValueMap>

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.