CodeForPhilly / CodeForPhilly/codeforphilly-ng

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

Ouverte
#30 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
TypeScript
Étoiles
1
Forks
1
Merge moyen
5 j 3 h
PR mergées (30 j)
9

Description

**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.

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.