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

Add more tests #10

Open
devshawn opened this issue Oct 23, 2018 · 2 comments
Open

Add more tests #10

devshawn opened this issue Oct 23, 2018 · 2 comments
Labels
good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed

Comments

@devshawn
Copy link
Owner

Currently, the client and the server both have very little tests. Any more added tests would be appreciated!

We utilize Jest for writing tests on both the server and the client. The README.md explains how to run the application, and "npm run test" or "lerna run test" will run the tests.
@devshawn devshawn added help wanted Extra attention is needed good first issue Good for newcomers hacktoberfest labels Oct 23, 2018
@danmakenoise
Copy link
Contributor

danmakenoise commented Oct 23, 2018 •
edited
Loading

I can take this! I'll write some tests for some of the server code.

@devshawn
Copy link
Owner Author

Awesome, @danmakenoise! 👍

@danmakenoise danmakenoise mentioned this issue Oct 24, 2018
Merged
mstruensee pushed a commit to mstruensee/password.live that referenced this issue Oct 26, 2018
test cleanup, es6'ified files for devshawn#10
7da076f
@mstruensee mstruensee mentioned this issue Oct 26, 2018
Merged
devshawn added a commit that referenced this issue Oct 26, 2018
@devshawn
Merge pull request #39 from mstruensee/mstruensee/#10
25839b2 
test cleanup, es6'ified files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@devshawn @danmakenoise