Skip to content

Commit

Permalink
bug: #586 adjust plugins to use file instead of string
Browse files Browse the repository at this point in the history
  • Loading branch information
rohwerj committed Aug 23, 2024
1 parent 0b05faf commit 1e4cac8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ package org.camunda.community.process_test_coverage.report.aggregator
import org.camunda.community.process_test_coverage.core.export.CoverageStateJsonExporter.combineCoverageStateResults
import org.camunda.community.process_test_coverage.core.export.CoverageStateJsonExporter.createCoverageStateResult
import org.camunda.community.process_test_coverage.core.export.CoverageStateJsonExporter.readCoverageStateResult
import org.camunda.community.process_test_coverage.core.export.CoverageStateResult
import org.camunda.community.process_test_coverage.report.CoverageReportUtil
import org.gradle.api.Plugin
import org.gradle.api.Project
Expand All @@ -39,7 +38,7 @@ class ReportAggregatorPlugin : Plugin<Project> {
val extension = project.extensions.create("aggregateProcessTestCoverage", ReportAggregatorPluginExtension::class.java)
project.task("aggregateProcessTestCoverage")
.doLast {
val outputDirectory = File(File(project.projectDir, extension.reportDirectory), extension.outputDirectory).path
val outputDirectory = File(File(project.projectDir, extension.reportDirectory), extension.outputDirectory)
project.allprojects
.asSequence()
.map {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ class ReportAggregatorMojo : AbstractMojo(), MavenReport {
.reduceOrNull { result1, result2 -> combineCoverageStateResults(result1, result2) }
?.let {
val report = readCoverageStateResult(it)
val aggregateReportDirectory = File(File(project.basedir, targetDirectory), outputDirectory).path
val aggregateReportDirectory = File(File(project.basedir, targetDirectory), outputDirectory)
CoverageReportUtil.writeReport(createCoverageStateResult(report.suites, report.models), false,
aggregateReportDirectory, "report.json"
) { result -> result }
Expand Down

0 comments on commit 1e4cac8

Please sign in to comment.