Snapshot testing for resolver tests
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 362
- Forks
- 75
- Avg merge
- 6d 9h
- Merged PRs (30d)
- 35
Description
Is your refactor request related to a problem? Please describe.
I think our resolver unit tests could benefit from snapshot testing, as writing them currently is a bit tedious imo. This is somewhat related to https://github.com/PLC-lang/rusty/issues/1001.
Describe the solution you'd like
Introduce a function similar to parse_and_validate_buffered but for annotations.
Contributor guide
No contributing guide indexed for this repository
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 reading the resolver unit tests and the existing parse_and_validate_buffered function. Determine how annotation parsing should parallel that entry point and what snapshot coverage is expected; the work is done when resolver annotation cases can use the new helper and their snapshots are maintained by the tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- compilers, testing-qa
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100