[jbossws-commits] JBossWS SVN: r9674 - framework/trunk/testsuite/test/java/org/jboss/test/ws/jaxws/smoke/tools.

[jbossws-commits at lists.jboss.org]

Author: richard.opalka at jboss.com
Date: 2009-03-24 09:29:09 -0400 (Tue, 24 Mar 2009)
New Revision: 9674

Modified:
   framework/trunk/testsuite/test/java/org/jboss/test/ws/jaxws/smoke/tools/WSRunClientTestCase.java
Log:
[JBWS-2327] adding javadoc comments how the test works

Modified: framework/trunk/testsuite/test/java/org/jboss/test/ws/jaxws/smoke/tools/WSRunClientTestCase.java
===================================================================
--- framework/trunk/testsuite/test/java/org/jboss/test/ws/jaxws/smoke/tools/WSRunClientTestCase.java	2009-03-24 11:26:26 UTC (rev 9673)
+++ framework/trunk/testsuite/test/java/org/jboss/test/ws/jaxws/smoke/tools/WSRunClientTestCase.java	2009-03-24 13:29:09 UTC (rev 9674)
@@ -42,7 +42,13 @@
    }
 
    /**
-    * Executing wsrunclient commandline script
+    * Executing wsrunclient commandline script.
+    * We're using simple trick here as we're running
+    * junit tests via wsrunclient here.
+    * The reason is we don't need to touch any test case
+    * plus we can add any test case to the execution anytime.
+    * Plus invoking the test classes via main method vs. junit runner is equivalent
+    * from wsrunclient point of view. 
     * @throws Exception if any error occurs
     */
    public void test() throws Exception
@@ -75,7 +81,12 @@
    }
 
    /**
-    * Prepares additional classpath containing junit lib, test classes directory and jbossws integration jars
+    * Prepares additional classpath containing junit lib, test classes directory and jbossws integration jars.
+    * The junit is needed, because we're running junit samples tests via wsrunclient.
+    * Test classes directory contains test cases we're executing via wsrunclient.
+    * Finally integration jars are needed because every test case extends JBossWSTest and it needs integration
+    * jars to deploy tested archives to the server programatically. Generally, users of wsrunclient
+    * will usually specify just test classes directory and test class to invoke main method on. 
     */
    private String prepareAdditionalClasspath()
    {
@@ -158,6 +169,10 @@
       return retVal;
    }
 
+   /**
+    * Detects whether we're running the distribution tests
+    * @return true if distro tests are executed, false otherwise
+    */
    private boolean isDistroTest()
    {
       return Boolean.getBoolean("binary.distribution");