On Mon, 13 Jul 2026 14:38:18 GMT, Andreas Chmielewski 
<[email protected]> wrote:

>> https://bugs.openjdk.org/browse/JDK-8387124
>> 
>> The test times out. This change increases the timeout based on the relative
>> execution time of DisabledAlgorithms.java.
>> 
>> Execution times on my system:
>>   DisabledAlgorithms.java            ~11 s (timeout = 480)
>>   BulkCipherDisabledAlgorithms.java  ~53 s
>> 
>> Using the same ratio would result in a timeout of approximately 2300 seconds.
>> To avoid setting such a high value, the timeout was reduced and set to 1200,
>> which should still provide sufficient margin.
>> 
>> ---------
>> - [x] I confirm that I make this contribution in accordance with the 
>> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai).
>
> Andreas Chmielewski has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Separated test

test/jdk/sun/security/ssl/SSLAlgorithmDecomposer/BulkCipherDecomposition.java 
line 94:

> 92:     }
> 93: 
> 94:     private static List<String> buildTests() throws 
> NoSuchAlgorithmException {

`buildTests` name is not quite correct in this test. Let's modify 
`getCipherSuites` to return the cipher suites' names, move 
`Security.setProperty("jdk.tls.disabledAlgorithms", "")` as the 1st line in 
`main`, and then remove `buildTests` method replacing it with `getCipherSuites`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/31812#discussion_r3571866719

Reply via email to