[jfx-tests] RFR: 8315896: Perspective lod tests fail because of minute difference in values [v2]

Jayathirth D V jdv at openjdk.org
Tue Oct 3 07:20:01 UTC 2023


> Two perspective lod tests fail because of minute differences in expected values.
> 
> Failing tests:
> test/scenegraph/fx3d/lod/PerspectiveLodCameraTest.java:
> test/scenegraph/fx3d/lod/PerspectiveLodGroupTest.java: 
> 
> Exception:
> test test.scenegraph.fx3d.lod.LodTests.sphereInitialLightOnTest(): failure
> org.jemmy.TimeoutExpiredException: State 'Expected 50180.140882730484, but was 50180.137157440186' has not been reached in 1000 milliseconds
> at org.jemmy.timing.Waiter.ensureValue(Waiter.java:93)
> at test.scenegraph.fx3d.lod.LodTestBase.checkLod(LodTestBase.java:117)
> at test.scenegraph.fx3d.lod.LodTests.sphereInitialLightOnTest(LodTests.java:98)
> at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:104)
> at java.base/java.lang.reflect.Method.invoke(Method.java:577)
> at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:59)
> at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
> at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:56)
> at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
> at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:299)
> at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:293)
> at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
> at java.base/java.lang.Thread.run(Thread.java:833) 
> 
> These little difference in values are seen both initial drawing and updates.
> These tests are not run from long time and change in product might change lod values.
> 
> With updated lod values i have verified that both the tests pass in OpenGL and Metal pipeline with both retina display(scale = 2) and external monitor(scale = 1).

Jayathirth D V has updated the pull request incrementally with one additional commit since the last revision:

  Add lod tolerance

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

Changes:
  - all: https://git.openjdk.org/jfx-tests/pull/6/files
  - new: https://git.openjdk.org/jfx-tests/pull/6/files/e6c3308b..7dcb21aa

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jfx-tests&pr=6&range=01
 - incr: https://webrevs.openjdk.org/?repo=jfx-tests&pr=6&range=00-01

  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jfx-tests/pull/6.diff
  Fetch: git fetch https://git.openjdk.org/jfx-tests.git pull/6/head:pull/6

PR: https://git.openjdk.org/jfx-tests/pull/6


More information about the openjfx-dev mailing list