From cc880a1cec02f665b03d1a196e6382f65faa25f8 Mon Sep 17 00:00:00 2001 From: Lorenzo Bettini Date: Wed, 15 May 2024 09:32:45 +0200 Subject: [PATCH] hardcoded ClassFileConstants.JDK21 To let it compile with older target platforms --- .../xtext/xbase/ui/tests/JavaVersionExtendedTest.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/org.eclipse.xtext.xbase.ui.tests/src/org/eclipse/xtext/xbase/ui/tests/JavaVersionExtendedTest.java b/org.eclipse.xtext.xbase.ui.tests/src/org/eclipse/xtext/xbase/ui/tests/JavaVersionExtendedTest.java index 3642e33d005..69ca18f34ea 100644 --- a/org.eclipse.xtext.xbase.ui.tests/src/org/eclipse/xtext/xbase/ui/tests/JavaVersionExtendedTest.java +++ b/org.eclipse.xtext.xbase.ui.tests/src/org/eclipse/xtext/xbase/ui/tests/JavaVersionExtendedTest.java @@ -29,7 +29,10 @@ public void testToJdtClassFileConstant() { assertEquals(ClassFileConstants.JDK10, JavaVersion.JAVA10.toJdtClassFileConstant()); assertEquals(ClassFileConstants.JDK11, JavaVersion.JAVA11.toJdtClassFileConstant()); assertEquals(ClassFileConstants.JDK17, JavaVersion.JAVA17.toJdtClassFileConstant()); - assertEquals(ClassFileConstants.JDK21, JavaVersion.JAVA21.toJdtClassFileConstant()); + // until we use the newer JDT as minimal requirement, we cannot refer to ClassFileConstants.JDK21 + // or the test won't compile with older target platforms + // assertEquals(ClassFileConstants.JDK21, JavaVersion.JAVA21.toJdtClassFileConstant()); + assertEquals(4259840, JavaVersion.JAVA21.toJdtClassFileConstant()); // try { // long value = ClassFileConstants.class.getField("JDK17").getLong(null); // assertEquals(value, JavaVersion.JAVA17.toJdtClassFileConstant());