Skip to content

test: cover preferred export format hook#634

Open
saurabhhhcodes wants to merge 2 commits into
utksh1:mainfrom
saurabhhhcodes:test/preferred-export-format-559
Open

test: cover preferred export format hook#634
saurabhhhcodes wants to merge 2 commits into
utksh1:mainfrom
saurabhhhcodes:test/preferred-export-format-559

Conversation

@saurabhhhcodes
Copy link
Copy Markdown
Contributor

Summary

  • add direct unit coverage for usePreferredExportFormat
  • assert empty default state, stored preference restoration, and savePreference persistence
  • keep the coverage isolated from page-level report components

Fixes #559

Validation

  • npm test -- --run testing/unit/hooks/usePreferredExportFormat.test.ts
  • git diff --check

@utksh1 utksh1 added level:beginner 20 pts difficulty label for small beginner-friendly PRs type:testing Testing work category bonus label area:frontend Frontend React/UI work labels Jun 7, 2026
Copy link
Copy Markdown
Owner

@utksh1 utksh1 left a comment

Choose a reason for hiding this comment

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

Please rebase and narrow this PR to the preferred-export-format hook coverage only. It currently carries the same CI baseline repair that is already handled by #629/#635, so merging it as-is would duplicate unrelated backend/workflow/test-baseline changes.

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 level:beginner 20 pts difficulty label for small beginner-friendly PRs type:testing Testing work category bonus label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TEST] Add usePreferredExportFormat hook coverage

2 participants