Skip to content

[7/N] Add unit tests for AboutScreen screen - #98

Open
karkir0003 wants to merge 7 commits into
csvistool:masterfrom
karkir0003:karthik/kar-15-add-unit-tests-for-aboutscreen-screen
Open

karkir0003 wants to merge 7 commits into
csvistool:masterfrom
karkir0003:karthik/kar-15-add-unit-tests-for-aboutscreen-screen

Conversation

@karkir0003

@karkir0003 karkir0003 commented Oct 3, 2026 •

Copy link
Copy Markdown
Contributor

🥞 Stack

  1. ⬅️ [1/N] Update CI to run frontend unit tests #91 (Update CI to run frontend unit tests)
  2. ⬅️ [2/N] Configure React Testing Library and Setup Tests #92 (Configure React Testing Library and Setup Tests)
  3. ⬅️ [3/N] Add unit tests for SideButton component #94 (Add unit tests for SideButton component)
  4. ⬅️ [4/N] Add unit tests for SearchFilter component #95 (Add unit tests for SearchFilter component)
  5. ⬅️ [5/N] Add unit tests for Header and Footer components #96 (Add unit tests for Header and Footer components)
  6. ⬅️ [6/N] Add unit tests for Blob mascot component #97 (Add unit tests for Blob mascot component)
  7. 👉 [7/N] Add unit tests for AboutScreen screen #98 — Add unit tests for AboutScreen screen (this PR)
  8. ➡️ [8/N] Add integration tests for HomeScreen screen #99 (Add integration tests for HomeScreen screen)

Issue: #93

Context

Adds unit test coverage for AboutScreen.js to ensure page layout, content, external attribution links, and theme propagation work reliably.

Changes Made

  • Introduced src/screens/__tests__/AboutScreen.test.js to unit test the AboutScreen component.

Test Plan

  • Ran npm test -- --watchAll=false
  • Ran npm run lint and npx prettier --check src/
  • Verified that the CI checks pass

@karkir0003 karkir0003 changed the title Add unit tests for AboutScreen screen [7/N] Add unit tests for AboutScreen screen Oct 3, 2026
This was referenced Oct 3, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant