Skip to content

WIP: bpo-1100942: Add datetime.time.strptime and datetime.date.strptime #5578

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.

Sign up for GitHub

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
matrixise wants to merge 16 commits into from
+288 −12
Closed

WIP: bpo-1100942: Add datetime.time.strptime and datetime.date.strptime #5578

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
ad5ffab
bpo-1100942: Add datetime.time.strptime and datetime.date.strptime
matrixise Feb 7, 2018
50606eb
Follow the PEP7 for the C Code
matrixise May 15, 2018
ab5030f
Remove unused formats
matrixise May 15, 2018
07fb326
Use subtest for the tests
matrixise Oct 17, 2018
010ac8b
Fix the signatures
matrixise Oct 25, 2018
5b58fc1
Update with the recommendation of @pganssle
matrixise Oct 25, 2018
8353a65
Update
matrixise Nov 4, 2018
cff74a9
Add use case where we generate an exception
matrixise Nov 11, 2018
f92a799
Add the Py_UNUSED(ignore) for datetime_gettime & datetime_getdate
matrixise Nov 11, 2018
4c4a7cc
Add a link to the strtime() and strptime() behaviour
matrixise Mar 6, 2019
b006172
Update the blurb entry and add an entry in the Doc/whatsnew/3.8.rst
matrixise Mar 6, 2019
ff7487f
Fix doc
matrixise Mar 6, 2019
a048f0e
Convert the _check_invalid_datetime_spec to the C-API
matrixise Mar 18, 2019
9ac0af4
Support tuple and list for the specs
matrixise Mar 18, 2019
a952bb7
_datetimemodule._check_invalid_datetime_specs only accepts tuple and not
matrixise Mar 24, 2019
412aea2
Fix the code with the recommendation of Martin Panter
matrixise Mar 24, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add use case where we generate an exception
  • Loading branch information
@matrixise
matrixise committed Mar 22, 2019
commit cff74a9540940415fb43b9d40b7c323a4c53bd2a
4 changes: 1 addition & 3 deletions Lib/test/datetimetester.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1031,7 +1031,6 @@ def test_strptime_valid_format(self):
def test_strptime_invalid_format(self):
tests = [
('2004-12-01 13:02:47.197', '%Y-%m-%d %H:%M:%S.%f'),
('2018-01-01 00:00', '%Y-%m-%d %H:%M'),
('2018-01-01', ''),
('01', '%M'),
('02', '%H'),
Copy link
Copy Markdown
Member

@pganssle pganssle Nov 4, 2018

Choose a reason for hiding this comment

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

This needs at least two more test cases:

    ('2018-01-01 00:00', '%Y-%m-%d %H:%M'),
    ('2018-01-01', ''),

Both should fail.

Expand Down Expand Up @@ -3134,8 +3133,7 @@ def test_strptime_invalid(self):
tests = [
('2004-12-01 13:02:47.197', '%Y-%m-%d %H:%M:%S.%f'),
('2004-12-01', '%Y-%m-%d'),
('1900-01-01 12:30', '%Y-%m-%d %H:%M'),
('12:30:15', '')
('12:30:15', ''),
]
for date_string, date_format in tests:
Copy link
Copy Markdown
Member

@pganssle pganssle May 15, 2018

Choose a reason for hiding this comment

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

I think it would be good to use with self.subTest for these parametrized tests.

Also, per the other comment I guess you need to add something like ('12:30:15', '')` to get full coverage.

Copy link
Copy Markdown
Member

@pganssle pganssle Nov 4, 2018

Choose a reason for hiding this comment

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

I think you need two more test cases:

    ('1900-01-01 12:30', '%Y-%m-%d %H:%M'),
    ('12:30:15', ''),

date.strptime has similarly missing tests.

Copy link
Copy Markdown
Member Author

@matrixise matrixise Nov 11, 2018

Choose a reason for hiding this comment

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

the test with ('1900-01-01 12:30', '%Y-%m-%d %H:%M') does not raise an exception but returns datetime.time(12, 30)

For the other test, yep, there is an issue.

Copy link
Copy Markdown
Member

@pganssle pganssle Nov 11, 2018

Choose a reason for hiding this comment

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

The fact that it doesn't raise an exception is an issue. I'm a bit surprised that it doesn't raise an exception on pure Python, that's a bug, because I'm pretty sure that:

datetime.time.strptime("1901-01-01 12:30", "%Y-%m-%d %H:%M") does raise an exception.

with self.subTest(date_string=date_string, date_format=date_format):
Expand Down