Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Fix intermittenttest_ci_fuzz_stdlib failures#145641

Open
chris-eibl wants to merge 1 commit intopython:mainfrom
chris-eibl:fix_test_ci_fuzz_stdlib
Open

Fix intermittenttest_ci_fuzz_stdlib failures#145641
chris-eibl wants to merge 1 commit intopython:mainfrom
chris-eibl:fix_test_ci_fuzz_stdlib

Conversation

@chris-eibl
Copy link
Member

@chris-eiblchris-eibl commentedMar 8, 2026•
edited
Loading

test_ci_fuzz_stdlib fails randomly since#145232,for example:

======================================================================ERROR: test_ci_fuzz_stdlib (test.test_tools.test_compute_changes.TestProcessChangedFiles.test_ci_fuzz_stdlib) (p=PosixPath('Modules/tomllib'))----------------------------------------------------------------------Traceback (most recent call last):  File "/home/runner/work/cpython/cpython-ro-srcdir/Lib/test/test_tools/test_compute_changes.py", line 57, in test_ci_fuzz_stdlib    result = process_changed_files({f})                                    ^UnboundLocalError: cannot access local variable 'f' where it is not associated with a value

in#145597.

Because theLIBRARY_FUZZER_PATHS is an unorderedfrozenset and contains an invalid entryModules\tomllib (#143749), it can happen that this is the first item when iterating.

Let's failtest_ci_fuzz_stdlib with an explicit error message if we detect an invalid entry.

Path("Lib/tarfile.py"),
# tomllib
Path("Modules/tomllib/"),
Path("Lib/tomllib/"),
Copy link
MemberAuthor

@chris-eiblchris-eiblMar 8, 2026•
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

I don't think this needs a new issue, but maybe reusing#143572 makes sense?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Since it's an internal tool, it doesn’t need a blurb/issue.

@StanFromIrelandStanFromIreland changed the titlefix intermittent test_ci_fuzz_stdlib failuresFix intermittenttest_ci_fuzz_stdlib failuresMar 8, 2026
Copy link
Member

@StanFromIrelandStanFromIreland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@StanFromIrelandStanFromIrelandStanFromIreland approved these changes

@hugovkhugovkAwaiting requested review from hugovkhugovk is a code owner

@sethmlarsonsethmlarsonAwaiting requested review from sethmlarson

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@chris-eibl@StanFromIreland
[8]ページ先頭
©2009-2026 Movatter.jp