[jfx-tests] RFR: 8315842: 3D tests fail because of edge pixel differences

Jayathirth D V jdv at openjdk.org
Mon Sep 11 05:21:12 UTC 2023


On Fri, 8 Sep 2023 12:15:35 GMT, Ajit Ghaisas <aghaisas at openjdk.org> wrote:

> A static constant can be added to `test.scenegraph.fx3d.utils.FX3DAbstractApp` public static final float COLOR_TOLERANCE = 0.05f;
> 
> This can easily be accessed in test classes without FX3DAbstractApp instance as - `Root.ROOT.getEnvironment().setProperty(ImageComparator.class, new GlassPixelImageComparator(new PixelEqualityRasterComparator(FX3DAbstractApp.COLOR_TOLERANCE)));`
> 
> This will allow us to tweak the tolerance value at a single place in future and not in all the 26 files.
> 
> This change is pretty safe as and just a sanity check would be needed. If a test does not use FX3DAbstractApp, then you can keep the hard-coded constant.

Thanks @aghaisas for your inputs.
I have updated the code to use common variable.

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

PR Comment: https://git.openjdk.org/jfx-tests/pull/5#issuecomment-1713179402


More information about the openjfx-dev mailing list