[CLI-322] Add '-' as an option char and implemented extensive tests

[Claudenw]

Fix for CLI-322

Adds kabab-format options and extensive option name testing.

[garydgregory]

@Claudenw
I think you mean "kabab" -> "kebab". Always run mvn before you push with the latest from git master.

[Claudenw]

@Claudenw I think you mean "kabab" -> "kebab". Always run mvn before you push with the latest from git master.

Indeed, I do not know how to spell. :(

I did run 'mvn' before the push and everything worked. Got "BUILD SUCCESS" result. Just tried again and same thing. I am addressing the issues from the build now.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9f6b23b) 91.90% compared to head (38e15c4) 91.93%.
Report is 4 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master     #217      +/-   ##
============================================
+ Coverage     91.90%   91.93%   +0.03%     
- Complexity      575      580       +5     
============================================
  Files            22       22              
  Lines          1247     1252       +5     
  Branches        210      211       +1     
============================================
+ Hits           1146     1151       +5     
  Misses           63       63              
  Partials         38       38              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Claudenw]

The build fails on 22-ea which fails with a spotbugs execution error, It seems like spotbugs is not yet supported on 22-ea.

[garydgregory]

RE 22-ea, that should be fixed by the next commons-parent release, so don't worry about that part.

[Claudenw]

@garydgregory Looks like it works. Can I get a review plz.

[garydgregory]

'ary' is a poor parameter name here, almost any else else is better: 'chars', 'array, 'buffer'...

[garydgregory]

Hi @Claudenw
The site needs matching updates to document this feature.

[Claudenw]

@garydgregory updated as requested

[garydgregory]

Hi @Claudenw
Run mvn locally before you push to catch these build failures.