On Wed, 14 Sep 2022 16:27:36 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

> Add `forceFail` implementation which allows passing the reason why the test 
> is failed.
> 
> I used the newly added method `forceFail(reason)` instead of throwing an 
> exception in `PrintAllPagesTest`. When the test is run outside of jtreg and 
> printing is cancelled, the the test UI becomes “hung”: clicking Pass or Fail 
> or closing windows does not stop the test.
> 
> I also converted the `ret` field to local variable.
> 
> In addition to this, I reduced the size of the test frame, its size of 
> 1000×800 doesn't fit even full HD screens when positioned close to the 
> instruction frame.

This pull request has now been integrated.

Changeset: 64b96e5c
Author:    Alexey Ivanov <aiva...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/64b96e5cf57023a5d55b4392074e3922ac7c0534
Stats:     26 lines in 2 files changed: 16 ins; 2 del; 8 mod

8293811: Provide a reason for PassFailJFrame.forceFail

Reviewed-by: honkar, jdv

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

PR: https://git.openjdk.org/jdk/pull/10269

Reply via email to