Skip to content

TST: Period tests reorg (gh14854) #15352

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.

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

Closed
wants to merge 1 commit into from
Closed

TST: Period tests reorg (gh14854) #15352

wants to merge 1 commit into from

Conversation

TrigonaMinima
Copy link

@jreback
Copy link
Contributor

jreback commented Feb 8, 2017

whoosh that test_period was huge. ok. ping when green.

@TrigonaMinima
Copy link
Author

hahah yeah. As a result, scalar/test_period.py and indexes/period/test_ops.py have gotten big as well.

@jreback
Copy link
Contributor

jreback commented Feb 8, 2017

yep
certainly can come back later and split things a bit

but let's wait on that for now

@jreback jreback added Period Period data type Testing pandas testing functions or related to the test suite labels Feb 8, 2017
@jreback jreback added this to the 0.20.0 milestone Feb 8, 2017
@jreback
Copy link
Contributor

jreback commented Feb 8, 2017

@TrigonaMinima going to merge shortly.

@codecov-io
Copy link

Codecov Report

Merging #15352 into master will increase coverage by <.01%.

@@            Coverage Diff             @@
##           master   #15352      +/-   ##
==========================================
+ Coverage   86.32%   86.32%   +<.01%     
==========================================
  Files         141      141              
  Lines       51163    51163              
==========================================
+ Hits        44166    44167       +1     
+ Misses       6997     6996       -1
Impacted Files Coverage Δ
pandas/tseries/index.py 95.42% <ø> (+0.09%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 704cdbf...b28eec7. Read the comment docs.

@jreback
Copy link
Contributor

jreback commented Feb 9, 2017

thanks again @TrigonaMinima !

if you are up for more :>

let's move the remainder of tseries/tests to pandas/tests/tseries

also let's remove tests/timeseries_legacy.py (and some of data can be removed as well).

BUT should move any useful tests to a proper location. IOW the pickle things you can simply remove (they are really old). but some of the freq tests might be useful.

@TrigonaMinima
Copy link
Author

TrigonaMinima commented Feb 9, 2017

Sure. I'll try to do it tonight.

I should also thank you. You are helping me alot throughout the way.

Is there something remaining to be done in this PR?

@jreback
Copy link
Contributor

jreback commented Feb 9, 2017

closed via 9ba1008

@TrigonaMinima no this is all good. thanks!

@jreback jreback closed this Feb 9, 2017
@TrigonaMinima TrigonaMinima deleted the gh14854-period branch February 9, 2017 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Period Period data type Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants