svn commit: r1326098 - in /webservices/commons/trunk/modules/axiom/modules: axiom-dom/src/test/java/org/apache/axiom/soap/impl/dom/ axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/

[veithen]

Author: veithen
Date: Sat Apr 14 12:44:03 2012
New Revision: 1326098

URL: http://svn.apache.org/viewvc?rev=1326098&view=rev
Log:
Resolved a couple of todo items that are no longer relevant because of the 
changes introduced by AXIOM-412.

Modified:
    
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/test/java/org/apache/axiom/soap/impl/dom/SOAPImplementationTest.java
    
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/TestIsCompleteAfterAddingIncompleteChild.java
    
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/TestSerializeAndConsumeWithIncompleteDescendant.java

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/test/java/org/apache/axiom/soap/impl/dom/SOAPImplementationTest.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/test/java/org/apache/axiom/soap/impl/dom/SOAPImplementationTest.java?rev=1326098&r1=1326097&r2=1326098&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/test/java/org/apache/axiom/soap/impl/dom/SOAPImplementationTest.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-dom/src/test/java/org/apache/axiom/soap/impl/dom/SOAPImplementationTest.java
 Sat Apr 14 12:44:03 2012
@@ -26,7 +26,6 @@ import org.apache.axiom.ts.soap.SOAPTest
 import org.apache.axiom.ts.soap.factory.TestGetDefaultFaultEnvelope;
 import org.apache.axiom.ts.soap.faultdetail.TestWSCommons202;
 import org.apache.axiom.ts.soap.faulttext.TestSetLang;
-import org.apache.axiom.ts.soap.xpath.TestXPathAppliedToSOAPEnvelope;
 import org.apache.axiom.ts.soap12.fault.TestMoreChildrenAddition;
 import 
org.apache.axiom.ts.soap12.faultcode.TestSetValueFromQNameWithExistingValue;
 
@@ -47,10 +46,6 @@ public class SOAPImplementationTest exte
         
builder.exclude(org.apache.axiom.ts.soap12.faultcode.TestSetValueFromQName.class);
         builder.exclude(TestSetValueFromQNameWithExistingValue.class);
         
-        // TODO: fails because DOOM's factories are not stateless
-        //       (and createOMDocument doesn't create a new document in this 
case)
-        builder.exclude(TestXPathAppliedToSOAPEnvelope.class, 
"(createDocument=true)");
-        
         return builder.build();
     }
 }

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/TestIsCompleteAfterAddingIncompleteChild.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/TestIsCompleteAfterAddingIncompleteChild.java?rev=1326098&r1=1326097&r2=1326098&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/TestIsCompleteAfterAddingIncompleteChild.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/TestIsCompleteAfterAddingIncompleteChild.java
 Sat Apr 14 12:44:03 2012
@@ -41,8 +41,7 @@ public class TestIsCompleteAfterAddingIn
         OMFactory factory = metaFactory.getOMFactory();
         OMElement incompleteElement = 
OMXMLBuilderFactory.createOMBuilder(factory,
                 new 
StringReader("<elem>text</elem>")).getDocumentElement(true);
-        // TODO: need to get the OMFactory again because for DOOM, it is 
stateful
-        OMDocument document = metaFactory.getOMFactory().createOMDocument();
+        OMDocument document = factory.createOMDocument();
         assertTrue(document.isComplete());
         document.addChild(incompleteElement);
         assertFalse(document.isComplete());

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/TestSerializeAndConsumeWithIncompleteDescendant.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/TestSerializeAndConsumeWithIncompleteDescendant.java?rev=1326098&r1=1326097&r2=1326098&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/TestSerializeAndConsumeWithIncompleteDescendant.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/om/document/TestSerializeAndConsumeWithIncompleteDescendant.java
 Sat Apr 14 12:44:03 2012
@@ -45,8 +45,7 @@ public class TestSerializeAndConsumeWith
         OMFactory factory = metaFactory.getOMFactory();
         OMElement incompleteElement = 
OMXMLBuilderFactory.createOMBuilder(factory,
                 new 
StringReader("<elem>text</elem>")).getDocumentElement(true);
-        // TODO: need to get the OMFactory again because for DOOM, it is 
stateful
-        OMDocument document = metaFactory.getOMFactory().createOMDocument();
+        OMDocument document = factory.createOMDocument();
         OMElement root = factory.createOMElement("root", null, document);
         root.addChild(incompleteElement);
         StringWriter out = new StringWriter();