[Japi-cvs] SF.net SVN: japi:[876] tools/todoScanner/trunk/src/tst/test/net/sf/japi/ tools/todoScanner/TodoScannerTest.java

[<chr...@us...>]

Revision: 876
          http://japi.svn.sourceforge.net/japi/?rev=876&view=rev
Author:   christianhujer
Date:     2009-02-15 11:59:43 +0000 (Sun, 15 Feb 2009)

Log Message:
-----------
 Improved Unit tests, e.g. add messages, bring expected and actual in right order, use assertEquals.

Modified Paths:
--------------
    tools/todoScanner/trunk/src/tst/test/net/sf/japi/tools/todoScanner/TodoScannerTest.java

Modified: tools/todoScanner/trunk/src/tst/test/net/sf/japi/tools/todoScanner/TodoScannerTest.java
===================================================================
--- tools/todoScanner/trunk/src/tst/test/net/sf/japi/tools/todoScanner/TodoScannerTest.java	2009-02-15 11:59:33 UTC (rev 875)
+++ tools/todoScanner/trunk/src/tst/test/net/sf/japi/tools/todoScanner/TodoScannerTest.java	2009-02-15 11:59:43 UTC (rev 876)
@@ -32,23 +32,14 @@
     public void testDefault() {
         final TodoScanner todoScanner = new TodoScanner();
         todoScanner.init();
-        Assert.assertFalse(todoScanner.isMatching("OTTODO"));
-        Assert.assertFalse(todoScanner.isMatching("XXXX"));
-        Assert.assertFalse(todoScanner.isMatching("FIXMEE"));
-        Assert.assertTrue(todoScanner.isMatching("TODO"));
-        Assert.assertTrue(todoScanner.isMatching("XXX"));
-        Assert.assertTrue(todoScanner.isMatching("FIXME"));
-        Assert.assertTrue(todoScanner.isMatching("// TODO:"));
-        Assert.assertTrue(todoScanner.isMatching("// XXX:"));
-        Assert.assertTrue(todoScanner.isMatching("// FIXME:"));
-        Assert.assertTrue(todoScanner.isMatching("//TODO:"));
-        Assert.assertTrue(todoScanner.isMatching("//XXX:"));
-        Assert.assertTrue(todoScanner.isMatching("//FIXME:"));
-        Assert.assertTrue(todoScanner.isMatching("/*TODO:"));
-        Assert.assertTrue(todoScanner.isMatching("/*XXX:"));
-        Assert.assertTrue(todoScanner.isMatching("/*FIXME:"));
-        Assert.assertTrue(todoScanner.isMatching("#TODO:"));
-        Assert.assertTrue(todoScanner.isMatching("#XXX:"));
-        Assert.assertTrue(todoScanner.isMatching("#FIXME:"));
+        final String[] mustNotMatch = { "OTTODO", "XXXX", "FIXMEE" };
+        final String[] mustMatch = { "TODO", "XXX", "FIXME", "// TODO:", "// XXX:", "// FIXME:", "//TODO:", "//XXX:", "//FIXME:", "/*TODO:", "/*XXX:", "/*FIXME:", "#TODO:", "#XXX:", "#FIXME:" };
+
+        for (final String text : mustNotMatch) {
+            Assert.assertFalse("Default todo scanner must not match \" + text + \".", todoScanner.isMatching(text));
+        }
+        for (final String text : mustMatch) {
+            Assert.assertTrue("Default todo scanner must not match \" + text + \".", todoScanner.isMatching(text));
+        }
     }
 }


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.