Skip to content

Datetime test coverage#7544

Merged
abalkin merged 3 commits intopython:masterfrom
abalkin:datetime-test-coverage
Jun 8, 2018
Merged

Datetime test coverage#7544
abalkin merged 3 commits intopython:masterfrom
abalkin:datetime-test-coverage

Conversation

@abalkin
Copy link
Copy Markdown
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
Copy Markdown

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

@miss-islington
Copy link
Copy Markdown
Contributor

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

@bedevere-bot
Copy link
Copy Markdown

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