Skip to content

Commit 6d99342

Browse files
committed
Add a step to the CI build that fails on warnings in the functional tests (doesn't fail the build)
1 parent 2cc7085 commit 6d99342

File tree

3 files changed

+11
-2
lines changed

3 files changed

+11
-2
lines changed

.github/workflows/gradle.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,3 +25,6 @@ jobs:
2525
run: chmod +x gradlew
2626
- name: Build with Gradle
2727
run: ./gradlew --info --stacktrace check
28+
- name: Fail on warning
29+
run: ./gradlew --info --stacktrace check --warning-mode=fail -PfailOnWarning
30+
continue-on-error: true

build.gradle

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,6 +104,8 @@ task functionalTest(type: Test) {
104104
showStandardStreams = System.env.CI == 'true'
105105
}
106106

107+
systemProperty 'failOnWarning', project.hasProperty('failOnWarning')
108+
107109
mustRunAfter crossScalaVersionTest
108110
}
109111
check.dependsOn functionalTest
@@ -206,4 +208,4 @@ idea.project.settings {
206208
taskTriggers {
207209
beforeBuild fixIdeaPluginClasspath, pluginUnderTestMetadata
208210
}
209-
}
211+
}

src/functionalTest/java/org/scoverage/ScoverageFunctionalTest.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,11 @@ private void configureArguments(String... arguments) {
127127
fullArguments.add("-PjunitVersion=5.3.2");
128128
fullArguments.add("-PjunitPlatformVersion=1.3.2");
129129
fullArguments.add("-PscalatestVersion=3.0.8");
130-
fullArguments.add("--warning-mode=all");
130+
if (Boolean.parseBoolean(System.getProperty("failOnWarning"))) {
131+
fullArguments.add("--warning-mode=fail");
132+
} else {
133+
fullArguments.add("--warning-mode=all");
134+
}
131135
fullArguments.addAll(Arrays.asList(arguments));
132136

133137
runner.withArguments(fullArguments);

0 commit comments

Comments
 (0)