changeset 530:81e43e3770e6

Fixing some small issues with level retrieval with the new file based log implementation.
author wbell
date Sat, 21 Mar 2009 19:05:34 +0000
parents ed37b6a02897
children 5b4a1f1872d3
files bitten/model.py bitten/tests/master.py
diffstat 2 files changed, 5 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/bitten/model.py
+++ b/bitten/model.py
@@ -791,12 +791,12 @@
                 log_lines = codecs.open(log_filename, "rb", "UTF-8").readlines()
             else:
                 log_lines = []
-            level_filename = log.filename + ".levels"
+            level_filename = log.get_log_file(log.filename + ".levels")
             if os.path.exists(level_filename):
-                log_levels = dict(enumerate(codecs.open(log.get_log_file(level_filename), "rb", "UTF-8").readlines()))
+                log_levels = dict(enumerate(codecs.open(level_filename, "rb", "UTF-8").readlines()))
             else:
                 log_levels = {}
-            log.messages = [(log_levels.get(line_num, BuildLog.UNKNOWN), line.rstrip("\n")) for line_num, line in enumerate(log_lines)]
+            log.messages = [(log_levels.get(line_num, BuildLog.UNKNOWN).rstrip("\n"), line.rstrip("\n")) for line_num, line in enumerate(log_lines)]
         else:
             log.messages = []
 
--- a/bitten/tests/master.py
+++ b/bitten/tests/master.py
@@ -370,8 +370,8 @@
             self.assertEqual('http://bitten.cmlenz.net/tools/python#unittest',
                              logs[0].generator)
             self.assertEqual(2, len(logs[0].messages))
-            self.assertEqual((u'', u'Doing stuff'), logs[0].messages[0])
-            self.assertEqual((u'', u'Ouch that hurt'), logs[0].messages[1])
+            self.assertEqual((u'info', u'Doing stuff'), logs[0].messages[0])
+            self.assertEqual((u'error', u'Ouch that hurt'), logs[0].messages[1])
 
     def test_process_build_step_success_with_report(self):
         recipe = """<build>
Copyright (C) 2012-2017 Edgewall Software