[Libguestfs] [PATCH supermin] build: use a custom test driver
Pino Toscano
ptoscano at redhat.com
Fri Nov 6 13:12:50 UTC 2015
On Friday 06 November 2015 13:00:03 Richard W.M. Jones wrote:
> On Fri, Nov 06, 2015 at 01:24:07PM +0100, Pino Toscano wrote:
> > Use a custom test driver for running the tests: based on the test-driver
> > provided by automake, it adds the running time of the test in each .trs
> > file.
> > ---
> > configure.ac | 1 +
> > guestfs-test-driver | 151 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>
> supermin-test-driver ..?
Can do.
> I applied this to the supermin tree to try and see how it worked,
> but I can't see what it's supposed to do. I still see the usual
> `PASS:' lines in the output.
Yes, that's expected. The difference is just in what gets in the
resulting .trs files, which now have a line like:
:guestfs-time: 1
which indicates how much time each test took; this way, we can parse it
and improve the resulting XML for junit (produced by
tests/automake2junit.ml in CI).
The plan is to apply the same also to libguestfs when switching to
parallel-tests, so we can get proper test results for CI even without
$RUN_OUTPUT_FILE.
> I don't know whether or not you use emacs, but:
>
> > +# Local Variables:
> > +# mode: shell-script
> > +# sh-indentation: 2
> > +# eval: (add-hook 'write-file-hooks 'time-stamp)
> > +# time-stamp-start: "scriptversion="
> > +# time-stamp-format: "%:y-%02m-%02d.%02H"
> > +# time-stamp-time-zone: "UTC"
> > +# time-stamp-end: "; # UTC"
> > +# End:
>
> are super annoying. It even popped up when I opened the reply to this
> email message.
This is part of the test-driver file in upstream automake (which gets
copied in the build tree). Should I remove them from the local copy?
--
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20151106/2924c321/attachment.sig>
More information about the Libguestfs
mailing list