On Wed, 8 Apr 2026 16:24:16 GMT, Daniel Fuchs <[email protected]> wrote:

>> May I get a review for this change that converts `java/net/DatagramSocket` 
>> TestNG tests to use JUnit.
>> 
>> Some of these tests now take advantage from the fact that Closeable 
>> arguments passed to a ParameterizedTest are automatically closed by JUnit.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Daniel Fuchs has updated the pull request with a new target base due to a 
> merge or a rebase. The pull request now contains nine commits:
> 
>  - Merge branch 'master' into junit-ds-8381359
>  - Remove unneeded import
>  - Merge
>  - Merge
>  - Merge branch 'master' into junit-ds-8381359
>  - Review feedback
>  - Apply suggestions from code review
>    
>    Co-authored-by: Marcono1234 <[email protected]>
>  - Apply suggestion from @Marcono1234
>    
>    Co-authored-by: Marcono1234 <[email protected]>
>  - 8381359: Refactor java/net/DatagramSocket TestNG tests to use JUnit

Marked as reviewed by vyazici (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/30502#pullrequestreview-4096912586

Reply via email to