Review request for JDK-8051540: Convert JAXP functin tests: org.xml.sax to jtreg (testNG) tests
huizhe wang
huizhe.wang at oracle.com
Wed Oct 15 18:09:03 UTC 2014
On 10/15/2014 8:35 AM, Joe Darcy wrote:
> Hi Joe,
>
> Before performing the full investigation, I'm now leaning toward have
> a "base" repo which would host files currently in the jdk, langtools,
> and hotspot repos and then another repo for all the other library code
> (jaxp, jaxws, corba).
Something like this?
otherlib repo/
jaxp/src and etc.
jaxws
corba
Currently, we are enjoying the benefit of being a repo by itself, that
is, using the whole repo as our workspace.
But I understand the need for consolidating, as long as it's in its own
root directory, I think it would be fine.
>
> Compared to moving around all the source files in the jdk repo, I
> think the other potential changes are fairly small!
True, I'll push the change then.
Thanks!
Joe
>
> HTH,
>
> -Joe
>
> On 10/15/2014 1:43 AM, huizhe wang wrote:
>> Joe,
>>
>> You mentioned recently that you were considering folding the jaxp
>> repo into some other repos. Would it matter if we put these
>> functional tests into the existing jaxp repo (<openjdk
>> forest>/jaxp/test)? Or, would jaxp tests be under whatever the
>> 'other' repo is?
>>
>> We had planned to migrate jaxp unit and functional tests to jaxp
>> repo/test, and then move jaxp tests currently under jdk/test to
>> jaxp/test as well.
>>
>> These patches are big. I hope we don't have to move them around :-)
>>
>> Thanks,
>> Joe
>>
>> On 10/14/2014 3:10 PM, Tristan Yan wrote:
>>> Hi Joe
>>> Could you be my sponsor to push this if you’re okay with the code.
>>> Thank you
>>> Tristan
>>>
>>>> On Aug 27, 2014, at 4:38 PM, huizhe wang <huizhe.wang at oracle.com
>>>> <mailto:huizhe.wang at oracle.com>> wrote:
>>>>
>>>>
>>>> On 8/27/2014 4:03 PM, Tristan Yan wrote:
>>>>> Hi Joe and others
>>>>>
>>>>> I updated the tests with putting them in jaxp repo. I also run
>>>>> these tests with security manager and they all passed
>>>>> http://cr.openjdk.java.net/~tyan/JDK-8051540/webrev01/
>>>>> <http://cr.openjdk.java.net/%7Etyan/JDK-8051540/webrev01/>
>>>>>
>>>>
>>>> Awesome.
>>>>
>>>>> Also I’d like to propose our way for handling jaxp tests run with
>>>>> security manager. The way we’d use is creating two targets for
>>>>> running jaxp tests. One is for normal run; which will run all the
>>>>> tests without security manager. One is secure run; the target only
>>>>> run the tests that have to be run with security manager. This
>>>>> could be easy to be handled with adding two targets in makefile.
>>>>> And for most of people they only care about the function. They
>>>>> only need run normal run target. We would run two targets for any
>>>>> of our formal tests like nightly, ci build and jprt tests.
>>>>
>>>> Yes, please coordinate with Frank and Eric so that all of the jaxp
>>>> tests share the same configuration.
>>>>
>>>>> For the tests which can not be run in secure mode(like tests for
>>>>> xsltc direct extension), we'd add testng group called
>>>>> “secure-hostile”. We won’t run these tests in secure mode by
>>>>> bypassing them in secure run target. By this way we could easily
>>>>> transform our tests as usual without additional effort.
>>>>
>>>> I had been previously updated them so that all of the tests were
>>>> capable of running with and without security manager. Sustaining
>>>> SQE had invested several month to incorporate the changes into that
>>>> hosted in Aurora. Please consider taking the patches from them if
>>>> you haven't already done so.
>>>>
>>>> Thanks,
>>>> Joe
>>>>
>>>>>
>>>>> Thank you
>>>>> Tristan
>>>>>
>>>>>> On Aug 19, 2014, at 10:32 AM, huizhe wang <huizhe.wang at oracle.com
>>>>>> <mailto:huizhe.wang at oracle.com>> wrote:
>>>>>>
>>>>>> By the way, the plan has been that all of the JAXP SQE and Unit
>>>>>> tests be migrated into [openjdk]/jaxp repo under jaxp/test. Tests
>>>>>> currently in the jdk repo shall be moved to jaxp/test as well. I
>>>>>> see that your webrev was generated in jdk9/dev/jdk. I hope it
>>>>>> doesn't mean you're checking tests into the jdk repo.
>>>>>>
>>>>>> Thanks,
>>>>>> Joe
>>>>>>
>>>>>> On 8/18/2014 4:42 PM, Tristan Yan wrote:
>>>>>>> Thanks Joe
>>>>>>> We intend to replace the base class with test library because
>>>>>>> that doesn’t look like a real base class but an utilities class.
>>>>>>> I haven’t tried to run these tests with security manager, I will
>>>>>>> run them with security manager then get back you soon.
>>>>>>> Thank you.
>>>>>>> Tristan
>>>>>>>
>>>>>>>> On Aug 18, 2014, at 4:32 PM, huizhe wang
>>>>>>>> <huizhe.wang at oracle.com <mailto:huizhe.wang at oracle.com>> wrote:
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>
>
More information about the core-libs-dev
mailing list