> 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
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:
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
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:
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
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
>
> 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
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
>
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
>
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
>>
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
>
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
>
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
-
Commit messages:
- Remove
17 matches
Mail list logo