From: Thomas Huth <thuth@redhat.com>
To: Wainer dos Santos Moschetta <wainersm@redhat.com>, qemu-devel@nongnu.org
Cc: qemu-trivial@nongnu.org, pbonzini@redhat.com
Subject: Re: [PATCH] docs/devel/testing.rst: Fix referencies to unit tests
Date: Thu, 18 Mar 2021 06:10:32 +0100 [thread overview]
Message-ID: <f52cb9c9-e962-ec3d-54e6-a058e5466b03@redhat.com> (raw)
In-Reply-To: <20210317194849.2262346-1-wainersm@redhat.com>
On 17/03/2021 20.48, Wainer dos Santos Moschetta wrote:
> With the recent move of the unit tests to tests/unit directory some
> instructions under the "Unit tests" section became imprecise, which
> are fixed by this change.
>
> Related-to: da668aa15b99 (tests: Move unit tests into a separate directory)
> Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
> ---
> docs/devel/testing.rst | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/docs/devel/testing.rst b/docs/devel/testing.rst
> index 1434a50cc4..1da4c4e4c4 100644
> --- a/docs/devel/testing.rst
> +++ b/docs/devel/testing.rst
> @@ -34,17 +34,17 @@ If you are writing new code in QEMU, consider adding a unit test, especially
> for utility modules that are relatively stateless or have few dependencies. To
> add a new unit test:
>
> -1. Create a new source file. For example, ``tests/foo-test.c``.
> +1. Create a new source file. For example, ``tests/unit/foo-test.c``.
>
> 2. Write the test. Normally you would include the header file which exports
> the module API, then verify the interface behaves as expected from your
> test. The test code should be organized with the glib testing framework.
> Copying and modifying an existing test is usually a good idea.
>
> -3. Add the test to ``tests/meson.build``. The unit tests are listed in a
> +3. Add the test to ``tests/unit/meson.build``. The unit tests are listed in a
> dictionary called ``tests``. The values are any additional sources and
> dependencies to be linked with the test. For a simple test whose source
> - is in ``tests/foo-test.c``, it is enough to add an entry like::
> + is in ``tests/unit/foo-test.c``, it is enough to add an entry like::
>
> {
> ...
>
Reviewed-by: Thomas Huth <thuth@redhat.com>
prev parent reply other threads:[~2021-03-18 5:12 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-17 19:48 [PATCH] docs/devel/testing.rst: Fix referencies to unit tests Wainer dos Santos Moschetta
2021-03-17 21:55 ` John Snow
2021-03-18 12:54 ` Wainer dos Santos Moschetta
2021-03-18 5:10 ` Thomas Huth [this message]
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=f52cb9c9-e962-ec3d-54e6-a058e5466b03@redhat.com \
--to=thuth@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-trivial@nongnu.org \
--cc=wainersm@redhat.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).