From: Andrew Cagney <cagney@redhat.com>
To: Mark Wielaard <mark@klomp.org>
Cc: frysk@sourceware.org
Subject: Re: Testing frysk.testbed.Funit*
Date: Tue, 31 Jul 2007 15:57:00 -0000 [thread overview]
Message-ID: <46AF5BDE.6000806@redhat.com> (raw)
In-Reply-To: <1185876171.3653.64.camel@dijkstra.wildebeest.org>
Mark Wielaard wrote:
> Hi Andrew,
>
> On Mon, 2007-07-30 at 11:47 -0400, Andrew Cagney wrote:
>
>> I'm looking at ways to more directly test the frysk.testbed.Funit*
>> classes (e.g., FunitExec, DetachedAckProcess) that wrap
>> PKGLIBDIR/funit-* utilities, but am finding that the most effective
>> route is to use frysk.proc's framework - duplicating the existing
>> frysk.proc tests exercising frysk.proc functionality that is effectively
>> testing that code.
>>
>> I could duplicate the tests but it seems redundant. Any thoughts on a
>> strategy?
>>
>
>
Mark,
you describe the current state of play; frysk.proc code is testing both
itself internally and the funit tools implicitly. There's nothing
directly testing units such as FunitExecOffspring; instead it is done
implicitly via frysk.proc. That is great when it works, but not so
great when tests fail as differentiating between an FunitExecOffspring
or frysk.proc breakage that caused the fail isn't possible.
As a contrasting example. Say we find the UI is crashing and track it
down to a dwarf binding bug. What we do is add a test-case to the dwarf
bindings testing the problem (which contains the root cause of the
problem); and then fix it. Is the core code is now being tested, we're
confident that our problem won't return. Is there an effective way to
do that here with the Funit* bindings?
Andrew
> I might be missing the exact cases you want to test. But can't you just
> audit the current frysk.proc tests to see if they cover all relevant
> cases already and if not add one or two tests to the existing proc tests
> so all cases are covered? That way you will also extend the real proc
> tests to handle more cases catching two birds with one stone (if that
> isn't a terribly political incorrect saying).
>
> Cheers,
>
> Mark
>
next prev parent reply other threads:[~2007-07-31 15:57 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-07-30 15:47 Andrew Cagney
2007-07-31 10:03 ` Mark Wielaard
2007-07-31 15:57 ` Andrew Cagney [this message]
2007-08-02 8:12 ` Mark Wielaard
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=46AF5BDE.6000806@redhat.com \
--to=cagney@redhat.com \
--cc=frysk@sourceware.org \
--cc=mark@klomp.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 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 read-only IMAP folder(s) and NNTP newsgroup(s).