lib/cli: unit tests

BodgeMaster commented

2022-07-14 02:01:40 +02:00

Owner

The argument parsing for tools/hexnet is really ugly and we are going to need more of that for other programs.

Yes, I’m just adding an issue for what I’ve been working on for hours now.

The argument parsing for tools/hexnet is really ugly and we are going to need more of that for other programs. Yes, I’m just adding an issue for what I’ve been working on for hours now.

BodgeMaster added the

TODO

Enhancement

labels 2022-07-14 02:01:40 +02:00

BodgeMaster self-assigned this 2022-07-14 02:01:40 +02:00

BodgeMaster commented

2022-07-14 05:28:02 +02:00

Poster

Owner

arguments parser is implemented, unit tests needed

BodgeMaster removed the

Enhancement

label 2022-07-14 05:28:27 +02:00

BodgeMaster removed their assignment 2022-07-14 05:28:54 +02:00

BodgeMaster added a new dependency 2022-07-14 05:34:32 +02:00

#28 tools/hexnet: use arguments parser in cli lib

BodgeMaster changed title from ~~lib/cli: Build a proper arguments parser~~ to lib/cli: unit tests

2022-07-14 05:34:49 +02:00

BodgeMaster commented

2022-07-15 11:39:59 +02:00

Poster

Owner

Unit test for valid input has been implemented.

Now we need unit tests that try to break things in all sorts of ways.

Unit test for valid input has been implemented. Now we need unit tests that try to break things in all sorts of ways.

BodgeMaster removed a dependency 2022-07-15 13:41:08 +02:00

#28 tools/hexnet: use arguments parser in cli lib

BodgeMaster commented

2022-07-15 19:54:06 +02:00

Poster

Owner

TODO-list of remaining unit tests:

unknown flags (standalone)
unknown flags (mixed with other input)
unknown unpositional argument (standalone)
unknown unpositional argument (mixed with other input)
incomplete unpositional arguments (standalone)
incomplete unpositional arguments (mixed with other input (unpositional, positional args; before and after))
too few positional arguments (standalone)
too few positional arguments (mixed with other input)
too many positional arguments (standalone)
too many positional arguments (mixed with other input)
check the produced error messages

TODO-list of remaining unit tests: - [x] unknown flags (standalone) - [x] unknown flags (mixed with other input) - [x] unknown unpositional argument (standalone) - [x] unknown unpositional argument (mixed with other input) - [x] incomplete unpositional arguments (standalone) - [x] incomplete unpositional arguments (mixed with other input (unpositional, positional args; before and after)) - [x] too few positional arguments (standalone) - [x] too few positional arguments (mixed with other input) - [x] too many positional arguments (standalone) - [x] too many positional arguments (mixed with other input) - [ ] check the produced error messages

BodgeMaster self-assigned this 2022-07-19 04:20:54 +02:00

BodgeMaster commented

2022-07-21 09:46:11 +02:00

Poster

Owner

can’t edit from this browser, implemented test for produced error messages

BodgeMaster closed this issue

2022-07-21 09:46:11 +02:00

lib/cli: unit tests #27