Skip to content

Commit 7aac62f

Browse files
Only apply bootstrap classpath modifications to instrumentation tests
1 parent 62f4162 commit 7aac62f

File tree

8 files changed

+36
-10
lines changed

8 files changed

+36
-10
lines changed
Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
apply from: "$rootDir/gradle/java.gradle"
2+
3+
dependencies {
4+
api libs.bytebuddy
5+
api libs.bytebuddyagent
6+
api libs.guava
7+
8+
api project(':dd-java-agent:testing')
9+
api project(':utils:test-utils')
10+
11+
implementation 'org.junit.platform:junit-platform-launcher:1.12.0'
12+
}

dd-java-agent/testing/src/main/groovy/datadog/trace/agent/test/BootstrapClasspathSetup.java renamed to dd-java-agent/instrumentation/testing/src/main/groovy/datadog/trace/agent/test/BootstrapClasspathSetup.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
import java.util.Set;
1414
import java.util.TreeSet;
1515
import java.util.jar.JarFile;
16-
import net.bytebuddy.agent.ByteBuddyAgent;
16+
import net.bytebuddy.ageAgentTestRunnerTestnt.ByteBuddyAgent;
1717
import org.junit.platform.launcher.LauncherSession;
1818
import org.junit.platform.launcher.LauncherSessionListener;
1919

Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
datadog.trace.agent.test.BootstrapClasspathSetup
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
package datadog.trace.agent.test
2+
3+
4+
import datadog.trace.bootstrap.Constants
5+
import datadog.trace.test.util.DDSpecification
6+
7+
class BootstrapClasspathSetupTest extends DDSpecification {
8+
9+
def "spock runner bootstrap prefixes correct for test setup"() {
10+
expect:
11+
BootstrapClasspathSetup.BOOTSTRAP_PACKAGE_PREFIXES_COPY == Constants.BOOTSTRAP_PACKAGE_PREFIXES
12+
}
13+
}

dd-java-agent/testing/build.gradle

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,3 @@ shadowJar {
8787
relocate "org.eclipse.jetty", "datadog.eclipse.jetty"
8888
}
8989

90-
tasks.withType(Test).configureEach {
91-
useJUnitPlatform()
92-
}

dd-java-agent/testing/src/test/groovy/AgentTestRunnerTest.groovy

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import com.google.common.reflect.ClassPath
22
import datadog.trace.agent.test.AgentTestRunner
3-
import datadog.trace.agent.test.BootstrapClasspathSetup
43
import datadog.trace.agent.test.utils.ClasspathUtils
54
import datadog.trace.api.GlobalTracer
65
import datadog.trace.api.Platform
@@ -35,11 +34,6 @@ class AgentTestRunnerTest extends AgentTestRunner {
3534
sharedSpanClass = AgentSpan
3635
}
3736

38-
def "spock runner bootstrap prefixes correct for test setup"() {
39-
expect:
40-
BootstrapClasspathSetup.BOOTSTRAP_PACKAGE_PREFIXES_COPY == Constants.BOOTSTRAP_PACKAGE_PREFIXES
41-
}
42-
4337
def "classpath setup"() {
4438
setup:
4539
boolean jfrSupported = isJFRSupported()

gradle/java_no_deps.gradle

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,12 @@ if (applyCodeCoverage) {
3535
apply from: "$rootDir/gradle/jacoco.gradle"
3636
}
3737

38+
if (project.path.startsWith(":dd-java-agent:instrumentation")) {
39+
project.dependencies {
40+
testImplementation project(':dd-java-agent:instrumentation:testing')
41+
}
42+
}
43+
3844
sourceCompatibility = JavaVersion.VERSION_1_8
3945
targetCompatibility = JavaVersion.VERSION_1_8
4046

settings.gradle

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -209,6 +209,9 @@ include ':dd-smoke-tests:iast-util:iast-util-17'
209209
// annotation processor for checking instrumentation advice
210210
include ':dd-java-agent:instrumentation-annotation-processor'
211211

212+
// instrumentation testing utilities
213+
include ':dd-java-agent:instrumentation:testing'
214+
212215
// instrumentation:
213216
include ':dd-java-agent:instrumentation:aerospike-4'
214217
include ':dd-java-agent:instrumentation:akka:akka-actor-2.5'

0 commit comments

Comments
 (0)