oblac / jodd

Jodd! Lightweight. Java. Zero dependencies. Use what you like.
https://jodd.org
BSD 2-Clause "Simplified" License
4.06k stars 723 forks source link

Add test for Option.toString #788

Closed lacinoire closed 2 years ago

lacinoire commented 2 years ago

PR Checklist

Please check if your PR fulfills the following requirements:

Current behavior

No test for Option.toString

New behavior?

Hey 😊 I want to contribute the following test:

Test that cli.option().longName("foo").hasArg().with(out::add).toString() is equal to "--foo". This tests the method Option.toString. This test is based on the test testOptionWithValue.

Curious to hear what you think!

Also, is the description I provided of the test helping you to answer to this pull request? Why (not)?

(I wrote this test as part of a research study at TU Delft. Find out more)