On Fri, 2 Sep 2022 06:11:47 GMT, Jayathirth D V <j...@openjdk.org> wrote:

>> Abhishek Kumar has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Review Comments fix
>
> test/jdk/javax/swing/JTree/TestTreeBackgroundColor.java line 84:
> 
>> 82:                 }
>> 83:             }
>> 84:             if (!passed) {
> 
> This test will pass even in the case of last pixel check is RED. We should 
> check for pixel data and whenever it is not RED we should bail out.
> 
> Also if test fails it will leave TreeBackgroundColorTestFail.png, we should 
> use temporary file and delete it on exit.

I have removed the writing of test image in case of fail operation and test 
condition changed to check if pixel color is not red then test will fail.

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

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

Reply via email to