Issue #82: Unit Tests for statistics.py #84

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

Merged

harry093 merged 9 commits into GeoscienceAustralia:master from suv27:feat(test-statistics)

Oct 25, 2019

Contributor

suv27 commented Oct 7, 2019 •

edited

Loading

Writing tests for the `statistics.py` and the methods:

How to Test

Create a virtual env python3 -m venv env
Enter the virtual environment source env/bin/activate
Upgrade pip pip install --upgrade pip
Install coverage -> pip install coverage
Install numpy -> pip install numpy
Run the test -> coverage run geodepy/tests/test_statistics.py
Run the report -> coverage report ../geodepy/geodepy/statistics.py
Run coverage html ../geodepy/geodepy/statistics.py
Open in Browser open htmlcov/index.html


          Issue GeoscienceAustralia#82: Preparing to test the functions, readin…

92af48d

…g docs

suv27 changed the title ~~Issue #82: Unit Tests for statistics.py~~ Issue #82: Unit Tests for statistics.py

suv27 added 5 commits

October 8, 2019 21:18


          issue GeoscienceAustralia#82: Test for rotation_matrix()

5325f40


          issue GeoscienceAustralia#82: Test for vcv_cart2local() & vcv_local2c…

5c473b2

…art()


          issue GeoscienceAustralia#82: removing bad code commit

c00a895


          issue GeoscienceAustralia#82: health check

f50682e


          Issue GeoscienceAustralia#82: finished tests for all the methods

e1c1f59

suv27 mentioned this pull request

Provide test coverage for the statistics module #82

Closed

6 tasks

suv27 added 3 commits

October 18, 2019 21:58

b384ac9


          issue GeoscienceAustralia#83: DONE

1836c76


          issue GeoscienceAustralia#83: fixing .gitignore

64d74ed

harry093 reviewed

View reviewed changes

geodepy/tests/test_statistics.py Show resolved Hide resolved

harry093 reviewed

View reviewed changes

geodepy/tests/test_statistics.py Show resolved Hide resolved

harry093 reviewed

View reviewed changes

geodepy/tests/test_statistics.py Show resolved Hide resolved

harry093 reviewed

View reviewed changes

geodepy/tests/test_statistics.py Show resolved Hide resolved

harry093 reviewed

View reviewed changes

geodepy/tests/test_statistics.py Show resolved Hide resolved

harry093 reviewed

View reviewed changes

geodepy/tests/test_statistics.py Show resolved Hide resolved

harry093 reviewed

View reviewed changes

geodepy/tests/test_statistics.py Show resolved Hide resolved

harry093 reviewed

View reviewed changes

geodepy/tests/test_statistics.py Show resolved Hide resolved

harry093 reviewed

View reviewed changes

Collaborator

harry093 left a comment

@Physsix27 thanks heaps for coding up these tests. Please have a look at the comments I have made in each of the files

suv27 commented

View reviewed changes

Contributor Author

suv27 left a comment

Can you review this PR

geodepy/tests/test_statistics.py Show resolved Hide resolved

geodepy/tests/test_statistics.py Show resolved Hide resolved

geodepy/tests/test_statistics.py Show resolved Hide resolved

geodepy/tests/test_statistics.py Show resolved Hide resolved

harry093 approved these changes

View reviewed changes

Collaborator

harry093 left a comment •

edited

Loading

Thanks @Physsix27 I appreciate your work on this. Closes #82

harry093 merged commit f9e38bb into GeoscienceAustralia:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet