Retire JavaFXTest which requires Java 8

[vogella]

JDT has dropped support for Java 8 in its recent release. Hence this test is or will become soon obsolete and should be removed.

Currently this test fails in a local build, requesting me to configure my toolchain.xml (which is not described in the Contributor documentation.

[eclipse-ls-bot]

Can one of the admins verify this patch?

[snjeza]

JDT has dropped support for Java 8 in its recent release. Hence this test is or will become soon obsolete and should be removed.

Could you show a JDT issue?

Currently this test fails in a local build, requesting me to configure my toolchain.xml

It shouldn't happen. See

eclipse.jdt.ls/org.eclipse.jdt.ls.tests/src/org/eclipse/jdt/ls/core/internal/javafx/JavaFXTest.java

Line 70 in 3698052

Assume.assumeThat("JavaSE-1.8 VM is not found, skipping test", java8vm, IsNull.notNullValue());

Could you try to run the tests with -DDetectVMInstallationsJob.disabled=true?

[testforstephen]

JDT has dropped support for Java 8 in its recent release.

If I remember correctly, JDT has dropped support for Java 7 and earlier, but Java 8 is still supported. See eclipse-jdt/eclipse.jdt.core#2551

[vogella]

@testforstephen ah, that is true, sorry for my misunderstanding, closing this one as wont fix.