Add basic unit-testing and CI

con / duct

A helper to run a command, capture stdout/stderr and details about running

https://pypi.org/project/con-duct/

MIT License

1 stars 2 forks source link

Add basic unit-testing and CI #6

Closed yarikoptic closed 3 months ago

yarikoptic commented 5 months ago

Candidates for unit-testing:

[x] without much of load, just printing to stdout, stderr and exiting with different exit statuses:
- [x] ensure that main portion of the code is running and producing expected files
- [x] ~parametrize through different values for the options and test if behaves accordingly~
[x] aggregation (should probably go to separate function)

overall just try following https://github.com/datalad/datalad-installer/blob/master/.github/workflows/test.yml as much as possible as to include type checking etc.

asmacdo commented 5 months ago

moving the matrix to to python version issue

yarikoptic commented 4 months ago

please update where testing is standing ATM since some tests were added, close this whenever all items are fulfilled

yarikoptic commented 4 months ago

should it be closed ?

asmacdo commented 4 months ago

No, I've got some testing but not all cases described above

yarikoptic commented 4 months ago

is there any item which could be "checked" already?

asmacdo commented 3 months ago

I opted not to use pytest parameters, IMO it is easier to read (and think about) like this.

yarikoptic commented 3 months ago

I opted not to use pytest parameters, IMO it is easier to read (and think about) like this.

as long as there is no severe code duplication, fine with me. But if it is the same test functionality with a few values change -- better be a parametric test for the sake of avoiding evil duplication