Python 3.12 inspecting a coroutine using getcoroutinestate returns the wrong state.

Bug report

Bug description:

Prior to Python 3.12, we can close a just created coroutine, thus, the following test passes:

from inspect import getcoroutinestate, CORO_CLOSED


async def do_nothing():
    pass


def test_immediate_close():
    coro = do_nothing()
    coro.close()
    assert getcoroutinestate(coro) == CORO_CLOSED

But in Python 3.12, the assertion fails and getcoroutinestate(coro) still is CORO_CREATED after the coro.close(). You can confirm it from https://github.com/gottadiveintopython/py312/actions/runs/6571303025/job/17850194150.

I don't know if this is an intensional change or not, but I just wanted to tell you.

CPython versions tested on:

3.12

Operating systems tested on:

Linux

Linked PRs

Same change on Windows. 3.13 like 3.12.

@markshannon

Bisected to f02fa64
cc @markshannon

The coroutine is closed, as coro.send(None) fails. It is just that getcoroutinestate returns the wrong value.
I've updated the title accordingly.

getcoroutinestate does this:

    if coroutine.cr_running:
        return CORO_RUNNING
    if coroutine.cr_suspended:
        return CORO_SUSPENDED
    if coroutine.cr_frame is None:
        return CORO_CLOSED
    return CORO_CREATED

coroutine.cr_frame returns None for frames which have state FRAME_CLEARED, but not for FRAME_COMPLETED.

pythongh-111058: Change coro.cr_frame to be None for a closed coroutine

gh-111058: Change coro.cr_frame/gen.gi_frame to be None for a closed …

pythongh-111058: Change coro.cr_frame/gen.gi_frame to be None for a c…

@gottadiveintopython

Thanks @gottadiveintopython for the report and Irit for fixing it!

[3.12] gh-111058: Change coro.cr_frame/gen.gi_frame to be None for a …

pythongh-111058: Change coro.cr_frame/gen.gi_frame to be None for a c…

Navigation Menu

Repository navigation

Python 3.12 inspecting a coroutine using `getcoroutinestate` returns the wrong state. #111058

Bug report

Bug description:

CPython versions tested on:

Operating systems tested on:

Linked PRs

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Python 3.12 inspecting a coroutine using getcoroutinestate returns the wrong state. #111058

Description

Bug report

Bug description:

CPython versions tested on:

Operating systems tested on:

Linked PRs

Activity

terryjreedy commented on Oct 19, 2023

Eclips4 commented on Oct 19, 2023

markshannon commented on Oct 31, 2023

iritkatriel commented on Nov 26, 2023

Eclips4 commented on Dec 1, 2023

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions

Python 3.12 inspecting a coroutine using `getcoroutinestate` returns the wrong state. #111058