update

Author: ArnavBalyan

parquet-cli/src/test/java/org/apache/parquet/cli/ShowSizeStatisticsCliTest.java

@@ -35,9 +35,7 @@ public void showSizeStatistics() throws Exception {
 
   @Test
   public void showsHelpMessage() throws Exception {
-    cli("help size-stats")
-        .ok()
-        .matchOutputFromFile("src/test/resources/cli-outputs/size-stats-help.txt");
+    cli("help size-stats").ok().matchOutputFromFile("src/test/resources/cli-outputs/size-stats-help.txt");
   }
 
   @Test

parquet-cli/src/test/java/org/apache/parquet/cli/testing/CapturingLogger.java

@@ -45,7 +45,6 @@ String dump() {
     return buf.toString();
   }
 
-
   // Since the CLI logic can call any console method, this is some needed delegator code to
   // ensure all methods are coverted and that the test harness does not miss anything.
   // Unfortunately slf4j API does not make this easy to do in a generic way, so we

parquet-cli/src/test/java/org/apache/parquet/cli/testing/CliResult.java

@@ -19,9 +19,10 @@
 package org.apache.parquet.cli.testing;
 
 import static org.junit.Assert.*;
+
+import java.nio.charset.StandardCharsets;
 import java.nio.file.Files;
 import java.nio.file.Paths;
-import java.nio.charset.StandardCharsets;
 
 public final class CliResult {
   public final int exitCode;
@@ -64,9 +65,7 @@ public CliResult lineCount(int expected) {
   }
 
   public CliResult matchOutputFromFile(String filePath) throws Exception {
-    String expected = new String(
-        Files.readAllBytes(Paths.get(filePath)),
-        StandardCharsets.UTF_8);
+    String expected = new String(Files.readAllBytes(Paths.get(filePath)), StandardCharsets.UTF_8);
     return outputContains(expected);
   }
 }

parquet-cli/src/test/java/org/apache/parquet/cli/testing/CliTestBase.java

@@ -20,6 +20,32 @@
 
 import org.apache.parquet.cli.commands.ParquetFileTest;
 
+/**
+ * Base class for CLI integration tests with an API for testing command output.
+ *
+ * Developer Usage Examples:
+ *
+ * // Basic command execution and assertion
+ * cli("schema file.parquet")
+ *     .ok()
+ *     .outputContains("int32_field", "int64_field");
+ *
+ * // Test help output
+ * cli("help size-stats")
+ *     .ok()
+ *     .matchOutputFromFile("expected-help.txt");
+ *
+ * // Test error conditions
+ * cli("invalid-command")
+ *     .fails(1)
+ *     .outputContains("Unknown command");
+ *
+ * // Test command with multiple arguments
+ * cli("size-stats parquetFile.getAbsolutePath()")
+ *     .ok()
+ *     .lineCount(8);
+ *
+ */
 public abstract class CliTestBase extends ParquetFileTest {
   private final CliHarness harness = new CliHarness();