Add unit tests for personUtils.ts
Los mantenedores suelen responder en 6 días
@mystic-fae ya está trabajando en esto.
Desde el 14/11/2026.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 62/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- react, typescript
- Área
- frontend, testing-qa
Línea de trabajo
Empieza con frontend/src/utils/personUtils.ts y revisa una prueba existente en la carpeta tests de utils vecina para copiar sus convenciones. Escribe pruebas para getPersonName, getPhotoCountText, formatPeopleTitle y facePhotoToImage, tal como aparecen en el issue. Está terminado cuando el nuevo archivo de pruebas pasa junto con el lint y la comprobación de tipos, sin cambiar los archivos fuente. El autor ya tiene una implementación que funciona, así que revisa los dos comentarios por si hay una respuesta del mantenedor antes de empezar.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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?
- Lenguaje dominante
- Python
- Estrellas
- 313
- Forks
- 755
- Merge medio
- 6 d 36 min
- PR fusionados (30 d)
- 8
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de AOSSIE-Org/PictoPy
-
enhancement possible-duplicate
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
AOSSIE-Org/PictoPy#1715 · 2 comentarios ·
Los mantenedores suelen responder en 6 días
-
BUG: /folders/all-folders takes over a second when folders contain both images and videosPosiblemente ocupada @Saurabhyadav0 la tomó hace 1 día. Abiertobug enhancement possible-duplicate
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
AOSSIE-Org/PictoPy#1708 · 3 comentarios ·
Los mantenedores suelen responder en 6 días
-
bug possible-duplicate
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
AOSSIE-Org/PictoPy#1702 · 2 comentarios ·
Los mantenedores suelen responder en 6 días
-
bug possible-duplicate
Dificultad 1/5 Menos de una hora Aptitud para principiantes 86/100
AOSSIE-Org/PictoPy#1700 · 3 comentarios ·
Los mantenedores suelen responder en 6 días
-
BUG: Share page returns HTTP 500 when the unlock cookie contains a non-ASCII characterQuizá libre de nuevo @swastiksharma45 la tomó hace 7 días y no hay ningún pull request abierto. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
AOSSIE-Org/PictoPy#1696 · 6 comentarios ·
Los mantenedores suelen responder en 6 días
Todos los issues de AOSSIE-Org/PictoPy
Issues similares
-
add entry to blockAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
StevenBlack/hosts#3299 ·
-
`garak --report` overwrites the input report when its name has no ".report"Posiblemente ocupada @0z5a la tomó hoy. Abiertoneeds-triage
Dificultad 2/5 Medio día Aptitud para principiantes 72/100
Los mantenedores suelen responder en 2 días
-
plot_graph crashes when a mesh node has no m2m edgePosiblemente ocupada @tygvi la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
mllam/neural-lam#883 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
future-agi/future-agi#4092 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
transcript analytics: a text whose only protected spans the prose pass redacts is withheld wholeAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
WingedGuardian/GENesis-AGI#4641 ·
Los mantenedores suelen responder en 1 día