Running Mauve tests with JUnit

Running Mauve tests with JUnit

[David Gilbert]

Hi All,

A while back there was some discussion about converting Mauve tests to 
run under JUnit.  I did some experimentation and have put together the 
following patch that shows one approach to making this work.  I created 
a new JUnitRunnable class which makes it possible to run a Mauve testlet 
under JUnit (as well as continuing to support the standard Mauve 
runners).  It requires that the existing testlets be modified to 'extend 
JUnitRunnable' (or, where this isn't possible, an alternative 
JUnitWrapper class can be used, see AttributeSetTests.java for an 
example).  The patch includes conversion of the tests in the 
javax.swing.text.* package so you can at least try it out - if this 
patch is approved, I also have done the conversion of all the Swing 
tests (roughly 25% of Mauve I think).

Here is a link to the patch, it's probably over the size limit for the 
mailing list (the ChangeLog entry is at the end of this e-mail):

http://www.object-refinery.com/classpath/diff.txt

I'm actually indifferent about whether or not this patch is accepted, I 
find Mauve slightly easier to work with relative to JUnit for the type 
of tests we write for GNU Classpath and plan to continue writing tests 
in the Mauve format.  Anyway, here's the pros and cons that I can think 
of, please feel free to add to the list:

Pros:
- JUnit is integrated with Eclipse, so it can be used to run Mauve tests 
easily for developers that work in Eclipse;
- most Java developers are familiar with JUnit, so Mauve may reach a 
wider audience;
- other tools can work with JUnit (although I haven't used any of them - 
does anyone know of a JUnit runner that can produce comparisons of two 
runs? I'm interested in (a) regression reports (e.g. CVS vs last 
release) and (b) comparison reports (GNU Classpath vs Sun JDK1.5));

Cons:
- a lot of work to modify the remaining tests in Mauve;
- one extra step to make Mauve tests that run with both JUnit and Mauve 
test runners - it's pretty easy, but still a mental step that new 
contributors to Mauve will have to learn;
- there is a possibility that some developers will write new tests 
against JUnit only, so we'll lose the ability to run all tests with the 
Mauve runners (which I think provide better detail on failures);

Oh, and someone mentioned that the JUnit licence is not GPL compatible, 
so I compiled the tests against the GPLed code here:

http://cvs.sourceforge.net/viewcvs.py/freenet/Contrib/junit/

I had to add a no-arg constructor for the TestCase class, then the tests 
compiled OK.  And I can use the SimpleTestRunner to run a single test 
(this could do with some more work though).

Please comment!

Regards,

Dave

P.S. Here is the ChangeLog entry:

2006-03-06  David Gilbert  <david.gilbert@object-refinery.com>

    * gnu/testlet/JUnitRunnable.java: New file,
    * gnu/testlet/JUnitTestHarness.java: Likewise,
    * gnu/testlet/JUnitTestWrapper.java: Likewise,
    * gnu/testlet/JUnitWrapper.java: Likewise,
    * gnu/testlet/javax/swing/text/PackageTestSuite.java: Likewise,
    * 
gnu/testlet/javax/swing/text/AbstractDocument/AbstractDocumentTest.java: 
Now extends JUnitRunnable,
    * 
gnu/testlet/javax/swing/text/AbstractDocument/AbstractDocumentTests.java: 
New file,
    * gnu/testlet/javax/swing/text/AbstractDocument/ElementChange.java: 
Now extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/AbstractDocument/ElementChange2.java: 
Likewise,
    * 
gnu/testlet/javax/swing/text/AbstractDocument/BranchElement/BranchElementTest.java: 
Updated header,
    * 
gnu/testlet/javax/swing/text/AbstractDocument/BranchElement/BranchElementTests.java: 
New file,
    * 
gnu/testlet/javax/swing/text/AbstractDocument/BranchElement/getElementIndexNullPointer.java: 
Now extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/AttributeSet/AttributeSetTests.java: 
New file,
    * gnu/testlet/javax/swing/text/BoxView/BoxViewTests.java: Likewise,
    * gnu/testlet/javax/swing/text/BoxView/spans.java: Now extends 
JUnitRunnable,
    * 
gnu/testlet/javax/swing/text/DefaultEditorKit/DefaultEditorKitTests.java: 
New file,
    * gnu/testlet/javax/swing/text/DefaultEditorKit/getActions.java: Now 
extends JUnitRunnable,
    * 
