Add tests for Datetime C API Macros

[pganssle]

BPO	36833
Nosy	@vstinner, @pganssle, @tonybaloney, @nanjekyejoannah, @SimiCode
PRs	bpo-36833: Add tests for Datetime C API Macros #14842

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/pganssle'
closed_at = <Date 2019-08-29.12:55:19.529>
created_at = <Date 2019-05-07.14:17:31.677>
labels = ['type-feature', 'tests', '3.9']
title = 'Add tests for Datetime C API Macros'
updated_at = <Date 2019-08-29.12:55:19.528>
user = 'https://github.com/pganssle'

bugs.python.org fields:

activity = <Date 2019-08-29.12:55:19.528>
actor = 'vstinner'
assignee = 'p-ganssle'
closed = True
closed_date = <Date 2019-08-29.12:55:19.529>
closer = 'vstinner'
components = ['Tests']
creation = <Date 2019-05-07.14:17:31.677>
creator = 'p-ganssle'
dependencies = []
files = []
hgrepos = []
issue_num = 36833
keywords = ['patch']
message_count = 4.0
messages = ['341731', '342235', '350771', '350772']
nosy_count = 5.0
nosy_names = ['vstinner', 'p-ganssle', 'anthony shaw', 'nanjekyejoannah', 'edison.abahurire']
pr_nums = ['14842']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue36833'
versions = ['Python 3.9']

[pganssle]

This is a child issue for bpo-36782, specifically for testing the macro-only datetime C API functions

Untested macros with no corresponding API module:

• PyDateTime_GET_YEAR

• PyDateTime_GET_MONTH

• PyDateTime_GET_DAY

• PyDateTime_DATE_GET_HOUR

• PyDateTime_DATE_GET_MINUTE

• PyDateTime_DATE_GET_SECOND

• PyDateTime_DATE_GET_MICROSECOND

• PyDateTime_TIME_GET_HOUR

• PyDateTime_TIME_GET_MINUTE

• PyDateTime_TIME_GET_SECOND

• PyDateTime_TIME_GET_MICROSECOND

• PyDateTime_DELTA_GET_DAYS

• PyDateTime_DELTA_GET_SECONDS

• PyDateTime_DELTA_GET_MICROSECONDS

For all of these, I think you can write one "wrapper function" that just extracts all fields as a tuple (4 C API wrapper functions - date, datetime, time, timedelta).

For each function, test with both the standard class and a subclass.

[simicode]

I'm now working on this as pbo-36782 awaits merging.

[vstinner]

New changeset 2c5fb17 by Victor Stinner (Joannah Nanjekye) in branch 'master':
bpo-36833: Add tests for Datetime C API Macros (GH-14842)
2c5fb17

[vstinner]

Thanks Joannah Nanjekye, well done ;-)