WebReflection / WebReflection/linkedom
Add proper tests for both assignedNodes and assignedElements
Nobody has claimed this yet.
- Dominant language
- HTML
- Stars
- 2.1k
- Forks
- 104
- PR merge metrics
- No merged PRs in 30d
Description
This MR was blocking devs but it looked OK(ish) to me ... I don't care much about slots but I want to enable devs ... I rather merge that in but I really would like to have tests around new features added to this project.
https://github.com/WebReflection/linkedom/pull/209
This is a one-off thing that won't likely be repeated in the future.
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
Read pull request #209 first to understand the assignedNodes and assignedElements features and the behavior that needs coverage. Then locate the project's existing test entry points and add tests for both APIs, with completion defined by coverage of their expected results.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- html
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100