Skip to content

Commit

Permalink
Merge branch 'jacoco' into 42-audit-get-automated-code-coverage-working
Browse files Browse the repository at this point in the history
  • Loading branch information
drernie committed Sep 3, 2024
2 parents 07883ef + 316c83b commit c6ee789
Show file tree
Hide file tree
Showing 3 changed files with 27 additions and 1 deletion.
4 changes: 4 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,10 @@ test-nextflow: clean nextflow-git compile check #coverage

test-all: clean compile-all check #coverage

coverage:
./gradlew jacocoTestReport
open plugins/nf-quilt/build/reports/jacoco/test/html/index.html

#
# Create packages
#
Expand Down
13 changes: 13 additions & 0 deletions build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -155,6 +155,19 @@ allprojects {
}
}

jacoco {
toolVersion = '0.8.12'
reportsDirectory = layout.buildDirectory.dir('reports/jacoco')
}

jacocoTestReport {
reports {
xml.required = true
csv.required = true
html.required = true
}
}

// this is required due to this IDEA bug
// https://youtrack.jetbrains.com/issue/IDEA-129282
sourceSets {
Expand Down
11 changes: 10 additions & 1 deletion plugins/nf-quilt/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ plugins {
}

useLatestVersions {
// A blacklist of dependencies to not update, in the format of group:name
// A blacklist of dependencies to update, in the format of group:name
updateBlacklist = [
'com.quiltdata:quiltcore',
'org.codehaus.groovy:groovy',
Expand Down Expand Up @@ -115,3 +115,12 @@ dependencies {
test {
useJUnitPlatform()
}

jacocoTestReport {
reports {
xml.required = true
csv.required = true
html.required = true
}
}

0 comments on commit c6ee789

Please sign in to comment.