Author: richard.opalka(a)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");