[overlord-commits] Overlord SVN: r585 - in cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src: test/org/jboss/tools/overlord/cdl/bpel/model/component and 1 other directory.

overlord-commits at lists.jboss.org overlord-commits at lists.jboss.org
Sun Apr 12 16:47:46 EDT 2009


Author: objectiser
Date: 2009-04-12 16:47:44 -0400 (Sun, 12 Apr 2009)
New Revision: 585

Modified:
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractCondition.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractInteraction.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/BPELActivity.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Catch.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Invoke.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/OnEvent.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/OnMessage.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Receive.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Reply.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Throw.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Variable.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractInteractionTest.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/BPELActivityTest.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/CatchTest.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ConditionTest.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/InvokeTest.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/OnEventTest.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/OnMessageTest.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ReceiveTest.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ReplyTest.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ThrowTest.java
   cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/VariableTest.java
Log:
Updated accessors to return null if attribute not defined, rather than empty string.

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractCondition.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractCondition.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractCondition.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -104,7 +104,8 @@
 	 * @return The expression language
 	 */
 	public String getExpressionLanguage() {
-		return(getDOMElement().getAttribute(EXPRESSION_LANGUAGE));
+		return(getDOMElement().hasAttribute(EXPRESSION_LANGUAGE)?
+				getDOMElement().getAttribute(EXPRESSION_LANGUAGE):null);
 	}
 	
 	/**

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractInteraction.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractInteraction.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractInteraction.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -69,7 +69,8 @@
 	 * @return The operation
 	 */
 	public String getOperation() {
-		return(getDOMElement().getAttribute(OPERATION));
+		return(getDOMElement().hasAttribute(OPERATION)?
+				getDOMElement().getAttribute(OPERATION):null);
 	}
 
 	/**
@@ -87,7 +88,8 @@
 	 * @return The partner link
 	 */
 	public String getPartnerLink() {
-		return(getDOMElement().getAttribute(PARTNER_LINK));
+		return(getDOMElement().hasAttribute(PARTNER_LINK)?
+				getDOMElement().getAttribute(PARTNER_LINK):null);
 	}
 
 	/**
@@ -105,7 +107,8 @@
 	 * @return The port type
 	 */
 	public String getPortType() {
-		return(getDOMElement().getAttribute(PORT_TYPE));
+		return(getDOMElement().hasAttribute(PORT_TYPE)?
+				getDOMElement().getAttribute(PORT_TYPE):null);
 	}
 
 	/**

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/BPELActivity.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/BPELActivity.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/BPELActivity.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -78,7 +78,8 @@
 	 * @return The name
 	 */
 	public String getName() {
-		return(getDOMElement().getAttribute(NAME));
+		return(getDOMElement().hasAttribute(NAME)?
+				getDOMElement().getAttribute(NAME):null);
 	}
 
 	/**

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Catch.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Catch.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Catch.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -111,7 +111,8 @@
 	 * @return The fault name
 	 */
 	public String getFaultName() {
-		return(getDOMElement().getAttribute(FAULT_NAME));
+		return(getDOMElement().hasAttribute(FAULT_NAME)?
+				getDOMElement().getAttribute(FAULT_NAME):null);
 	}
 
 	/**
@@ -129,7 +130,8 @@
 	 * @return The fault variable
 	 */
 	public String getFaultVariable() {
-		return(getDOMElement().getAttribute(FAULT_VARIABLE));
+		return(getDOMElement().hasAttribute(FAULT_VARIABLE)?
+				getDOMElement().getAttribute(FAULT_VARIABLE):null);
 	}
 
 	private BPELElement m_activity;

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Invoke.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Invoke.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Invoke.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -70,7 +70,8 @@
 	 * @return The input variable
 	 */
 	public String getInputVariable() {
-		return(getDOMElement().getAttribute(INPUT_VARIABLE));
+		return(getDOMElement().hasAttribute(INPUT_VARIABLE)?
+				getDOMElement().getAttribute(INPUT_VARIABLE):null);
 	}
 
 	/**
@@ -88,7 +89,8 @@
 	 * @return The output variable
 	 */
 	public String getOutputVariable() {
-		return(getDOMElement().getAttribute(OUTPUT_VARIABLE));
+		return(getDOMElement().hasAttribute(OUTPUT_VARIABLE)?
+				getDOMElement().getAttribute(OUTPUT_VARIABLE):null);
 	}
 
 	/**

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/OnEvent.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/OnEvent.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/OnEvent.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -79,7 +79,8 @@
 	 * @return The operation
 	 */
 	public String getOperation() {
-		return(getDOMElement().getAttribute(OPERATION));
+		return(getDOMElement().hasAttribute(OPERATION)?
+				getDOMElement().getAttribute(OPERATION):null);
 	}
 
 	/**
@@ -97,7 +98,8 @@
 	 * @return The partner link
 	 */
 	public String getPartnerLink() {
-		return(getDOMElement().getAttribute(PARTNER_LINK));
+		return(getDOMElement().hasAttribute(PARTNER_LINK)?
+				getDOMElement().getAttribute(PARTNER_LINK):null);
 	}
 
 	/**
@@ -115,7 +117,8 @@
 	 * @return The port type
 	 */
 	public String getPortType() {
-		return(getDOMElement().getAttribute(PORT_TYPE));
+		return(getDOMElement().hasAttribute(PORT_TYPE)?
+				getDOMElement().getAttribute(PORT_TYPE):null);
 	}
 
 	/**
@@ -133,7 +136,8 @@
 	 * @return The variable
 	 */
 	public String getVariable() {
-		return(getDOMElement().getAttribute(VARIABLE));
+		return(getDOMElement().hasAttribute(VARIABLE)?
+				getDOMElement().getAttribute(VARIABLE):null);
 	}
 
 	/**

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/OnMessage.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/OnMessage.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/OnMessage.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -75,7 +75,8 @@
 	 * @return The operation
 	 */
 	public String getOperation() {
-		return(getDOMElement().getAttribute(OPERATION));
+		return(getDOMElement().hasAttribute(OPERATION)?
+				getDOMElement().getAttribute(OPERATION):null);
 	}
 
 	/**
@@ -93,7 +94,8 @@
 	 * @return The partner link
 	 */
 	public String getPartnerLink() {
-		return(getDOMElement().getAttribute(PARTNER_LINK));
+		return(getDOMElement().hasAttribute(PARTNER_LINK)?
+				getDOMElement().getAttribute(PARTNER_LINK):null);
 	}
 
 	/**
@@ -111,7 +113,8 @@
 	 * @return The port type
 	 */
 	public String getPortType() {
-		return(getDOMElement().getAttribute(PORT_TYPE));
+		return(getDOMElement().hasAttribute(PORT_TYPE)?
+				getDOMElement().getAttribute(PORT_TYPE):null);
 	}
 
 	/**
@@ -129,7 +132,8 @@
 	 * @return The variable
 	 */
 	public String getVariable() {
-		return(getDOMElement().getAttribute(VARIABLE));
+		return(getDOMElement().hasAttribute(VARIABLE)?
+				getDOMElement().getAttribute(VARIABLE):null);
 	}
 
 	/**

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Receive.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Receive.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Receive.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -71,7 +71,8 @@
 	 * @return The variable
 	 */
 	public String getVariable() {
-		return(getDOMElement().getAttribute(VARIABLE));
+		return(getDOMElement().hasAttribute(VARIABLE)?
+				getDOMElement().getAttribute(VARIABLE):null);
 	}
 
 	/**

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Reply.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Reply.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Reply.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -70,7 +70,8 @@
 	 * @return The variable
 	 */
 	public String getVariable() {
-		return(getDOMElement().getAttribute(VARIABLE));
+		return(getDOMElement().hasAttribute(VARIABLE)?
+				getDOMElement().getAttribute(VARIABLE):null);
 	}
 
 	/**
@@ -88,7 +89,8 @@
 	 * @return The fault name
 	 */
 	public String getFaultName() {
-		return(getDOMElement().getAttribute(FAULT_NAME));
+		return(getDOMElement().hasAttribute(FAULT_NAME)?
+				getDOMElement().getAttribute(FAULT_NAME):null);
 	}
 
 	/**

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Throw.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Throw.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Throw.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -70,7 +70,8 @@
 	 * @return The fault name
 	 */
 	public String getFaultName() {
-		return(getDOMElement().getAttribute(FAULT_NAME));
+		return(getDOMElement().hasAttribute(FAULT_NAME)?
+				getDOMElement().getAttribute(FAULT_NAME):null);
 	}
 
 	/**
@@ -88,7 +89,8 @@
 	 * @return The fault variable
 	 */
 	public String getFaultVariable() {
-		return(getDOMElement().getAttribute(FAULT_VARIABLE));
+		return(getDOMElement().hasAttribute(FAULT_VARIABLE)?
+				getDOMElement().getAttribute(FAULT_VARIABLE):null);
 	}
 
 	/**

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Variable.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Variable.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/java/org/jboss/tools/overlord/cdl/bpel/model/component/Variable.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -70,7 +70,8 @@
 	 * @return The name
 	 */
 	public String getName() {
-		return(getDOMElement().getAttribute(NAME));
+		return(getDOMElement().hasAttribute(NAME)?
+				getDOMElement().getAttribute(NAME):null);
 	}
 
 	/**
@@ -88,7 +89,8 @@
 	 * @return The type
 	 */
 	public String getType() {
-		return(getDOMElement().getAttribute(TYPE));
+		return(getDOMElement().hasAttribute(TYPE)?
+				getDOMElement().getAttribute(TYPE):null);
 	}
 
 	/**

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractInteractionTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractInteractionTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/AbstractInteractionTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -92,6 +92,10 @@
 		
 		TestAbstractInteraction component=new TestAbstractInteraction(model);		
 		
+		if (component.getOperation() != null) {
+			fail("Should be null");
+		}
+		
 		component.setOperation(op);
 		
 		if (component.getOperation().equals(op) == false) {
@@ -107,6 +111,10 @@
 		
 		TestAbstractInteraction component=new TestAbstractInteraction(model);		
 		
+		if (component.getPortType() != null) {
+			fail("Should be null");
+		}
+		
 		component.setPortType(pt);
 		
 		if (component.getPortType().equals(pt) == false) {
@@ -122,6 +130,10 @@
 		
 		TestAbstractInteraction component=new TestAbstractInteraction(model);		
 		
+		if (component.getPartnerLink() != null) {
+			fail("Should be null");
+		}
+		
 		component.setPartnerLink(pl);
 		
 		if (component.getPartnerLink().equals(pl) == false) {

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/BPELActivityTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/BPELActivityTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/BPELActivityTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -77,6 +77,10 @@
 		
 		TestBPELActivity component=new TestBPELActivity(model);		
 		
+		if (component.getName() != null) {
+			fail("Should be null");
+		}
+		
 		component.setName(name);
 		
 		if (component.getName().equals(name) == false) {

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/CatchTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/CatchTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/CatchTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -83,6 +83,10 @@
 		
 		Catch component=new Catch(model);		
 		
+		if (component.getFaultVariable() != null) {
+			fail("Should be null");
+		}
+		
 		component.setFaultVariable(var);
 		
 		if (component.getFaultVariable().equals(var) == false) {
@@ -98,6 +102,10 @@
 		
 		Catch component=new Catch(model);		
 		
+		if (component.getFaultName() != null) {
+			fail("Should be null");
+		}
+		
 		component.setFaultName(fault);
 		
 		if (component.getFaultName().equals(fault) == false) {

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ConditionTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ConditionTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ConditionTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -39,6 +39,10 @@
 		
 		String expr="Test Expression";
 		
+		if (condition.getExpression() != null) {
+			fail("Should be null");
+		}
+		
 		condition.setExpression(expr);
 		
 		String result=condition.getExpression();
@@ -100,6 +104,10 @@
 		
 		String lang="Test Language";
 		
+		if (condition.getExpressionLanguage() != null) {
+			fail("Should be null");
+		}
+		
 		condition.setExpressionLanguage(lang);
 		
 		String result=condition.getExpressionLanguage();

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/InvokeTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/InvokeTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/InvokeTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -85,6 +85,10 @@
 		
 		Invoke component=new Invoke(model);		
 		
+		if (component.getInputVariable() != null) {
+			fail("Should be null");
+		}
+		
 		component.setInputVariable(var);
 		
 		if (component.getInputVariable().equals(var) == false) {
@@ -100,6 +104,10 @@
 		
 		Invoke component=new Invoke(model);		
 		
+		if (component.getOutputVariable() != null) {
+			fail("Should be null");
+		}
+		
 		component.setOutputVariable(var);
 		
 		if (component.getOutputVariable().equals(var) == false) {

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/OnEventTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/OnEventTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/OnEventTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -103,6 +103,10 @@
 		
 		OnEvent component=new OnEvent(model);		
 		
+		if (component.getVariable() != null) {
+			fail("Should be null");
+		}
+		
 		component.setVariable(var);
 		
 		if (component.getVariable().equals(var) == false) {
@@ -118,6 +122,10 @@
 		
 		OnEvent component=new OnEvent(model);		
 		
+		if (component.getOperation() != null) {
+			fail("Should be null");
+		}
+		
 		component.setOperation(op);
 		
 		if (component.getOperation().equals(op) == false) {
@@ -133,6 +141,10 @@
 		
 		OnEvent component=new OnEvent(model);		
 		
+		if (component.getPortType() != null) {
+			fail("Should be null");
+		}
+		
 		component.setPortType(pt);
 		
 		if (component.getPortType().equals(pt) == false) {
@@ -148,6 +160,10 @@
 		
 		OnEvent component=new OnEvent(model);		
 		
+		if (component.getPartnerLink() != null) {
+			fail("Should be null");
+		}
+		
 		component.setPartnerLink(pl);
 		
 		if (component.getPartnerLink().equals(pl) == false) {

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/OnMessageTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/OnMessageTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/OnMessageTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -108,6 +108,10 @@
 		
 		OnMessage component=new OnMessage(model);		
 		
+		if (component.getVariable() != null) {
+			fail("Should be null");
+		}
+		
 		component.setVariable(var);
 		
 		if (component.getVariable().equals(var) == false) {
@@ -123,6 +127,10 @@
 		
 		OnMessage component=new OnMessage(model);		
 		
+		if (component.getOperation() != null) {
+			fail("Should be null");
+		}
+		
 		component.setOperation(op);
 		
 		if (component.getOperation().equals(op) == false) {
@@ -138,6 +146,10 @@
 		
 		OnMessage component=new OnMessage(model);		
 		
+		if (component.getPortType() != null) {
+			fail("Should be null");
+		}
+		
 		component.setPortType(pt);
 		
 		if (component.getPortType().equals(pt) == false) {
@@ -153,6 +165,10 @@
 		
 		OnMessage component=new OnMessage(model);		
 		
+		if (component.getPartnerLink() != null) {
+			fail("Should be null");
+		}
+		
 		component.setPartnerLink(pl);
 		
 		if (component.getPartnerLink().equals(pl) == false) {

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ReceiveTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ReceiveTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ReceiveTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -85,6 +85,10 @@
 		
 		Receive component=new Receive(model);		
 		
+		if (component.getVariable() != null) {
+			fail("Should be null");
+		}
+		
 		component.setVariable(var);
 		
 		if (component.getVariable().equals(var) == false) {

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ReplyTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ReplyTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ReplyTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -85,6 +85,10 @@
 		
 		Reply component=new Reply(model);		
 		
+		if (component.getVariable() != null) {
+			fail("Should be null");
+		}
+		
 		component.setVariable(var);
 		
 		if (component.getVariable().equals(var) == false) {
@@ -100,6 +104,10 @@
 		
 		Reply component=new Reply(model);		
 		
+		if (component.getFaultName() != null) {
+			fail("Should be null");
+		}
+		
 		component.setFaultName(fault);
 		
 		if (component.getFaultName().equals(fault) == false) {

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ThrowTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ThrowTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/ThrowTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -85,6 +85,10 @@
 		
 		Throw component=new Throw(model);		
 		
+		if (component.getFaultVariable() != null) {
+			fail("Should be null");
+		}
+		
 		component.setFaultVariable(var);
 		
 		if (component.getFaultVariable().equals(var) == false) {
@@ -100,6 +104,10 @@
 		
 		Throw component=new Throw(model);		
 		
+		if (component.getFaultName() != null) {
+			fail("Should be null");
+		}
+		
 		component.setFaultName(fault);
 		
 		if (component.getFaultName().equals(fault) == false) {

Modified: cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/VariableTest.java
===================================================================
--- cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/VariableTest.java	2009-04-12 20:18:00 UTC (rev 584)
+++ cdl/trunk/tools/plugins/org.jboss.tools.overlord.cdl.bpel/src/test/org/jboss/tools/overlord/cdl/bpel/model/component/VariableTest.java	2009-04-12 20:47:44 UTC (rev 585)
@@ -85,6 +85,10 @@
 		
 		Variable component=new Variable(model);		
 		
+		if (component.getName() != null) {
+			fail("Should be null");
+		}
+		
 		component.setName(var);
 		
 		if (component.getName().equals(var) == false) {
@@ -100,6 +104,10 @@
 		
 		Variable component=new Variable(model);		
 		
+		if (component.getType() != null) {
+			fail("Should be null");
+		}
+		
 		component.setType(type);
 		
 		if (component.getType().equals(type) == false) {




More information about the overlord-commits mailing list