unofficial mirror of guile-devel@gnu.org 
 help / color / mirror / Atom feed
From: Mike Gran <spk121@yahoo.com>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: guile-devel@gnu.org
Subject: Re: On white-box tests
Date: Wed, 19 Aug 2009 07:28:43 -0700	[thread overview]
Message-ID: <1250692123.18373.1014.camel@localhost.localdomain> (raw)
In-Reply-To: <87my5v3o66.fsf@gnu.org>

On Wed, 2009-08-19 at 15:53 +0200, Ludovic Courtès wrote:
> Mike Gran <spk121@yahoo.com> writes:
> 
> > On Wed, 2009-08-19 at 10:38 +0200, Ludovic Courtès wrote:

> I just wanted to hear what you and others thought about the issue,
> because I think unit tests are a crucial part of software development.

OK.  To say something slightly more cogent.  I think when a current
phase of development centers around modifying low-level code, it is
useful to have a set of low-level tests.  If those tests fail, it
reminds the developer that s/he has modified something upon which other
routines rely.

I wrote the string-internals tests to indicate to me when I'd done
something that had unexpected side-effects.  They are intentionally
white-box; they are intentionally reflexive.

There is a danger that those tests, should they remain, could be seen as
indicating software requirements, which they do not.  The software
requirement specification for Scheme (RnRS) is high level and leaves
much of the implementation detail unspecified.

I think it is a good idea to leave them in, probably with comments that
express that they test the implementation, not the specification.   I
also think that it is a good idea to segregate them from tests that
exercise the actual software requirements.

But, I can see an equally valid argument for stripping them out once
strings are no longer in flux, for example at release 2.0, assuming it
is bug free ;-) or perhaps 2.1.

Thanks,

Mike




> 
> Thanks,
> Ludo'.
> 
> 
> 





  reply	other threads:[~2009-08-19 14:28 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <E1Majpl-0008Ga-T9@cvs.savannah.gnu.org>
2009-08-19  8:38 ` On white-box tests Ludovic Courtès
2009-08-19 12:14   ` Mike Gran
2009-08-19 13:53     ` Ludovic Courtès
2009-08-19 14:28       ` Mike Gran [this message]
2009-08-19 14:41         ` Ludovic Courtès

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

  List information: https://www.gnu.org/software/guile/

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

  git send-email \
    --in-reply-to=1250692123.18373.1014.camel@localhost.localdomain \
    --to=spk121@yahoo.com \
    --cc=guile-devel@gnu.org \
    --cc=ludo@gnu.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).