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 Tests #41

Closed
chadstewart opened this issue Feb 3, 2023 · 1 comment
Closed

Add Tests #41

chadstewart opened this issue Feb 3, 2023 · 1 comment
Labels
duplicate This issue or pull request already exists enhancement New feature or request

Comments

@chadstewart
Copy link
Collaborator

Feature Request

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I have an issue when [...]

Want to add a test suite to the application.

Describe the solution you'd like
A clear and concise description of what you want to happen. Add any considered drawbacks.

As of right now, this issue will remain vague as I don't have a great idea on how I want to structure the test suite. The application is mostly split between separating markup (display logic) from business logic. Because of that, I'm not sure if it'd add much value to test the display logic through unit tests. The business logic seems like a no-brainer for now but I'd like some time to think about and get opinions on what's the best way to write tests that'll add value to the project.

I also want to leverage Cypress for testing as well.

Note: This issue will need to be tackled after issue #29.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Currently n/a

Teachability, Documentation, Adoption, Migration Strategy
If you can, explain how users will be able to use this and possibly write out a version in the docs.
Maybe a screenshot or design?

Currently n/a

@chadstewart
Copy link
Collaborator Author

As this issue is essentially a duplicate of issue #30, I'm going to close the issue and add a comment to that issue that remarks on some of the thoughts I brought up here.

@chadstewart chadstewart added duplicate This issue or pull request already exists enhancement New feature or request labels Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant