Patchwork run-test: include test name in the return output

login
register
mail settings
Submitter Pierre-Yves David
Date May 4, 2015, 10:24 p.m.
Message ID <4bb7ebe34af5251078b7.1430778273@marginatus.alto.octopoid.net>
Download mbox | patch
Permalink /patch/8878/
State Accepted
Headers show

Comments

Pierre-Yves David - May 4, 2015, 10:24 p.m.
# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@fb.com>
# Date 1426796120 25200
#      Thu Mar 19 13:15:20 2015 -0700
# Node ID 4bb7ebe34af5251078b76a3a30e41585c59298e7
# Parent  e9edd53770fb77a9787a3e6592a3bf0a29c1bd80
run-test: include test name in the return output

Otherwise using --verbose and --jobs give useless output
Augie Fackler - May 5, 2015, 2:04 p.m.
On Mon, May 04, 2015 at 03:24:33PM -0700, Pierre-Yves David wrote:
> # HG changeset patch
> # User Pierre-Yves David <pierre-yves.david@fb.com>
> # Date 1426796120 25200
> #      Thu Mar 19 13:15:20 2015 -0700
> # Node ID 4bb7ebe34af5251078b76a3a30e41585c59298e7
> # Parent  e9edd53770fb77a9787a3e6592a3bf0a29c1bd80
> run-test: include test name in the return output

Queued with a tweaked description, thanks.

>
> Otherwise using --verbose and --jobs give useless output
>
> diff --git a/tests/run-tests.py b/tests/run-tests.py
> --- a/tests/run-tests.py
> +++ b/tests/run-tests.py
> @@ -618,11 +618,11 @@ class Test(unittest.TestCase):
>              f = open(self.errpath, 'wb')
>              for line in self._out:
>                  f.write(line)
>              f.close()
>
> -        vlog("# Ret was:", self._ret)
> +        vlog("# Ret was:", self._ret, '(%s)' % self.name)
>
>      def _run(self, env):
>          # This should be implemented in child classes to run tests.
>          raise SkipTest('unknown test type')
>
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel@selenic.com
> http://selenic.com/mailman/listinfo/mercurial-devel

Patch

diff --git a/tests/run-tests.py b/tests/run-tests.py
--- a/tests/run-tests.py
+++ b/tests/run-tests.py
@@ -618,11 +618,11 @@  class Test(unittest.TestCase):
             f = open(self.errpath, 'wb')
             for line in self._out:
                 f.write(line)
             f.close()
 
-        vlog("# Ret was:", self._ret)
+        vlog("# Ret was:", self._ret, '(%s)' % self.name)
 
     def _run(self, env):
         # This should be implemented in child classes to run tests.
         raise SkipTest('unknown test type')