[java-opensaml COMMIT] /trunk/opensaml-saml-impl/src/test/java/org/opensaml/saml/common/binding/artifact/BasicSAMLArt...

noreply at shibboleth.net noreply at shibboleth.net
Sun Jul 7 17:37:42 EDT 2013


Author: scantor
Date: Sun Jul  7 17:37:41 2013
New Revision: 3401

URL: http://svn.shibboleth.net/view/java-opensaml?rev=3401&view=rev
Log:
Fix document element handling in unit test.

Modified:
    trunk/opensaml-saml-impl/src/test/java/org/opensaml/saml/common/binding/artifact/BasicSAMLArtifactMapEntryFactoryTest.java

Modified: trunk/opensaml-saml-impl/src/test/java/org/opensaml/saml/common/binding/artifact/BasicSAMLArtifactMapEntryFactoryTest.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-saml-impl/src/test/java/org/opensaml/saml/common/binding/artifact/BasicSAMLArtifactMapEntryFactoryTest.java?rev=3401&r1=3400&r2=3401&view=diff
==============================================================================
--- trunk/opensaml-saml-impl/src/test/java/org/opensaml/saml/common/binding/artifact/BasicSAMLArtifactMapEntryFactoryTest.java (original)
+++ trunk/opensaml-saml-impl/src/test/java/org/opensaml/saml/common/binding/artifact/BasicSAMLArtifactMapEntryFactoryTest.java Sun Jul  7 17:37:41 2013
@@ -53,7 +53,6 @@
         factory = new BasicSAMLArtifactMapEntryFactory();
         
         samlObject = (SAMLObject) unmarshallElement("/data/org/opensaml/saml/saml1/core/SignedAssertion.xml");
-        samlObject.getDOM().getOwnerDocument().appendChild(samlObject.getDOM());
     }
 
     @Test
@@ -88,6 +87,7 @@
         Assert.assertEquals(basicEntry.getRelyingPartyId(), newEntry.getRelyingPartyId());
 
         Document origDocument = samlObject.getDOM().getOwnerDocument();
+        origDocument.appendChild(samlObject.getDOM());
         Document newDocument = newEntry.getSamlMessage().getDOM().getOwnerDocument();
         XMLAssertTestNG.assertXMLIdentical(new Diff(origDocument, newDocument), true);
     }



More information about the commits mailing list