Skip to content

gh-126018: Avoid aborting due to unnecessary assert in sys.audit #126020

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.

Sign up for GitHub

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

Merged
JelleZijlstra merged 7 commits into from
Oct 27, 2024
Merged

gh-126018: Avoid aborting due to unnecessary assert in sys.audit #126020

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
ba8ac4d
Remove an assert that sys.audit's args[0] passes PyUnicode_Check.
devdanzin Oct 26, 2024
620f682
Test that passing a non-string to sys.audit doesn't trigger an assert…
devdanzin Oct 26, 2024
b775f5d
📜🤖 Added by blurb_it.
blurb-it[bot] Oct 26, 2024
6234c5d
Apply suggestions from code review
devdanzin Oct 26, 2024
a6edd2c
Apply suggestions from code review by @Eclips4.
devdanzin Oct 27, 2024
57cc87c
Fail test_assert_unicode if no exception is raised by sys.audit(9) (t…
devdanzin Oct 27, 2024
c42c4d6
Apply review suggestion for better description of issue in NEWS.
devdanzin Oct 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fail test_assert_unicode if no exception is raised by sys.audit(9) (t… 
…hanks @JelleZijlstra).
  • Loading branch information
@devdanzin
devdanzin committed Oct 27, 2024
commit 57cc87c3c935320e7a5db10f46a4f4ec1c0e12d3
2 changes: 2 additions & 0 deletions Lib/test/audit-tests.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -574,6 +574,8 @@ def test_assert_unicode():
sys.audit(9)
except TypeError:
pass
Comment thread
devdanzin marked this conversation as resolved.
Comment thread
devdanzin marked this conversation as resolved.
else:
raise RuntimeError("Expected sys.audit(9) to fail.")


if __name__ == "__main__":
Expand Down