[Aptitude-svn-commit] r3684 - in branches/aptitude-0.3/aptitude: . src/generic/tests

[Daniel Burrows]

Author: dburrows
Date: Tue Jul 26 03:26:52 2005
New Revision: 3684

Modified:
   branches/aptitude-0.3/aptitude/ChangeLog
   branches/aptitude-0.3/aptitude/src/generic/tests/main.cc
Log:
Report errors properly.

Modified: branches/aptitude-0.3/aptitude/ChangeLog
==============================================================================
--- branches/aptitude-0.3/aptitude/ChangeLog	(original)
+++ branches/aptitude-0.3/aptitude/ChangeLog	Tue Jul 26 03:26:52 2005
@@ -1,5 +1,11 @@
 2005-07-25  Daniel Burrows  <dburrows at debian.org>
 
+	* src/generic/tests/main.cc:
+
+	  As I suspected, the return value of runner.run() was reversed;
+	  fix it so the test harness actually reports errors to its
+	  caller.
+
 	* src/generic/tests/Makefile.am:
 
 	  Run the tests on "make check".

Modified: branches/aptitude-0.3/aptitude/src/generic/tests/main.cc
==============================================================================
--- branches/aptitude-0.3/aptitude/src/generic/tests/main.cc	(original)
+++ branches/aptitude-0.3/aptitude/src/generic/tests/main.cc	Tue Jul 26 03:26:52 2005
@@ -28,7 +28,7 @@
 
   runner.addTest(registry.makeTest());
 
-  bool wasSuccessful = runner.run("", false);
+  bool wasSuccessful = (runner.run("", false) == 0);
 
   return wasSuccessful;
 }