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

Complete unittests #6

Closed
dizak opened this issue Apr 25, 2018 · 2 comments
Closed

Complete unittests #6

dizak opened this issue Apr 25, 2018 · 2 comments
Labels
Milestone

Comments

@dizak
Copy link
Owner

dizak commented Apr 25, 2018

commit

7d9ab00

Proposed change

The aim is to test as much as possible. Though the test coverage will increase as the legacy code will be finally removed, there are still methods with no tests at all.

At the moment, nosetests --with-coverage --cover-package prowler gives:

Name                      Stmts   Miss  Cover
---------------------------------------------
prowler.py                    4      0   100%
prowler/apis.py             126     58    54%
prowler/databases.py        186     72    61%
prowler/errors.py             8      0   100%
prowler/genome.py           198    183     8%
prowler/interactions.py     134    124     7%
prowler/network.py           34     22    35%
prowler/profiles.py          64     11    83%
prowler/stats.py            367    278    24%
prowler/templater.py         26     17    35%
prowler/utils.py             45     33    27%
---------------------------------------------
TOTAL                      1192    798    33%
----------------------------------------------------------------------
Ran 28 tests in 1.830s

OK
@dizak dizak added the update label Apr 25, 2018
@dizak dizak added this to the v0.1.0 milestone Apr 25, 2018
@dizak
Copy link
Owner Author

dizak commented May 2, 2018

Some of the files from test_data directory are not used anymore (or should not be) as the strategy moved to utilizing pickled objects as a reference.

@dizak
Copy link
Owner Author

dizak commented May 3, 2018

#22 resolves this for now as further tests should go along with the other aspects prowler restructures and enhancements.

@dizak dizak closed this as completed May 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant