> 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision: - Merge remote-tracking branch 'upstream/master' into lod_tests - Add lod tolerance - 8315896: Perspective lod tests fail because of minute difference in values ------------- Changes: - all: https://git.openjdk.org/jfx-tests/pull/6/files - new: https://git.openjdk.org/jfx-tests/pull/6/files/7dcb21aa..70895516 Webrevs: - full: https://webrevs.openjdk.org/?repo=jfx-tests&pr=6&range=02 - incr: https://webrevs.openjdk.org/?repo=jfx-tests&pr=6&range=01-02 Stats: 437 lines in 186 files changed: 277 ins; 16 del; 144 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
