RFR: 8337723: Remove redundant tests from com/sun/security/sasl/gsskerb [v3]

Fernando Guallini fguallini at openjdk.org
Tue Feb 11 13:14:19 UTC 2025


On Fri, 7 Feb 2025 20:22:52 GMT, Rajan Halade <rhalade at openjdk.org> wrote:

>> Fernando Guallini has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits:
>> 
>>  - Merge branch 'master' into 8337723
>>  - Merge branch 'master' into 8337723
>>  - refactoring
>>  - Merge branch 'master' into 8337723
>>  - remov gsskerb tests
>
> test/jdk/sun/security/krb5/auto/SaslBasic.java line 118:
> 
>> 116: 
>> 117:         if (!Arrays.equals(hello, token)) {
>> 118:             throw new Exception("Client message altered");
> 
> please update these final exceptions from test to `RuntimeException`

Updated to RuntimeException

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20464#discussion_r1950829860


More information about the security-dev mailing list