Cleanup compiler test. remove unnecessary isJavaAtLeast conditions

Author: xuwei-k

compiler/test/dotty/tools/dotc/CompilationTests.scala

@@ -51,8 +51,7 @@ class CompilationTests {
       else Nil
     )
 
-    if scala.util.Properties.isJavaAtLeast("16") then
-      tests ::= compileFilesInDir("tests/pos-java16+", defaultOptions.and("-Wsafe-init"))
+    tests ::= compileFilesInDir("tests/pos-java16+", defaultOptions.and("-Wsafe-init"))
 
     aggregateTests(tests*).checkCompile()
   }

compiler/test/dotty/tools/dotc/classpath/MultiReleaseJarTest.scala

@@ -19,7 +19,6 @@ class MultiReleaseJarTest extends dotty.tools.backend.jvm.DottyBytecodeTest {
 
   @Test
   def mrJar(): Unit = {
-    if (!Properties.isJavaAtLeast("9")) { println("skipping mrJar() on old JDK"); return }
 
     // The test fails if the same jar file gets reused. This might be a caching problem in our classpath implementation
 
@@ -61,8 +60,7 @@ class MultiReleaseJarTest extends dotty.tools.backend.jvm.DottyBytecodeTest {
       assertEquals(Set("foo1", "bar1"), apiMethods(jar1, "8"))
       assertEquals(Set("foo1", "foo2", "bar1"), apiMethods(jar2, "9"))
 
-      if Properties.isJavaAtLeast("10") then
-        assertEquals(Set("foo1", "foo2", "bar1", "bar2"), apiMethods(jar3, "10"))
+      assertEquals(Set("foo1", "foo2", "bar1", "bar2"), apiMethods(jar3, "10"))
     } finally
       List(jar1, jar2, jar3).forall(path =>
         try Files.deleteIfExists(path)
@@ -72,7 +70,6 @@ class MultiReleaseJarTest extends dotty.tools.backend.jvm.DottyBytecodeTest {
 
   @Test
   def ctSymTest(): Unit = {
-    if (!Properties.isJavaAtLeast("9")) { println("skipping mrJar() on old JDK"); return }
 
     def classExists(className: String, release: String): Boolean = {
       given ctx: Context = initCtx.fresh

compiler/test/dotty/tools/io/ZipArchiveTest.scala

@@ -46,22 +46,19 @@ class ZipArchiveTest {
   }
 
   private val bootClassLoader: ClassLoader = {
-    if (!util.Properties.isJavaAtLeast("9")) null
-    else {
-      try {
-        MethodHandles
-          .lookup()
-          .findStatic(
-            classOf[ClassLoader],
-            "getPlatformClassLoader",
-            MethodType.methodType(classOf[ClassLoader])
-          )
-          .invoke()
-          .asInstanceOf[ClassLoader]
-      } catch {
-        case _: Throwable =>
-          null
-      }
+    try {
+      MethodHandles
+        .lookup()
+        .findStatic(
+          classOf[ClassLoader],
+          "getPlatformClassLoader",
+          MethodType.methodType(classOf[ClassLoader])
+        )
+        .invoke()
+        .asInstanceOf[ClassLoader]
+    } catch {
+      case _: Throwable =>
+        null
     }
   }