Skip to content

Commit ff7292f

Browse files
committed
Clean up BuildInsightsPrinter
1 parent 0599e23 commit ff7292f

File tree

1 file changed

+12
-11
lines changed

1 file changed

+12
-11
lines changed

components/fetch-build-scan-data-cmdline-tool/src/main/java/com/gradle/enterprise/cli/BuildInsightsPrinter.java

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,9 @@
44
import com.gradle.enterprise.model.BuildTimeMetrics;
55

66
import java.util.List;
7-
import java.util.stream.Collectors;
7+
import java.util.stream.Stream;
8+
9+
import static java.util.stream.Collectors.joining;
810

911
final class BuildInsightsPrinter {
1012

@@ -19,33 +21,32 @@ static void printInsights(List<BuildScanData> buildScanData) {
1921
}
2022

2123
private static void printBuildScanDataHeader() {
22-
List<String> labels = BuildScanDataFields.ordered().map(f -> f.label).collect(Collectors.toList());
23-
System.out.println(String.join(",", labels));
24+
printRow(BuildScanDataFields.ordered().map(f -> f.label));
2425
}
2526

2627
private static void printBuildScanData(List<BuildScanData> buildScanData) {
2728
buildScanData.forEach(BuildInsightsPrinter::printBuildScanData);
2829
}
2930

3031
private static void printBuildScanData(BuildScanData buildScanData) {
31-
List<String> values = BuildScanDataFields.ordered().map(f -> f.value.apply(buildScanData)).collect(Collectors.toList());
32-
System.out.println(String.join(",", values));
32+
printRow(BuildScanDataFields.ordered().map(f -> f.value.apply(buildScanData)));
3333
}
3434

3535
private static void printBuildTimeMetricsHeader() {
36-
List<String> labels = BuildTimeMetricsFields.ordered().map(f -> f.label).collect(Collectors.toList());
37-
System.out.println(String.join(",", labels));
36+
printRow(BuildTimeMetricsFields.ordered().map(f -> f.label));
3837
}
3938

4039
private static void printBuildTimeMetrics(List<BuildScanData> buildScanData) {
4140
final BuildTimeMetrics buildTimeData = BuildTimeMetrics.from(buildScanData.get(0), buildScanData.get(1));
42-
List<String> values;
4341
if (buildTimeData == null) {
44-
values = BuildTimeMetricsFields.ordered().map(f -> "").collect(Collectors.toList());
42+
printRow(BuildTimeMetricsFields.ordered().map(f -> ""));
4543
} else {
46-
values = BuildTimeMetricsFields.ordered().map(f -> f.value.apply(buildTimeData)).collect(Collectors.toList());
44+
printRow(BuildTimeMetricsFields.ordered().map(f -> f.value.apply(buildTimeData)));
4745
}
48-
System.out.println(String.join(",", values));
46+
}
47+
48+
private static void printRow(Stream<String> values) {
49+
System.out.println(values.collect(joining(",")));
4950
}
5051

5152
private BuildInsightsPrinter() {

0 commit comments

Comments
 (0)