Skip to content

test: migrate FilesList header specs to testing Pinia#7945

Open
ColumbusLabs wants to merge 1 commit into
LibreSign:mainfrom
ColumbusLabs:test/7940-testing-pinia
Open

test: migrate FilesList header specs to testing Pinia#7945
ColumbusLabs wants to merge 1 commit into
LibreSign:mainfrom
ColumbusLabs:test/7940-testing-pinia

Conversation

@ColumbusLabs

Copy link
Copy Markdown

Resolves: #7940

📝 Summary

  • migrate the FilesList table header component tests from real Pinia instances to createTestingPinia()
  • provide isolated store state through initialState for sorting, file, and selection scenarios
  • keep sorting actions stubbed when tests only verify calls
  • preserve real file and selection action behavior where rendering or state transitions are under test

🧪 How to test

npm run ts:check
npx vitest run \
  src/tests/views/FilesList/FilesListTableHeaderButton.spec.ts \
  src/tests/views/FilesList/FilesListTableHeader.spec.ts
npm run lint
npm test

Local results with Node 24.18.0:

  • 31/31 affected tests passed
  • 3,040/3,040 frontend tests passed
  • TypeScript check passed
  • ESLint passed

✅ Checklist

  • I have read and followed the contribution guide.
  • The change is limited to the two test suites named in the issue.
  • Existing test intent and application behavior are preserved.
  • The commit includes the required DCO sign-off.

🤖 AI (if applicable)

  • The content of this PR was partially or fully generated using AI

OpenAI Codex assisted with repository inspection, the test migration, and verification. I reviewed the changes and take responsibility for the submission.

Signed-off-by: ColumbusLabs <287001685+ColumbusLabs@users.noreply.github.com>
@github-project-automation github-project-automation Bot moved this to 0. Needs triage in Roadmap Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 0. Needs triage

Development

Successfully merging this pull request may close these issues.

[FEATURE]: Migrate FilesList table header tests to createTestingPinia()

1 participant