ReferenceField has no component tests and only 42.4% line coverage
Maintainers usually reply within 2 days
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 72/100
- Issue type
- Feature
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- python, react, typescript
Research direction
Start with frontend/src/components/Parameters/ReferenceField.tsx, especially lines 31–76, and read the existing component tests to follow their interaction-testing conventions. Add tests for selecting repeated references, reordering and removing entries, and disabled boundary controls; done means these user behaviors are covered without treating a target coverage percentage as the goal.
Written by the indexing model from the issue text.
Description
Is your feature request related to a problem? Please describe.
ReferenceField has limited direct component coverage, leaving its user-facing ordering/removal and disabled-boundary behavior unverified.
Describe the solution you'd like
Add interaction tests for selecting/adding repeated references, moving entries up/down, removing entries, and disabled boundary controls.
Describe alternatives you've considered, if relevant.
Prioritize observable user behavior over targeting a particular coverage percentage.
Additional context
Measured line coverage is 42.4%; no claim is made that uncovered lines are all bugs. Source: ReferenceField. Tested commit d3446e045960b5ebee4641aa70fdeb03af186587, PyRIT 1.2.0.dev0, offline test/coverage scope. Evidence: files/evidence/round4/coverage-referencefield.json; local path is not an attachment.
- Dominant language
- Python
- Stars
- 4.7k
- Forks
- 1k
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 439
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing 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.
More from microsoft/PyRIT
-
Description-only preset edits split comma-containing exact goals into separate goalsPossibly taken @MRX-72 claimed this 1 day ago. Openhelp wanted
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 2 days
-
GUI rejects valid empty-string SearchReplaceConverter settings, preventing deletion convertersPossibly taken @Darshika140 claimed this 1 day ago. Openhelp wanted
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
microsoft/PyRIT#3749 · 1 comment ·
Maintainers usually reply within 2 days
-
HTML scenario export replaces an intentionally empty converted message with the original textPossibly taken @VISHNU0906 claimed this 1 day ago. Openhelp wanted
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
microsoft/PyRIT#3741 · 1 comment · 1 assignee ·
Maintainers usually reply within 2 days
-
Development toolchain includes an audited vulnerable Zod versionPossibly taken @ppcvote claimed this today. Openhelp wanted
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
microsoft/PyRIT#3725 · 1 comment ·
Maintainers usually reply within 2 days
-
TextTarget is described as using API-key authentication even though it needs no credentialsPossibly taken @pcbeingused333 claimed this 1 day ago. Openhelp wanted
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Maintainers usually reply within 2 days
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Nixtla/mlforecast#823 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
microsoft/ML-For-Beginners#1061 ·
-
bot-found documentation priority: P3
Difficulty 2/5 Under an hour Newbie friendliness 82/100
madenvel/KalinkaPlayer#503 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 63/100
EleutherAI/lm-evaluation-harness#4508 ·
Maintainers usually reply within 1 day