From 696aec651b6f8ee85a7d51a1692b6eed8e36727e Mon Sep 17 00:00:00 2001 From: Alexander Kriegisch Date: Thu, 14 Dec 2023 20:17:51 +0700 Subject: [PATCH] Remove some TODOs from tests after upstream JDT Core fixes Signed-off-by: Alexander Kriegisch --- tests/features1919/java19/SwitchPatternPreview3OK.java | 2 +- tests/features1920/java20/SwitchPatternPreview4OK.java | 2 +- tests/features1921/java21/SwitchPatternPreview4OK.java | 2 -- tests/features199/java18/SwitchPatternPreview2OK.java | 2 ++ 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/features1919/java19/SwitchPatternPreview3OK.java b/tests/features1919/java19/SwitchPatternPreview3OK.java index 7395f08e92..2718aace12 100644 --- a/tests/features1919/java19/SwitchPatternPreview3OK.java +++ b/tests/features1919/java19/SwitchPatternPreview3OK.java @@ -157,7 +157,7 @@ static void defaultCanAppearBeforeConstantLabel(Integer i) { * https://bugs.eclipse.org/bugs/show_bug.cgi?id=579360 * https://github.com/eclipse-jdt/eclipse.jdt.core/issues/587 * - * TODO: reactivate when implemented or move to preview 4 with Java 20. + * DONE: reactivated after fix in Java 21 */ /* sealed interface I permits A, B {} diff --git a/tests/features1920/java20/SwitchPatternPreview4OK.java b/tests/features1920/java20/SwitchPatternPreview4OK.java index f5518d4960..c070b9cc9e 100644 --- a/tests/features1920/java20/SwitchPatternPreview4OK.java +++ b/tests/features1920/java20/SwitchPatternPreview4OK.java @@ -130,7 +130,7 @@ static void nullCanAppearAfterConstantLabel(Integer i) { * https://bugs.eclipse.org/bugs/show_bug.cgi?id=579360 * https://github.com/eclipse-jdt/eclipse.jdt.core/issues/587 * - * TODO: reactivate when implemented or move to preview 5 with Java 21. + * DONE: reactivated after fix in Java 21 */ /* sealed interface I permits D, E {} diff --git a/tests/features1921/java21/SwitchPatternPreview4OK.java b/tests/features1921/java21/SwitchPatternPreview4OK.java index 40854a0078..3fe6d98203 100644 --- a/tests/features1921/java21/SwitchPatternPreview4OK.java +++ b/tests/features1921/java21/SwitchPatternPreview4OK.java @@ -129,8 +129,6 @@ static void nullCanAppearAfterConstantLabel(Integer i) { * https://openjdk.java.net/jeps/420#2--Exhaustiveness-of-switch-expressions-and-statements * https://bugs.eclipse.org/bugs/show_bug.cgi?id=579360 * https://github.com/eclipse-jdt/eclipse.jdt.core/issues/587 - * - * TODO: reactivate when implemented or move to preview 5 with Java 21. */ sealed interface I permits D, E {} final static class D implements I {} diff --git a/tests/features199/java18/SwitchPatternPreview2OK.java b/tests/features199/java18/SwitchPatternPreview2OK.java index c17fdc12b5..b0efd8553b 100644 --- a/tests/features199/java18/SwitchPatternPreview2OK.java +++ b/tests/features199/java18/SwitchPatternPreview2OK.java @@ -97,6 +97,8 @@ static void defaultCanAppearBeforeConstantLabel(Integer i) { * https://openjdk.java.net/jeps/420#2--Exhaustiveness-of-switch-expressions-and-statements * https://bugs.eclipse.org/bugs/show_bug.cgi?id=579360 * https://github.com/eclipse-jdt/eclipse.jdt.core/issues/587 + * + * DONE: reactivated after fix in Java 21 */ /* sealed interface I permits A, B {}