diff --git a/dd-java-agent-ittests/pom.xml b/dd-java-agent-ittests/pom.xml index 095bf4ad64..65652d3e7b 100644 --- a/dd-java-agent-ittests/pom.xml +++ b/dd-java-agent-ittests/pom.xml @@ -139,57 +139,21 @@ org.apache.maven.plugins maven-surefire-plugin - 2.20 + 2.19 - true + false *Test.java - 3 - false - -Xmx1g -Xms1g -XX:MaxPermSize=512m -Dorg.jboss.byteman.verbose=true + + + -javaagent:${M2_REPO}/com/datadoghq/dd-java-agent/${project.version}/dd-java-agent-${project.version}.jar - target/FORK_DIRECTORY_${surefire.forkNumber} + - - org.apache.maven.plugins - maven-shade-plugin - 2.4.3 - - - package - - shade - - - - - - - - - - - otarules.btm - - - - io.opentracing.contrib.agent.AnnotationsTracingAgent - io.opentracing.contrib.agent.AnnotationsTracingAgent - - true - true - ./${project.artifactId}.jar - - - - - - - diff --git a/dd-java-agent-ittests/src/test/java/com/datadoghq/trace/instrument/AAgentIntegration.java b/dd-java-agent-ittests/src/test/java/com/datadoghq/trace/instrument/AAgentIntegration.java new file mode 100644 index 0000000000..fde44f6220 --- /dev/null +++ b/dd-java-agent-ittests/src/test/java/com/datadoghq/trace/instrument/AAgentIntegration.java @@ -0,0 +1,22 @@ +package com.datadoghq.trace.instrument; + +import org.junit.Before; + +import io.opentracing.mock.MockTracer; +import io.opentracing.util.GlobalTracer; + +public class AAgentIntegration { + + protected static MockTracer tracer = new MockTracer(); + + @Before + public void beforeTest() throws Exception { + try{ + GlobalTracer.register(tracer); + }catch(Exception e){ + //DO NOTHING IF ALREADY REGISTERED + } + tracer.reset(); + } + +} diff --git a/dd-java-agent-ittests/src/test/java/com/datadoghq/trace/instrument/ApacheHTTPClientTest.java b/dd-java-agent-ittests/src/test/java/com/datadoghq/trace/instrument/ApacheHTTPClientTest.java index b99f6057d5..4587d8a991 100644 --- a/dd-java-agent-ittests/src/test/java/com/datadoghq/trace/instrument/ApacheHTTPClientTest.java +++ b/dd-java-agent-ittests/src/test/java/com/datadoghq/trace/instrument/ApacheHTTPClientTest.java @@ -9,20 +9,9 @@ import org.apache.http.HttpResponse; import org.apache.http.client.HttpClient; import org.apache.http.client.methods.HttpGet; import org.apache.http.impl.client.HttpClientBuilder; -import org.junit.Before; import org.junit.Test; -import io.opentracing.mock.MockTracer; -import io.opentracing.util.GlobalTracer; - -public class ApacheHTTPClientTest { - - protected MockTracer tracer = new MockTracer(); - - @Before - public void beforeTest() throws Exception { - GlobalTracer.register(tracer); - } +public class ApacheHTTPClientTest extends AAgentIntegration{ @Test public void test() throws Exception{ diff --git a/dd-java-agent-ittests/src/test/java/io/opentracing/contrib/agent/TraceAnnotationsManagerTest.java b/dd-java-agent-ittests/src/test/java/io/opentracing/contrib/agent/TraceAnnotationsManagerTest.java index 5c4d590fb2..5788b5d57d 100644 --- a/dd-java-agent-ittests/src/test/java/io/opentracing/contrib/agent/TraceAnnotationsManagerTest.java +++ b/dd-java-agent-ittests/src/test/java/io/opentracing/contrib/agent/TraceAnnotationsManagerTest.java @@ -5,16 +5,13 @@ import static org.assertj.core.api.Assertions.assertThat; import org.junit.Before; import org.junit.Test; -import io.opentracing.mock.MockTracer; -import io.opentracing.util.GlobalTracer; - -public class TraceAnnotationsManagerTest { - - protected MockTracer tracer = new MockTracer(); +import com.datadoghq.trace.instrument.AAgentIntegration; +public class TraceAnnotationsManagerTest extends AAgentIntegration{ + @Before public void beforeTest() throws Exception { - GlobalTracer.register(tracer); + super.beforeTest(); TraceAnnotationsManager.loadAnnotationsRules(ClassLoader.getSystemClassLoader()); }