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

Add test for Argument Clinic converters #79759

Closed
serhiy-storchaka opened this issue Dec 24, 2018 · 5 comments
Closed

Add test for Argument Clinic converters #79759

serhiy-storchaka opened this issue Dec 24, 2018 · 5 comments
Labels
3.7 (EOL) end of life 3.8 (EOL) end of life tests Tests in the Lib/test dir topic-argument-clinic type-feature A feature request or enhancement

Comments

@serhiy-storchaka
Copy link
Member

BPO 35578
Nosy @larryhastings, @serhiy-storchaka, @pablogsal
PRs
  • bpo-35578: Add an example file for testing Argument Clinic converters. #11306
  • bpo-35578: Add an example file for testing Argument Clinic converters. #11306
  • bpo-35578: Add an example file for testing Argument Clinic converters. #11306
  • [3.7] bpo-35578: Add an example file for testing Argument Clinic converters. (GH-11306) #11311
  • [3.7] bpo-35578: Add an example file for testing Argument Clinic converters. (GH-11306) #11311
  • [3.7] bpo-35578: Add an example file for testing Argument Clinic converters. (GH-11306) #11311
  • 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 = None
    closed_at = None
    created_at = <Date 2018-12-24.16:41:25.367>
    labels = ['3.7', '3.8', 'type-feature', 'tests', 'expert-argument-clinic']
    title = 'Add test for Argument Clinic converters'
    updated_at = <Date 2018-12-29.20:34:50.829>
    user = 'https://github.com/serhiy-storchaka'

    bugs.python.org fields:

    activity = <Date 2018-12-29.20:34:50.829>
    actor = 'pablogsal'
    assignee = 'none'
    closed = False
    closed_date = None
    closer = None
    components = ['Tests', 'Argument Clinic']
    creation = <Date 2018-12-24.16:41:25.367>
    creator = 'serhiy.storchaka'
    dependencies = []
    files = []
    hgrepos = []
    issue_num = 35578
    keywords = ['patch', 'patch', 'patch']
    message_count = 4.0
    messages = ['332493', '332504', '332506', '332732']
    nosy_count = 3.0
    nosy_names = ['larry', 'serhiy.storchaka', 'pablogsal']
    pr_nums = ['11306', '11306', '11306', '11311', '11311', '11311']
    priority = 'normal'
    resolution = None
    stage = 'patch review'
    status = 'open'
    superseder = None
    type = 'enhancement'
    url = 'https://bugs.python.org/issue35578'
    versions = ['Python 3.7', 'Python 3.8']

    @serhiy-storchaka
    Copy link
    Member Author

    Currently Argument Clinic converters are tested by running Argument Clinic on the CPython source tree. If it generates incorrect code, then it contains bugs. But not all combinations of standard converters and options are used in the stdlib.

    The programming interface of Argument Clinic is complex, and it is hard to write tests for testing only specific functionality. The simplest way of testing Argument Clinic is to write a C file containing declarations and generated code for all test cases. Although this does not allow to test error cases.

    The proposed PR adds Lib/test/clinic_test.c which contains tests for all standard converters. It will be extended in bpo-20180 (PR bpo-9828) and bpo-23867.

    @serhiy-storchaka serhiy-storchaka added 3.7 (EOL) end of life 3.8 (EOL) end of life tests Tests in the Lib/test dir topic-argument-clinic type-feature A feature request or enhancement labels Dec 24, 2018
    @serhiy-storchaka
    Copy link
    Member Author

    New changeset 837c7dc by Serhiy Storchaka in branch 'master':
    bpo-35578: Add an example file for testing Argument Clinic converters. (GH-11306)
    837c7dc

    @serhiy-storchaka
    Copy link
    Member Author

    New changeset 74d914e by Serhiy Storchaka in branch '3.7':
    [3.7] bpo-35578: Add an example file for testing Argument Clinic converters. (GH-11306) (GH-11311)
    74d914e

    @pablogsal
    Copy link
    Member

    Is there anything left in this issue or we can close it?

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    @erlend-aasland
    Copy link
    Contributor

    Tests for clinic converters were significantly improved in gh-96178. We can close this now.

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    3.7 (EOL) end of life 3.8 (EOL) end of life tests Tests in the Lib/test dir topic-argument-clinic type-feature A feature request or enhancement
    Projects
    None yet
    Development

    No branches or pull requests

    3 participants