Bug 2279996

Summary: python-sphinxcontrib-openapi fails to build with Python 3.13: AssertionError: assert Entity == Content
Product: [Fedora] Fedora Reporter: Karolina Surma <ksurma>
Component: python-sphinxcontrib-openapiAssignee: Orphan Owner <extras-orphan>
Status: CLOSED WONTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: fboucher, fti-bugs, ksurma, mhroncok
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2024-08-23 13:00:24 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 2244836, 2260875, 2260877, 2300529    

Description Karolina Surma 2024-05-10 12:22:25 UTC
python-sphinxcontrib-openapi fails to build with Python 3.13.0b1.

________________ test_render_response_content_status_code[422] _________________

testrenderer = <sphinxcontrib.openapi.renderers._httpdomain.HttpdomainRenderer object at 0x7fbf1409a530>
status_code = '422', status_text = 'Unprocessable Entity'

    @pytest.mark.parametrize(
        ["status_code", "status_text"],
        [
            pytest.param("201", "Created", id="201"),
            pytest.param("307", "Temporary Redirect", id="307"),
            pytest.param("422", "Unprocessable Entity", id="422"),
        ],
    )
    def test_render_response_content_status_code(testrenderer, status_code, status_text):
        """Path response's example is rendered with proper status code."""
    
        markup = textify(
            testrenderer.render_response_content(
                {
                    "text/csv": {
                        "example": textwrap.dedent(
                            """\
                            foo,baz
                            bar,42
                            """
                        )
                    }
                },
                status_code,
            )
        )
>       assert markup == textwrap.dedent(
            f"""\
            .. sourcecode:: http
    
               HTTP/1.1 {status_code} {status_text}
               Content-Type: text/csv
    
               foo,baz
               bar,42
            """.rstrip()
        )
E       AssertionError: assert '.. sourcecod...az\n   bar,42' == '.. sourcecod...az\n   bar,42'
E         Skipping 42 identical leading characters in diff, use -v to show
E         - ocessable Entity
E         ?           ^  ^ -
E         + ocessable Content
E         ?           ^^  ^^
E              Content-Type: text/csv
E           
E              foo,baz
E              bar,42

tests/renderers/httpdomain/test_render_response_content.py:465: AssertionError
______________ test_render_response_content_status_code_int[422] _______________

testrenderer = <sphinxcontrib.openapi.renderers._httpdomain.HttpdomainRenderer object at 0x7fbf14c5c7d0>
status_code = '422', status_text = 'Unprocessable Entity'

    @pytest.mark.parametrize(
        ["status_code", "status_text"],
        [
            pytest.param("201", "Created", id="201"),
            pytest.param("307", "Temporary Redirect", id="307"),
            pytest.param("422", "Unprocessable Entity", id="422"),
        ],
    )
    def test_render_response_content_status_code_int(
        testrenderer, status_code, status_text
    ):
        """Path response's example is rendered with proper status code."""
    
        markup = textify(
            testrenderer.render_response_content(
                {
                    "text/csv": {
                        "example": textwrap.dedent(
                            """\
                            foo,baz
                            bar,42
                            """
                        )
                    }
                },
                status_code,
            )
        )
>       assert markup == textwrap.dedent(
            f"""\
            .. sourcecode:: http
    
               HTTP/1.1 {status_code} {status_text}
               Content-Type: text/csv
    
               foo,baz
               bar,42
            """.rstrip()
        )
E       AssertionError: assert '.. sourcecod...az\n   bar,42' == '.. sourcecod...az\n   bar,42'
E         Skipping 42 identical leading characters in diff, use -v to show
E         - ocessable Entity
E         ?           ^  ^ -
E         + ocessable Content
E         ?           ^^  ^^
E              Content-Type: text/csv
E           
E              foo,baz
E              bar,42

tests/renderers/httpdomain/test_render_response_content.py:547: AssertionError

=========================== short test summary info ============================
FAILED tests/renderers/httpdomain/test_render_response.py::test_render_response_content_custom[422]
FAILED tests/renderers/httpdomain/test_render_response_content.py::test_render_response_content_status_code[422]
FAILED tests/renderers/httpdomain/test_render_response_content.py::test_render_response_content_status_code_int[422]

https://docs.python.org/3.13/whatsnew/3.13.html