gnu/testlet/javax/swing/text/DefaultFormatter/DefaultFormatterTests.java: 
New file,
    * gnu/testlet/javax/swing/text/DefaultFormatter/getValueClass.java: 
Now extends JUnitRunnable,
    * 
gnu/testlet/javax/swing/text/DefaultStyledDocument/DefaultStyledDocumentTests.java: 
New file,
    * 
gnu/testlet/javax/swing/text/DefaultStyledDocument/insertString.java: 
Now extends JUnitRunnable,
    * 
gnu/testlet/javax/swing/text/DefaultStyledDocument/ElementBuffer/ElementBufferTests.java: 
New file,
    * 
gnu/testlet/javax/swing/text/DefaultStyledDocument/ElementBuffer/insert.java: 
Now extends JUnitRunnable,
    * 
gnu/testlet/javax/swing/text/ElementIterator/ElementIteratorTest.java: 
Likewise,
    * 
gnu/testlet/javax/swing/text/ElementIterator/ElementIteratorTests.java: 
New file,
    * gnu/testlet/javax/swing/text/FlowView/FlowViewTests.java: New file,
    * gnu/testlet/javax/swing/text/FlowView/getFlowAxis.java: Now 
extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/GapContent/GapContentTest.java: Likewise,
    * gnu/testlet/javax/swing/text/GapContent/GapContentTests.java: New 
file,
    * gnu/testlet/javax/swing/text/GapContent/PositionTest.java: Now 
extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/GapContent/constructors.java: Likewise,
    * gnu/testlet/javax/swing/text/GapContent/createPosition.java: Likewise,
    * gnu/testlet/javax/swing/text/GapContent/getChars.java: Likewise,
    * gnu/testlet/javax/swing/text/GapContent/getString.java: Likewise,
    * gnu/testlet/javax/swing/text/GapContent/insertString.java: Likewise,
    * gnu/testlet/javax/swing/text/GapContent/length.java: Likewise,
    * gnu/testlet/javax/swing/text/GapContent/remove.java: Likewise,
    * 
gnu/testlet/javax/swing/text/InternationalFormatter/InternationalFormatterTest.java: 
Likewise,
    * 
gnu/testlet/javax/swing/text/InternationalFormatter/InternationalFormatterTests.java: 
New file,
    * gnu/testlet/javax/swing/text/MaskFormatter/MaskFormatterTest.java: 
Now extends JUnitRunnable,
    * 
gnu/testlet/javax/swing/text/MaskFormatter/MaskFormatterTests.java: New 
file,
    * gnu/testlet/javax/swing/text/PlainDocument/PlainDocumentTest.java: 
Now extends JUnitRunnable,
    * 
gnu/testlet/javax/swing/text/PlainDocument/PlainDocumentTests.java: New 
file,
    * gnu/testlet/javax/swing/text/PlainDocument/createPosition.java: 
Now extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/PlainDocument/getLength.java: Likewise,
    * gnu/testlet/javax/swing/text/PlainDocument/getRootElements.java: 
Likewise,
    * gnu/testlet/javax/swing/text/PlainDocument/getText.java: Likewise,
    * gnu/testlet/javax/swing/text/PlainDocument/insertString.java: 
Likewise,
    * gnu/testlet/javax/swing/text/PlainDocument/multipleLeafs.java: 
Likewise,
    * gnu/testlet/javax/swing/text/PlainDocument/remove.java: Likewise,
    * gnu/testlet/javax/swing/text/PlainDocument/removeJoinesLines.java: 
Likewise,
    * gnu/testlet/javax/swing/text/Segment/SegmentTests.java: New file,
    * gnu/testlet/javax/swing/text/Segment/clone.java: Now extends 
JUnitRunnable,
    * gnu/testlet/javax/swing/text/Segment/constructors.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/first.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/getBeginIndex.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/getEndIndex.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/getIndex.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/isPartialReturn.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/last.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/next.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/previous.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/setIndex.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/setPartialReturn.java: Likewise,
    * gnu/testlet/javax/swing/text/Segment/toString(), Likewise,
    * 
gnu/testlet/javax/swing/text/SimpleAttributeSet/SimpleAttributeSetTests.java: 
New file,
    * 
gnu/testlet/javax/swing/text/StringContent/BadLocationExceptionTest.java: 
Now extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/StringContent/StringContentTest.java: 
Likewise,
    * 
