Reduce flakyness of Resource tests

instrumentation/resources/library/build.gradle.kts

@@ -69,3 +69,21 @@ tasks {
     )
   }
 }
+
+testing {
+  suites {
+    // Security Manager tests involve setup that can poison the environment for other tests
+    val testSecurityManager by registering(JvmTestSuite::class) {
+      dependencies {
+        implementation(project(":instrumentation:resources:library"))
+        implementation("io.opentelemetry:opentelemetry-sdk-common")
+      }
+    }
+  }
+}
+
+tasks {
+  check {
+    dependsOn(testing.suites)
+  }
+}

instrumentation/resources/library/src/main/java/io/opentelemetry/instrumentation/resources/ProcessResource.java

@@ -23,13 +23,13 @@
 public final class ProcessResource {
 
   // copied from ProcessIncubatingAttributes
-  private static final AttributeKey<List<String>> PROCESS_COMMAND_ARGS =
+  static final AttributeKey<List<String>> PROCESS_COMMAND_ARGS =
       AttributeKey.stringArrayKey("process.command_args");
-  private static final AttributeKey<String> PROCESS_COMMAND_LINE =
+  static final AttributeKey<String> PROCESS_COMMAND_LINE =
       AttributeKey.stringKey("process.command_line");
-  private static final AttributeKey<String> PROCESS_EXECUTABLE_PATH =
+  static final AttributeKey<String> PROCESS_EXECUTABLE_PATH =
       AttributeKey.stringKey("process.executable.path");
-  private static final AttributeKey<Long> PROCESS_PID = AttributeKey.longKey("process.pid");
+  static final AttributeKey<Long> PROCESS_PID = AttributeKey.longKey("process.pid");
 
   // Note: This pattern doesn't support file paths with spaces in them.
   // Important: This is statically used in buildResource, so must be declared/initialized first.

instrumentation/resources/library/src/test/java/io/opentelemetry/instrumentation/resources/HostResourceTest.java

@@ -11,12 +11,7 @@
 import io.opentelemetry.sdk.resources.Resource;
 import io.opentelemetry.semconv.SchemaUrls;
 import io.opentelemetry.semconv.incubating.HostIncubatingAttributes;
-import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.TestInstance;
-import org.junit.jupiter.api.condition.EnabledOnJre;
-import org.junit.jupiter.api.condition.JRE;
-import org.junit.jupiter.api.extension.ExtendWith;
 
 class HostResourceTest {
   @Test
@@ -30,18 +25,4 @@ void shouldCreateRuntimeAttributes() {
     assertThat(attributes.get(HostIncubatingAttributes.HOST_NAME)).isNotBlank();
     assertThat(attributes.get(HostIncubatingAttributes.HOST_ARCH)).isNotBlank();
   }
-
-  @Nested
-  @TestInstance(TestInstance.Lifecycle.PER_CLASS)
-  @ExtendWith(SecurityManagerExtension.class)
-  @EnabledOnJre(
-      value = {JRE.JAVA_8, JRE.JAVA_11, JRE.JAVA_16},
-      disabledReason = "Java 17 deprecates security manager for removal")
-  static class SecurityManagerEnabled {
-    @Test
-    void empty() {
-      Attributes attributes = HostResource.buildResource().getAttributes();
-      assertThat(attributes.asMap()).containsOnlyKeys(HostIncubatingAttributes.HOST_NAME);
-    }
-  }
 }

instrumentation/resources/library/src/test/java/io/opentelemetry/instrumentation/resources/OsResourceTest.java

@@ -10,12 +10,7 @@
 import io.opentelemetry.sdk.resources.Resource;
 import io.opentelemetry.semconv.SchemaUrls;
 import io.opentelemetry.semconv.incubating.OsIncubatingAttributes;
-import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.TestInstance;
-import org.junit.jupiter.api.condition.EnabledOnJre;
-import org.junit.jupiter.api.condition.JRE;
-import org.junit.jupiter.api.extension.ExtendWith;
 import org.junitpioneer.jupiter.SetSystemProperty;
 
 class OsResourceTest {
@@ -138,17 +133,4 @@ void unknown() {
     assertThat(resource.getAttribute(OsIncubatingAttributes.OS_TYPE)).isNull();
     assertThat(resource.getAttribute(OsIncubatingAttributes.OS_DESCRIPTION)).isNotEmpty();
   }
-
-  @Nested
-  @TestInstance(TestInstance.Lifecycle.PER_CLASS)
-  @ExtendWith(SecurityManagerExtension.class)
-  @EnabledOnJre(
-      value = {JRE.JAVA_8, JRE.JAVA_11, JRE.JAVA_16},
-      disabledReason = "Java 17 deprecates security manager for removal")
-  static class SecurityManagerEnabled {
-    @Test
-    void empty() {
-      assertThat(OsResource.buildResource()).isEqualTo(Resource.empty());
-    }
-  }
 }

instrumentation/resources/library/src/test/java/io/opentelemetry/instrumentation/resources/ProcessResourceTest.java

@@ -11,12 +11,7 @@
 import io.opentelemetry.sdk.resources.Resource;
 import io.opentelemetry.semconv.SchemaUrls;
 import io.opentelemetry.semconv.incubating.ProcessIncubatingAttributes;
-import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.TestInstance;
-import org.junit.jupiter.api.condition.EnabledOnJre;
-import org.junit.jupiter.api.condition.JRE;
-import org.junit.jupiter.api.extension.ExtendWith;
 import org.junitpioneer.jupiter.SetSystemProperty;
 
 class ProcessResourceTest {
@@ -52,18 +47,4 @@ void windows() {
     // With Java 9+ and a compiled jar, ResourceAttributes.PROCESS_COMMAND_ARGS
     // will be set instead of ResourceAttributes.PROCESS_COMMAND_LINE
   }
-
-  @Nested
-  @TestInstance(TestInstance.Lifecycle.PER_CLASS)
-  @ExtendWith(SecurityManagerExtension.class)
-  @EnabledOnJre(
-      value = {JRE.JAVA_8, JRE.JAVA_11, JRE.JAVA_16},
-      disabledReason = "Java 17 deprecates security manager for removal")
-  static class SecurityManagerEnabled {
-    @Test
-    void empty() {
-      Attributes attributes = ProcessResource.buildResource().getAttributes();
-      assertThat(attributes.asMap()).containsOnlyKeys(ProcessIncubatingAttributes.PROCESS_PID);
-    }
-  }
 }

instrumentation/resources/library/src/test/java/io/opentelemetry/instrumentation/resources/ProcessRuntimeResourceTest.java

@@ -11,12 +11,7 @@
 import io.opentelemetry.sdk.resources.Resource;
 import io.opentelemetry.semconv.SchemaUrls;
 import io.opentelemetry.semconv.incubating.ProcessIncubatingAttributes;
-import org.junit.jupiter.api.Nested;
 import org.junit.jupiter.api.Test;
-import org.junit.jupiter.api.TestInstance;
-import org.junit.jupiter.api.condition.EnabledOnJre;
-import org.junit.jupiter.api.condition.JRE;
-import org.junit.jupiter.api.extension.ExtendWith;
 
 class ProcessRuntimeResourceTest {
   @Test
@@ -32,17 +27,4 @@ void shouldCreateRuntimeAttributes() {
     assertThat(attributes.get(ProcessIncubatingAttributes.PROCESS_RUNTIME_DESCRIPTION))
         .isNotBlank();
   }
-
-  @Nested
-  @TestInstance(TestInstance.Lifecycle.PER_CLASS)
-  @ExtendWith(SecurityManagerExtension.class)
-  @EnabledOnJre(
-      value = {JRE.JAVA_8, JRE.JAVA_11, JRE.JAVA_16},
-      disabledReason = "Java 17 deprecates security manager for removal")
-  static class SecurityManagerEnabled {
-    @Test
-    void empty() {
-      assertThat(ProcessRuntimeResource.buildResource()).isEqualTo(Resource.empty());
-    }
-  }
 }

instrumentation/resources/library/src/testSecurityManager/java/io/opentelemetry/instrumentation/resources/SecurityManagerResourceTest.java

@@ -0,0 +1,46 @@
+/*
+ * Copyright The OpenTelemetry Authors
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+package io.opentelemetry.instrumentation.resources;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+import io.opentelemetry.api.common.Attributes;
+import io.opentelemetry.sdk.resources.Resource;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.TestInstance;
+import org.junit.jupiter.api.condition.EnabledOnJre;
+import org.junit.jupiter.api.condition.JRE;
+import org.junit.jupiter.api.extension.ExtendWith;
+
+@TestInstance(TestInstance.Lifecycle.PER_CLASS)
+@ExtendWith(SecurityManagerExtension.class)
+@EnabledOnJre(
+    value = {JRE.JAVA_8, JRE.JAVA_11, JRE.JAVA_16},
+    disabledReason = "Java 17 deprecates security manager for removal")
+class SecurityManagerResourceTest {
+
+  @Test
+  void hostResourceTestEmpty() {
+    Attributes attributes = HostResource.buildResource().getAttributes();
+    assertThat(attributes.asMap()).containsOnlyKeys(HostResource.HOST_NAME);
+  }
+
+  @Test
+  void osResourceEmpty() {
+    assertThat(OsResource.buildResource()).isEqualTo(Resource.empty());
+  }
+
+  @Test
+  void processResourceEmpty() {
+    Attributes attributes = ProcessResource.buildResource().getAttributes();
+    assertThat(attributes.asMap()).containsOnlyKeys(ProcessResource.PROCESS_PID);
+  }
+
+  @Test
+  void processRuntimeResourceEmpty() {
+    assertThat(ProcessRuntimeResource.buildResource()).isEqualTo(Resource.empty());
+  }
+}