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

Datetime test coverage #7544

Merged
merged 3 commits into from
Jun 8, 2018
Merged

Conversation

abalkin
Copy link
Member

@abalkin abalkin commented Jun 8, 2018

No description provided.

abalkin added 3 commits June 8, 2018 15:49
The added test checks a case with UTC offest expressed in an integer
number of seconds.
Check that a greater than  comparison of a naive  datetime instance with
an aware one raises a TypeError.
@bedevere-bot
Copy link

@abalkin: Please replace # with GH- in the commit message next time. Thanks!

@miss-islington
Copy link
Contributor

Thanks @abalkin for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-7551 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 8, 2018
* Added a test case for strftime("%z").

The added test checks a case with UTC offest expressed in an integer
number of seconds.

* Added a test comparing naive and aware datetimes.

Check that a greater than  comparison of a naive  datetime instance with
an aware one raises a TypeError.

* Test datetime in fold or in gap comparison both ways.
(cherry picked from commit 4c3e39f)

Co-authored-by: Alexander Belopolsky <abalkin@users.noreply.github.com>
abalkin added a commit that referenced this pull request Jun 9, 2018
* Added a test case for strftime("%z").

The added test checks a case with UTC offest expressed in an integer
number of seconds.

* Added a test comparing naive and aware datetimes.

Check that a greater than  comparison of a naive  datetime instance with
an aware one raises a TypeError.

* Test datetime in fold or in gap comparison both ways.
(cherry picked from commit 4c3e39f)

Co-authored-by: Alexander Belopolsky <abalkin@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants