Skip to content

test_argparse.py trailing spaces within test strings removed by make patchcheck if file is modified #91984

@AbhigyanBose

Description

@AbhigyanBose

Bug report
There are two strings in test_argparse.py which have lines ending with blank spaces. When the file is modified and make patchcheck run, it removes these trailing spaces from within the strings. This causes the test cases that these strings are a part of to fail. The strings in question are:

This issue is further discussed over on #91841

Test Environment

CPython versions tested on:
-3,11(pulled latest main branch from python/cpython)
Operating system and architecture:
-Ubuntu LTS (20.04), 64-Bit

Proposed Fix
In both of these cases the lines are followed by a newline. We could append '\n' to the lines and remove the next line. As follows:

                foo         \n

This would solve the issue since the last character will no longer be a blank space. The strings would be essentially the same as before, thus the test cases will not be affected. (Confirmed this locally.)

Metadata

Metadata

Assignees

No one assigned

    Labels

    type-bugAn unexpected behavior, bug, or error

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      pFad - Phonifier reborn

      Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

      Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


      Alternative Proxies:

      Alternative Proxy

      pFad Proxy

      pFad v3 Proxy

      pFad v4 Proxy