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.

Sign up for GitHub

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

Jump to bottom

Testing #84

Open
rgraue opened this issue Jun 20, 2024 · 0 comments
Open

Testing #84

rgraue opened this issue Jun 20, 2024 · 0 comments
Labels
energy-calculator FE

Comments

@rgraue
Copy link
Collaborator

rgraue commented Jun 20, 2024

Description

We need to add testing to this repo. Both unit and react component wise.

There is too much untested code to do this all in one shot. go through in chunks and create tests and or refactor existing test.

link issue in PR

Technical Notes

  • jest or vitest for js code
  • cypress for react component

Risks

[Add Risks]

Related Issues Or Discussions

[...Issues]
sfishel18 added a commit to sfishel18/home-energy-calculator that referenced this issue Jun 24, 2024
@sfishel18
initial integration of Vitest for unit tests
4cd2c1c 
- no configuration needed! Vitest automatically checks for a Vite config and uses that
- add one example unit test to show what they look like
- update README

issue: climate-cooperative#84
@sfishel18 sfishel18 mentioned this issue Jun 24, 2024
Merged
sfishel18 added a commit to sfishel18/home-energy-calculator that referenced this issue Jul 4, 2024
@sfishel18
updates all files to conform to prettier rules
a724fcb 
also updates lint-staged config so that formatting is automatically
fixed as part of the pre-commit hook

issue climate-cooperative#84
@sfishel18 sfishel18 mentioned this issue Jul 4, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
energy-calculator FE
Projects
Climate Cooperative Board
Status: Development
Milestone
No milestone
Development

No branches or pull requests
1 participant
@rgraue