Remove some TODOs from tests after upstream JDT Core fixes

Signed-off-by: Alexander Kriegisch <[email protected]>
eclipse-aspectj · Dec 14, 2023 · 696aec6 · 696aec6
1 parent 3b81ea1
commit 696aec6
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git 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<T> permits A, B {}

diff --git 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<T> permits D, E {}

diff --git 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<T> permits D, E {}
   final static class D<X> implements I<String> {}

diff --git 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<T> permits A, B {}