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

chore: implement tests for the frontend #135

Open
davidlag0 opened this issue Jan 3, 2023 · 1 comment
Open

chore: implement tests for the frontend #135

davidlag0 opened this issue Jan 3, 2023 · 1 comment
Labels
enhancement Enhancement of the code, not introducing new features. priority-medium This issue may be useful, and needs some attention.

Comments

@davidlag0
Copy link

@dec0dOS
This is an issue to track the enablement of unit testing for the frontend portion of the codebase. I started working to enable them with Jest and React Testing Library. I ran into issues that seemed related to ESM (EcmaScript Modules) so at that moment I thought it would be easier to upgrade to React 18 to get access to a more recent version of Jest through react-scripts. However, it would seem safer to have unit tests in place to provide more confidence of not breaking anything with the upgrade to React 18 so I would like to revisit how Jest can be configured to work with the current codebase before the upgrade to React 18.

@davidlag0 davidlag0 added the enhancement Enhancement of the code, not introducing new features. label Jan 3, 2023
@davidlag0
Copy link
Author

I got tests working this morning with the HomeLoggedOutcomponent. I'll work from smaller components to larger ones that encapsulate them. I need to build more tests for HomeLoggedOut to fully test the component. My initial goal was to get the testing process and commands to work first without errors or warnings. I'll submit pull requests as I move along.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of the code, not introducing new features. priority-medium This issue may be useful, and needs some attention.
Projects
None yet
Development

No branches or pull requests

2 participants