From c211d61f852600c19b3db8d6aa1a7f619866c207 Mon Sep 17 00:00:00 2001 From: Monty Taylor Date: Wed, 20 Jun 2012 10:24:22 -0700 Subject: [PATCH] Don't output times for skipped tests. Brian Waldon hates it when skipped tests show times. Change-Id: I69abed90ed5fdf5bd1d2b8c725d4913660fa0123 --- openstack/nose_plugin.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/openstack/nose_plugin.py b/openstack/nose_plugin.py index 46ac21c..8086f44 100644 --- a/openstack/nose_plugin.py +++ b/openstack/nose_plugin.py @@ -65,7 +65,8 @@ class Openstack(plugins.Plugin): else: return None, parts[0] - def _writeResult(self, test, long_result, color, short_result): + def _writeResult(self, test, long_result, color, short_result, + show_elapsed=True): if isinstance(test, suite.ContextSuite): return @@ -79,7 +80,7 @@ class Openstack(plugins.Plugin): if self.show_all: self.colorizer.write(long_result, color) - if self.show_elapsed: + if self.show_elapsed and show_elapsed: color = self._get_color(elapsed) self.colorizer.write(" %.2f" % elapsed, color) self.stream.writeln() @@ -123,7 +124,7 @@ class Openstack(plugins.Plugin): name = self._get_name(test) self.times[name].append(time.time()) - self._writeResult(test, "SKIP", "blue", "S") + self._writeResult(test, "SKIP", "blue", "S", show_elapsed=False) self._result_addSkip(test, reason) def _print_errors(self):