Skip to content

Commit 7212cde

Browse files
author
Rob Langley
committed
Plots - prepare for multiple series types
1 parent 59e48db commit 7212cde

File tree

3 files changed

+8
-4
lines changed

3 files changed

+8
-4
lines changed

job-dsl-core/src/main/groovy/javaposse/jobdsl/dsl/helpers/publisher/PlotContext.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ class PlotContext implements Context {
6464
void propertiesFile(String fileName, Closure plotSeriesClosure = null) {
6565
checkArgument(!Strings.isNullOrEmpty(fileName), 'fileName must not be null or empty')
6666

67-
PlotSeriesContext plotSeriesContext = new PlotSeriesContext(fileName)
67+
PlotSeriesContext plotSeriesContext = new PlotSeriesContext(fileName, 'properties', 'PropertiesSeries')
6868
ContextHelper.executeInContext(plotSeriesClosure, plotSeriesContext)
6969

7070
dataSeriesList << plotSeriesContext

job-dsl-core/src/main/groovy/javaposse/jobdsl/dsl/helpers/publisher/PlotSeriesContext.groovy

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,13 @@ import javaposse.jobdsl.dsl.Context
55
class PlotSeriesContext implements Context {
66
final String fileName
77
String label
8+
String fileType
9+
String seriesType
810

9-
PlotSeriesContext(String fileName) {
11+
PlotSeriesContext(String fileName, String fileType, String seriesType) {
1012
this.fileName = fileName
13+
this.fileType = fileType
14+
this.seriesType = seriesType
1115
}
1216

1317
void label(String label) {

job-dsl-core/src/main/groovy/javaposse/jobdsl/dsl/helpers/publisher/PublisherContext.groovy

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -376,10 +376,10 @@ class PublisherContext implements Context {
376376
yaxis(plot.yAxis ?: '')
377377
series {
378378
plot.dataSeriesList.each { PlotSeriesContext data ->
379-
'hudson.plugins.plot.PropertiesSeries' {
379+
"hudson.plugins.plot.${data.seriesType}" {
380380
file(data.fileName)
381381
label(data.label ?: '')
382-
fileType('properties')
382+
fileType(data.fileType)
383383
}
384384
}
385385
}

0 commit comments

Comments
 (0)