test: migrate FilesList header specs to testing Pinia

[ColumbusLabs]

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.

[vitormattos]

Thanks for the contribution! The migration in FilesListTableHeaderButton.spec.ts is mostly aligned with the issue, but I think the PR needs another simplification pass.

The current diff is quite large for migrating only two test suites. Please keep the changes focused on:

• replacing real Pinia with createTestingPinia();
• providing pre-mount state through initialState;
• selectively preserving only the action behavior required by existing tests;
• using Testing Pinia action spies where the test only verifies a call.

Please avoid unrelated restructuring, additional local abstractions, or cleanup that is not required by the migration. The existing test intent and behavioral coverage should remain unchanged.

The main blocking point is the suite-wide stubActions: false, because it executes substantially more real store behavior than these component tests require.

Please compare the final diff with main and revert any new local types, JSDoc, helper structure, or test rewrites that are not strictly required for the createTestingPinia() migration. The resulting diff should be materially smaller than the current one and should satisfy every acceptance criterion from issue #7940.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
see 173 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[vitormattos]

/backport to stable34

[vitormattos]

/backport to stable33

[vitormattos]

/backport to stable32

[vitormattos]

Thanks! The previous review points have been addressed.

[backportbot-libresign]

The backport to stable32 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable32
git pull origin stable32

# Create the new backport branch
git checkout -b backport/7945/stable32

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick 54e03932 bca74aae

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/7945/stable32

Error: Failed to push branch backport/7945/stable32: remote: Invalid username or token. Password authentication is not supported for Git operations.
fatal: Authentication failed for 'https://github.com/LibreSign/libresign.git/'

---

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

[vitormattos]

/backport to stable32