gnu/testlet/javax/swing/text/StringContent/StringContentTests.java: New 
file,
    * gnu/testlet/javax/swing/text/StringContent/constructors.java: Now 
extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/StringContent/createPosition.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StringContent/getChars.java: Likewise,
    * gnu/testlet/javax/swing/text/StringContent/getString.java: Likewise,
    * gnu/testlet/javax/swing/text/StringContent/insertString.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StringContent/insertUndo.java: Likewise,
    * gnu/testlet/javax/swing/text/StringContent/length.java: Likewise,
    * gnu/testlet/javax/swing/text/StringContent/remove.java: Likewise,
    * gnu/testlet/javax/swing/text/StringContent/removeUndo.java: Likewise,
    * gnu/testlet/javax/swing/text/StringContent/stickyPosition.java: 
Likewise,
    * 
gnu/testlet/javax/swing/text/StyleConstants/StyleConstantsTests.java: 
New file,
    * gnu/testlet/javax/swing/text/StyleConstants/constants.java: Now 
extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/StyleConstants/getAlignment.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getBackground.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getBidiLevel.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getComponent.java: 
Likewise,
    * 
gnu/testlet/javax/swing/text/StyleConstants/getFirstLineIndent.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getFontFamily.java: 
Liekwise,
    * gnu/testlet/javax/swing/text/StyleConstants/getFontSize.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getForeground.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getIcon.java: Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getLeftIndent.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getLineSpacing.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getRightIndent.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getSpaceAbove.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getSpaceBelow.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/getTabSet.java: Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/isBold.java: Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/isItalic.java: Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/isStrikeThrough.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/isSubscript.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/isSuperscript.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/isUnderline.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setAlignment.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setBackground.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setBidiLevel.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setBold.java: Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setComponent.java: 
Likewise,
    * 
gnu/testlet/javax/swing/text/StyleConstants/setFirstLineIndent.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setFontFamily.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setFontSize.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setForeground.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setIcon.java: Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setItalic.java: Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setLeftIndent.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setLineSpacing.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setRightIndent.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setSpaceAbove.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setSpaceBelow.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setStrikeThrough.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setSubscript.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setSuperscript.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setTabSet.java: Likewise,
    * gnu/testlet/javax/swing/text/StyleConstants/setUnderline.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleContext/NamedStyleInit.java: 
Likewise,
    * 
gnu/testlet/javax/swing/text/StyleContext/NamedStyleSetResolveParent.java: 
Likewise,
    * gnu/testlet/javax/swing/text/StyleContext/StyleContextTests.java: 
New file,
    * gnu/testlet/javax/swing/text/StyleContext/addAttribute.java: Now 
extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/StyleContext/addStyle.java: Likewise,
    * 
gnu/testlet/javax/swing/text/StyledEditorKit/StyledEditorKitTests.java: 
New file,
    * 
gnu/testlet/javax/swing/text/StyledEditorKit/createInputAttributesTest.java: 
Now extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/TextAction/TextActionTests.java: New 
file,
    * gnu/testlet/javax/swing/text/TextAction/augmentList.java: Now 
extends JUnitRunnable,
    * gnu/testlet/javax/swing/text/View/ViewTests.java: New file,
    * gnu/testlet/javax/swing/text/View/getAlignment.java: Now extends 
JUnitRunnable,
    * gnu/testlet/javax/swing/text/View/getMaximumSpan.java: Likewise,
    * gnu/testlet/javax/swing/text/View/getMinimumSpan.java: Likewise,
    * gnu/testlet/javax/swing/text/View/getResizeWeight.java: Likewise.   

Re: Running Mauve tests with JUnit

[David Gilbert]

I didn't get any feedback about this...anyone think it is a good/bad idea?

Regards,

Dave

David Gilbert wrote:

