AcademySoftwareFoundation / AcademySoftwareFoundation/rez

modify rez-selftest -s arg

Open
#751 0 comments 0 reactions 0 assignees View on GitHub
enhancement selftest
Dominant language
Python
Stars
1.1k
Forks
374
Avg merge
9d 12h
Merged PRs (30d)
5

Description

Update the rez-selftest -s arg to take multiple shells, and for each of those, assert that the shell is available, rather than skip over it if not.

Right now it isn't that clear whether a given shell is actually getting tested in the CI; for eg, rez could think the shell is unavailable (which in itself may be a bug). So the way I'd see it working is, we would do the apt-get install etc of the shells we want to test, then pass those shells to rez-selftest also, which would fail if any of those shells are not available / rez doesn't know about them.

Contributor guide

Open the contributing guide

Research direction

Start at the rez-selftest -s argument handling and review how the CI installs and passes shells to it. Update the behavior to accept multiple shells and fail when any requested shell is unavailable or unrecognized; done means each installed CI shell is explicitly asserted rather than skipped.

Written by the indexing model from the issue text.

Assessment

Tech stack
python, shell
Domain
ci-cd, cli, testing-qa
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.