ImperialCollegeLondon / pyrealm

Development of the pyrealm package, providing an integrated toolbox for modelling plant productivity, growth and demography using Python.
https://pyrealm.readthedocs.io/
MIT License
23 stars 9 forks source link

Split the tests directory into unit and regression #151

Closed surbhigoel77 closed 11 months ago

surbhigoel77 commented 11 months ago

Description

Restructured the tests directory and split it into unit and regression modules. The segregation would facilitate a structured execution of the tests and would also provide easier navigation. Also added a README.md to empty folders to make them trackable by git.

Fixes #143

Type of change

Key checklist

Further checks

surbhigoel77 commented 11 months ago

This looks generally good for me. I am just wondering whether we want to use these as packages, in which case we need a init.py ( see https://docs.python.org/3/tutorial/modules.html , section 6.4). If we don't need it at the moment or in the foreseeable future it's fine as it is.

I will add init.py to the unit/regression subfolders to allow us the flexibility of using them as packages in future.

codecov-commenter commented 11 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ee9eb7f) 94.09% compared to head (1091eac) 94.09%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #151 +/- ## ======================================== Coverage 94.09% 94.09% ======================================== Files 21 21 Lines 1134 1134 ======================================== Hits 1067 1067 Misses 67 67 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

davidorme commented 11 months ago

Looks good to me too.