public inbox for frysk@sourceware.org
 help / color / mirror / Atom feed
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
>   

  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).