Skip to content

Conversation

@abhiraj75
Copy link

@abhiraj75 abhiraj75 commented Jan 12, 2026

Refactored the accounts/pages/tests/create.spec.js test suite to use Vue Testing Library (VTL) and align tests with real user interactions instead of component internals.

The new tests focus on core user workflows:

Visiting the Create Account page

Interacting with form fields and checkboxes

Submitting the form with missing data

Handling URL-prefilled email values

Tests that depend on Vuetify-specific components (e.g. dropdowns) are explicitly skipped with clear comments, following Testing Library principles and keeping the suite resilient ahead of the planned Vuetify removal.

Manual verification

Ran the updated test suite locally using pnpm test accounts/pages/tests/create.spec.js (from the frontend folder)

Screenshot 2026-01-13 at 4 15 14 AM

Verified all non-skipped tests pass

Confirmed skipped tests are documented and justified

No UI layout or visual changes were made.

References

Closes #5633

Refactors: accounts/pages/tests/create.spec.js

Reviewer guidance

To review:

Navigate to accounts/pages/tests/create.spec.js

Verify tests:

Use @testing-library/vue

Follow Testing Library query priority (roles, labels)

Describe behavior from a user’s point of view

Run:

pnpm test accounts/pages/tests/create.spec.js (from the frontend folder)

Notes for reviewers

Skipped tests are intentional and documented due to known limitations with Vuetify components in VTL

No production code was changed

This PR is scoped strictly to test refactoring, per the issue’s out-of-scope requirements

@learning-equality-bot
Copy link

👋 Thanks for contributing!

We will assign a reviewer within the next two weeks. In the meantime, please ensure that:

  • You ran pre-commit locally
  • All issue requirements are satisfied
  • The contribution is aligned with our Contributing guidelines. Pay extra attention to Using generative AI. Pull requests that don't follow the guidelines will be closed.

We'll be in touch! 😊

@abhiraj75
Copy link
Author

Some submission-related tests are intentionally skipped due to current Vuetify components (v-select / v-autocomplete) not being reliably testable with userEvent, I’ve added inline notes for re-enabling them after the migration.

Could you please confirm if this approach aligns with expectations, or if you’d prefer a different way of handling the skipped workflows? Thanks!

@abhiraj75 abhiraj75 changed the title test(accounts): migrate create account page tests to Vue Testing Library [Remove Vuetify from Studio] Convert 'Create an account' unit tests to Vue Testing Library Jan 13, 2026
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.

[Remove Vuetify from Studio] Convert 'Create an account' unit tests to Vue Testing Library

1 participant