For the build logs, see:
https://copr-be.cloud.fedoraproject.org/results/@python/python3.13/fedora-rawhide-x86_64/07434158-python-sphinxcontrib-openapi/

For all our attempts to build python-sphinxcontrib-openapi with Python 3.13, see:
https://copr.fedorainfracloud.org/coprs/g/python/python3.13/package/python-sphinxcontrib-openapi/

Testing and mass rebuild of packages is happening in copr.
You can follow these instructions to test locally in mock if your package builds with Python 3.13:
https://copr.fedorainfracloud.org/coprs/g/python/python3.13/

Let us know here if you have any questions.

Python 3.13 is planned to be included in Fedora 41.
To make that update smoother, we're building Fedora packages with all pre-releases of Python 3.13.
A build failure prevents us from testing all dependent packages (transitive [Build]Requires),
so if this package is required a lot, it's important for us to get it fixed soon.

We'd appreciate help from the people who know this package best,
but if you don't want to work on this now, let us know so we can try to work around it on our side.

Comment 1 Karolina Surma 2024-06-12 15:07:43 UTC
*** Bug 2291924 has been marked as a duplicate of this bug. ***

Comment 2 Fedora Fails To Install 2024-06-19 19:59:29 UTC
Hello,

Please note that this comment was generated automatically by https://pagure.io/releng/blob/main/f/scripts/ftbfs-fti/follow-policy.py
If you feel that this output has mistakes, please open an issue at https://pagure.io/releng/

This package fails to install and maintainers are advised to take one of the following actions:

 - Fix this bug and close this bugzilla once the update makes it to the repository.
   (The same script that posted this comment will eventually close this bugzilla
   when the fixed package reaches the repository, so you don't have to worry about it.)

or

 - Move this bug to ASSIGNED if you plan on fixing this, but simply haven't done so yet.

or

 - Orphan the package if you no longer plan to maintain it.


If you do not take one of these actions, the process at https://docs.fedoraproject.org/en-US/fesco/Fails_to_build_from_source_Fails_to_install/#_package_removal_for_long_standing_ftbfs_and_fti_bugs will continue.
This package may be orphaned in 7+ weeks.
This is the first reminder (step 3) from the policy.

Don't hesitate to ask for help on https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/ if you are unsure how to fix this bug.

Comment 3 Fedora Fails To Install 2024-07-13 08:01:33 UTC
Hello,

Please note that this comment was generated automatically by https://pagure.io/releng/blob/main/f/scripts/ftbfs-fti/follow-policy.py
If you feel that this output has mistakes, please open an issue at https://pagure.io/releng/

This package fails to install and maintainers are advised to take one of the following actions:

 - Fix this bug and close this bugzilla once the update makes it to the repository.
   (The same script that posted this comment will eventually close this bugzilla
   when the fixed package reaches the repository, so you don't have to worry about it.)

or

 - Move this bug to ASSIGNED if you plan on fixing this, but simply haven't done so yet.

or

 - Orphan the package if you no longer plan to maintain it.


If you do not take one of these actions, the process at https://docs.fedoraproject.org/en-US/fesco/Fails_to_build_from_source_Fails_to_install/#_package_removal_for_long_standing_ftbfs_and_fti_bugs will continue.
This package may be orphaned in 4+ weeks.
This is the second reminder (step 4) from the policy.

Don't hesitate to ask for help on https://lists.fedoraproject.org/archives/list/devel@lists.fedoraproject.org/ if you are unsure how to fix this bug.

Comment 4 Fedora Fails To Install 2024-08-13 00:02:27 UTC
This package has been orphaned.

You can pick it up at https://src.fedoraproject.org/rpms/python-sphinxcontrib-openapi by clicking button "Take". If nobody picks it up, it will be retired and removed from a distribution.

Comment 5 Fedora Admin user for bugzilla script actions 2024-08-22 18:44:32 UTC
This package has changed maintainer in Fedora. Reassigning to the new maintainer of this component.

Comment 6 Miro HronĨok 2024-08-23 13:00:24 UTC
Automation has figured out the package is retired in rawhide.

If you like it to be unretired, please open a ticket at https://pagure.io/releng/new_issue?template=package_unretirement

Comment 7 Red Hat Bugzilla 2024-12-22 04:25:11 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 120 days