Skip to content

test: add AppShell mobile navigation coverage#736

Merged
utksh1 merged 2 commits into
utksh1:mainfrom
kunal-9090:codex/issue-552-appshell-tests
Jun 10, 2026
Merged

test: add AppShell mobile navigation coverage#736
utksh1 merged 2 commits into
utksh1:mainfrom
kunal-9090:codex/issue-552-appshell-tests

Conversation

@kunal-9090

Copy link
Copy Markdown
Contributor

Linked issue

Closes #552

GSSoC

  • Source: official GSSoC issues page
  • Labels: level:intermediate, type:testing, priority:medium

Summary

  • Adds focused AppShell unit coverage for mobile drawer open/close behavior.
  • Verifies route navigation closes the mobile drawer via the existing location effect.
  • Covers the persistent mobile bottom navigation shortcuts without relying on snapshots.

Validation

  • Attempted: npm test -- AppShell.test.tsx
  • Blocked: node_modules is not installed in this checkout.

@utksh1 utksh1 added level:intermediate 35 pts difficulty label for moderate contributor PRs type:testing Testing work category bonus label area:frontend Frontend React/UI work gssoc:approved Admin validation: approved for GSSoC scoring labels Jun 10, 2026

@utksh1 utksh1 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. This adds focused AppShell mobile navigation coverage using real MemoryRouter behavior and user-event interactions, with green CI.

@utksh1 utksh1 merged commit 069046e into utksh1:main Jun 10, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:frontend Frontend React/UI work gssoc:approved Admin validation: approved for GSSoC scoring level:intermediate 35 pts difficulty label for moderate contributor PRs type:testing Testing work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TEST] Add AppShell coverage for mobile nav toggle and route-change close behavior

2 participants