CodeForPhilly / CodeForPhilly/codeforphilly-ng

Add screen smoke tests for the public-screens detail/feed/static screens

Abierto
#30 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
TypeScript
Estrellas
1
Forks
1
Merge medio
5 d 3 h
PR fusionados (30 d)
9

Descripción

**Context:** PR #28 (`feat/public-screens`) shipped screen smoke tests for Home, ProjectsIndex, ProjectDetail, HelpWantedIndex — the screens with the most spec-bound Display Rules. The remaining screens were exercised only via the build + browser walkthrough, not unit tests.

**Add smoke tests for:**

- [ ] `PersonDetail` — renders fullName + "Member since {month yyyy}" + projects list sorted by maintainer / joinedAt
- [ ] `TagDetail` — renders the three sections (Projects / Help-wanted / Members), hides Members on `event` namespace
- [ ] `ProjectUpdatesFeed` — renders cards reverse-chron, filtered empty state
- [ ] `ProjectBuzzFeed` — same shape, external-link headlines
- [ ] `Volunteer` — hero CTA toggles on auth state, live project count uses /api/projects?perPage=1
- [ ] `Sponsor` — static content + copy-to-clipboard button

Each test: render with `renderScreen()` from `apps/web/tests/test-utils.tsx`, mock `fetch` with `mockOk` / `mockPaginated`, assert two or three load-bearing Display Rules from the spec.

Scoped small enough for a single follow-up PR.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.