Reorder tests after discovering underlying faults

Author: Matt

dd-java-agent/agent-bootstrap/src/main/java/datadog/trace/bootstrap/Agent.java

@@ -430,7 +430,7 @@ public static void patchJPSAccess(Instrumentation inst) {
       // Unclear if supported for J9, may need to revisit
       try {
         Class.forName("datadog.trace.util.JPMSJPSAccess")
-            .getMethod("patchModuleAccess")
+            .getMethod("patchModuleAccess", Instrumentation.class)
             .invoke(inst);
       } catch (Exception e) {
         log.debug(

dd-smoke-tests/profiling-integration-tests/src/test/java/datadog/smoketest/JFRBasedProfilingIntegrationTest.java

@@ -507,8 +507,6 @@ void testShutdown(final TestInfo testInfo) throws Exception {
             assertTrue(
                 targetProcess.waitFor(
                     duration + PROFILING_UPLOAD_TIMEOUT_SECONDS + 1, TimeUnit.SECONDS));
-            assertTrue(
-                checkLogLines(logFilePath, it -> it.contains("Successfully invoked jvmstat")));
           } finally {
             if (targetProcess != null) {
               targetProcess.destroyForcibly();

internal-api/internal-api-9/build.gradle

@@ -41,10 +41,6 @@ forbiddenApisMain {
   failOnMissingClasses = false
 }
 
-forbiddenApisMain_java11 {
-  failOnMissingClasses = false
-}
-
 idea {
   module {
     jdkName = '11'

internal-api/internal-api-9/src/main_java11/java/datadog/trace/util/JPMSJPSAccess.java renamed to internal-api/internal-api-9/src/main/java/datadog/trace/util/JPMSJPSAccess.java

@@ -1,11 +1,21 @@
 package datadog.trace.util
 
 import datadog.trace.test.util.DDSpecification
+import net.bytebuddy.agent.ByteBuddyAgent
 
 class PidHelperTest extends DDSpecification {
 
   def "PID is available everywhere we test"() {
     expect:
     !PidHelper.getPid().isEmpty()
   }
+
+  def "JPS via jvmstat is used when possible"() {
+    when:
+    def inst = ByteBuddyAgent.install()
+    JPMSJPSAccess.patchModuleAccess(inst)
+
+    then:
+    JPSUtils.VMPids != null
+  }
 }