Author: tmortagne
Date: 2008-01-31 14:23:54 +0100 (Thu, 31 Jan 2008)
New Revision: 7235

Modified:
   
xwiki-products/xwiki-enterprise-manager/trunk/distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/selenium/AllTests.java
   
xwiki-products/xwiki-enterprise-manager/trunk/distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/selenium/xem/StartTest.java
Log:
Fix javadoc

Modified: 
xwiki-products/xwiki-enterprise-manager/trunk/distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/selenium/AllTests.java
===================================================================
--- 
xwiki-products/xwiki-enterprise-manager/trunk/distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/selenium/AllTests.java
  2008-01-31 13:17:56 UTC (rev 7234)
+++ 
xwiki-products/xwiki-enterprise-manager/trunk/distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/selenium/AllTests.java
  2008-01-31 13:23:54 UTC (rev 7235)
@@ -30,10 +30,10 @@
 
 /**
  * A class listing all the Selenium Functional tests to execute. We need such 
a class (rather than
- * letting the JUnit Runner discover the different TestCases classes by 
itself) because we want
- * to start/stop XWiki before and after the tests start (but only once).
- *
- * @version $Id: $ 
+ * letting the JUnit Runner discover the different TestCases classes by 
itself) because we want to
+ * start/stop XWiki before and after the tests start (but only once).
+ * 
+ * @version $Id: $
  */
 public class AllTests extends TestCase
 {
@@ -43,6 +43,13 @@
     {
         TestSuite suite = new TestSuite();
 
+        // TODO: I don't like listing tests here as it means we can add a new 
TestCase class and
+        // forget to add it here and the tests won't be run but we'll not know 
about it and we'll
+        // think the tests are all running fine. I haven't found a simple 
solution to this yet
+        // (there are complex solutions like searching for all tests by 
parsing the source tree).
+        // I think there are TestSuite that do this out there but I haven't 
looked for them yet.
+
+        // Just try to access wiki ti see if it's starting correctly.
         addTestCase(suite, StartTest.class);
 
         return new XWikiTestSetup(suite);

Modified: 
xwiki-products/xwiki-enterprise-manager/trunk/distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/selenium/xem/StartTest.java
===================================================================
--- 
xwiki-products/xwiki-enterprise-manager/trunk/distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/selenium/xem/StartTest.java
     2008-01-31 13:17:56 UTC (rev 7234)
+++ 
xwiki-products/xwiki-enterprise-manager/trunk/distribution-test/selenium-tests/src/test/it/com/xpn/xwiki/it/selenium/xem/StartTest.java
     2008-01-31 13:23:54 UTC (rev 7235)
@@ -26,8 +26,8 @@
 import junit.framework.Test;
 
 /**
- * Verify deletion of pages.
- *
+ * Minimum test that just try to access wiki.
+ * 
  * @version $Id: $
  */
 public class StartTest extends AbstractXWikiTestCase
@@ -38,7 +38,7 @@
         suite.addTestSuite(StartTest.class, AlbatrossSkinExecutor.class);
         return suite;
     }
-    
+
     public void testStart()
     {
         open("/xwiki/bin/view/Main/");

_______________________________________________
notifications mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/notifications

Reply via email to