Ping: [JAXP] RFR: 8173602: TESTBUG: javax/xml/jaxp/unittest/transform/TransformerTest.java needs refactoring
Daniel Fuchs
daniel.fuchs at oracle.com
Fri Feb 3 12:54:22 UTC 2017
Hi Christoph,
I had a look at your patch, it seems OK.
I didn't know that testng supported @Test
annotations on methods of inner classes, but
that looks useful.
The patch is a bit difficult to review because the diff
seems a bit lost sometime - but as far as I could see
what was there before seems to be there after too :-)
I noticed you changed the calls for pretty-printing
the results - using public APIs now. Hopefully that
doesn't matter to test (I mean that's hopefully not
what the test was testing).
I ran your new test locally on my machine, and sent it
through our test system as well, and everything passed,
so I guess that's a +1 for me.
Will you push that to the new 10 forest or did you
plan to push it in 9?
best regards,
-- daniel
On 03/02/17 10:11, Langer, Christoph wrote:
> Hi,
>
> Ping - anyone out there who could have a look at my JAXP test update?
>
> Thanks
> Christoph
>
> From: Langer, Christoph
> Sent: Montag, 30. Januar 2017 15:28
> To: core-libs-dev at openjdk.java.net
> Subject: [JAXP] RFR: 8173602: TESTBUG: javax/xml/jaxp/unittest/transform/TransformerTest.java needs refactoring
>
> Hi,
>
> please review a test fix for javax/xml/jaxp/unittest/transform/TransformerTest.java:
> Bug: https://bugs.openjdk.java.net/browse/JDK-8173602
> Webrev: http://cr.openjdk.java.net/~clanger/webrevs/8173602.0/
>
> The main motivation for this refactoring is to remove the dependency to the JDK internal classes com.sun.org.apache.xml.internal.serialize.OutputFormat and com.sun.org.apache.xml.internal.serialize.XMLSerializer.
>
> Generally, TransformerTest.java contains several subtests which do similar things (test XALAN transformations) but the implementations differ. I created a common template with helper methods which all tests take advantage of now. I also introduced data sources to some tests to make them more readable and adaptable.
>
> Thanks
> Christoph
>
More information about the core-libs-dev
mailing list