microsoft / microsoft/winget-cli
CONTRIBUTING.md: define testing requirements for contributions
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 26.4k
- Forks
- 1.8k
- Avg merge
- 1d 11h
- Merged PRs (30d)
- 15
Description
Part of #6393.
Problem
The Testing section of CONTRIBUTING.md is currently a single, aspirational sentence ("Testing is a key component in the development workflow.") with no actual guidance or requirement. doc/Developing.md explains how to run the unit tests but never states that contributions are expected to add or update tests.
Proposed change
Replace the stub Testing section in CONTRIBUTING.md with explicit guidance that:
- Fixes and features are expected to add or extend automated tests in the
AppInstallerCLITestsproject (and E2E tests inAppInstallerCLIE2ETestswhere relevant). - Describes at a high level how to build and run the tests, linking to
doc/Developing.md("Running Unit Tests"). - Notes that PRs without appropriate test coverage may be asked to add it before review completes.
Notes for engineers
Please confirm the exact bar we want to set: unit vs. E2E expectations, when tests can be reasonably omitted (e.g., pure doc changes), and any coverage guidance.
Acceptance criteria
CONTRIBUTING.mdTesting section states the expectation to add/update tests and links to how to run them.
(Created with GitHub Copilot assistance.)
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.
Research direction
Start with the Testing section in CONTRIBUTING.md, then read the “Running Unit Tests” section of doc/Developing.md and review the AppInstallerCLITests and AppInstallerCLIE2ETests references in the issue. Done means the CONTRIBUTING.md section states when contributions should add or update tests, links to the test-running guidance, and addresses the stated exceptions and expectations.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- cli, documentation, testing-qa
- Issue type
- Documentation
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 74/100