> Hi All,
>
> A while back there was some discussion about converting Mauve tests to 
> run under JUnit.  I did some experimentation and have put together the 
> following patch that shows one approach to making this work.  I 
> created a new JUnitRunnable class which makes it possible to run a 
> Mauve testlet under JUnit (as well as continuing to support the 
> standard Mauve runners).  It requires that the existing testlets be 
> modified to 'extend JUnitRunnable' (or, where this isn't possible, an 
> alternative JUnitWrapper class can be used, see AttributeSetTests.java 
> for an example).  The patch includes conversion of the tests in the 
> javax.swing.text.* package so you can at least try it out - if this 
> patch is approved, I also have done the conversion of all the Swing 
> tests (roughly 25% of Mauve I think).
>
> Here is a link to the patch, it's probably over the size limit for the 
> mailing list (the ChangeLog entry is at the end of this e-mail):
>
> http://www.object-refinery.com/classpath/diff.txt
>
> I'm actually indifferent about whether or not this patch is accepted, 
> I find Mauve slightly easier to work with relative to JUnit for the 
> type of tests we write for GNU Classpath and plan to continue writing 
> tests in the Mauve format.  Anyway, here's the pros and cons that I 
> can think of, please feel free to add to the list:
>
> Pros:
> - JUnit is integrated with Eclipse, so it can be used to run Mauve 
> tests easily for developers that work in Eclipse;
> - most Java developers are familiar with JUnit, so Mauve may reach a 
> wider audience;
> - other tools can work with JUnit (although I haven't used any of them 
> - does anyone know of a JUnit runner that can produce comparisons of 
> two runs? I'm interested in (a) regression reports (e.g. CVS vs last 
> release) and (b) comparison reports (GNU Classpath vs Sun JDK1.5));
>
> Cons:
> - a lot of work to modify the remaining tests in Mauve;
> - one extra step to make Mauve tests that run with both JUnit and 
> Mauve test runners - it's pretty easy, but still a mental step that 
> new contributors to Mauve will have to learn;
> - there is a possibility that some developers will write new tests 
> against JUnit only, so we'll lose the ability to run all tests with 
> the Mauve runners (which I think provide better detail on failures);
>
> Oh, and someone mentioned that the JUnit licence is not GPL 
> compatible, so I compiled the tests against the GPLed code here:
>
> http://cvs.sourceforge.net/viewcvs.py/freenet/Contrib/junit/
>
> I had to add a no-arg constructor for the TestCase class, then the 
> tests compiled OK.  And I can use the SimpleTestRunner to run a single 
> test (this could do with some more work though).
>
> Please comment!
>
> Regards,
>
> Dave
>
> P.S. Here is the ChangeLog entry:
>
> 2006-03-06  David Gilbert  <david.gilbert@object-refinery.com>
>
>    * gnu/testlet/JUnitRunnable.java: New file,
>    * gnu/testlet/JUnitTestHarness.java: Likewise,
>    * gnu/testlet/JUnitTestWrapper.java: Likewise,
>    * gnu/testlet/JUnitWrapper.java: Likewise,
>    * gnu/testlet/javax/swing/text/PackageTestSuite.java: Likewise,
>    * 
> gnu/testlet/javax/swing/text/AbstractDocument/AbstractDocumentTest.java: 
> Now extends JUnitRunnable,
>    * 
> gnu/testlet/javax/swing/text/AbstractDocument/AbstractDocumentTests.java: 
> New file,
>    * gnu/testlet/javax/swing/text/AbstractDocument/ElementChange.java: 
> Now extends JUnitRunnable,
>    * 
> gnu/testlet/javax/swing/text/AbstractDocument/ElementChange2.java: 
> Likewise,
>    * 
> gnu/testlet/javax/swing/text/AbstractDocument/BranchElement/BranchElementTest.java: 
> Updated header,
>    * 
> gnu/testlet/javax/swing/text/AbstractDocument/BranchElement/BranchElementTests.java: 
> New file,
>    * 
> gnu/testlet/javax/swing/text/AbstractDocument/BranchElement/getElementIndexNullPointer.java: 
> Now extends JUnitRunnable,
>    * gnu/testlet/javax/swing/text/AttributeSet/AttributeSetTests.java: 
> New file,
>    * gnu/testlet/javax/swing/text/BoxView/BoxViewTests.java: Likewise,
>    * gnu/testlet/javax/swing/text/BoxView/spans.java: Now extends 
> JUnitRunnable,
>    * 
> gnu/testlet/javax/swing/text/DefaultEditorKit/DefaultEditorKitTests.java: 
> New file,
>    * gnu/testlet/javax/swing/text/DefaultEditorKit/getActions.java: 
> Now extends JUnitRunnable,
>    * 
> gnu/testlet/javax/swing/text/DefaultFormatter/DefaultFormatterTests.java: 
> New file,
>    * gnu/testlet/javax/swing/text/DefaultFormatter/getValueClass.java: 
> Now extends JUnitRunnable,
>    * 
> gnu/testlet/javax/swing/text/DefaultStyledDocument/DefaultStyledDocumentTests.java: 
> New file,
>    * 
> gnu/testlet/javax/swing/text/DefaultStyledDocument/insertString.java: 
> Now extends JUnitRunnable,
>    * 
> gnu/testlet/javax/swing/text/DefaultStyledDocument/ElementBuffer/ElementBufferTests.java: 
> New file,
>    * 
> gnu/testlet/javax/swing/text/DefaultStyledDocument/ElementBuffer/insert.java: 
> Now extends JUnitRunnable,
>    * 
> gnu/testlet/javax/swing/text/ElementIterator/ElementIteratorTest.java: 
> Likewise,
>    * 
> gnu/testlet/javax/swing/text/ElementIterator/ElementIteratorTests.java: 
> New file,
>    * gnu/testlet/javax/swing/text/FlowView/FlowViewTests.java: New file,
>    * gnu/testlet/javax/swing/text/FlowView/getFlowAxis.java: Now 
> extends JUnitRunnable,
>    * gnu/testlet/javax/swing/text/GapContent/GapContentTest.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/GapContent/GapContentTests.java: New 
> file,
>    * gnu/testlet/javax/swing/text/GapContent/PositionTest.java: Now 
> extends JUnitRunnable,
>    * gnu/testlet/javax/swing/text/GapContent/constructors.java: Likewise,
>    * gnu/testlet/javax/swing/text/GapContent/createPosition.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/GapContent/getChars.java: Likewise,
>    * gnu/testlet/javax/swing/text/GapContent/getString.java: Likewise,
>    * gnu/testlet/javax/swing/text/GapContent/insertString.java: Likewise,
>    * gnu/testlet/javax/swing/text/GapContent/length.java: Likewise,
>    * gnu/testlet/javax/swing/text/GapContent/remove.java: Likewise,
>    * 
> gnu/testlet/javax/swing/text/InternationalFormatter/InternationalFormatterTest.java: 
> Likewise,
>    * 
> gnu/testlet/javax/swing/text/InternationalFormatter/InternationalFormatterTests.java: 
> New file,
>    * 
> gnu/testlet/javax/swing/text/MaskFormatter/MaskFormatterTest.java: Now 
> extends JUnitRunnable,
>    * 
> gnu/testlet/javax/swing/text/MaskFormatter/MaskFormatterTests.java: 
> New file,
>    * 
> gnu/testlet/javax/swing/text/PlainDocument/PlainDocumentTest.java: Now 
> extends JUnitRunnable,
>    * 
> gnu/testlet/javax/swing/text/PlainDocument/PlainDocumentTests.java: 
> New file,
>    * gnu/testlet/javax/swing/text/PlainDocument/createPosition.java: 
> Now extends JUnitRunnable,
>    * gnu/testlet/javax/swing/text/PlainDocument/getLength.java: Likewise,
>    * gnu/testlet/javax/swing/text/PlainDocument/getRootElements.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/PlainDocument/getText.java: Likewise,
>    * gnu/testlet/javax/swing/text/PlainDocument/insertString.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/PlainDocument/multipleLeafs.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/PlainDocument/remove.java: Likewise,
>    * 
> gnu/testlet/javax/swing/text/PlainDocument/removeJoinesLines.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/Segment/SegmentTests.java: New file,
>    * gnu/testlet/javax/swing/text/Segment/clone.java: Now extends 
> JUnitRunnable,
>    * gnu/testlet/javax/swing/text/Segment/constructors.java: Likewise,
>    * gnu/testlet/javax/swing/text/Segment/first.java: Likewise,
>    * gnu/testlet/javax/swing/text/Segment/getBeginIndex.java: Likewise,
>    * gnu/testlet/javax/swing/text/Segment/getEndIndex.java: Likewise,
>    * gnu/testlet/javax/swing/text/Segment/getIndex.java: Likewise,
>    * gnu/testlet/javax/swing/text/Segment/isPartialReturn.java: Likewise,
>    * gnu/testlet/javax/swing/text/Segment/last.java: Likewise,
>    * gnu/testlet/javax/swing/text/Segment/next.java: Likewise,
>    * gnu/testlet/javax/swing/text/Segment/previous.java: Likewise,
>    * gnu/testlet/javax/swing/text/Segment/setIndex.java: Likewise,
>    * gnu/testlet/javax/swing/text/Segment/setPartialReturn.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/Segment/toString(), Likewise,
>    * 
> gnu/testlet/javax/swing/text/SimpleAttributeSet/SimpleAttributeSetTests.java: 
> New file,
>    * 
> gnu/testlet/javax/swing/text/StringContent/BadLocationExceptionTest.java: 
> Now extends JUnitRunnable,
>    * 
> gnu/testlet/javax/swing/text/StringContent/StringContentTest.java: 
> Likewise,
>    * 
> gnu/testlet/javax/swing/text/StringContent/StringContentTests.java: 
> New file,
>    * gnu/testlet/javax/swing/text/StringContent/constructors.java: Now 
> extends JUnitRunnable,
>    * gnu/testlet/javax/swing/text/StringContent/createPosition.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StringContent/getChars.java: Likewise,
>    * gnu/testlet/javax/swing/text/StringContent/getString.java: Likewise,
>    * gnu/testlet/javax/swing/text/StringContent/insertString.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StringContent/insertUndo.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StringContent/length.java: Likewise,
>    * gnu/testlet/javax/swing/text/StringContent/remove.java: Likewise,
>    * gnu/testlet/javax/swing/text/StringContent/removeUndo.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StringContent/stickyPosition.java: 
> Likewise,
>    * 
> gnu/testlet/javax/swing/text/StyleConstants/StyleConstantsTests.java: 
> New file,
>    * gnu/testlet/javax/swing/text/StyleConstants/constants.java: Now 
> extends JUnitRunnable,
>    * gnu/testlet/javax/swing/text/StyleConstants/getAlignment.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getBackground.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getBidiLevel.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getComponent.java: 
> Likewise,
>    * 
> gnu/testlet/javax/swing/text/StyleConstants/getFirstLineIndent.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getFontFamily.java: 
> Liekwise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getFontSize.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getForeground.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getIcon.java: Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getLeftIndent.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getLineSpacing.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getRightIndent.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getSpaceAbove.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getSpaceBelow.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/getTabSet.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/isBold.java: Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/isItalic.java: Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/isStrikeThrough.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/isSubscript.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/isSuperscript.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/isUnderline.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setAlignment.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setBackground.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setBidiLevel.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setBold.java: Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setComponent.java: 
> Likewise,
>    * 
> gnu/testlet/javax/swing/text/StyleConstants/setFirstLineIndent.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setFontFamily.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setFontSize.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setForeground.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setIcon.java: Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setItalic.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setLeftIndent.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setLineSpacing.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setRightIndent.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setSpaceAbove.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setSpaceBelow.java: 
> Likewise,
>    * 
> gnu/testlet/javax/swing/text/StyleConstants/setStrikeThrough.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setSubscript.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setSuperscript.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setTabSet.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleConstants/setUnderline.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleContext/NamedStyleInit.java: 
> Likewise,
>    * 
> gnu/testlet/javax/swing/text/StyleContext/NamedStyleSetResolveParent.java: 
> Likewise,
>    * gnu/testlet/javax/swing/text/StyleContext/StyleContextTests.java: 
> New file,
>    * gnu/testlet/javax/swing/text/StyleContext/addAttribute.java: Now 
> extends JUnitRunnable,
>    * gnu/testlet/javax/swing/text/StyleContext/addStyle.java: Likewise,
>    * 
> gnu/testlet/javax/swing/text/StyledEditorKit/StyledEditorKitTests.java: 
> New file,
>    * 
> gnu/testlet/javax/swing/text/StyledEditorKit/createInputAttributesTest.java: 
> Now extends JUnitRunnable,
>    * gnu/testlet/javax/swing/text/TextAction/TextActionTests.java: New 
> file,
>    * gnu/testlet/javax/swing/text/TextAction/augmentList.java: Now 
> extends JUnitRunnable,
>    * gnu/testlet/javax/swing/text/View/ViewTests.java: New file,
>    * gnu/testlet/javax/swing/text/View/getAlignment.java: Now extends 
> JUnitRunnable,
>    * gnu/testlet/javax/swing/text/View/getMaximumSpan.java: Likewise,
>    * gnu/testlet/javax/swing/text/View/getMinimumSpan.java: Likewise,
>    * gnu/testlet/javax/swing/text/View/getResizeWeight.java: Likewise.  
>
>

Re: Running Mauve tests with JUnit

[Michael Koch]

On Wed, Mar 08, 2006 at 10:40:55AM +0000, David Gilbert wrote:
> I didn't get any feedback about this...anyone think it is a good/bad idea?

I think this is a good idea. JUnit is widely known and supported well by
IDEs. Please go ahead.


Cheers,
Michael
-- 
Escape the Java Trap with GNU Classpath!
http://www.gnu.org/philosophy/java-trap.html

Join the community at http://planet.classpath.org/

Re: Running Mauve tests with JUnit

[David Gilbert]

Hi Dalibor,

I had meant to keep this on the Mauve lists, but I'll reply to the 
Classpath list also...

Dalibor Topic wrote:

>On Wed, Mar 08, 2006 at 10:40:55AM +0000, David Gilbert wrote:
>  
>
>>I didn't get any feedback about this...anyone think it is a good/bad idea?
>>    
>>
>excellent idea in my opinion. Have you looked at graydon's junit mauve
>bridge at
>http://sources.redhat.com/ml/mauve-discuss/2003-q4/msg00003.html ?
>
>  
>
I hadn't seen Graydon's bridge class, thanks for the link (and I should 
do more research next time).  Looking over it, it has the advantage that 
it doesn't require any existing Mauve testlets to be modified (and we 
have a lot of testlets), but the disadvantage that it doesn't buy you 
much in terms of integration with IDEs (you still have to generate the 
test list ['classes'] file, for instance, which is the major stumbling 
block that people seem to have when trying to run Mauve). 

By modifying the Mauve testlets in the way that I proposed, you can (for 
example) run a single test in Eclipse just by selecting the source file 
and clicking 'Run as --> JUnit test'.  I figured that was the sort of 
thing people were expecting.

JUnit does seem to me to be less flexible in terms of selecting subsets 
of tests, and it's approach of reporting a pass/fail for each test 
method (only) makes it, in my opinion, less suitable for the type of 
testing we are doing on GNU Classpath.  But I was careful in the 
"conversion" to retain the Mauve testlets so that we can continue 
running the tests in the traditional (Mauve) way.

>If we do something like that. I'd like to see the junit code from
>freenet merged in, to keep it simple to run mauve without external
>dependencies.
>  
>
Agreed.  I didn't have much trouble getting the tests to compile against 
the freenet code (a  basic GPLed implementation of the JUnit API for 
those that don't know what it is) but didn't get any meaningful output 
from running the tests against it yet.  I don't think that will be too 
hard to resolve.

Regards,

Dave

Re: Running Mauve tests with JUnit

[Dalibor Topic]

On Wed, 2006-03-08 at 13:21 +0000, David Gilbert wrote:
> Hi Dalibor,
> 
> I had meant to keep this on the Mauve lists, but I'll reply to the 
> Classpath list also...
> 

Sorry about the cross-posting :/ I've taken the classpath list out
again.

> I hadn't seen Graydon's bridge class, thanks for the link (and I should 
> do more research next time).  Looking over it, it has the advantage that 
> it doesn't require any existing Mauve testlets to be modified (and we 
> have a lot of testlets), but the disadvantage that it doesn't buy you 
> much in terms of integration with IDEs (you still have to generate the 
> test list ['classes'] file, for instance, which is the major stumbling 
> block that people seem to have when trying to run Mauve). 

OK, thanks for the explanation. I wasn't familiar with Graydon's code
either, just remembered it was sitting in my mail box.

> By modifying the Mauve testlets in the way that I proposed, you can (for 
> example) run a single test in Eclipse just by selecting the source file 
> and clicking 'Run as --> JUnit test'.  I figured that was the sort of 
> thing people were expecting.

That sounds very cool. 

Would it be possible to make that work without having to modify the
existing tests, by (just trowing random ideas here) using a proxy to
delegate to the junit Test runner, and having Testlet implement both the
junit.Test interface and the Testlet interface?

(I assume that a Junit test is recognized by an IDE by looking whether a
class implements the junit.Test interface? Or do IDEs look for a class
extending TestCase/TestSuite? I don't use IDEs much, so I hope the
questions are not too stupid.)

> Agreed.  I didn't have much trouble getting the tests to compile against 
> the freenet code (a  basic GPLed implementation of the JUnit API for 
> those that don't know what it is) but didn't get any meaningful output 
> from running the tests against it yet.  I don't think that will be too 
> hard to resolve.

Great, thanks for looking at that code. Sounds like it should be good
enough for the basic needs.

cheers,
dalibor topic