Add more unit tests to package manager chart
Open
Nobody has claimed this yet.
team: package manager
- Dominant language
- Markdown
- Stars
- 46
- Forks
- 40
- Avg merge
- 4h 3m
- Merged PRs (30d)
- 6
Description
Workbench has added a variety of tests, which you can see in
- https://github.com/rstudio/helm/tree/main/charts/rstudio-workbench/lint
- https://github.com/rstudio/helm/tree/main/charts/rstudio-workbench/tests
- https://github.com/rstudio/helm/tree/main/charts/rstudio-workbench/ci
Talk to Workbench about this experience, and add some to https://github.com/rstudio/helm/tree/main/charts/rstudio-pm
A good first step would be to bootstrap a basic boilerplate unit test
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
Compare the existing tests in charts/rstudio-workbench/lint, charts/rstudio-workbench/tests, and charts/rstudio-workbench/ci with the charts/rstudio-pm chart. Start by identifying the testing setup and bootstrap a basic unit test there; done means rstudio-pm has an initial unit-test structure with useful coverage comparable to the referenced Workbench chart.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- helm
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100