[#noissue] Remove pinpoint-commons dependency of bootstrap module

pinpoint-apm · Jan 3, 2025 · f3b008d · f3b008d
1 parent 0db115e
commit f3b008d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/...tstrap-java9/src/main/java/com/navercorp/pinpoint/bootstrap/java9/module/ModuleUtils.java b/...tstrap-java9/src/main/java/com/navercorp/pinpoint/bootstrap/java9/module/ModuleUtils.java
@@ -32,6 +32,6 @@ private static int readJavaVersion() {
     }
 
     public static boolean jvmVersionUpper(int version) {
-        return getJavaVersion() <= version;
+        return getJavaVersion() >= version;
     }
 }
diff --git a/...ap-java9/src/test/java/com/navercorp/pinpoint/bootstrap/java9/module/ModuleUtilsTest.java b/...ap-java9/src/test/java/com/navercorp/pinpoint/bootstrap/java9/module/ModuleUtilsTest.java
@@ -9,10 +9,10 @@ class ModuleUtilsTest {
 
     @Test
     void isJava11JvmVersionUpper() {
-        assertFalse(ModuleUtils.jvmVersionUpper(7));
-        assertFalse(ModuleUtils.jvmVersionUpper(8));
-
+        assertTrue(ModuleUtils.jvmVersionUpper(7));
+        assertTrue(ModuleUtils.jvmVersionUpper(8));
         assertTrue(ModuleUtils.jvmVersionUpper(11));
-        assertTrue(ModuleUtils.jvmVersionUpper(17));
+
+        assertFalse(ModuleUtils.jvmVersionUpper(17));
     }
 }