Review all existing tests
Open
@mawad-amd is already working on this.
Since Feb 4, 2026.
duplicate
examples
iris
test
- Dominant language
- Python
- Stars
- 202
- Forks
- 47
- Avg merge
- 6d 11h
- Merged PRs (30d)
- 4
Description
Review all existing tests
We like tests but we like tests too much. We have 2,454 tests (see https://github.com/ROCm/iris/pull/348 for reference).
Your job: Review the entire test suite and tell us what's redundant, duplicate, unnecessarily time-consuming, or just plain wasteful.
What we want:
A response with:
- Summary of findings (what's bloated, what's redundant, what's slow)
- Specific examples of problematic tests with reasoning
- A plan for reducing test count while maintaining coverage
- Estimated time/cost savings
No code changes. Just analysis and recommendations.
Tools:
- Check
.github/workflows/to understand the CI testing structure (60 test matrix jobs across 5 directories, 4 rank configs, 3 install methods) - Use
ghCLI to check CI run times and patterns from recent PRs (start with #348) - Look at test execution logs for timing data
- Dig through the test code in
tests/
Give us specific recommendations with reasoning. No handwaving - we need actionable insights.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.