Skip to content

Commit cc7510a

Browse files
committed
chore: More laziness on some Gradle jvmArgs
1 parent 55e6ae2 commit cc7510a

File tree

6 files changed

+57
-7
lines changed

6 files changed

+57
-7
lines changed

dd-smoke-tests/spring-security/build.gradle

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,13 @@ dependencies {
3232

3333

3434
tasks.withType(Test).configureEach {
35-
jvmArgs "-Ddatadog.smoketest.springboot.shadowJar.path=${tasks.jar.archiveFile.get()}"
35+
jvmArgumentProviders.add(new CommandLineArgumentProvider() {
36+
@Override
37+
Iterable<String> asArguments() {
38+
def jarTask = tasks.named('jar', Jar).get()
39+
return ["-Ddatadog.smoketest.springboot.shadowJar.path=${jarTask.archiveFile.get()}"]
40+
}
41+
})
3642
}
3743

3844
tasks.named("processResources", ProcessResources) {

dd-smoke-tests/springboot-jetty-jsp/build.gradle

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import org.springframework.boot.gradle.tasks.bundling.BootWar
2+
13
plugins {
24
id 'java'
35
id 'war'
@@ -34,5 +36,12 @@ dependencies {
3436

3537
tasks.withType(Test).configureEach {
3638
dependsOn "war", "bootWar"
37-
jvmArgs "-Ddatadog.smoketest.springboot.war.path=${tasks.bootWar.archiveFile.get().getAsFile()}"
39+
40+
jvmArgumentProviders.add(new CommandLineArgumentProvider() {
41+
@Override
42+
Iterable<String> asArguments() {
43+
def bootWarTask = tasks.named('bootWar', BootWar).get()
44+
return ["-Ddatadog.smoketest.springboot.war.path=${bootWarTask.archiveFile.get().getAsFile()}"]
45+
}
46+
})
3847
}

dd-smoke-tests/springboot-jpa/build.gradle

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import org.springframework.boot.gradle.tasks.bundling.BootWar
2+
13
plugins {
24
id 'java'
35
id 'war'
@@ -27,7 +29,14 @@ dependencies {
2729

2830
tasks.withType(Test).configureEach {
2931
dependsOn "bootWar"
30-
jvmArgs "-Ddatadog.smoketest.springboot.bootWar.path=${tasks.bootWar.archiveFile.get()}"
32+
33+
jvmArgumentProviders.add(new CommandLineArgumentProvider() {
34+
@Override
35+
Iterable<String> asArguments() {
36+
def bootWarTask = tasks.named('bootWar', BootWar).get()
37+
return ["-Ddatadog.smoketest.springboot.bootWar.path=${bootWarTask.archiveFile.get()}"]
38+
}
39+
})
3140
}
3241

3342
tasks.withType(GroovyCompile).configureEach {

dd-smoke-tests/springboot-tomcat-jsp/build.gradle

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import org.springframework.boot.gradle.tasks.bundling.BootWar
2+
13
plugins {
24
id 'java'
35
id 'war'
@@ -34,5 +36,12 @@ dependencies {
3436

3537
tasks.withType(Test).configureEach {
3638
dependsOn "war", "bootWar"
37-
jvmArgs "-Ddatadog.smoketest.springboot.war.path=${tasks.bootWar.archiveFile.get().getAsFile()}"
39+
40+
jvmArgumentProviders.add(new CommandLineArgumentProvider() {
41+
@Override
42+
Iterable<String> asArguments() {
43+
def bootWarTask = tasks.named('bootWar', BootWar).get()
44+
return ["-Ddatadog.smoketest.springboot.war.path=${bootWarTask.archiveFile.get().getAsFile()}"]
45+
}
46+
})
3847
}

dd-smoke-tests/springboot-tomcat/build.gradle

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import org.springframework.boot.gradle.tasks.bundling.BootWar
2+
13
plugins {
24
id 'war'
35
id 'org.springframework.boot' version '2.5.12'
@@ -110,6 +112,15 @@ tasks.matching({it.name.startsWith('compileTest')}).configureEach {
110112

111113
tasks.withType(Test).configureEach {
112114
dependsOn "war", "bootWar", "unzip"
113-
jvmArgs "-Ddatadog.smoketest.springboot.war.path=${tasks.bootWar.archiveFile.get().getAsFile()}"
114-
jvmArgs "-Ddatadog.smoketest.tomcatDir=${buildDir}/apache-${serverName}-${serverVersion}"
115+
116+
jvmArgumentProviders.add(new CommandLineArgumentProvider() {
117+
@Override
118+
Iterable<String> asArguments() {
119+
def bootWarTask = tasks.named('bootWar', BootWar).get()
120+
return [
121+
"-Ddatadog.smoketest.springboot.war.path=${bootWarTask.archiveFile.get().getAsFile()}",
122+
"-Ddatadog.smoketest.tomcatDir=${layout.buildDirectory.get()}/apache-${serverName}-${serverVersion}"
123+
]
124+
}
125+
})
115126
}

dd-smoke-tests/tracer-flare/build.gradle

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,5 +14,11 @@ dependencies {
1414
tasks.withType(Test).configureEach {
1515
dependsOn "jar"
1616

17-
jvmArgs "-Ddatadog.smoketest.tracer-flare.jar.path=${tasks.jar.archiveFile.get()}"
17+
jvmArgumentProviders.add(new CommandLineArgumentProvider() {
18+
@Override
19+
Iterable<String> asArguments() {
20+
def jarTask = tasks.named('jar', Jar).get()
21+
return ["-Ddatadog.smoketest.tracer-flare.jar.path=${jarTask.archiveFile.get()}"]
22+
}
23+
})
1824
}

0 commit comments

Comments
 (0)