-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Increase test coverage for Components which are missing few methods #90
Comments
Could you guide me related to this contribution? As I am new to open source. |
Hi @ayushsinha25 , So we have introduced a unit test framework as part of #56 and there are many unit tests written in the project link. These tests cover many classes but some of the methods are not getting covered as described at https://app.codecov.io/gh/SasanLabs/VulnerableApp-facade/tree/main/facade-app/src/Components. Now what we need to do is find out what all conditions as not getting covered and cover those via writing the unit test which is adding methods to the already written tests. example: so we need to cover them. thanks, |
Hey, can I work on this PR? |
@trishitchar @ayushsinha25 There are multiple classes, you guys can pick one of them and work on it. I have assigned to you. please remove assignment, if not interested. |
Can you please elaborate. I'm also a newbie in open source. |
please have a look at #90 (comment) and let me know if you have any questions. |
As part of #56 we have introduced a unit testing framework and now we are looking for adding more unit test for the project.
we are missing few unit tests for https://github.com/SasanLabs/VulnerableApp-facade/tree/main/facade-app/src/Components package.
Have a look at codecov report for finding what all is missing and cover that as part of this issue.
Codecov report: https://app.codecov.io/gh/SasanLabs/VulnerableApp-facade/tree/main/facade-app/src/Components
The text was updated successfully, but these errors were encountered: