From fca59df9050b475b118941302c3adf66c4bb0f3b Mon Sep 17 00:00:00 2001
From: antelder <antelder@13f79535-47bb-0310-9956-ffa450edef68>
Date: Thu, 3 Nov 2011 14:45:25 +0000
Subject: TUSCANY-3972 JMSBIndingProcessor.writeDestinationProperties should
 handle Destination Type

git-svn-id: http://svn.us.apache.org/repos/asf/tuscany@1197162 13f79535-47bb-0310-9956-ffa450edef68
---
 .../org/apache/tuscany/sca/binding/jms/JMSBindingProcessor.java  | 9 ++++-----
 .../sca/binding/jms/JMSBindingProcessorWriteTestCase.java        | 2 +-
 2 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/sca-java-2.x/trunk/modules/binding-jms/src/main/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessor.java b/sca-java-2.x/trunk/modules/binding-jms/src/main/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessor.java
index a428af8812..de5842c9c3 100644
--- a/sca-java-2.x/trunk/modules/binding-jms/src/main/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessor.java
+++ b/sca-java-2.x/trunk/modules/binding-jms/src/main/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessor.java
@@ -1421,11 +1421,10 @@ public class JMSBindingProcessor extends BaseStAXArtifactProcessor implements St
             writer.writeAttribute("jndiName", destinationName);            
         }
 
-        // Type not handled yet
-        // String destinationType = jmsBinding.getDestinationType();
-        // if ( destinationType != null && destinationType.length() > 0) {
-        //     writer.writeAttribute("type", destinationType);            
-        // }
+        String destinationType = jmsBinding.getDestinationType();
+        if ( destinationType != null && destinationType.length() > 0) {
+            writer.writeAttribute("type", destinationType);            
+        }
 
         String destinationCreate = jmsBinding.getDestinationCreate();
         if ( destinationCreate != null && destinationCreate.length() > 0 &&
diff --git a/sca-java-2.x/trunk/modules/binding-jms/src/test/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessorWriteTestCase.java b/sca-java-2.x/trunk/modules/binding-jms/src/test/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessorWriteTestCase.java
index db138aa30d..5fdbd57338 100644
--- a/sca-java-2.x/trunk/modules/binding-jms/src/test/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessorWriteTestCase.java
+++ b/sca-java-2.x/trunk/modules/binding-jms/src/test/java/org/apache/tuscany/sca/binding/jms/JMSBindingProcessorWriteTestCase.java
@@ -404,7 +404,7 @@ public class JMSBindingProcessorWriteTestCase extends TestCase {
                        "<component name=\"HelloWorldComponent\">" + 
                            "<service name=\"HelloWorldService\">" +
                              "<binding.jms><operationProperties name=\"JKL\" /> " +
-                               "<destination jndiName=\"AAA\">" +
+                               "<destination jndiName=\"AAA\" type=\"queue\">" +
                                  "<property name=\"AAAProp\" type=\"string\"></property>" +
                                "</destination> " +
                                "<connectionFactory jndiName=\"ABC\" /> "+
-- 
cgit v1.2.3