Skip to content

[Remove Vuetify from Studio] Convert 'Reset password' unit tests to Vue Testing Library聽#5637

Closed

Description

@MisRob

馃檪 Looking for an issue? Welcome! This issue is open for contribution. If this is the first time you鈥檙e requesting an issue, please:

  • Read Contributing guidelines carefully. Pay extra attention to Using generative AI. Pull requests and comments that don鈥檛 follow the guidelines won鈥檛 be answered.
  • Confirm that you鈥檝e read the guidelines in your comment.

Overview

Complexity: Low

Context

The test suite accounts/pages/__tests__/resetPassword.spec.js currently relies heavily on internal implementation details and uses Vue Test Utils in a way that couples tests tightly to component internals. This creates several issues:

  • Brittle and unreliable tests (break when implementation changes even if behavior remains correct, or incorrectly pass while actual user interactions fail)
  • Maintenance burden (refactoring component's internal code requires updating tests)

Related page:

The Change

Refactor accounts/pages/__tests__/resetPassword.spec.js test suite to use Vue Testing Library (VTL) and write tests in a way that reflects how a user interacts with the application.

Do not refactor any other areas of the codebase.

Hot to Get There

  • Navigate to /accounts/#/reset-password URL
  • Submit the form with missing or invalid data
  • Submit the form with correct data
  • To display the error banner, temporarily modify code

Value Add

Prepares the test suite for easier updates during the upcoming refactoring to remove Vuetify from this page.

Out of Scope

Do not refactor any other areas of the codebase.

Acceptance Criteria

References

Sub-issue of #5060

Activity

  1. LightCreator1007 commented on Jan 13, 2026

    @LightCreator1007
    Contributor

    Hello @MisRob could I work on thisss.

  2. sharma-anushka commented on Jan 13, 2026

    @sharma-anushka
    Contributor

    Hi @MisRob, I would also like to work on this if its available. Ive read the guidelines.

  3. Ananya-Kaushal137 commented on Jan 13, 2026

    @Ananya-Kaushal137

    Hello @MisRob, I鈥檇 like to work on this issue. I鈥檓 confident I can handle it efficiently . If it's available , please assign me.

  4. vtushar06 commented on Jan 13, 2026

    @vtushar06
    Contributor

    Hii @MisRob, I would like to work on this, if possible.

  5. Rajeev529 commented on Jan 13, 2026

    @Rajeev529
    Contributor

    hii @MisRob Could you please assign this issue to me? Ive understood the issue in the reset password tests鈥攃urrently they depend on component internals like setData, wrapper.vm, and submitting via refs instead of real user interactions. i can refactor them using vue testing Library to make them user-centric and stable.

  6. abhiraj75 commented on Jan 13, 2026

    @abhiraj75
    Contributor

    Greetings @MisRob , I would love to work to on this as I already solved a similar issue which was VTU to VTL conversion(#5633)

  7. MisRob commented on Jan 13, 2026

    @MisRob
    MemberAuthor

    Thanks all! I can only assign one of you :) If the assignee is open to collaboration, perhaps some of you can connect and resolve together.

  8. vtushar06 commented on Jan 15, 2026

    @vtushar06
    Contributor

    Sure @MisRob,
    And @LightCreator1007 if you have any doubt while resolving this issue feel free to connect before you request review from maintainers, I just worked some similar issues in past I may able to help you out.

  9. learning-equality-bot commented on Jan 15, 2026

    @learning-equality-bot
    Contributor

    Hi! 馃憢

    Thanks so much for your interest! This issue is already assigned. Visit Contributing guidelines to learn about the contributing process and how to find suitable issues. If there are no unassigned 'help wanted' issues available, please wait until new ones are added.

    We really appreciate your willingness to help. 馃槉

    Are you preparing for Google Summer of Code? See our GSoC guidelines.

  10. added a commit that references this issue on Jan 15, 2026
    515389f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions