Author: pekka.klarck
Date: Sun Apr  5 09:51:14 2009
New Revision: 1676

Modified:
   trunk/src/robot/serializing/reportserializers.py

Log:
In test details table, use only 'longname' with suites and 'name' with tests. Old behavior was to use 'mediumname' with both.

Modified: trunk/src/robot/serializing/reportserializers.py
==============================================================================
--- trunk/src/robot/serializing/reportserializers.py    (original)
+++ trunk/src/robot/serializing/reportserializers.py Sun Apr 5 09:51:14 2009
@@ -43,8 +43,8 @@
     def _start_suite_or_test_row(self, item, type_):
         self._writer.start('tr', {'class': '%s_row' % type_})
         self._writer.start('td', {'class': 'col_name'}, newline=False)
-        elem, attrs = self._get_name_params(item, type_)
-        self._writer.element(elem, item.mediumname, attrs, newline=False)
+        name, elem, attrs = self._get_name_params(item, type_)
+        self._writer.element(elem, name, attrs, newline=False)
         self._writer.end('td')
         self._writer.element('td', item.htmldoc, {'class': 'col_doc'},
                              escape=False)
@@ -56,7 +56,12 @@
attrs['href'] = '%s#%s_%s' % (item.linkpath, type_, item.linkname)
         else:
             elem = 'span'
-        return elem, attrs
+        return self._get_name(item, type_), elem, attrs
+
+    def _get_name(self, item, type_):
+        if type_ == 'suite':
+            return item.longname
+        return item.name

     def _end_test_row(self, test):
self._writer.element('td', ', '.join(test.tags), {'class': 'col_tags'})
@@ -112,7 +117,7 @@
     def start_test(self, test):
         self._set_test_link(test)
         self._test_row(test)
-
+
     def _set_suite_link(self, suite):
         # linkpath and linkname are also used when TagStats are serialized.
         # This is rather ugly and should be refactored at some point.
@@ -186,6 +191,9 @@
         self._tag_row(stat)
         for test in stat.tests:
             self._test_row(test)
+
+    def _get_name(self, stat, type_is_ignored):
+        return stat.mediumname

     def _tag_row(self, stat):
         self._writer.start('tr', {'class': 'tag_row'})

Reply via email to