[savara-commits] savara SVN: r526 - branches/experimental/2.0.x/bundles/org.savara.bpel.tests/src/test/java/org/savara/bpel/generator.

[do-not-reply at jboss.org]

Author: objectiser
Date: 2011-01-02 19:14:10 -0500 (Sun, 02 Jan 2011)
New Revision: 526

Modified:
   branches/experimental/2.0.x/bundles/org.savara.bpel.tests/src/test/java/org/savara/bpel/generator/GeneratorTest.java
Log:


Modified: branches/experimental/2.0.x/bundles/org.savara.bpel.tests/src/test/java/org/savara/bpel/generator/GeneratorTest.java
===================================================================
--- branches/experimental/2.0.x/bundles/org.savara.bpel.tests/src/test/java/org/savara/bpel/generator/GeneratorTest.java	2011-01-02 21:10:02 UTC (rev 525)
+++ branches/experimental/2.0.x/bundles/org.savara.bpel.tests/src/test/java/org/savara/bpel/generator/GeneratorTest.java	2011-01-03 00:14:10 UTC (rev 526)
@@ -38,14 +38,32 @@
     public static Test suite() {
         TestSuite suite = new TestSuite("Choreography->BPEL Generator Tests");
         
+        /*
+         * TWO ISSUES:
+         * 
+         * 1) Additional invoke being generated in first choice path, which
+         * appears to be related to the 'when' message signature - the problem
+         * being that the second instance of the invoke is created from the
+         * interaction inside the run sub-protocol.
+         * 
+         * 2) Additional 'myRole' on partnerlink - seems to be related to
+         * Broker rather tha Buyer which would have expected.
+         * 
         suite.addTest(new ChoreographyToBPELTester("ESBBroker", "Buyer"));
+        */
+        
+        // ISSUES:
+        // Need to deal with case when top level protocol is not projected, but
+        // the role to be projected belongs to a nested protocol. What happens
+        // if two separate nested protocols use the same role name? Possibly
+        // this needs to be reported as an unhandled case for now.
+        
+        suite.addTest(new ChoreographyToBPELTester("ESBBroker", "SupplierTxnProcessor"));
+        //suite.addTest(new ChoreographyToBPELTester("ESBBroker", "CreditAgency"));
         suite.addTest(new ChoreographyToBPELTester("ESBBroker", "Broker"));
         
         /*
-        suite.addTest(new ChoreographyToBPELTester("ESBBroker", "CreditAgency"));
         suite.addTest(new ChoreographyToBPELTester("ESBBroker",
-        								"SupplierTxnProcessor"));
-        suite.addTest(new ChoreographyToBPELTester("ESBBroker",
 										"SupplierQuoteEngine"));
         */
         
@@ -131,7 +149,7 @@
     				org.scribble.protocol.model.Role role=null;
     				
     				// Obtain role from definition
-     				Protocol defn=model.getProtocol();
+     				//Protocol defn=model.getProtocol();
     				    				
     				// Check if subpath definition defined
     				/*
@@ -140,7 +158,7 @@
     				}
     				*/
     				
-    				java.util.List<org.scribble.protocol.model.Role> roles=defn.getRoles();
+    				java.util.List<org.scribble.protocol.model.Role> roles=model.getRoles();
     					//new org.scribble.model.Role(m_role);
     				
     				for (int i=0; role == null && i < roles.size(); i++) {