> HI all,
>
> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>
> The fix adds a no-arg constructor which TestNG requires.
>
> The change allows the test to run with the current jtreg as well as the
> upcoming jtreg
>
>
> Best
> Lance
Lance Andersen has updated t
On Fri, 14 May 2021 08:02:29 GMT, Alan Bateman wrote:
>> Lance Andersen has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Adjust imports and spacing
>
> test/jdk/tools/jmod/hashes/HashesTest.java line 389:
>
>> 387: ModuleReferenc
On Thu, 13 May 2021 20:38:20 GMT, Lance Andersen wrote:
>> HI all,
>>
>> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>>
>> The fix adds a no-arg constructor which TestNG requires.
>>
>> The change allows the test to run with the current jtreg as well as the
>>
On Thu, 13 May 2021 20:38:20 GMT, Lance Andersen wrote:
>> HI all,
>>
>> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>>
>> The fix adds a no-arg constructor which TestNG requires.
>>
>> The change allows the test to run with the current jtreg as well as the
>>
> HI all,
>
> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>
> The fix adds a no-arg constructor which TestNG requires.
>
> The change allows the test to run with the current jtreg as well as the
> upcoming jtreg
>
>
> Best
> Lance
Lance Andersen has updated t
On Thu, 13 May 2021 18:53:25 GMT, Mandy Chung wrote:
>> Lance Andersen has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Update copyright typo
>
> test/jdk/tools/jmod/hashes/HashesTest.java line 90:
>
>> 88: }
>> 89: this.
On Thu, 13 May 2021 18:26:52 GMT, Lance Andersen wrote:
>> HI all,
>>
>> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>>
>> The fix adds a no-arg constructor which TestNG requires.
>>
>> The change allows the test to run with the current jtreg as well as the
>>
On Thu, 13 May 2021 18:37:21 GMT, Lance Andersen wrote:
>> HI all,
>>
>> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>>
>> The fix adds a no-arg constructor which TestNG requires.
>>
>> The change allows the test to run with the current jtreg as well as the
>>
> HI all,
>
> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>
> The fix adds a no-arg constructor which TestNG requires.
>
> The change allows the test to run with the current jtreg as well as the
> upcoming jtreg
>
>
> Best
> Lance
Lance Andersen has updated t
On Thu, 13 May 2021 10:49:21 GMT, Lance Andersen wrote:
> HI all,
>
> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>
> The fix adds a no-arg constructor which TestNG requires.
>
> The change allows the test to run with the current jtreg as well as the
> upcomin
> HI all,
>
> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>
> The fix adds a no-arg constructor which TestNG requires.
>
> The change allows the test to run with the current jtreg as well as the
> upcoming jtreg
>
>
> Best
> Lance
Lance Andersen has updated t
On Thu, 13 May 2021 10:49:21 GMT, Lance Andersen wrote:
> HI all,
>
> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>
> The fix adds a no-arg constructor which TestNG requires.
>
> The change allows the test to run with the current jtreg as well as the
> upcomin
On Thu, 13 May 2021 11:24:34 GMT, Chris Hegarty wrote:
> The non-static state in this test class is initialized for each of the static
> testXXX scenarios. An alternative could be to move said state (four fields)
> into a static inner class, and have each of the testXXX scenarios create an
> i
On Thu, 13 May 2021 11:11:23 GMT, Alan Bateman wrote:
>> HI all,
>>
>> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>>
>> The fix adds a no-arg constructor which TestNG requires.
>>
>> The change allows the test to run with the current jtreg as well as the
>> u
On Thu, 13 May 2021 10:49:21 GMT, Lance Andersen wrote:
> HI all,
>
> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>
> The fix adds a no-arg constructor which TestNG requires.
>
> The change allows the test to run with the current jtreg as well as the
> upcomin
On Thu, 13 May 2021 10:49:21 GMT, Lance Andersen wrote:
> HI all,
>
> Please review the fix to HashesTest.java to support running on TestNG 7.4.
>
> The fix adds a no-arg constructor which TestNG requires.
>
> The change allows the test to run with the current jtreg as well as the
> upcomin
16 matches
Mail list logo