On Fri, 30 May 2025 09:06:00 GMT, Volkan Yazici <vyaz...@openjdk.org> wrote:

>> p-nima has updated the pull request incrementally with one additional commit 
>> since the last revision:
>> 
>>   apply review feedback
>
> test/jdk/java/net/httpclient/HttpClientRetryLimitTest.java line 113:
> 
>> 111:                 }
>> 112:                 e.printStackTrace();
>> 113:             }
> 
> AFAICT, you should be using `assertThrows` as follows:
> 
> 
> IOException exception = assertThrows(...);
> assertEquals(exception.message(), "too many authentication attempts. Limit: " 
> + RETRY_LIMIT);
> assertEquals(requestCount.get(), RETRY_LIMIT > 0 ? RETRY_LIMIT : 1);

The changes have been made in 2d0325649e4d0f67e25aa30ba36c1c2555bc59b9- We 
decrement the request count because we authenticate the request at least once 
before the limit is checked. A call to math.max can be done but I think the 
ternary operator helps to reduce the overhead of the call, wdyt?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25490#discussion_r2132339249

Reply via email to