All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
	Eduardo Habkost <ehabkost@redhat.com>,
	Michael Roth <mdroth@linux.vnet.ibm.com>,
	Don Slutz <dslutz@verizon.com>,
	qemu-devel@nongnu.org, Anthony Liguori <aliguori@amazon.com>
Subject: Re: [Qemu-devel] [PULL 03/12] test-qdev-global-props: Run tests on subprocess
Date: Thu, 18 Sep 2014 20:38:27 +0300	[thread overview]
Message-ID: <20140918173827.GA27433@redhat.com> (raw)
In-Reply-To: <541B1108.6090708@redhat.com>

On Thu, Sep 18, 2014 at 07:06:16PM +0200, Paolo Bonzini wrote:
> Il 18/09/2014 18:29, Michael Roth ha scritto:
> > <snip>
> > 
> >> >  int main(int argc, char **argv)
> >> >  {
> >> >      g_test_init(&argc, &argv, NULL);
> >> > @@ -174,9 +200,20 @@ int main(int argc, char **argv)
> >> >      type_register_static(&static_prop_type);
> >> >      type_register_static(&dynamic_prop_type);
> >> > 
> >> > -    g_test_add_func("/qdev/properties/static/default", test_static_prop);
> >> > -    g_test_add_func("/qdev/properties/static/global", test_static_globalprop);
> >> > -    g_test_add_func("/qdev/properties/dynamic/global", test_dynamic_globalprop);
> >> > +    g_test_add_func("/qdev/properties/static/default/subprocess",
> >> > +                    test_static_prop_subprocess);
> >> > +    g_test_add_func("/qdev/properties/static/default",
> >> > +                    test_static_prop);
> > Since in the code above test_static_prop is actually the test that re-runs
> > /qdev/properties/static/default/subprocess under g_test_trap_subprocess, aren't
> > the tests (or test function implementations) backwards?
> > 
> 
> No, it's correct.  The parent test is the parent function, the child
> test is the subprocess function.  The child test is automagically
> skipped by GTest, I don't know how that works.
> 
> Paolo

Based on the "subprocess" string in the path.

  reply	other threads:[~2014-09-18 17:35 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-09-14 18:41 [Qemu-devel] [PULL 00/12] pci, pc, virtio, misc bugfixes Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 01/12] hw/machine: Free old values of string properties Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 02/12] test-qdev-global-props: Trivial comment fix Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 03/12] test-qdev-global-props: Run tests on subprocess Michael S. Tsirkin
2014-09-18 16:29   ` Michael Roth
2014-09-18 17:06     ` Paolo Bonzini
2014-09-18 17:38       ` Michael S. Tsirkin [this message]
2014-09-18 17:27     ` Eduardo Habkost
2014-09-18 17:44       ` Michael Roth
2014-09-14 18:41 ` [Qemu-devel] [PULL 04/12] test-qdev-global-props: Initialize not_used=true for all props Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 05/12] test-qdev-global-props: Test handling of hotpluggable and non-device types Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 06/12] qdev: Rename qdev_prop_check_global() to qdev_prop_check_globals() Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 07/12] qdev: Move global validation to a single function Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 08/12] Revert "rng-egd: remove redundant free" Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 09/12] virtio-net: drop assert on vm stop Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 10/12] Revert "virtio: don't call device on !vm_running" Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 11/12] virtio-pci: enable bus master for old guests Michael S. Tsirkin
2014-09-14 18:41 ` [Qemu-devel] [PULL 12/12] vhost-user: fix VIRTIO_NET_F_MRG_RXBUF negotiation Michael S. Tsirkin
2014-09-16  7:06   ` Linhaifeng
2014-09-16 15:57   ` Michael S. Tsirkin
2014-09-15 20:30 ` [Qemu-devel] [PULL 00/12] pci, pc, virtio, misc bugfixes Peter Maydell
2014-09-16 14:43   ` Michael S. Tsirkin
2014-09-16 14:07     ` Paolo Bonzini
2014-09-16 14:52       ` Michael S. Tsirkin
2014-09-18 16:37         ` Michael Roth

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140918173827.GA27433@redhat.com \
    --to=mst@redhat.com \
    --cc=aliguori@amazon.com \
    --cc=dslutz@verizon.com \
    --cc=ehabkost@redhat.com \
    --cc=mdroth@linux.vnet.ibm.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.