From: Thierry Parmentelat <thierry.parmentelat@inria.fr>
Date: Wed, 29 Jan 2014 09:59:37 +0000 (+0100)
Subject: cosmetic improvement for the daily mail
X-Git-Tag: tests-5.3-2~21
X-Git-Url: http://git.onelab.eu/?p=tests.git;a=commitdiff_plain;h=0e2563af12488bea959a6c3b76ee74d396a46070

cosmetic improvement for the daily mail
---

diff --git a/system/Substrate.py b/system/Substrate.py
index b75ed5c..304abf1 100644
--- a/system/Substrate.py
+++ b/system/Substrate.py
@@ -882,15 +882,17 @@ class TestInstance:
         # first letter : '=', unless build is running : '*'
         double = '*' if self.pids else '='
         # second letter : '=' if fine, 'W' for warnings (only ignored steps) 'B' for broken
-        double += self.second_letter()
+        letter2 = self.second_letter()
+        double += letter2
         msg = " %s %s =="%(double,self.buildname)
         if not self.pids:       pass
         elif len(self.pids)==1: msg += " (pid=%s)"%self.pids[0]
         else:                   msg += " !!!pids=%s!!!"%self.pids
         msg += " @%s"%self.pretty_timestamp()
-        if self.broken_steps:
+        if letter2 != '=':
+            msg = 'BROKEN' if letter2 == 'B' else 'WARNING'
             # sometimes we have an empty plcindex
-            msg += " [BROKEN=" + " ".join( [ "%s@%s"%(s,i) if i else s for (i,s) in self.broken_steps ] ) + "]"
+            msg += " [%s="%msg + " ".join( [ "%s@%s"%(s,i) if i else s for (i,s) in self.broken_steps ] ) + "]"
         return msg
 
 class TestBox (Box):