search

slac207 / cs207project

MIT License
0 stars 4 forks source link

Write tests for ArrayTimeSeries.py #21

Closed sanoke closed 7 years ago

sanoke commented 7 years ago

From Project Spec 2,

Make sure that any doctests and tests you wrote before to test all kinds of sequences as input still pass. (If you didn't write any, how do you know your timeseries class from last time is any good? We'll be testing your code by running our own tests as well!)

@jsm09 How is this going? Do you need any of us to help?

jsm09 commented 7 years ago

Sure if you have time feel free to help out with this. Then I can get started on the re-parenting part sooner.

jsm09 commented 7 years ago

@sanoke I pushed some first pass tests (with comments). Please take a look and add to them (or get someone else to if you like)! I'd appreciate the help.

jsm09 commented 7 years ago

@sanoke Done with this item. Please go ahead and check them over, or add any that I may have missed. Thanks!

sanoke commented 7 years ago

I'll have a look tomorrow evening @jsm09, if that's not too late.

On Sun, Oct 23, 2016 at 8:47 PM, Laura Ware notifications@github.com wrote:

@sanoke https://github.com/sanoke I pushed some first pass tests (with comments). Please take a look and add to them (or get someone else to if you like)! I'd appreciate the help.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/slac207/cs207project/issues/21#issuecomment-255626727, or mute the thread https://github.com/notifications/unsubscribe-auth/AAYSVtofuUdZsUN98-MzvD-INCKHvEvnks5q3AA8gaJpZM4KeCes .

jsm09 commented 7 years ago

Closing this issue @sanoke