public inbox for mauve-discuss@sourceware.org
 help / color / mirror / Atom feed
From: Thomas Zander <zander@javalobby.org>
To: mauve-discuss@sources.redhat.com
Cc: Jeff Sturm <jsturm@one-point.com>,
	Michael Koch <konqueror@gmx.de>,
	GNU Classpath Project <classpath@gnu.org>
Subject: Re: Eclipse and Classpath
Date: Wed, 28 Apr 2004 21:56:00 -0000	[thread overview]
Message-ID: <200404282354.50797.zander@javalobby.org> (raw)
In-Reply-To: <877jvzhoef.fsf@fleche.redhat.com>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Wednesday 28 April 2004 21:51, Tom Tromey wrote:
> Jeff> (What about Mauve?  Much of Eclipse's usefulness for us comes from
> Jeff> interactively running the test suite so we get a very quick
> Jeff> edit/compile/test/debug cycle.  But eclipse only knows about
> junit-style Jeff> tests, not the gnu.testlet classes in Mauve.)
>
> I think Graydon changed Mauve to use JUnit at one point, but it never
> went in.  I think there was some confusion about some points I made.
>
> It's time to revisit this (various folks pointed this out to me at the
> recent RH meeting).  IMO anything that makes Mauve easier to hack is a
> good thing.  My only real requirement is that we still be able to run
> the gcj test suite; I don't think there is any substantial problem
> here.

IIRC the biggest problem with using JUnit is that it depends of rather 
advanced JVM features; like reflection.

Its technically possible to have a mauve test case extend a 
mauve-base-class which can be changed on run-time (probably using a 
singleton) to either use the very-simple-mauve test framework, or the 
JUnit ones. Using a facade pattern in that baseclass.
Its still needed to call each test in one long test() method if you want to 
use the mauve version, but other then that it would integrate with JUnit 
quite nicely.

Naturally all tests will need to be refactored to do this..
- - extending a base class instead of implementing Testlet
- - using the assertTrue() style checking instead of the check() versions.

Thoughts?

- -- 
Thomas
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFAkCgpCojCW6H2z/QRAp0LAKCxgcsGMzDHdMPDDVDTPUBNPCQjGgCgkU8j
qAf1uGZxFCYoTpSJreFpitY=
=nRwQ
-----END PGP SIGNATURE-----

  reply	other threads:[~2004-04-28 21:56 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <Pine.LNX.4.44.0404241127510.5951-100000@ops2.one-point.com>
2004-04-28 20:02 ` Tom Tromey
2004-04-28 21:56   ` Thomas Zander [this message]
2004-04-30 23:22     ` Tom Tromey
     [not found] <OFD30D19BA.CBC8235A-ON85256E84.007C2292-85256E84.007C73EC@us.ibm.com>
2004-04-28 22:54 ` Archie Cobbs
2004-04-29  5:09   ` C. Brian Jones

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=200404282354.50797.zander@javalobby.org \
    --to=zander@javalobby.org \
    --cc=classpath@gnu.org \
    --cc=jsturm@one-point.com \
    --cc=konqueror@gmx.de \
    --cc=mauve-discuss@sources.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 read-only IMAP folder(s) and NNTP newsgroup(s).