Re: svn commit: r570831 - in /incubator/cxf/trunk: rt/transports/jms/src/main/java/org/apache/cxf/transport/jms/ rt/transports/jms/src/main/resources/schemas/wsdl/ rt/transports/jms/src/test/java/org/apache/cxf/transport/jms/ systests/src/test/java/org/apa...

[Ulhas Bhole <ul...@iona.com>]

Hi Glen,

System.out is not really required , it's copy+paste in the test but is
good for debugging at the server side in case the test fails so left it
that way.

-- Ulhas

Glen Mazza wrote:
> Am Mittwoch, den 29.08.2007, 15:00 +0000 schrieb ulhasbhole@apache.org:
>   
>> Author: ulhasbhole
>> Date: Wed Aug 29 08:00:40 2007
>> New Revision: 570831
>>
>>     
>
>   
>> Added: incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jms/GreeterByteMessageImpl.java
>> URL: http://svn.apache.org/viewvc/incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jms/GreeterByteMessageImpl.java?rev=570831&view=auto
>> ==============================================================================
>> --- incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jms/GreeterByteMessageImpl.java (added)
>> +++ incubator/cxf/trunk/systests/src/test/java/org/apache/cxf/systest/jms/GreeterByteMessageImpl.java Wed Aug 29 08:00:40 2007
>> @@ -0,0 +1,97 @@
>>     
>
>   
>> +package org.apache.cxf.systest.jms;
>> +
>> +
>> +@WebService(serviceName = "HWByteMsgService", 
>> +            portName = "HWSByteMsgPort",
>> +            endpointInterface = "org.apache.cxf.hello_world_jms.HelloWorldPortType",
>> +            targetNamespace = "http://cxf.apache.org/hello_world_jms",
>> +            wsdlLocation = "testutils/jms_test.wsdl")
>> +public class GreeterByteMessageImpl implements HelloWorldPortType {
>> +    @Resource
>> +    protected WebServiceContext wsContext;
>> +    public String greetMe(String me) {
>> +        MessageContext mc = wsContext.getMessageContext();
>> +        JMSMessageHeadersType headers =
>> +            (JMSMessageHeadersType) mc.get(JMSConstants.JMS_SERVER_REQUEST_HEADERS);
>> +        System.out.println("get the message headers JMSCorrelationID" + headers.getJMSCorrelationID());
>> +        System.out.println("Reached here :" + me);
>>     
>
> This looks more like a debugging statement than an actual output test message.  Was retention of this line of code your actual intent?
>
>
>   
>> +        // set reply header custom property
>> +        JMSPropertyType testProperty = new JMSPropertyType();
>> +        testProperty.setName("Test_Prop");
>> +        testProperty.setValue("some return value "  + me);
>> +        
>>     
>
> Same here.
>
> Regards,
> Glen
>
>   


----------------------------
IONA Technologies PLC (registered in Ireland)
Registered Number: 171387
Registered Address: The IONA Building, Shelbourne Road, Dublin 4, Ireland