diff --git a/aeron-cluster/src/main/java/io/aeron/cluster/ClusterTool.java b/aeron-cluster/src/main/java/io/aeron/cluster/ClusterTool.java index b8f029e6cd..a793c1a241 100644 --- a/aeron-cluster/src/main/java/io/aeron/cluster/ClusterTool.java +++ b/aeron-cluster/src/main/java/io/aeron/cluster/ClusterTool.java @@ -508,10 +508,11 @@ public static boolean invalidateLatestSnapshot(final PrintStream out, final File * * @param out to print the operation result. * @param clusterDir where the cluster is running. + * @return true if the snapshot was successfully described false otherwise. */ - public static void describeLatestConsensusModuleSnapshot(final PrintStream out, final File clusterDir) + public static boolean describeLatestConsensusModuleSnapshot(final PrintStream out, final File clusterDir) { - BACKWARD_COMPATIBLE_OPERATIONS.describeLatestConsensusModuleSnapshot(clusterDir, out, null); + return BACKWARD_COMPATIBLE_OPERATIONS.describeLatestConsensusModuleSnapshot(clusterDir, out, null) == SUCCESS; } /** diff --git a/aeron-system-tests/src/test/java/io/aeron/cluster/ClusterToolTest.java b/aeron-system-tests/src/test/java/io/aeron/cluster/ClusterToolTest.java index 5140b28aec..602cdfe2b3 100644 --- a/aeron-system-tests/src/test/java/io/aeron/cluster/ClusterToolTest.java +++ b/aeron-system-tests/src/test/java/io/aeron/cluster/ClusterToolTest.java @@ -103,9 +103,9 @@ void shouldDescribeLatestConsensusModuleSnapshot() leader.consensusModule().context().clusterDir(), capturingPrintStream.resetAndGetPrintStream())); - ClusterTool.describeLatestConsensusModuleSnapshot( + assertTrue(ClusterTool.describeLatestConsensusModuleSnapshot( capturingPrintStream.resetAndGetPrintStream(), - leader.consensusModule().context().clusterDir()); + leader.consensusModule().context().clusterDir())); assertThat( capturingPrintStream.flushAndGetContent(),