[Boost-build] Boost Build test case count.

Subject: [Boost-build] Boost Build test case count.
From: Jurko Gospodnetiæ (jurko.gospodnetic_at_[hidden])
Date: 2009-02-22 11:13:08

   Hi.

   I was wondering why Boost Build's test system (run by test_all.py) 
reports the total number of tests run as 'PASS: ##' and the total number 
of failed tests as 'FAIL: ##'.

   Would it not be better to report the number of successfully passed 
tests as 'PASS: ##'?

   Attached is a patch implementing this change.

   Description:
----------------------
   Makes the Boost Build testing system correctly report the number of 
passed test cases instead of always reporting all test cases as passed.
----------------------

   I do not feel comfortable applying the patch directly since I've been 
away from Boost Build for a while I'm afraid to mess up the new test 
system and its integration to the Boost Library test suite. Although, at 
a second glance this value does not seem to be used with xml result 
logging and my hunch is that that is the system used to deliver the test 
results to the Boost Library test reporting framework.

   Hope this helps.

   Best regards,
     Jurko Gospodnetiæ

Index: tools/build/v2/test/test_all.py
===================================================================
--- tools/build/v2/test/test_all.py	(revision 51391)
+++ tools/build/v2/test/test_all.py	(working copy)
@@ -80,8 +80,8 @@
 </run>
 </test-log>
 """
-
-        pass_count = pass_count + 1
+        if passed:
+            ++pass_count
         sys.stdout.flush()  # Makes testing under emacs more entertaining.
 
     # Erase the file on success.