Fix validation test for Java 21 (#1422)

jacoco · Mar 31, 2023 · c561c13 · c561c13
1 parent 461ebf3
commit c561c13
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/...test.validation.java20/src/org/jacoco/core/test/validation/java20/RecordPatternsTest.java b/...test.validation.java20/src/org/jacoco/core/test/validation/java20/RecordPatternsTest.java
@@ -12,6 +12,8 @@
  *******************************************************************************/
 package org.jacoco.core.test.validation.java20;
 
+import org.jacoco.core.test.validation.JavaVersion;
+import org.jacoco.core.test.validation.Source.Line;
 import org.jacoco.core.test.validation.ValidationTestBase;
 import org.jacoco.core.test.validation.java20.targets.RecordPatternsTarget;
 
@@ -24,4 +26,22 @@ public RecordPatternsTest() {
 		super(RecordPatternsTarget.class);
 	}
 
+	public void assertSwitchStatementCase(Line line) {
+		if (JavaVersion.current().isBefore("21")) {
+			assertFullyCovered(line);
+		} else {
+			// https://github.com/openjdk/jdk/commit/138cdc9283ae8f3367e51f0fe7e27833118dd7cb
+			assertPartlyCovered(line);
+		}
+	}
+
+	public void assertSwitchStatementDefault(Line line) {
+		if (JavaVersion.current().isBefore("21")) {
+			assertPartlyCovered(line);
+		} else {
+			// https://github.com/openjdk/jdk/commit/138cdc9283ae8f3367e51f0fe7e27833118dd7cb
+			assertFullyCovered(line);
+		}
+	}
+
 }
diff --git a/...ation.java20/src/org/jacoco/core/test/validation/java20/targets/RecordPatternsTarget.java b/...ation.java20/src/org/jacoco/core/test/validation/java20/targets/RecordPatternsTarget.java
@@ -31,8 +31,8 @@ private static void instanceofOperator(Object o) {
 
 	private static void switchStatement(Object p) {
 		switch (p) { // assertFullyCovered(0, 2)
-		case Point(int x, int y) -> nop(x + y); // assertFullyCovered()
-		default -> nop(); // assertPartlyCovered()
+		case Point(int x, int y) -> nop(x + y); // assertSwitchStatementCase()
+		default -> nop(); // assertSwitchStatementDefault()
 		} // assertEmpty()
 	}