Skip to content

TST: test_assert_index check index attributes #13242

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
5 tasks done
sinhrks opened this issue May 20, 2016 · 7 comments
Closed
5 tasks done

TST: test_assert_index check index attributes #13242

sinhrks opened this issue May 20, 2016 · 7 comments
Labels
Testing pandas testing functions or related to the test suite

Comments

@sinhrks
Copy link
Member

sinhrks commented May 20, 2016

@sinhrks sinhrks added Testing pandas testing functions or related to the test suite Master Tracker High level tracker for similar issues labels May 20, 2016
@sinhrks sinhrks added this to the 0.18.2 milestone May 20, 2016
@sinhrks
Copy link
Member Author

sinhrks commented May 20, 2016

I already have a branch which checks PeriodIndex.freq currently fails in empty resampling (being fixed by #13079).

https://github.com/sinhrks/pandas/tree/test_periodindex

@max-sixty
Copy link
Contributor

How about comparing the attr dictionaries (._get_attribute_dict()), rather than having the separate lookup above?
That could also include name

@sinhrks
Copy link
Member Author

sinhrks commented May 20, 2016

There are mandatory and optional check. For example, our tests do something like assert_frame_equal(df.a, df.b, check_names=False). Also, we have pickle tests to skip some attribute check like Categorical.ordered

jreback pushed a commit that referenced this issue May 22, 2016
xref to #13242
closes #13251

Author: sinhrks <[email protected]>

Closes #13250 from sinhrks/period_test and squashes the following commits:

33a04f3 [sinhrks] TST/ERR: Add Period ops tests / fix error message
@jreback
Copy link
Contributor

jreback commented May 22, 2016

@sinhrks I checked the freq boxes which I think closed by #13250

@sinhrks
Copy link
Member Author

sinhrks commented May 22, 2016

Thanks. Because #13250 only covers PeriodIndex, modified the boxes.

@jreback
Copy link
Contributor

jreback commented May 22, 2016

ok great!

@jorisvandenbossche jorisvandenbossche modified the milestones: 0.20.0, 0.19.0 Aug 21, 2016
@jreback jreback modified the milestones: 0.20.0, Next Major Release Mar 23, 2017
@jreback jreback modified the milestones: Next Major Release, High Level Issue Tracking Sep 24, 2017
@TomAugspurger TomAugspurger removed the Master Tracker High level tracker for similar issues label Jul 6, 2018
@TomAugspurger TomAugspurger removed this from the High Level Issue Tracking milestone Jul 6, 2018
@jschendel jschendel added this to the Contributions Welcome milestone Nov 21, 2018
@mroeschke
Copy link
Member

Looks like all the above are implemented. Going to close this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing pandas testing functions or related to the test suite
Projects
None yet
Development

No branches or pull requests

7 participants