RFR: 8266460: java.io tests fail on null stream with upgraded jtreg/TestNG

[Lance Andersen]

On Tue, 4 May 2021 22:36:29 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> Hi all,
>> 
>> This fix addresses a change in TestNG behavior for the Before/AfterGroups annotations that was introduced  via  https://github.com/cbeust/testng/pull/2167.   The tests have been verified against the latest TestNG release and continue to run with the current TestNG release used by jtreg
>
> test/jdk/java/io/InputStream/NullInputStream.java line 33:
> 
>> 31: import java.io.InputStream;
>> 32: 
>> 33: import static org.testng.Assert.*;
> 
> Would it not be more standard to put the new imports just after this import? Same comment applies in the other files.

That's IntelliJ magic.  I can update if you prefer and can let IntelliJ optimize all of the imports

-------------

PR: https://git.openjdk.java.net/jdk/pull/3866