Skip to content
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

TST: Add unit tests for older timezone issues #21491

Merged
merged 6 commits into from Jun 18, 2018

Conversation

mroeschke
Copy link
Member

@mroeschke mroeschke commented Jun 15, 2018

In the spirit of your #21407 (comment) @jreback, cleaned up some old timezone issues (but not necessarily covered by #21407) that appear to have been solved

@mroeschke mroeschke added Testing pandas testing functions or related to the test suite Timezones Timezone data dtype labels Jun 15, 2018
@jreback
Copy link
Contributor

jreback commented Jun 15, 2018

@mroeschke looks awesome!

so can you either add these issue numbers to the whatsnew for #21407, or if its a 'different' issue (even though it was solved via the same), add an appropriate whatnew e.g. a resampling fixed deserves its own whatsnew (in bug fixes for all of these i think)

@jreback jreback added this to the 0.23.2 milestone Jun 15, 2018
@codecov
Copy link

codecov bot commented Jun 16, 2018

Codecov Report

Merging #21491 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #21491   +/-   ##
=======================================
  Coverage   91.92%   91.92%           
=======================================
  Files         153      153           
  Lines       49587    49587           
=======================================
  Hits        45583    45583           
  Misses       4004     4004
Flag Coverage Δ
#multiple 90.32% <ø> (ø) ⬆️
#single 41.89% <ø> (ø) ⬆️

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 9e982e1...6d20d6a. Read the comment docs.

@mroeschke
Copy link
Member Author

Thanks @jreback. Yeah I don't think #21407 was directly related to any of these older issues, so made separate entries for each.

@jreback jreback merged commit 89418a3 into pandas-dev:master Jun 18, 2018
@jreback
Copy link
Contributor

jreback commented Jun 18, 2018

thanks @mroeschke !

@mroeschke mroeschke deleted the test_old_timezone_issues branch June 18, 2018 22:44
- Bug in :class:`Timestamp` constructor where passing an invalid timezone offset designator (``Z``) would not raise a ``ValueError``(:issue:`8910`)
- Bug in :meth:`Timestamp.replace` where replacing at a DST boundary would retain an incorrect offset (:issue:`7825`)
- Bug in :meth:`DatetimeIndex.reindex` when reindexing a tz-naive and tz-aware :class:`DatetimeIndex` (:issue:`8306`)
- Bug in :meth:`DatetimeIndex.resample` when downsampling across a DST boundary (:issue:`8531`)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we should put those in 0.23.2, as we didn't fix them in there.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see #21407

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These all didn't get fixed by #21407 per se. These issues may have been fixed by other prior PR's in other releases. What should our whatsnew policy be on cleaning up these older issues that have already been fixed?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if it is needed to list them at all. But if we list them, I would rather add them eg to the 0.24.0 (or 0.23.0) file than to the 0.23.2 file. For those bug fix releases, the list is much shorter and it is often more valuable to actually look which bugs/regressions have been fixed compared to 0.23.0 (in the 0.24.0 or 0.23.0 files the list is huge anyways ...)

@jreback
Copy link
Contributor

jreback commented Jun 21, 2018

it’s fine to move some to 0.24 whatsnew
though if they r indeed fixed by #21407 are ok here

@mroeschke
Copy link
Member Author

Sounds good. I have another timezones issues PR in the works so I'll keep that in mind

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 Timezones Timezone data dtype
Projects
None yet
3 participants