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

Increase the test coverage #20

Open
AlexITC opened this issue Oct 1, 2018 · 3 comments
Open

Increase the test coverage #20

AlexITC opened this issue Oct 1, 2018 · 3 comments
Labels
alerts-server Changes required on the alerts-server project good first issue Hacktoberfest

Comments

@AlexITC
Copy link
Owner

AlexITC commented Oct 1, 2018 •
edited
Loading

Increase the backend test coverage.

Expected behavior

The test coverage is higher.

Actual behavior

The test coverage is not as good as it can be.

Steps to reproduce the behavior
@AlexITC
Copy link
Owner Author

AlexITC commented Oct 1, 2018

A PR per file is ok too.

@AlexITC AlexITC added the alerts-server Changes required on the alerts-server project label Oct 6, 2018
@mghildiy
Copy link

user.scala has 93 percent coverage as per codacy report.
But I am not able to locate test file for same.

@AlexITC
Copy link
Owner Author

AlexITC commented Oct 21, 2018

Codacy shows the file path: alerts-server/app/com/alexitc/coinalerts/models/user.scala

In fact, the uncovered piece (UserPassword.writes) is never used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
alerts-server Changes required on the alerts-server project good first issue Hacktoberfest
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@AlexITC @mghildiy