nodejs / nodejs/core-validate-commit
Add Tests for command line arguments
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 55
- Forks
- 62
- Avg merge
- 4d 9h
- Merged PRs (30d)
- 1
Description
When i was adding in the tests for the --list-subsystem flag, i noticed that there were no tests for the other command line flags.
Now that there is a test for list-subsystems(which has affected the code coverage), there should probably be tests for the other commands
Instead of creating separate issues for each command, this issue is to track the addition of those other tests.
- usage
- version
- validate-metadata
- tap
- list
- list-subsystems
I'm happy to continue working on adding these items :)
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 by reviewing the existing tests for the --list-subsystems flag and the command-line entry points for usage, version, validate-metadata, tap, and list. Add coverage for each unchecked command and run the test suite; the issue is done when all listed commands have tests comparable to list-subsystems.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- cli, testing-qa
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100