Author: veithen
Date: Sat Jan 22 13:54:00 2011
New Revision: 1062155
URL: http://svn.apache.org/viewvc?rev=1062155&view=rev
Log:
Improved the SOAPFault tests so that the assertions about child elements not
only compare local names, but QNames.
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/SOAPSpec.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetCode.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetCodeWithParser.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetDetail.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetDetailWithParser.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetReason.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetReasonWithParser.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetRole.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetRoleWithParser.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetCode.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetDetail.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetReason.java
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetRole.java
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/SOAPSpec.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/SOAPSpec.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/SOAPSpec.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/SOAPSpec.java
Sat Jan 22 13:54:00 2011
@@ -18,6 +18,8 @@
*/
package org.apache.axiom.ts.soap;
+import javax.xml.namespace.QName;
+
import org.apache.axiom.om.OMMetaFactory;
import org.apache.axiom.soap.SOAP11Constants;
import org.apache.axiom.soap.SOAP11Version;
@@ -47,22 +49,6 @@ public abstract class SOAPSpec {
public String getEnvelopeNamespaceURI() {
return SOAP11Constants.SOAP_ENVELOPE_NAMESPACE_URI;
}
-
- public String getFaultCodeLocalName() {
- return SOAP11Constants.SOAP_FAULT_CODE_LOCAL_NAME;
- }
-
- public String getFaultReasonLocalName() {
- return SOAP11Constants.SOAP_FAULT_STRING_LOCAL_NAME;
- }
-
- public String getFaultRoleLocalName() {
- return SOAP11Constants.SOAP_FAULT_ACTOR_LOCAL_NAME;
- }
-
- public String getFaultDetailLocalName() {
- return SOAP11Constants.SOAP_FAULT_DETAIL_LOCAL_NAME;
- }
};
public static final SOAPSpec SOAP12 = new
SOAPSpec(SOAP12Version.getSingleton()) {
@@ -81,22 +67,6 @@ public abstract class SOAPSpec {
public String getEnvelopeNamespaceURI() {
return SOAP12Constants.SOAP_ENVELOPE_NAMESPACE_URI;
}
-
- public String getFaultCodeLocalName() {
- return SOAP12Constants.SOAP_FAULT_CODE_LOCAL_NAME;
- }
-
- public String getFaultReasonLocalName() {
- return SOAP12Constants.SOAP_FAULT_REASON_LOCAL_NAME;
- }
-
- public String getFaultRoleLocalName() {
- return SOAP12Constants.SOAP_FAULT_ROLE_LOCAL_NAME;
- }
-
- public String getFaultDetailLocalName() {
- return SOAP12Constants.SOAP_FAULT_DETAIL_LOCAL_NAME;
- }
};
private final SOAPVersion version;
@@ -109,10 +79,22 @@ public abstract class SOAPSpec {
public abstract SOAPFactory getFactory(OMMetaFactory metaFactory);
public abstract SOAPFactory getAltFactory(OMMetaFactory metaFactory);
public abstract String getEnvelopeNamespaceURI();
- public abstract String getFaultCodeLocalName();
- public abstract String getFaultReasonLocalName();
- public abstract String getFaultRoleLocalName();
- public abstract String getFaultDetailLocalName();
+
+ public QName getFaultCodeQName() {
+ return version.getFaultCodeQName();
+ }
+
+ public QName getFaultReasonQName() {
+ return version.getFaultReasonQName();
+ }
+
+ public QName getFaultRoleQName() {
+ return version.getFaultRoleQName();
+ }
+
+ public QName getFaultDetailQName() {
+ return version.getFaultDetailQName();
+ }
public String getNextRoleURI() {
return version.getNextRoleURI();
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetCode.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetCode.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetCode.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetCode.java
Sat Jan 22 13:54:00 2011
@@ -38,6 +38,6 @@ public class TestGetCode extends SOAPTes
"Fault Test:- After calling setCode method, Fault has no code",
soapFault.getCode());
assertEquals("Fault Test:- Fault code local name mismatch",
- spec.getFaultCodeLocalName(),
soapFault.getCode().getLocalName());
+ spec.getFaultCodeQName(), soapFault.getCode().getQName());
}
}
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetCodeWithParser.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetCodeWithParser.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetCodeWithParser.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetCodeWithParser.java
Sat Jan 22 13:54:00 2011
@@ -35,6 +35,6 @@ public class TestGetCodeWithParser exten
soapFaultWithParser.getCode());
assertEquals(
"Fault Test with parser: - Fault code local name mismatch",
- spec.getFaultCodeLocalName(),
soapFaultWithParser.getCode().getLocalName());
+ spec.getFaultCodeQName(),
soapFaultWithParser.getCode().getQName());
}
}
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetDetail.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetDetail.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetDetail.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetDetail.java
Sat Jan 22 13:54:00 2011
@@ -38,6 +38,6 @@ public class TestGetDetail extends SOAPT
"Fault Test:- After calling setDetail method, Fault has no
detail",
soapFault.getDetail());
assertEquals("Fault Test:- Fault detail local name mismatch",
- spec.getFaultDetailLocalName(),
soapFault.getDetail().getLocalName());
+ spec.getFaultDetailQName(), soapFault.getDetail().getQName());
}
}
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetDetailWithParser.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetDetailWithParser.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetDetailWithParser.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetDetailWithParser.java
Sat Jan 22 13:54:00 2011
@@ -35,6 +35,6 @@ public class TestGetDetailWithParser ext
soapFaultWithParser.getDetail());
assertEquals(
"Fault Test with parser: - Fault detail local name mismatch",
- spec.getFaultDetailLocalName(),
soapFaultWithParser.getDetail().getLocalName());
+ spec.getFaultDetailQName(),
soapFaultWithParser.getDetail().getQName());
}
}
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetReason.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetReason.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetReason.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetReason.java
Sat Jan 22 13:54:00 2011
@@ -38,6 +38,6 @@ public class TestGetReason extends SOAPT
"Fault Test:- After calling setReason method, Fault has no
reason",
soapFault.getReason());
assertEquals("Fault Test:- Fault reason local name mismatch",
- spec.getFaultReasonLocalName(),
soapFault.getReason().getLocalName());
+ spec.getFaultReasonQName(), soapFault.getReason().getQName());
}
}
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetReasonWithParser.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetReasonWithParser.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetReasonWithParser.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetReasonWithParser.java
Sat Jan 22 13:54:00 2011
@@ -35,6 +35,6 @@ public class TestGetReasonWithParser ext
soapFaultWithParser.getReason());
assertEquals(
"Fault Test with parser: - Fault reason local name mismatch",
- spec.getFaultReasonLocalName(),
soapFaultWithParser.getReason().getLocalName());
+ spec.getFaultReasonQName(),
soapFaultWithParser.getReason().getQName());
}
}
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetRole.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetRole.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetRole.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetRole.java
Sat Jan 22 13:54:00 2011
@@ -38,6 +38,6 @@ public class TestGetRole extends SOAPTes
"Fault Test:- After calling setRole method, Fault has no role",
soapFault.getRole());
assertEquals("Fault Test:- Fault role local name mismatch",
- spec.getFaultRoleLocalName(),
soapFault.getRole().getLocalName());
+ spec.getFaultRoleQName(), soapFault.getRole().getQName());
}
}
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetRoleWithParser.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetRoleWithParser.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetRoleWithParser.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestGetRoleWithParser.java
Sat Jan 22 13:54:00 2011
@@ -35,6 +35,6 @@ public class TestGetRoleWithParser exten
soapFaultWithParser.getRole());
assertEquals(
"Fault Test with parser: - Fault role local name mismatch",
- spec.getFaultRoleLocalName(),
soapFaultWithParser.getRole().getLocalName());
+ spec.getFaultRoleQName(),
soapFaultWithParser.getRole().getQName());
}
}
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetCode.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetCode.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetCode.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetCode.java
Sat Jan 22 13:54:00 2011
@@ -35,7 +35,7 @@ public class TestSetCode extends SOAPTes
"Fault Test:- After calling setCode method, Fault has no code",
soapFault.getCode());
assertEquals("Fault Test:- Code local name mismatch",
- spec.getFaultCodeLocalName(),
soapFault.getCode().getLocalName());
+ spec.getFaultCodeQName(), soapFault.getCode().getQName());
try {
soapFault.setCode(altSoapFactory.createSOAPFaultCode());
fail("SOAPFaultCode should not be set in to a SOAPFault for a
different SOAP version");
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetDetail.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetDetail.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetDetail.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetDetail.java
Sat Jan 22 13:54:00 2011
@@ -35,7 +35,7 @@ public class TestSetDetail extends SOAPT
"Fault Test:- After calling setDetail method, Fault has no
detail",
soapFault.getDetail());
assertEquals("Fault Test:- Fault detail local name mismatch",
- spec.getFaultDetailLocalName(),
soapFault.getDetail().getLocalName());
+ spec.getFaultDetailQName(), soapFault.getDetail().getQName());
try {
soapFault.setDetail(altSoapFactory.createSOAPFaultDetail());
fail("SOAPFaultDetail should not be set in to a SOAPFault for a
different SOAP version");
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetReason.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetReason.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetReason.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetReason.java
Sat Jan 22 13:54:00 2011
@@ -35,7 +35,7 @@ public class TestSetReason extends SOAPT
"Fault Test:- After calling setReason method, Fault has no
reason",
soapFault.getReason());
assertEquals("Fault Test:- Fault reason local name mismatch",
- spec.getFaultReasonLocalName(),
soapFault.getReason().getLocalName());
+ spec.getFaultReasonQName(), soapFault.getReason().getQName());
try {
soapFault.setReason(altSoapFactory.createSOAPFaultReason());
fail("SOAPFaultReason should not be set in to a SOAPFault for a
different SOAP version");
Modified:
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetRole.java
URL:
http://svn.apache.org/viewvc/webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetRole.java?rev=1062155&r1=1062154&r2=1062155&view=diff
==============================================================================
---
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetRole.java
(original)
+++
webservices/commons/trunk/modules/axiom/modules/axiom-testsuite/src/main/java/org/apache/axiom/ts/soap/fault/TestSetRole.java
Sat Jan 22 13:54:00 2011
@@ -35,7 +35,7 @@ public class TestSetRole extends SOAPTes
"Fault Test:- After calling setRole method, Fault has no role",
soapFault.getRole());
assertEquals("Fault Test:- Fault role local name mismatch",
- spec.getFaultRoleLocalName(),
soapFault.getRole().getLocalName());
+ spec.getFaultRoleQName(), soapFault.getRole().getQName());
try {
soapFault.setRole(altSoapFactory.createSOAPFaultRole());
fail("SOAPFaultRole should not be set in to a SOAPFault for a
different SOAP version");