[Clirr-devel] CVS: clirr/core/src/test/net/sf/clirr/core/internal/checks TestDiffListener.java,1.1,1.2

[Simon K. <s_k...@us...>]

Update of /cvsroot/clirr/clirr/core/src/test/net/sf/clirr/core/internal/checks
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv852

Modified Files:
	TestDiffListener.java 
Log Message:
The ApiDifference.toString() method no longer reports a human-readable
message, because a resource bundle must be selected to generate one. So
here we can't rely on toString() to generate a nice error message on
test case failure. Instead we build one using the default translation.


Index: TestDiffListener.java
===================================================================
RCS file: /cvsroot/clirr/clirr/core/src/test/net/sf/clirr/core/internal/checks/TestDiffListener.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- TestDiffListener.java	11 Jul 2004 10:29:24 -0000	1.1
+++ TestDiffListener.java	16 Jul 2004 09:09:23 -0000	1.2
@@ -38,7 +38,24 @@
 
                 if (!found)
                 {
-                    TestCase.fail("expected diff " + expected + " was not generated: " + diffs);
+                    // build a useful failure message
+                    MessageTranslator translator = new MessageTranslator();
+
+                    StringBuffer buf = new StringBuffer();
+                    buf.append("Expected diff " + expected + " was not generated.");
+                    buf.append(" Actual diffs generated were: ");
+                    for(Iterator diffIter = diffs.iterator(); diffIter.hasNext();)
+                    {
+                        ApiDifference diff = (ApiDifference) diffIter.next();
+
+                        buf.append(diff.toString(translator));
+                        if (diffIter.hasNext())
+                        {
+                            buf.append(", ");
+                        }
+                    }
+
+                    TestCase.fail(buf.toString());
                 }
             }