You are viewing a plain text version of this content. The canonical link for it is here.
Posted to java-commits@axis.apache.org by ve...@apache.org on 2017/12/25 19:58:15 UTC

svn commit: r1819260 - /axis/axis2/java/core/trunk/modules/jaxws-integration/test/org/apache/axis2/jaxws/provider/soapbinding/tests/SOAPBindingProviderTests.java

Author: veithen
Date: Mon Dec 25 19:58:14 2017
New Revision: 1819260

URL: http://svn.apache.org/viewvc?rev=1819260&view=rev
Log:
Eliminate an instance of the catch-and-fail testing anti-pattern.

Modified:
    axis/axis2/java/core/trunk/modules/jaxws-integration/test/org/apache/axis2/jaxws/provider/soapbinding/tests/SOAPBindingProviderTests.java

Modified: axis/axis2/java/core/trunk/modules/jaxws-integration/test/org/apache/axis2/jaxws/provider/soapbinding/tests/SOAPBindingProviderTests.java
URL: http://svn.apache.org/viewvc/axis/axis2/java/core/trunk/modules/jaxws-integration/test/org/apache/axis2/jaxws/provider/soapbinding/tests/SOAPBindingProviderTests.java?rev=1819260&r1=1819259&r2=1819260&view=diff
==============================================================================
--- axis/axis2/java/core/trunk/modules/jaxws-integration/test/org/apache/axis2/jaxws/provider/soapbinding/tests/SOAPBindingProviderTests.java (original)
+++ axis/axis2/java/core/trunk/modules/jaxws-integration/test/org/apache/axis2/jaxws/provider/soapbinding/tests/SOAPBindingProviderTests.java Mon Dec 25 19:58:14 2017
@@ -75,47 +75,34 @@ public class SOAPBindingProviderTests ex
         return getTestSetup(new TestSuite(SOAPBindingProviderTests.class));
     }
     
-    public void testSoap11Request(){
-        try{
-            System.out.println("---------------------------------------");
-            System.out.println("test: " + getName());
-            
-            Dispatch<SOAPMessage> dispatch=getDispatch();
-            String soapMessage = getSOAP11Message();
-            MessageFactory factory = MessageFactory.newInstance();
-            SOAPMessage message = factory.createMessage(null, new ByteArrayInputStream(soapMessage.getBytes()));
-            Object obj = dispatch.invoke(message);
-            assertTrue(obj!=null && obj instanceof SOAPMessage);
-            assertTrue(getVersionURI(message).equals(SOAP11_NS_URI));
-        }catch(Exception e){
-            System.out.println("Failure while sending soap 11 request");
-            System.out.println(e.getMessage());
-            fail();
-        }
+    public void testSoap11Request() throws Exception {
+        System.out.println("---------------------------------------");
+        System.out.println("test: " + getName());
+        
+        Dispatch<SOAPMessage> dispatch=getDispatch();
+        String soapMessage = getSOAP11Message();
+        MessageFactory factory = MessageFactory.newInstance();
+        SOAPMessage message = factory.createMessage(null, new ByteArrayInputStream(soapMessage.getBytes()));
+        Object obj = dispatch.invoke(message);
+        assertTrue(obj!=null && obj instanceof SOAPMessage);
+        assertTrue(getVersionURI(message).equals(SOAP11_NS_URI));
     }
 
-    public void testSoap12Request(){
-        try{
-            System.out.println("---------------------------------------");
-            System.out.println("test: " + getName());
-            
-            Dispatch<SOAPMessage> dispatch=getDispatch();
-            String soapMessage = getSOAP12Message();
-            System.out.println("soap message ="+soapMessage);
-            MessageFactory factory = MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
-            MimeHeaders header = new MimeHeaders();
-            header.addHeader("Content-Type", "application/soap+xml");
-            SOAPMessage message = factory.createMessage(header, new ByteArrayInputStream(soapMessage.getBytes()));
-            Object obj = dispatch.invoke(message);
-            assertTrue(obj!=null && obj instanceof SOAPMessage);
-            assertTrue(getVersionURI(message).equals(SOAP12_NS_URI));
-            System.out.println("Provider endpoint was able to receive both SOAP 11 and SOAP 12 request");
-        }catch(Exception e){
-            System.out.println("Expecting that endpoint will be able to receive soap 12 and soap 11 request");
-            System.out.println(e.getMessage());
-            fail();
-            
-        }
+    public void testSoap12Request() throws Exception {
+        System.out.println("---------------------------------------");
+        System.out.println("test: " + getName());
+        
+        Dispatch<SOAPMessage> dispatch=getDispatch();
+        String soapMessage = getSOAP12Message();
+        System.out.println("soap message ="+soapMessage);
+        MessageFactory factory = MessageFactory.newInstance(SOAPConstants.SOAP_1_2_PROTOCOL);
+        MimeHeaders header = new MimeHeaders();
+        header.addHeader("Content-Type", "application/soap+xml");
+        SOAPMessage message = factory.createMessage(header, new ByteArrayInputStream(soapMessage.getBytes()));
+        Object obj = dispatch.invoke(message);
+        assertTrue(obj!=null && obj instanceof SOAPMessage);
+        assertTrue(getVersionURI(message).equals(SOAP12_NS_URI));
+        System.out.println("Provider endpoint was able to receive both SOAP 11 and SOAP 12 request");
     }
     
     private Dispatch<SOAPMessage> getDispatch(){