Re: RFR: tests for JDK-8032884

2014-04-03 Thread Chris Hegarty
On 3 Apr 2014, at 09:58, Miroslav Kos wrote: > Hi Chris, > the test moved: http://cr.openjdk.java.net/~mkos/8032884/jdk.02/ Pushed to jdk9/dev http://hg.openjdk.java.net/jdk9/dev/jdk/rev/f6c3cb79d761 -Chris. > Thanks > Miran > > > > On 02/04/14 15:47, Chris Hegarty wrote: >> On 02/04/14 1

Re: RFR: tests for JDK-8032884

2014-04-03 Thread Miroslav Kos
Hi Chris, the test moved: http://cr.openjdk.java.net/~mkos/8032884/jdk.02/ Thanks Miran On 02/04/14 15:47, Chris Hegarty wrote: On 02/04/14 14:32, Miroslav Kos wrote: In the test, I need to do following steps: 1. compile schema file using xjc tool 2. compile generated sources 3. load genera

Re: RFR: tests for JDK-8032884

2014-04-02 Thread Chris Hegarty
On 02/04/14 14:32, Miroslav Kos wrote: In the test, I need to do following steps: 1. compile schema file using xjc tool 2. compile generated sources 3. load generated class and via reflection API verify generated methods In which case, what you have is fine then. Although I know how to run t

Re: RFR: tests for JDK-8032884

2014-04-02 Thread Miroslav Kos
In the test, I need to do following steps: 1. compile schema file using xjc tool 2. compile generated sources 3. load generated class and via reflection API verify generated methods Although I know how to run tool directly from the test, it seems to me not very elegant to run compiler directly

Re: RFR: tests for JDK-8032884

2014-04-01 Thread Chris Hegarty
The tests looks fine to me. Trivially, you could eliminate the shell script and run xic from ProcessBuilder? Also, does it make sense to move the test to jdk/test/javax/xml/ws/xjc? With the other jaxws tests, or is xjc different. -Chris,. On 1 Apr 2014, at 16:21, Miroslav Kos wrote: > Hi e

RFR: tests for JDK-8032884

2014-04-01 Thread Miroslav Kos
Hi everybody, I'd like to ask for review of following tests: http://cr.openjdk.java.net/~mkos/8032884/jdk.01/ Original bug (already fixed): Bug: https://bugs.openjdk.java.net/browse/JDK-8032884 Issue fixing the bug (bulk update): https://bugs.openjdk.java.net/browse/JDK-8036030 The test veri