Skip to content

gh-113932: assert SyntaxWarning in test_compile.TestSpecifics.test_…#113933

Merged
iritkatriel merged 1 commit into
python:mainfrom
Eclips4:issue-113932
Jan 11, 2024
Merged

gh-113932: assert SyntaxWarning in test_compile.TestSpecifics.test_…#113933
iritkatriel merged 1 commit into
python:mainfrom
Eclips4:issue-113932

Conversation

@Eclips4

@Eclips4 Eclips4 commented Jan 11, 2024

Copy link
Copy Markdown
Member

…condition_expression_with_redundant_comparisons_compiles

….test_condition_expression_with_redundant_comparisons_compiles
@Eclips4 Eclips4 requested a review from iritkatriel January 11, 2024 06:07
@bedevere-app bedevere-app Bot added tests Tests in the Lib/test dir awaiting review labels Jan 11, 2024
@Eclips4 Eclips4 added skip news and removed tests Tests in the Lib/test dir awaiting review labels Jan 11, 2024
@Eclips4 Eclips4 added tests Tests in the Lib/test dir awaiting review labels Jan 11, 2024

@iritkatriel iritkatriel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Did you try running this test with -We ? Does the assertion still work?

@iritkatriel

Copy link
Copy Markdown
Member

My point is: the title of the PR says 'suppress warning', but the PR actually does more - it asserts the warning.

@Eclips4

Eclips4 commented Jan 11, 2024

Copy link
Copy Markdown
Member Author

Did you try running this test with -We ? Does the assertion still work?

Yes, it's works.

My point is: the title of the PR says 'suppress warning', but the PR actually does more - it asserts the warning.

How's about "Make test_x more accurate"?

@iritkatriel iritkatriel changed the title gh-113932: Suppress SyntaxWarning in test_compile.TestSpecifics.test_… gh-113932: assert SyntaxWarning in test_compile.TestSpecifics.test_… Jan 11, 2024
@iritkatriel iritkatriel merged commit 9f08833 into python:main Jan 11, 2024
@Eclips4 Eclips4 deleted the issue-113932 branch January 11, 2024 11:36
kulikjak pushed a commit to kulikjak/cpython that referenced this pull request Jan 22, 2024
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants