Add unit tests for personUtils.ts
Maintainers usually reply within 6 days
@mystic-fae is already working on this.
Since Nov 14, 2026.
Assessment
- Difficulty
- 1/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 62/100
- Issue type
- Feature
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- react, typescript
- Domain
- frontend, testing-qa
Research direction
Start with frontend/src/utils/personUtils.ts and look at an existing test in a sibling utils tests folder to copy its conventions. Write tests for getPersonName, getPhotoCountText, formatPeopleTitle and facePhotoToImage as listed in the issue. Done when the new test file passes along with lint and type-check, without changing source files. The author already has a working implementation, so check the two comments for a maintainer reply before starting.
Written by the indexing model from the issue text.
Description
Description: frontend/src/utils/personUtils.ts has no test coverage, while other utilities in src/utils/ have tests in tests/.
Proposal:
Add src/utils/tests/personUtils.test.ts covering:
- getPersonName: name vs. ID fallback, including empty string
- getPhotoCountText: singular/plural for photos and videos
- formatPeopleTitle: empty, one, two and three or more names in both match modes
- facePhotoToImage: default fields and null handling
Notes: No changes to existing source files. I already have a working implementation (21 tests passing, along with lint, type-check and the full suite)
Would it be okay if I open a PR for this?
- Dominant language
- Python
- Stars
- 313
- Forks
- 755
- Avg merge
- 6d 36m
- Merged PRs (30d)
- 8
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the 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 AOSSIE-Org/PictoPy
-
enhancement possible-duplicate
Difficulty 1/5 1-3 hours Newbie friendliness 78/100
AOSSIE-Org/PictoPy#1715 · 2 comments ·
Maintainers usually reply within 6 days
-
BUG: /folders/all-folders takes over a second when folders contain both images and videosPossibly taken @Saurabhyadav0 claimed this 1 day ago. Openbug enhancement possible-duplicate
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
AOSSIE-Org/PictoPy#1708 · 3 comments ·
Maintainers usually reply within 6 days
-
bug possible-duplicate
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
AOSSIE-Org/PictoPy#1702 · 2 comments ·
Maintainers usually reply within 6 days
-
bug possible-duplicate
Difficulty 1/5 Under an hour Newbie friendliness 86/100
AOSSIE-Org/PictoPy#1700 · 3 comments ·
Maintainers usually reply within 6 days
-
BUG: Share page returns HTTP 500 when the unlock cookie contains a non-ASCII characterMay be free again @swastiksharma45 claimed this 7 days ago, and no pull request is open. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
AOSSIE-Org/PictoPy#1696 · 6 comments ·
Maintainers usually reply within 6 days
All issues in AOSSIE-Org/PictoPy
Similar issues
-
add entry to blockOpen
Difficulty 1/5 Under an hour Newbie friendliness 65/100
StevenBlack/hosts#3299 ·
-
`garak --report` overwrites the input report when its name has no ".report"Possibly taken @0z5a claimed this today. Openneeds-triage
Difficulty 2/5 Half a day Newbie friendliness 72/100
Maintainers usually reply within 2 days
-
plot_graph crashes when a mesh node has no m2m edgePossibly taken @tygvi claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
mllam/neural-lam#883 · 1 comment ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 62/100
future-agi/future-agi#4092 · 1 comment ·
Maintainers usually reply within 1 day
-
transcript analytics: a text whose only protected spans the prose pass redacts is withheld wholeOpen
Difficulty 1/5 Under an hour Newbie friendliness 82/100
WingedGuardian/GENesis-AGI#4641 ·
Maintainers usually reply within 1 day