-
-
Notifications
You must be signed in to change notification settings - Fork 301
Convert trash modal unit tests to Vue Testing Library #5790
Copy link
Copy link
Open
Labels
DEV: frontendTAG: tech update / debtTAG: unit testscommunity-contribution-in-progressgood first issueSelf-contained, straightforward, low-complexitySelf-contained, straightforward, low-complexityhelp wantedOpen source contributors welcomeOpen source contributors welcome
Metadata
Metadata
Assignees
Labels
DEV: frontendTAG: tech update / debtTAG: unit testscommunity-contribution-in-progressgood first issueSelf-contained, straightforward, low-complexitySelf-contained, straightforward, low-complexityhelp wantedOpen source contributors welcomeOpen source contributors welcome
Type
Fields
Give feedbackNo fields configured for issues without a type.
🙂 Looking for an issue? Welcome! This issue is open for contribution. If this is the first time you’re requesting an issue, please:
Overview
Complexity: Low
Context
See #5789
The Change
Refactor
channelEdit/views/trash/__tests__/trashModal.spec.jstest suite to use Vue Testing Library (VTL) and write tests in a way that reflects how a user interacts with the application. In the pull request description, include a screenshot or screen recording of the feature covered by these tests.Do not refactor any other areas of the codebase.
How to Get There
user@a.comwith passwordaOut of Scope
Do not refactor any other areas of the codebase.
Acceptance Criteria
render,screenqueries, anduserEvent. Follow Testing Library principles and Testing Library query priority recommendation@vue/test-utilsimports should remain in migrated filespnpm test <filename>References
Sub-issue of #5789
AI usage
Issue drafted & iteratively adjusted in collaboration with Claude. Final version carefully reviewed by @MisRob.