[pypy-svn] r68605 - in pypy/build/bot2/pypybuildbot: . test

[arigo at codespeak.net]

Author: arigo
Date: Sat Oct 17 19:33:59 2009
New Revision: 68605

Modified:
   pypy/build/bot2/pypybuildbot/summary.py
   pypy/build/bot2/pypybuildbot/test/test_summary.py
Log:
Fix the color of the "-" signs.


Modified: pypy/build/bot2/pypybuildbot/summary.py
==============================================================================
--- pypy/build/bot2/pypybuildbot/summary.py	(original)
+++ pypy/build/bot2/pypybuildbot/summary.py	Sat Oct 17 19:33:59 2009
@@ -419,17 +419,15 @@
         if self.SUCCESS_LINE:
             success = []
             for label, outcome_set in by_label:
-                symbol = "+"
-                clazz = "success"
                 if outcome_set.failed:
-                    clazz = "failed"
                     symbol = html.a("-",
                         id="a%dc%d" % (a_num, 1<<len(success)),
                         href="javascript:togglestate(%d,%d)" % (
-                                       a_num, 1<<len(success)))
-                success.append([" ",
-                                html.span(symbol,
-                                          class_="failSummary "+clazz)])
+                                       a_num, 1<<len(success)),
+                        class_="failSummary failed")
+                else:
+                    symbol = html.span("+", class_="failSummary success")
+                success.append([" ", symbol])
             success.append("  success\n")
             lines.append(success)
             

Modified: pypy/build/bot2/pypybuildbot/test/test_summary.py
==============================================================================
--- pypy/build/bot2/pypybuildbot/test/test_summary.py	(original)
+++ pypy/build/bot2/pypybuildbot/test/test_summary.py	Sat Oct 17 19:33:59 2009
@@ -491,7 +491,7 @@
 
         assert 'TEST1' in out
         assert ':-)' in out
-        assert '\n <span class="failSummary failed"><a href="javascript:togglestate(1,1)" id="a1c1">-</a></span> <span class="failSummary success">+</span>  success' in out
+        assert '\n <a class="failSummary failed" href="javascript:togglestate(1,1)" id="a1c1">-</a> <span class="failSummary success">+</span>  success' in out
 
     def test_two_builds_samerev(self):
         builder = status_builder.BuilderStatus('builder0')