1994 - adds pip install tests

[wyli]

Fixes #1994

• adds package min_tests for pre-merge ci/cd
• adds package full unit tests for nightly ci/cd

Status

Ready

tested:

• https://github.com/Project-MONAI/MONAI/runs/2647583616
• https://github.com/Project-MONAI/MONAI/runs/2647355573?check_suite_focus=true

Types of changes

• Non-breaking change (fix or new feature that would not break existing functionality).
• New tests added to cover the changes.
• Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
• Quick tests passed locally by running ./runtests.sh --quick --unittests.
• In-line docstrings updated.

[rijobro]

Am I correct in understanding that this PR doesn't install from pypi, but from tar.gz? Do you think this changes the "realness" of the test at all?

[wyli]

Am I correct in understanding that this PR doesn't install from pypi, but from tar.gz? Do you think this changes the "realness" of the test at all?

yes this is quite realistic, except that "pip install requirements-dev.txt" might hide some dependency errors. but we need it to construct a testing environment. we should consider refactoring requirements*.txt.

for the tar.gz, another approach is to upload the tar.gz to testPyPI server and pip install from there. this pipeline will then depend on the availability of the test PyPI server, and for now I don't think it's really needed.

[rijobro]

Ok, it sounds good to me. I can't really offer an opinion as to best practice, but happy to approve.

Has the workflow been run? Looking at the cron jobs and can't see cron-pip anywhere.

[wyli]

thanks, the cron will be available after merging the config. I had a test separately here: https://github.com/Project-MONAI/MONAI/actions/runs/867438967

[rijobro]

Great, thanks!