RFR: JDK-8222323: ChildAlwaysOnTopTest.java fails with "RuntimeException: Failed to unset alwaysOnTop" [v2]

[Harshitha Onkar]

On Thu, 5 May 2022 15:54:33 GMT, Alexander Zuev <kizune at openjdk.org> wrote:

>> Harshitha Onkar has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   added changes to continue testing and exception thrown at end in case of failure
>
> test/jdk/java/awt/Window/MultiWindowApp/ChildAlwaysOnTopTest.java line 243:
> 
>> 241:             System.out.println("Case "+ caseNo + errorMsg[scenarioNo - 1]
>> 242:                     +" Expected Color: "+ expectedColor +" vs Actual Color: "
>> 243:                     + actualColor);
> 
> Here instead of just printing out the error i would save it into the static StringBuilder that i would allocate at the beginning of the test. Then when we are concluding testing i would just check if the buffer is empty and if it is not i would throw an RuntimeException with the buffer content. This way we would have all the errors reported by the JavaTest suite in the HTML report and this will make it into the whatever testing environment report we are running tests in. Also that would replace the errorFlag variable since emptiness of the buffer will be an indication of test pass.

@azuev-java Thank you and will incorporate the suggested changes.

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

PR: https://git.openjdk.java.net/jdk/pull/8509