CodeForPhilly / CodeForPhilly/codeforphilly-ng

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

Open
#30 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
1
Forks
1
Avg merge
5d 3h
Merged PRs (30d)
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.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.