search

ArtesiaWater / hydropandas

Module for loading observation data into custom DataFrames
https://hydropandas.readthedocs.io
MIT License
52 stars 11 forks source link

Test should return `None` #102

Closed OnnoEbbens closed 6 months ago

OnnoEbbens commented 1 year ago

In the newest version of pytest a warning is issued when a test does not return None. I changed most of the tests so they return None. I could not change all of them because some of our tests call other tests and expect a return value. Tests calling other tests is probably bad practice and should be changed.

martinvonk commented 1 year ago

We can implement pytest.fixtures here as well. Same as Pastas and SPEI.