cocoon-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jo...@apache.org
Subject svn commit: r597538 - /cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/test/java/org/apache/cocoon/forms/datatype/FlowJXPathSelectionListTestCase.java
Date Fri, 23 Nov 2007 04:00:15 GMT
Author: joerg
Date: Thu Nov 22 20:00:14 2007
New Revision: 597538

URL: http://svn.apache.org/viewvc?rev=597538&view=rev
Log:
even better fix (removing the namespace declaration as attribute on expected document): If
it's expected document that's broken, fix that!

Modified:
    cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/test/java/org/apache/cocoon/forms/datatype/FlowJXPathSelectionListTestCase.java

Modified: cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/test/java/org/apache/cocoon/forms/datatype/FlowJXPathSelectionListTestCase.java
URL: http://svn.apache.org/viewvc/cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/test/java/org/apache/cocoon/forms/datatype/FlowJXPathSelectionListTestCase.java?rev=597538&r1=597537&r2=597538&view=diff
==============================================================================
--- cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/test/java/org/apache/cocoon/forms/datatype/FlowJXPathSelectionListTestCase.java
(original)
+++ cocoon/trunk/blocks/cocoon-forms/cocoon-forms-impl/src/test/java/org/apache/cocoon/forms/datatype/FlowJXPathSelectionListTestCase.java
Thu Nov 22 20:00:14 2007
@@ -69,8 +69,7 @@
         Document expected = this.parser.parse(expectedSource.getInputStream());
         Document destDocument = dest.getDocument();
         // FIXME: Why is the namespace declaration available as attribute on the expected
document?
-        //        That's the reason why we have to add it to destDocument as well.
-        destDocument.getDocumentElement().setAttribute("xmlns:" + FormsConstants.INSTANCE_PREFIX,
FormsConstants.INSTANCE_NS);
+        expected.getDocumentElement().removeAttribute("xmlns:" + FormsConstants.INSTANCE_PREFIX);
         assertEqual("Test if generated list matches expected", expected, destDocument);
     }
     
@@ -97,8 +96,7 @@
         Document expected = this.parser.parse(expectedSource.getInputStream());
         Document destDocument = dest.getDocument();
         // FIXME: Why is the namespace declaration available as attribute on the expected
document?
-        //        That's the reason why we have to add it to destDocument as well.
-        destDocument.getDocumentElement().setAttribute("xmlns:" + FormsConstants.INSTANCE_PREFIX,
FormsConstants.INSTANCE_NS);
+        expected.getDocumentElement().removeAttribute("xmlns:" + FormsConstants.INSTANCE_PREFIX);
         assertEqual("Test if generated list matches expected", expected, destDocument);
     }
     



Mime
View raw message