Author: kiwiwings
Date: Sun Feb  6 03:18:13 2022
New Revision: 1897797

URL: http://svn.apache.org/viewvc?rev=1897797&view=rev
Log:
XMLBEANS-596 - Workaround JDK-8139836 (only  happening in JDK8 and seems to be 
fixed in 1.8.0_312)
replace a few more try, fail, catch constructs with assertThrows

Modified:
    xmlbeans/trunk/src/test/java/dom/checkin/AttributeTest.java
    xmlbeans/trunk/src/test/java/dom/checkin/CommentTest.java
    xmlbeans/trunk/src/test/java/dom/checkin/DocumentTest.java
    xmlbeans/trunk/src/test/java/dom/checkin/ElementTest.java
    xmlbeans/trunk/src/test/java/dom/checkin/TextTest.java
    xmlbeans/trunk/src/test/java/dom/common/CharacterDataTest.java
    xmlbeans/trunk/src/test/java/dom/common/NodeWithChildrenTest.java

Modified: xmlbeans/trunk/src/test/java/dom/checkin/AttributeTest.java
URL: 
http://svn.apache.org/viewvc/xmlbeans/trunk/src/test/java/dom/checkin/AttributeTest.java?rev=1897797&r1=1897796&r2=1897797&view=diff
==============================================================================
--- xmlbeans/trunk/src/test/java/dom/checkin/AttributeTest.java (original)
+++ xmlbeans/trunk/src/test/java/dom/checkin/AttributeTest.java Sun Feb  6 
03:18:13 2022
@@ -19,7 +19,6 @@ package dom.checkin;
 
 import dom.common.DomUtils;
 import dom.common.NodeWithChildrenTest;
-import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.w3c.dom.*;
@@ -30,11 +29,12 @@ import static org.junit.jupiter.api.Asse
 public class AttributeTest extends NodeWithChildrenTest {
 
     public AttributeTest() {
-        String sDTD = "<?xml version=\"1.0\"?>" +
-                      "<!DOCTYPE foodoc [" +
-                      "<!ELEMENT foo>" +
-                      "<!ATTLIST foo at_spec CDATA \"0\">" +
-                      "]>";
+        String sDTD =
+            "<?xml version=\"1.0\"?>" +
+            "<!DOCTYPE foodoc [" +
+            "<!ELEMENT foo>" +
+            "<!ATTLIST foo at_spec CDATA \"0\">" +
+            "]>";
 
         sXml = "<foo xmlns:extra=\"bea.org\" xmlns:myns=\"uri:foo\" 
at0=\"val0\" myns:at0=\"val01\" at2=\"val2\" at3=\"val3\" at4=\"val4\">some 
text</foo>";
         if (bDTD) {
@@ -101,22 +101,18 @@ public class AttributeTest extends NodeW
     }
 
     @Test
-    protected void testAppendChild() {
+    public void testAppendChild() {
         //elt
-        Node newChild = m_doc.createElement("foo");
-        try {
-            m_node.appendChild(newChild);
-            Assertions.fail("Cannot append an element children to attributes " 
+
-                            m_node.getChildNodes().getLength());
-        } catch (DOMException de) {
-            assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
-        }
+        Node newChild1 = m_doc.createElement("foo");
+        DOMException de = assertThrows(DOMException.class, () -> 
m_node.appendChild(newChild1),
+            "Cannot append an element children to attributes ");
+        assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
 
-        newChild = m_doc.createTextNode("foobar");
-        m_node.appendChild(newChild);
+        Node newChild2 = m_doc.createTextNode("foobar");
+        m_node.appendChild(newChild2);
         assertEquals(2, m_node.getChildNodes().getLength());
 
-//TODO
+        // TODO
         m_node.normalize();
         assertEquals(1, m_node.getChildNodes().getLength());
         assertEquals("val01foobar", ((Text) m_node.getFirstChild()).getData());
@@ -182,20 +178,16 @@ public class AttributeTest extends NodeW
     }
 
     @Test
-    protected void testInsertBefore() {
-        Node newChild = m_doc.createElement("foo");
+    public void testInsertBefore() {
+        Node newChild1 = m_doc.createElement("foo");
         assertEquals(1, m_node.getChildNodes().getLength());
 
         Node textNode = m_node.getFirstChild();
+        DOMException de = assertThrows(DOMException.class, () -> 
m_node.insertBefore(newChild1, textNode));
+        assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
 
-        try {
-            m_node.insertBefore(newChild, textNode);
-        } catch (DOMException de) {
-            assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
-        }
-
-        newChild = m_doc.createTextNode("foo");
-        m_node.insertBefore(newChild, textNode);
+        Node newChild2 = m_doc.createTextNode("foo");
+        m_node.insertBefore(newChild2, textNode);
         assertEquals("foo", m_node.getFirstChild().getNodeValue());
         assertEquals("val01", m_node.getLastChild().getNodeValue());
     }
@@ -211,23 +203,17 @@ public class AttributeTest extends NodeW
 
     @Test
     void testReplaceChild() {
-
         //assertFalse(m_node.hasChildNodes());
-        Node newChild = m_doc.createElement("foo");
-        assertEquals(1, m_node.getChildNodes().getLength());
-        try {
-            m_node.replaceChild(newChild, m_node.getFirstChild());
-            Assertions.fail("can not put an element under an attr");
-        } catch (DOMException de) {
-            assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
-        }
-        newChild = m_doc.createTextNode("realnewval");
+        Node newChild1 = m_doc.createElement("foo");
         assertEquals(1, m_node.getChildNodes().getLength());
-        m_node.replaceChild(newChild, m_node.getFirstChild());
-        if (!"realnewval".equals(((Attr) m_node).getValue())) {
-            Assertions.fail(" Expected realnewval but got " + ((Attr) 
m_node).getValue());
-        }
+        DOMException de = assertThrows(DOMException.class, () -> 
m_node.replaceChild(newChild1, m_node.getFirstChild()),
+            "can not put an element under an attr");
+        assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
 
+        Node newChild2 = m_doc.createTextNode("realnewval");
+        assertEquals(1, m_node.getChildNodes().getLength());
+        m_node.replaceChild(newChild2, m_node.getFirstChild());
+        assertEquals("realnewval", ((Attr) m_node).getValue());
     }
 
     @Test
@@ -274,7 +260,7 @@ public class AttributeTest extends NodeW
     }
 
     @Test
-    protected void testInsertBeforeDocFrag() {
+    public void testInsertBeforeDocFrag() {
         DocumentFragment child = m_doc.createDocumentFragment();
         child.appendChild(m_doc.createTextNode("foo1"));
         Node target = m_node.getFirstChild();
@@ -282,54 +268,41 @@ public class AttributeTest extends NodeW
     }
 
     @Test
-    protected void testAppendChildDocFrag() {
+    public void testAppendChildDocFrag() {
         DocumentFragment child = m_doc.createDocumentFragment();
         child.appendChild(m_doc.createTextNode("foo"));
         super.testAppendChild(child);
     }
 
     @Test
-    protected void testReplaceChildDocFrag() {
-
+    public void testReplaceChildDocFrag() {
         DocumentFragment child = m_doc.createDocumentFragment();
         child.appendChild(m_doc.createElement("foo"));
         child.appendChild(m_doc.createElement("foobar"));
         Node toReplace = m_node.getFirstChild();
-        try {
-            super.testReplaceChild(child, toReplace);
-            Assertions.fail("cannot insert element in attr");
-        } catch (DOMException de) {
-            assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
-        }
-
+        DOMException de = assertThrows(DOMException.class, () -> 
super.testReplaceChild(child, toReplace), "cannot insert element in attr");
+        assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
     }
 
     @Test
-    protected void testInsertBeforeNullTarget() {
-        Node child = m_doc.createElementNS("org.foo.www", "foonode");
-        try {
-            super.testInsertBefore(child, null);
-            Assertions.fail("cannot insert element in attr");
-        } catch (DOMException de) {
-            assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
-        }
-        child = m_doc.createTextNode("foonode");
-        super.testInsertBefore(child, null);
+    public void testInsertBeforeNullTarget() {
+        Node child1 = m_doc.createElementNS("org.foo.www", "foonode");
+        DOMException de = assertThrows(DOMException.class, () -> 
super.testInsertBefore(child1, null), "cannot insert element in attr");
+        assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
+        Node child2 = m_doc.createTextNode("foonode");
+        super.testInsertBefore(child2, null);
     }
 
     @Test
     void testInsertExistingNode() {
-        Node toInsert = m_doc.getFirstChild();
+        Node toInsert1 = m_doc.getFirstChild();
         //elt under attr
-        try {
-            super.testInsertExistingNode(toInsert);
-            Assertions.fail("Shouldn't work for attrs");
-        } catch (DOMException de) {
-            assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
-        }
-        toInsert = m_doc.getFirstChild().getFirstChild(); //some text
+        DOMException de = assertThrows(DOMException.class, () -> 
super.testInsertExistingNode(toInsert1), "Shouldn't work for attrs");
+        assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
 
-        super.testInsertBefore(toInsert, m_node.getFirstChild());
+        //some text
+        Node toInsert2 = m_doc.getFirstChild().getFirstChild();
+        super.testInsertBefore(toInsert2, m_node.getFirstChild());
         assertEquals("some text", m_node.getFirstChild().getNodeValue());
         assertEquals(2, m_node.getChildNodes().getLength());
     }
@@ -372,13 +345,8 @@ public class AttributeTest extends NodeW
     void testInsertBeforeInvalidRefNode() {
         Node child = m_doc.createTextNode("foonode");
         Node target = m_doc.createElement("foo");
-        try {
-            super.testInsertBefore(child, target);
-            Assertions.fail("Insert cannot happen");
-        } catch (DOMException de) {
-            System.err.println(de.getMessage() + " " + de.code);
-            assertEquals(DOMException.NOT_FOUND_ERR, de.code);
-        }
+        DOMException de = assertThrows(DOMException.class, () -> 
super.testInsertBefore(child, target), "Insert cannot happen");
+        assertEquals(DOMException.NOT_FOUND_ERR, de.code);
     }
 
     @Test
@@ -387,12 +355,8 @@ public class AttributeTest extends NodeW
         assertNull(at.getPrefix(), "L1 prefix null");
         assertNull(at.getLocalName(), "L1 LocalName null");
         assertNull(at.getNamespaceURI(), "L1 Uri null");
-        try {
-            at.setPrefix("foo");
-            Assertions.fail("L1 prefix null");
-        } catch (DOMException de) {
-            assertEquals(DOMException.NAMESPACE_ERR, de.code);
-        }
+        DOMException de = assertThrows(DOMException.class, () -> 
at.setPrefix("foo"), "L1 prefix null");
+        assertEquals(DOMException.NAMESPACE_ERR, de.code);
     }
 
     @Test

Modified: xmlbeans/trunk/src/test/java/dom/checkin/CommentTest.java
URL: 
http://svn.apache.org/viewvc/xmlbeans/trunk/src/test/java/dom/checkin/CommentTest.java?rev=1897797&r1=1897796&r2=1897797&view=diff
==============================================================================
--- xmlbeans/trunk/src/test/java/dom/checkin/CommentTest.java (original)
+++ xmlbeans/trunk/src/test/java/dom/checkin/CommentTest.java Sun Feb  6 
03:18:13 2022
@@ -24,6 +24,7 @@ import org.w3c.dom.Text;
 import xmlcursor.common.Common;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNull;
 
 
 public class CommentTest extends CharacterDataTest {
@@ -61,7 +62,7 @@ public class CommentTest extends Charact
     @Test
     void testPreviousSibling() {
         Node prSibling = m_node.getPreviousSibling();
-        assertEquals(null, prSibling);
+        assertNull(prSibling);
     }
 
     @Test

Modified: xmlbeans/trunk/src/test/java/dom/checkin/DocumentTest.java
URL: 
http://svn.apache.org/viewvc/xmlbeans/trunk/src/test/java/dom/checkin/DocumentTest.java?rev=1897797&r1=1897796&r2=1897797&view=diff
==============================================================================
--- xmlbeans/trunk/src/test/java/dom/checkin/DocumentTest.java (original)
+++ xmlbeans/trunk/src/test/java/dom/checkin/DocumentTest.java Sun Feb  6 
03:18:13 2022
@@ -105,19 +105,19 @@ public class DocumentTest extends NodeWi
     }
 
     @Test
-    protected void testAppendChild() {
+    public void testAppendChild() {
         DOMException de = assertThrows(DOMException.class, 
super::testAppendChild);
         assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
     }
 
     @Test
-    protected void testInsertBefore() {
+    public void testInsertBefore() {
         DOMException de = assertThrows(DOMException.class, 
super::testInsertBefore);
         assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
     }
 
     @Test
-    protected void testInsertBeforeNullTarget() {
+    public void testInsertBeforeNullTarget() {
         DOMException de = assertThrows(DOMException.class, 
super::testInsertBeforeNullTarget);
         assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
     }
@@ -191,13 +191,13 @@ public class DocumentTest extends NodeWi
     }
 
     @Test
-    protected void testInsertBeforeDocFrag() {
+    public void testInsertBeforeDocFrag() {
         DOMException de = assertThrows(DOMException.class, 
super::testInsertBeforeDocFrag);
         assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
     }
 
     @Test
-    protected void testAppendChildDocFrag() {
+    public void testAppendChildDocFrag() {
         DOMException de = assertThrows(DOMException.class, 
super::testAppendChildDocFrag);
         assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
     }
@@ -220,7 +220,7 @@ public class DocumentTest extends NodeWi
     }
 
     @Test
-    protected void testReplaceChildDocFrag() {
+    public void testReplaceChildDocFrag() {
         DOMException de = assertThrows(DOMException.class, 
super::testReplaceChildDocFrag);
         assertEquals(DOMException.HIERARCHY_REQUEST_ERR, de.code);
     }

Modified: xmlbeans/trunk/src/test/java/dom/checkin/ElementTest.java
URL: 
http://svn.apache.org/viewvc/xmlbeans/trunk/src/test/java/dom/checkin/ElementTest.java?rev=1897797&r1=1897796&r2=1897797&view=diff
==============================================================================
--- xmlbeans/trunk/src/test/java/dom/checkin/ElementTest.java (original)
+++ xmlbeans/trunk/src/test/java/dom/checkin/ElementTest.java Sun Feb  6 
03:18:13 2022
@@ -355,31 +355,21 @@ public class ElementTest extends NodeWit
     @Test
     void testRemoveAttributeNode_DNE() {
         //DNE
-        Node removed;
-        Attr remove = ((Element) m_node).getAttributeNode("at3");
-        try {
-            removed = ((Element) m_node).removeAttributeNode(remove);
-            Assertions.fail("removing Non existing attr");
-        } catch (DOMException de) {
-            assertEquals(DOMException.NOT_FOUND_ERR, de.code);
-        }
+        Attr remove1 = ((Element) m_node).getAttributeNode("at3");
+        DOMException de1 = assertThrows(DOMException.class, () -> ((Element) 
m_node).removeAttributeNode(remove1),
+            "removing Non existing attr");
+        assertEquals(DOMException.NOT_FOUND_ERR, de1.code);
 
-        remove = null;
-        try {
-            removed = ((Element) m_node).removeAttributeNode(remove);
-            Assertions.fail("removing Non existing attr");
-        } catch (DOMException de) {
-            assertEquals(DOMException.NOT_FOUND_ERR, de.code);
-        }
+        Attr remove2 = null;
+        DOMException de2 = assertThrows(DOMException.class, () -> ((Element) 
m_node).removeAttributeNode(remove2),
+            "removing Non existing attr");
+        assertEquals(DOMException.NOT_FOUND_ERR, de2.code);
 
         //differentParent
-        remove = m_doc.getDocumentElement().getAttributeNode("attr0");
-        try {
-            removed = ((Element) m_node).removeAttributeNode(remove);
-            Assertions.fail("removing Non existing attr");
-        } catch (DOMException de) {
-            assertEquals(DOMException.NOT_FOUND_ERR, de.code);
-        }
+        Attr remove3 = m_doc.getDocumentElement().getAttributeNode("attr0");
+        DOMException de3 = assertThrows(DOMException.class, () -> ((Element) 
m_node).removeAttributeNode(remove3),
+            "removing Non existing attr");
+        assertEquals(DOMException.NOT_FOUND_ERR, de3.code);
     }
 
     @Test

Modified: xmlbeans/trunk/src/test/java/dom/checkin/TextTest.java
URL: 
http://svn.apache.org/viewvc/xmlbeans/trunk/src/test/java/dom/checkin/TextTest.java?rev=1897797&r1=1897796&r2=1897797&view=diff
==============================================================================
--- xmlbeans/trunk/src/test/java/dom/checkin/TextTest.java (original)
+++ xmlbeans/trunk/src/test/java/dom/checkin/TextTest.java Sun Feb  6 03:18:13 
2022
@@ -18,7 +18,6 @@ package dom.checkin;
 
 
 import dom.common.CharacterDataTest;
-import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.w3c.dom.DOMException;
@@ -27,6 +26,7 @@ import org.w3c.dom.Text;
 import xmlcursor.common.Common;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
 
 
 public class TextTest extends CharacterDataTest {
@@ -72,24 +72,14 @@ public class TextTest extends CharacterD
 
     @Test
     void testSplitTextNegative() {
-        try {
-            ((Text) m_node).splitText(-1);
-            Assertions.fail("Deleting OOB chars");
-        }
-        catch (DOMException de) {
-            assertEquals(de.code, DOMException.INDEX_SIZE_ERR);
-        }
+        DOMException de = assertThrows(DOMException.class, () -> ((Text) 
m_node).splitText(-1), "Deleting OOB chars");
+        assertEquals(de.code, DOMException.INDEX_SIZE_ERR);
     }
 
     @Test
     void testSplitTextLarge() {
-        try {
-            ((Text) m_node).splitText(((Text) m_node).getLength() + 1);
-            Assertions.fail("Deleting OOB chars");
-        }
-        catch (DOMException de) {
-            assertEquals(de.code, DOMException.INDEX_SIZE_ERR);
-        }
+        DOMException de = assertThrows(DOMException.class, () -> ((Text) 
m_node).splitText(((Text) m_node).getLength() + 1), "Deleting OOB chars");
+        assertEquals(de.code, DOMException.INDEX_SIZE_ERR);
     }
 
     @Test

Modified: xmlbeans/trunk/src/test/java/dom/common/CharacterDataTest.java
URL: 
http://svn.apache.org/viewvc/xmlbeans/trunk/src/test/java/dom/common/CharacterDataTest.java?rev=1897797&r1=1897796&r2=1897797&view=diff
==============================================================================
--- xmlbeans/trunk/src/test/java/dom/common/CharacterDataTest.java (original)
+++ xmlbeans/trunk/src/test/java/dom/common/CharacterDataTest.java Sun Feb  6 
03:18:13 2022
@@ -23,8 +23,7 @@ import org.w3c.dom.CharacterData;
 import org.w3c.dom.DOMException;
 import org.w3c.dom.Node;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.*;
 
 @Disabled
 public abstract class CharacterDataTest extends NodeTest {
@@ -125,24 +124,20 @@ public abstract class CharacterDataTest
     private void _testDeleteData(int offset, int count) {
         String sData = ((CharacterData) m_node).getData();
         int nDataLen = sData.length();
-        if (offset < 0 || offset > nDataLen || count < 0)
-            try {
-                ((CharacterData) m_node).deleteData(offset, count);
-                Assertions.fail("Deleting OOB chars");
-            } catch (DOMException de) {
-                assertEquals(de.code, DOMException.INDEX_SIZE_ERR);
-            }
-        else {
+        if (offset < 0 || offset > nDataLen || count < 0) {
+            DOMException de = assertThrows(DOMException.class, () -> 
((CharacterData) m_node).deleteData(offset, count), "Deleting OOB chars");
+            assertEquals(de.code, DOMException.INDEX_SIZE_ERR);
+        } else {
             ((CharacterData) m_node).deleteData(offset, count);
-            if (count == 0)
+            if (count == 0) {
                 assertEquals(sData, ((CharacterData) m_node).getData());
-            else if (offset + count == nDataLen || (offset + count) > nDataLen)
+            } else if (offset + count == nDataLen || (offset + count) > 
nDataLen) {
                 assertEquals("", ((CharacterData) m_node).getData());
-            else if (offset == 0) {
+            } else if (offset == 0) {
                 assertEquals(sData.substring(count), ((CharacterData) 
m_node).getData());
-
-            } else
+            } else {
                 assertEquals(sData.substring(0, offset) + 
sData.substring(offset + count, sData.length() - (offset + count)), 
((CharacterData) m_node).getData());
+            }
         }
     }
 
@@ -270,27 +265,23 @@ public abstract class CharacterDataTest
     private void _testReplaceData(int offset, int count, String newStr) {
         String sData = ((CharacterData) m_node).getData();
         int nDataLen = sData.length();
-        if (offset < 0 || offset > nDataLen || count < 0)
-            try {
-                ((CharacterData) m_node).replaceData(offset, count, newStr);
-                Assertions.fail("Deleting OOB chars");
-            } catch (DOMException de) {
-                assertEquals(de.code, DOMException.INDEX_SIZE_ERR);
-            }
-        else {
+        if (offset < 0 || offset > nDataLen || count < 0) {
+            DOMException de = assertThrows(DOMException.class, () -> 
((CharacterData) m_node).replaceData(offset, count, newStr), "Deleting OOB 
chars");
+            assertEquals(de.code, DOMException.INDEX_SIZE_ERR);
+        } else {
             ((CharacterData) m_node).replaceData(offset, count, newStr);
-            if (count == 0)
+            if (count == 0) {
                 assertEquals(sData, ((CharacterData) m_node).getData());
-            else if (newStr == null)
+            } else if (newStr == null) {
                 assertEquals("", ((CharacterData) m_node).getData());
-            else if (offset + count == nDataLen || (offset + count) > 
nDataLen) {
+            } else if (offset + count == nDataLen || (offset + count) > 
nDataLen) {
                 String sOld = sData.substring(0, offset);
                 assertEquals(sOld + newStr, ((CharacterData) 
m_node).getData());
-
-            } else if (offset == 0)
+            } else if (offset == 0) {
                 assertEquals(sData.substring(count, sData.length() - count) + 
newStr, ((CharacterData) m_node).getData());
-            else
+            } else {
                 assertEquals(sData.substring(0, offset) + newStr + 
sData.substring(offset + count), ((CharacterData) m_node).getData());
+            }
         }
 
     }
@@ -356,12 +347,9 @@ public abstract class CharacterDataTest
 
     @Test
     protected void testSetPrefix() {
-        try {
-            m_node.setPrefix("foobar");
-            Assertions.fail("Can't set prefix on node other than Element or 
Attribute");
-        } catch (DOMException de) {
-            assertEquals(DOMException.NAMESPACE_ERR, de.code);
-        }
+        DOMException de = assertThrows(DOMException.class, () -> 
m_node.setPrefix("foobar"),
+            "Can't set prefix on node other than Element or Attribute");
+        assertEquals(DOMException.NAMESPACE_ERR, de.code);
     }
 }
 

Modified: xmlbeans/trunk/src/test/java/dom/common/NodeWithChildrenTest.java
URL: 
http://svn.apache.org/viewvc/xmlbeans/trunk/src/test/java/dom/common/NodeWithChildrenTest.java?rev=1897797&r1=1897796&r2=1897797&view=diff
==============================================================================
--- xmlbeans/trunk/src/test/java/dom/common/NodeWithChildrenTest.java (original)
+++ xmlbeans/trunk/src/test/java/dom/common/NodeWithChildrenTest.java Sun Feb  
6 03:18:13 2022
@@ -160,7 +160,7 @@ public abstract class NodeWithChildrenTe
     }
 
     @Test
-    protected void testReplaceChildDocFrag() {
+    public void testReplaceChildDocFrag() {
         DocumentFragment child = m_doc.createDocumentFragment();
         child.appendChild(m_doc.createElement("foo"));
         child.appendChild(m_doc.createElement("foobar"));
@@ -169,7 +169,7 @@ public abstract class NodeWithChildrenTe
     }
 
     @Test
-    protected void testInsertBefore() {
+    public void testInsertBefore() {
         Node target = m_node.getFirstChild();
         Node child = m_doc.createElementNS("org.foo.www", "foonode");
         assertNotNull(target);
@@ -177,7 +177,7 @@ public abstract class NodeWithChildrenTe
     }
 
     @Test
-    protected void testInsertBeforeNullTarget() {
+    public void testInsertBeforeNullTarget() {
         Node child = m_doc.createElementNS("org.foo.www", "foonode");
         super.testInsertBefore(child, null);
     }
@@ -252,7 +252,7 @@ public abstract class NodeWithChildrenTe
     }
 
     @Test
-    protected void testInsertBeforeDocFrag() {
+    public void testInsertBeforeDocFrag() {
         DocumentFragment child = m_doc.createDocumentFragment();
         child.appendChild(m_doc.createElement("foo1"));
         Node target = m_node.getFirstChild();
@@ -260,7 +260,7 @@ public abstract class NodeWithChildrenTe
     }
 
     @Test
-    protected void testAppendChild() {
+    public void testAppendChild() {
         Node newNode = m_doc.createElement("foo");
         super.testAppendChild(newNode);
     }
@@ -284,7 +284,7 @@ public abstract class NodeWithChildrenTe
 
     //append doc frag
     @Test
-    protected void testAppendChildDocFrag() {
+    public void testAppendChildDocFrag() {
         DocumentFragment child = m_doc.createDocumentFragment();
         child.appendChild(m_doc.createElement("foo"));
         super.testAppendChild(child);



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to