comment

trask · trask · commit e1cbc8005ad5 · 2024-10-23T18:50:42.000-07:00
diff --git a/smoke-tests/apps/JmxMetric/src/smokeTest/java/com/microsoft/applicationinsights/smoketest/JmxMetricTest.java b/smoke-tests/apps/JmxMetric/src/smokeTest/java/com/microsoft/applicationinsights/smoketest/JmxMetricTest.java
@@ -217,4 +217,9 @@ static class Tomcat8Java21Test extends JmxMetricTest {}
 
   @Environment(TOMCAT_8_JAVA_23)
   static class Tomcat8Java23Test extends JmxMetricTest {}
+
+  // this smoke test doesn't work on OpenJ9 because it specifically
+  // targets JMX metrics present in Hotspot JVM
+  // TODO (trask) remove the test dependency on Hotspot, or create a
+  // second test to target OpenJ9
 }

Original file line number	Diff line number	Diff line change
`@@ -217,4 +217,9 @@ static class Tomcat8Java21Test extends JmxMetricTest {}`
`217`	`217`
`218`	`218`	`@Environment(TOMCAT_8_JAVA_23)`
`219`	`219`	`static class Tomcat8Java23Test extends JmxMetricTest {}`
	`220`	`+`
	`221`	`+ // this smoke test doesn't work on OpenJ9 because it specifically`
	`222`	`+ // targets JMX metrics present in Hotspot JVM`
	`223`	`+ // TODO (trask) remove the test dependency on Hotspot, or create a`
	`224`	`+ // second test to target OpenJ9`
`220`	`225`	`}`