search

RJT1990 / pyflux

Open source time series library for Python
BSD 3-Clause "New" or "Revised" License
2.1k stars 240 forks source link

Adding more unit tests and maybe property based testing #40

Open springcoil opened 7 years ago

springcoil commented 7 years ago

I know your feature list is long but maybe some property based testing a la Hypothesis would be good.

RJT1990 commented 7 years ago

Makes a lot of sense. I want to complete coverage first with the basic smoke tests; from there can go into more detail with property based tests. Will become more important with new model aggregation methods coming in next few versions.

springcoil commented 7 years ago

Do you use pytest for the tests?

On Tue, Sep 6, 2016 at 4:12 PM, Ross Taylor notifications@github.com wrote:

Makes a lot of sense. I want to complete coverage first with the basic smoke tests; from there can go into more detail with property based tests. Will become more important with new model aggregation methods coming in next few versions.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/RJT1990/pyflux/issues/40#issuecomment-244982783, or mute the thread https://github.com/notifications/unsubscribe-auth/AA8DiJy2RQZSN7XhGnms4Az5WVG4-zRKks5qnYLDgaJpZM4J17ve .

Peadar Coyle Skype: springcoilarch www.twitter.com/springcoil peadarcoyle.wordpress.com

RJT1990 commented 7 years ago

Yup - right now using pytest; is nice and lightweight.