On Sat, 26 Apr 2025 01:28:53 GMT, Manukumar V S <m...@openjdk.org> wrote:

>> The javadoc for PassFailJFrame.forcePass suggests an anti-pattern of 
>> forcibly passing the test if a resource is unavailable.
>> 
>> If a resource is unavailable or a feature isn't supported, the test should 
>> throw jtreg.SkippedException.
>> 
>> PassFailJFrame.forcePass should be used in semi-automatic tests when the 
>> test determines that all the conditions for passing the test are met.
>> Please refer: JDK-8355366 and https://github.com/openjdk/jdk/pull/24820
>> 
>> Testing
>> This is a javadoc change, so not testing required.
>
> Manukumar V S has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Updated line number in code link

test/jdk/java/awt/regtesthelpers/PassFailJFrame.java line 1306:

> 1304:      * <p>A sample usage can be found in
> 1305:      * <a 
> href="https://github.com/openjdk/jdk/blob/7283c8b/test/jdk/java/awt/FileDialog/SaveFileNameOverrideTest.java#L84";>{@code
> 1306:      * SaveFileNameOverrideTest.java}</a>

Javadoc update LGTM. Maybe the doc for forceFail() can be updated similarly?
@aivanov-jdk What do you suggest?

It can be done as a part of this PR change or a separate PR. Since it is a 
similar doc change, I think it can be done as part of this PR by the changing 
the JBS title appropriately  - _Update javadocs for PassFailJFrame's 
forcePass() & forceFail() methods_

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24837#discussion_r2064692150

Reply via email to