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

Adds tests. #26

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Adds tests. #26

wants to merge 3 commits into from

Conversation

dblock
Copy link

@dblock dblock commented Dec 4, 2024

I used this tutorial to make https://github.com/dblock/sparta-social, a beginning of a decentralized exercise activity tracker. Before I add tests to my implementation I thought maybe the example app would like test coverage to encourage good practices, and let users work with tests as they modify the code?

I've only implemented a basic test that renders the login page here, but I would be happy to do a 100% test coverage before merging this if it's of interest.

Test

@socialatm
Copy link

@dblock I can't speak for anyone else but I would love to have tests.... Thanks

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

Successfully merging this pull request may close these issues.

2 participants