Skip to content

Commit b736fcf

Browse files
committed
Full test coverage for HelpFormatter.rtrim()
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/cli/branches/cli-1.x@661638 13f79535-47bb-0310-9956-ffa450edef68
1 parent d36adeb commit b736fcf

1 file changed

Lines changed: 21 additions & 3 deletions

File tree

src/test/org/apache/commons/cli/HelpFormatterTest.java

Lines changed: 21 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -222,7 +222,8 @@ public int compare(Object o1, Object o2) {
222222
assertEquals("usage: app [-c] [-b] [-a]" + EOL, bytesOut.toString());
223223
}
224224

225-
public void testPrintOptionGroupUsage() {
225+
public void testPrintOptionGroupUsage()
226+
{
226227
OptionGroup group = new OptionGroup();
227228
group.addOption(OptionBuilder.create("a"));
228229
group.addOption(OptionBuilder.create("b"));
@@ -239,7 +240,8 @@ public void testPrintOptionGroupUsage() {
239240
assertEquals("usage: app [-a | -b | -c]" + EOL, out.toString());
240241
}
241242

242-
public void testPrintRequiredOptionGroupUsage() {
243+
public void testPrintRequiredOptionGroupUsage()
244+
{
243245
OptionGroup group = new OptionGroup();
244246
group.addOption(OptionBuilder.create("a"));
245247
group.addOption(OptionBuilder.create("b"));
@@ -257,7 +259,8 @@ public void testPrintRequiredOptionGroupUsage() {
257259
assertEquals("usage: app -a | -b | -c" + EOL, out.toString());
258260
}
259261

260-
public void testPrintOptionWithEmptyArgNameUsage() {
262+
public void testPrintOptionWithEmptyArgNameUsage()
263+
{
261264
Option option = new Option("f", true, null);
262265
option.setArgName("");
263266
option.setRequired(true);
@@ -273,4 +276,19 @@ public void testPrintOptionWithEmptyArgNameUsage() {
273276
assertEquals("usage: app -f" + EOL, out.toString());
274277
}
275278

279+
public void testRtrim()
280+
{
281+
HelpFormatter formatter = new HelpFormatter()
282+
{
283+
public String rtrim(String s)
284+
{
285+
return super.rtrim(s);
286+
}
287+
};
288+
289+
assertEquals(null, formatter.rtrim(null));
290+
assertEquals("", formatter.rtrim(""));
291+
assertEquals(" foo", formatter.rtrim(" foo "));
292+
}
293+
276294
}

0 commit comments

Comments
 (0)