Skip to content

Commit 0a1c9e3

Browse files
committed
Fix performance tool flag declaration in test
For some reasons, after PicoCLI upgrade to 4.6.3, using explicitly "true" to set the monitoring flags would trigger a validation error when launching the test (only on Maven!). This commit fixes this strange issue by just adding the flag to the command line.
1 parent 2eb70d8 commit 0a1c9e3

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

src/test/java/com/rabbitmq/stream/perf/StreamPerfTestTest.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -321,9 +321,9 @@ void monitoringShouldReturnValidEndpoint() throws Exception {
321321
run(
322322
builder()
323323
.deleteStreams()
324-
.monitoring(true)
324+
.monitoring()
325325
.monitoringPort(monitoringPort)
326-
.prometheus(true));
326+
.prometheus());
327327
waitUntilStreamExists(s);
328328
waitOneSecond();
329329

@@ -520,8 +520,8 @@ ArgumentsBuilder producerNames(String pattern) {
520520
return this;
521521
}
522522

523-
ArgumentsBuilder monitoring(boolean monitoring) {
524-
arguments.put("monitoring", String.valueOf(monitoring));
523+
ArgumentsBuilder monitoring() {
524+
arguments.put("monitoring", "");
525525
return this;
526526
}
527527

@@ -530,8 +530,8 @@ ArgumentsBuilder monitoringPort(int port) {
530530
return this;
531531
}
532532

533-
ArgumentsBuilder prometheus(boolean prometheus) {
534-
arguments.put("prometheus", String.valueOf(prometheus));
533+
ArgumentsBuilder prometheus() {
534+
arguments.put("prometheus", "");
535535
return this;
536536
}
537537

0 commit comments

Comments
 (0)