[xwiki-notifications] r7483 - xwiki-platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/render/markup

[SVN]

Author: sdumitriu
Date: 2008-02-11 11:23:23 +0100 (Mon, 11 Feb 2008)
New Revision: 7483

Modified:
   
xwiki-platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/render/markup/StrikethroughTest.java
   
xwiki-platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/render/markup/UnderlineTest.java
Log:
[cleanup] Improve tests


Modified: 
xwiki-platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/render/markup/StrikethroughTest.java
===================================================================
--- 
xwiki-platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/render/markup/StrikethroughTest.java
       2008-02-11 00:05:27 UTC (rev 7482)
+++ 
xwiki-platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/render/markup/StrikethroughTest.java
       2008-02-11 10:23:23 UTC (rev 7483)
@@ -141,6 +141,10 @@
         }
         tests.add(text);
         expects.add(text);
+        long startTime = System.currentTimeMillis();
         test(tests, expects);
+        // Even on very slow systems this should not take more than one 
second. Putting 10 seconds,
+        // just to be sure we don't get false test errors.
+        assertTrue(System.currentTimeMillis() - startTime < 10000);
     }
 }

Modified: 
xwiki-platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/render/markup/UnderlineTest.java
===================================================================
--- 
xwiki-platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/render/markup/UnderlineTest.java
   2008-02-11 00:05:27 UTC (rev 7482)
+++ 
xwiki-platform/core/trunk/xwiki-core/src/test/java/com/xpn/xwiki/render/markup/UnderlineTest.java
   2008-02-11 10:23:23 UTC (rev 7483)
@@ -55,7 +55,8 @@
         ArrayList tests = new ArrayList();
         ArrayList expects = new ArrayList();
         tests.add("More __underlines__ on a __line__");
-        expects.add("More <em class=\"underline\">underlines</em> on a <em 
class=\"underline\">line</em>");
+        expects
+            .add("More <em class=\"underline\">underlines</em> on a <em 
class=\"underline\">line</em>");
         test(tests, expects);
     }
 
@@ -84,9 +85,11 @@
         ArrayList tests = new ArrayList();
         ArrayList expects = new ArrayList();
         tests.add("__Eeny__meeny__miny__moe__");
-        expects.add("<em class=\"underline\">Eeny</em>meeny<em 
class=\"underline\">miny</em>moe__");
+        expects
+            .add("<em class=\"underline\">Eeny</em>meeny<em 
class=\"underline\">miny</em>moe__");
         tests.add("__ Eeny__meeny__miny__moe__");
-        expects.add("__ Eeny<em class=\"underline\">meeny</em>miny<em 
class=\"underline\">moe</em>");
+        expects
+            .add("__ Eeny<em class=\"underline\">meeny</em>miny<em 
class=\"underline\">moe</em>");
         test(tests, expects);
     }
 
@@ -125,6 +128,10 @@
         }
         tests.add(text);
         expects.add(text);
+        long startTime = System.currentTimeMillis();
         test(tests, expects);
+        // Even on very slow systems this should not take more than one 
second. Putting 10 seconds,
+        // just to be sure we don't get false test errors.
+        assertTrue(System.currentTimeMillis() - startTime < 10000);
     }
 }

_______________________________________________
notifications mailing list
notifications@xwiki.org
http://lists.xwiki.org/mailman/listinfo/notifications