Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add unit tests for Formats tab #994

Closed
alanb2718 opened this issue Aug 2, 2024 · 2 comments
Closed

Add unit tests for Formats tab #994

alanb2718 opened this issue Aug 2, 2024 · 2 comments
Assignees
Labels

Comments

@alanb2718
Copy link
Collaborator

No description provided.

@alanb2718 alanb2718 self-assigned this Sep 30, 2024
@alanb2718
Copy link
Collaborator Author

Mostly completed via #1071. Unfortunately though it's difficult to test the Accordion part of the popup Format Form because onMount doesn't work correctly for components when doing unit tests. (These are places to enter the information for a translation.) I worked around this for the Account unit tests by factoring out a component that consisted of the accordion item, which could then be tested separately. This just tested the contents of the accordion item, which was all that was needed for the Account tab. For the Format Form, doing this would only test a small and not that interesting part of the functionality, since it would just test the initial contents of the translation, but wouldn't let you test editing and saving (as far as I can see). So the accordion part of the popup is currently not tested.

@alanb2718
Copy link
Collaborator Author

Now completed with #1152.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant