Author: veithen
Date: Thu Apr 12 20:46:13 2012
New Revision: 1325504

URL: http://svn.apache.org/viewvc?rev=1325504&view=rev
Log:
AXIOM-69: Ensure that the test case actually tests what it is expected to test 
(a relevant part of it was actually unreachable code).

Modified:
    
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/omdom/document/TestImportNode.java

Modified: 
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/omdom/document/TestImportNode.java
URL: 
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/omdom/document/TestImportNode.java?rev=1325504&r1=1325503&r2=1325504&view=diff
==============================================================================
--- 
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/omdom/document/TestImportNode.java
 (original)
+++ 
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/omdom/document/TestImportNode.java
 Thu Apr 12 20:46:13 2012
@@ -71,10 +71,10 @@ public class TestImportNode extends Axio
             NamedNodeMap map = ele.getAttributes();
             Iterator attIterator = omele.getAllAttributes();
             OMAttribute omattribute;
-            while (attIterator != null && attIterator.hasNext() && map == 
null) {
+            while (attIterator.hasNext()) {
                 omattribute = (OMAttribute) attIterator.next();
                 Node node = map.getNamedItemNS(
-                        omattribute.getNamespace().getNamespaceURI(),
+                        omattribute.getNamespaceURI(),
                         omattribute.getLocalName());
                 if (node.getNodeType() == Node.ATTRIBUTE_NODE) {
                     Attr attr = (Attr) node;


Reply via email to