prepare moving TestApp to separate sourceSet

Author: SylvainJuge

jmx-scraper/src/integrationTest/java/io/opentelemetry/contrib/jmxscraper/JmxConnectorBuilderTest.java

@@ -76,7 +76,7 @@ private static void testConnector(ConnectorSupplier connectorSupplier) {
           .satisfies(
               connection -> {
                 try {
-                  ObjectName name = new ObjectName(TestApp.OBJECT_NAME);
+                  ObjectName name = new ObjectName("io.opentelemetry.test:name=TestApp");
                   Object value = connection.getAttribute(name, "IntValue");
                   assertThat(value).isEqualTo(42);
                 } catch (Exception e) {

jmx-scraper/src/integrationTest/java/io/opentelemetry/contrib/jmxscraper/TestApp.java

@@ -9,14 +9,11 @@
 import javax.management.MBeanServer;
 import javax.management.ObjectName;
 
-@SuppressWarnings("all")
 public class TestApp implements TestAppMXBean {
 
-  public static final String APP_STARTED_MSG = "app started";
-  public static final String OBJECT_NAME = "io.opentelemetry.test:name=TestApp";
-
   private volatile boolean running;
 
+  @SuppressWarnings("BusyWait")
   public static void main(String[] args) {
     TestApp app = TestApp.start();
     while (app.isRunning()) {
@@ -34,13 +31,13 @@ static TestApp start() {
     TestApp app = new TestApp();
     MBeanServer mbs = ManagementFactory.getPlatformMBeanServer();
     try {
-      ObjectName objectName = new ObjectName(OBJECT_NAME);
+      ObjectName objectName = new ObjectName("io.opentelemetry.test:name=TestApp");
       mbs.registerMBean(app, objectName);
     } catch (Exception e) {
       throw new RuntimeException(e);
     }
     app.running = true;
-    System.out.println(APP_STARTED_MSG);
+    System.out.println("app started");
     return app;
   }
 

jmx-scraper/src/integrationTest/java/io/opentelemetry/contrib/jmxscraper/TestAppContainer.java

@@ -21,7 +21,7 @@
 import org.testcontainers.shaded.com.google.errorprone.annotations.CanIgnoreReturnValue;
 import org.testcontainers.utility.MountableFile;
 
-/** Test container that allows to execute {@link TestApp} in an isolated container */
+/** Test container that allows to execute TestApp in an isolated container */
 public class TestAppContainer extends GenericContainer<TestAppContainer> {
 
   private final Map<String, String> properties;
@@ -38,7 +38,7 @@ public TestAppContainer() {
 
     this.withCopyFileToContainer(MountableFile.forHostPath(appJar), "/app.jar")
         .waitingFor(
-            Wait.forLogMessage(TestApp.APP_STARTED_MSG + "\\n", 1)
+            Wait.forLogMessage("app started\\n", 1)
                 .withStartupTimeout(Duration.ofSeconds(5)))
         .withCommand("java", "-jar", "/app.jar");
   }