Moved default value to ReporterConfig instead of Command line interpretation

Author: pesse

src/main/java/org/utplsql/cli/RunPicocliCommand.java

@@ -99,7 +99,7 @@ public class RunPicocliCommand implements IRunCommand {
  private List<Format> reporters = new ArrayList<>();
 
  static class Format {
- @Option(names = {"-f", "--format"}, required = true, description = "Enables specified format reporting", defaultValue = "ut_documentation_reporter")
+ @Option(names = {"-f", "--format"}, required = true, description = "Enables specified format reporting")
  String format;
  @Option(names = {"-o"}, description = "Outputs format to file")
  String outputFile;

src/main/java/org/utplsql/cli/config/ReporterConfig.java

@@ -1,5 +1,7 @@
 package org.utplsql.cli.config;
 
+import org.utplsql.api.reporter.CoreReporters;
+
 import java.beans.ConstructorProperties;
 
 public class ReporterConfig {
@@ -10,7 +12,11 @@ public class ReporterConfig {
 
  @ConstructorProperties({"name", "output", "forceToScreen"})
  public ReporterConfig(String name, String output, Boolean forceToScreen) {
- this.name = name;
+ if ( name != null ) {
+ this.name = name;
+ } else {
+ this.name = CoreReporters.UT_DOCUMENTATION_REPORTER.name();
+ }
  this.output = output;
  if (forceToScreen != null) this.forceToScreen = forceToScreen;
  }

src/test/java/org/utplsql/cli/PicocliRunCommandTest.java

@@ -172,7 +172,7 @@ void outputWithDefaultReporter() throws Exception {
  assertNotNull( config.getReporters() );
 
  ReporterConfig reporterConfig = config.getReporters()[0];
- assertEquals("ut_documentation_reporter", reporterConfig.getName());
+ assertEquals("ut_documentation_reporter", reporterConfig.getName().toLowerCase());
  assertEquals("output1.txt", reporterConfig.getOutput());
  assertFalse(reporterConfig.isForceToScreen());
  }