[Japi-cvs] SF.net SVN: japi:[870] libs/lang/trunk/src/tst/test/net/sf/japi/lang

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

Revision: 870
          http://japi.svn.sourceforge.net/japi/?rev=870&view=rev
Author:   christianhujer
Date:     2009-02-15 11:58:45 +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:
--------------
    libs/lang/trunk/src/tst/test/net/sf/japi/lang/PropertyComparatorTest.java
    libs/lang/trunk/src/tst/test/net/sf/japi/lang/SuperClassIteratorTest.java

Modified: libs/lang/trunk/src/tst/test/net/sf/japi/lang/PropertyComparatorTest.java
===================================================================
--- libs/lang/trunk/src/tst/test/net/sf/japi/lang/PropertyComparatorTest.java	2009-02-15 11:58:33 UTC (rev 869)
+++ libs/lang/trunk/src/tst/test/net/sf/japi/lang/PropertyComparatorTest.java	2009-02-15 11:58:45 UTC (rev 870)
@@ -37,7 +37,7 @@
         final String s4 = "nice";
         final String s5 = "day";
         final PropertyComparator<Integer, String> lengthComparator = new PropertyComparator<Integer, String>(String.class, "length", null);
-        Assert.assertTrue(lengthComparator.compare(s1, s2) == 0);
+        Assert.assertSame(0, lengthComparator.compare(s1, s2));
         Assert.assertTrue(lengthComparator.compare(s2, s3) > 0);
         Assert.assertTrue(lengthComparator.compare(s3, s4) < 0);
         Assert.assertTrue(lengthComparator.compare(s4, s5) > 0);

Modified: libs/lang/trunk/src/tst/test/net/sf/japi/lang/SuperClassIteratorTest.java
===================================================================
--- libs/lang/trunk/src/tst/test/net/sf/japi/lang/SuperClassIteratorTest.java	2009-02-15 11:58:33 UTC (rev 869)
+++ libs/lang/trunk/src/tst/test/net/sf/japi/lang/SuperClassIteratorTest.java	2009-02-15 11:58:45 UTC (rev 870)
@@ -70,7 +70,7 @@
     public void testIterable() {
         for (final Class c : new SuperClassIterable(String.class)) {
             if (!c.equals(String.class) && !c.equals(Object.class)) {
-                Assert.assertTrue("Expected String.class or Object.class.", false);
+                Assert.fail("Expected String.class or Object.class.");
             }
         }
     }


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