Do not run JFR specific tests on J9

Author: jbachorik

dd-java-agent/agent-profiling/profiling-controller-openjdk/src/test/java/com/datadog/profiling/controller/openjdk/OpenJdkControllerTest.java

@@ -12,14 +12,17 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assumptions.assumeFalse;
 
 import com.datadog.profiling.controller.ControllerContext;
 import com.datadog.profiling.controller.jfr.JfpUtilsTest;
+import datadog.trace.api.Platform;
 import datadog.trace.api.profiling.RecordingData;
 import datadog.trace.bootstrap.config.provider.ConfigProvider;
 import java.util.Properties;
 import jdk.jfr.Recording;
 import org.junit.jupiter.api.Assumptions;
+import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.junit.jupiter.MockitoExtension;
@@ -29,6 +32,11 @@ public class OpenJdkControllerTest {
 
   private static final String TEST_NAME = "recording name";
 
+  @BeforeAll
+  static void setupSpec() {
+    assumeFalse(Platform.isJ9());
+  }
+
   @Test
   public void testCreateContinuousRecording() throws Exception {
     Properties props = getConfigProperties();
@@ -194,7 +202,7 @@ public void testAllocationProfilerIsStillOverriddenThroughConfig() throws Except
 
   @Test
   public void testNativeProfilerIsDisabledOnUnsupportedVersion() throws Exception {
-    Assumptions.assumeFalse(isNativeMethodSampleAvailable());
+    assumeFalse(isNativeMethodSampleAvailable());
     Properties props = getConfigProperties();
 
     ConfigProvider configProvider = ConfigProvider.withPropertiesOverride(props);

dd-java-agent/agent-profiling/profiling-controller-openjdk/src/test/java/com/datadog/profiling/controller/openjdk/OpenJdkOngoingRecordingTest.java

@@ -1,15 +1,18 @@
 package com.datadog.profiling.controller.openjdk;
 
 import static org.junit.jupiter.api.Assertions.*;
+import static org.junit.jupiter.api.Assumptions.assumeFalse;
 import static org.mockito.Mockito.never;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
 import com.datadog.profiling.controller.ControllerContext;
+import datadog.trace.api.Platform;
 import datadog.trace.api.profiling.RecordingData;
 import java.time.Instant;
 import jdk.jfr.Recording;
 import jdk.jfr.RecordingState;
+import org.junit.jupiter.api.Assumptions;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
@@ -34,6 +37,7 @@ public class OpenJdkOngoingRecordingTest {
 
   @BeforeEach
   public void setup() {
+    assumeFalse(Platform.isJ9());
     when(recording.getState()).thenReturn(RecordingState.RUNNING);
     when(recording.getName()).thenReturn(TEST_NAME);
 

dd-java-agent/agent-profiling/profiling-controller-openjdk/src/test/java/com/datadog/profiling/controller/openjdk/OpenJdkRecordingDataTest.java

@@ -4,14 +4,17 @@
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assumptions.assumeFalse;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
+import datadog.trace.api.Platform;
 import datadog.trace.api.profiling.ProfilingSnapshot;
 import java.io.IOException;
 import java.io.InputStream;
 import java.time.Instant;
 import jdk.jfr.Recording;
+import org.junit.jupiter.api.Assumptions;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
@@ -42,6 +45,7 @@ public class OpenJdkRecordingDataTest {
 
   @BeforeEach
   public void setup() throws IOException {
+    assumeFalse(Platform.isJ9());
     when(recording.getStream(start, end)).thenReturn(stream);
     when(recording.getStream(customStart, customEnd)).thenReturn(customStream);
     when(recording.getStartTime()).thenReturn(start);

dd-java-agent/instrumentation/exception-profiling/src/test/groovy/datadog/trace/bootstrap/instrumentation/jfr/exceptions/KnownExcludesForkedTest.groovy

@@ -1,16 +1,19 @@
 import com.zaxxer.hikari.pool.ProxyLeakTask
 import datadog.trace.agent.test.AgentTestRunner
+import datadog.trace.api.Platform
 import datadog.trace.bootstrap.instrumentation.jfr.InstrumentationBasedProfiling
 import jdk.jfr.Recording
 import org.openjdk.jmc.common.item.Attribute
 import org.openjdk.jmc.common.item.IAttribute
 import org.openjdk.jmc.common.item.ItemFilters
 import org.openjdk.jmc.common.unit.UnitLookup
 import org.openjdk.jmc.flightrecorder.JfrLoaderToolkit
+import spock.lang.Requires
 import spock.lang.Shared
 
 import java.nio.file.Files
 
+@Requires({!Platform.isJ9()})
 class KnownExcludesForkedTest extends AgentTestRunner {
   private static final IAttribute<String> TYPE =
   Attribute.attr("type", "type", "Exception type", UnitLookup.PLAIN_TEXT)

dd-java-agent/instrumentation/exception-profiling/src/test/java/datadog/trace/bootstrap/instrumentation/jfr/exceptions/ExceptionHistogramTest.java

@@ -5,6 +5,7 @@
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assumptions.assumeFalse;
 
 import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.ImmutableSortedMap;
@@ -16,9 +17,12 @@
 import java.util.Properties;
 import java.util.concurrent.Phaser;
 import java.util.stream.Stream;
+
+import datadog.trace.api.Platform;
 import jdk.jfr.FlightRecorder;
 import jdk.jfr.Recording;
 import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.Assumptions;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.openjdk.jmc.common.item.Aggregators;
@@ -60,6 +64,7 @@ public boolean equals(final Object obj) {
 
   @BeforeEach
   public void setup() {
+    assumeFalse(Platform.isJ9());
     recording = new Recording();
     recording.enable("datadog.ExceptionCount");
     recording.start();

dd-java-agent/instrumentation/java-directbytebuffer/src/test/groovy/DirectAllocationTrackingTest.groovy

@@ -1,9 +1,11 @@
 import datadog.trace.agent.test.AgentTestRunner
+import datadog.trace.api.Platform
 import datadog.trace.bootstrap.instrumentation.api.AgentTracer
 import datadog.trace.bootstrap.instrumentation.jfr.InstrumentationBasedProfiling
 import jdk.jfr.FlightRecorder
 import jdk.jfr.Recording
 import jdk.jfr.consumer.RecordingFile
+import spock.lang.Requires
 
 import java.nio.ByteBuffer
 import java.nio.channels.FileChannel
@@ -14,6 +16,7 @@ import java.util.stream.Collectors
 
 import static datadog.trace.agent.test.utils.TraceUtils.runUnderTrace
 
+@Requires({!Platform.isJ9()})
 class DirectAllocationTrackingTest extends AgentTestRunner {
 
   Recording recording