Skip to content

test_monitoring has duplicated tests #106193

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

Jump to bottom
Closed
sobolevn opened this issue Jun 28, 2023 · 3 comments · Fixed by #109139
Closed

test_monitoring has duplicated tests #106193

sobolevn opened this issue Jun 28, 2023 · 3 comments · Fixed by #109139
Assignees
@sobolevn
Labels
3.12 only security fixes 3.13 bugs and security fixes tests Tests in the Lib/test dir type-bug An unexpected behavior, bug, or error

Comments

@sobolevn
Copy link
Member

sobolevn commented Jun 28, 2023 •
edited by bedevere-app bot
Loading

Based on python/core-workflow#505 by @hugovk

Lib/test/test_monitoring.py:973:5: F811 redefinition of unused 'test_line_then_instruction' from line 950
Lib/test/test_monitoring.py:978:5: F811 redefinition of unused 'test_instruction_then_line' from line 955

Before rename: Ran 49 tests in 0.014s
After rename:

======================================================================
ERROR: test_instruction_then_line (test.test_monitoring.TestInstallIncrementallly.test_instruction_then_line)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/sobolev/Desktop/cpython/Lib/test/test_monitoring.py", line 956, in test_instruction_then_line
    recorders = [ InstructionRecorder, LineRecorderLowNoise ]
                                       ^^^^^^^^^^^^^^^^^^^^
NameError: name 'LineRecorderLowNoise' is not defined

======================================================================
ERROR: test_line_then_instruction (test.test_monitoring.TestInstallIncrementallly.test_line_then_instruction)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/sobolev/Desktop/cpython/Lib/test/test_monitoring.py", line 953, in test_line_then_instruction
    recorders = recorders, must_include = self.EXPECTED_LI)
                                          ^^^^^^^^^^^^^^^^
AttributeError: 'TestInstallIncrementallly' object has no attribute 'EXPECTED_LI'

----------------------------------------------------------------------
Ran 51 tests in 0.045s

FAILED (errors=2)
test test_monitoring failed
test_monitoring failed (2 errors)

== Tests result: FAILURE ==

1 test failed:
    test_monitoring

Total duration: 94 ms
Tests result: FAILURE

Based on the missing runtime parts in these tests, we can just remove the first two ones, I guess.

Both of these tests were introduced in 411b169 by @markshannon

Linked PRs
@sobolevn sobolevn added type-bug An unexpected behavior, bug, or error tests Tests in the Lib/test dir labels Jun 28, 2023
@sobolevn sobolevn self-assigned this Jun 28, 2023
@sobolevn
Copy link
Member Author

No, they cannot be just deleted, because they test different things.

@sobolevn sobolevn mentioned this issue Jun 28, 2023
Closed
@sobolevn
Copy link
Member Author

Introduced in b32a075#diff-15fb0cb29709c04c449bacd69e4ab2208372424d8390c26258eb83b2363c3cf1

Looks like it never had parts that are missing right now.

@hugovk hugovk mentioned this issue Sep 8, 2023
Merged
@hugovk hugovk added 3.12 only security fixes 3.13 bugs and security fixes labels Sep 8, 2023
@hugovk
Copy link
Member

hugovk commented Sep 8, 2023

Please see PR #109139.

markshannon pushed a commit that referenced this issue Oct 12, 2023
@hugovk
gh-106193: Rename and fix duplicated tests in test_monitoring (GH-1…
ea530f2 
…09139)
@bedevere-app bedevere-app bot mentioned this issue Oct 15, 2023
Merged
hugovk added a commit to hugovk/cpython that referenced this issue Oct 15, 2023
@hugovk
[3.12] pythongh-106193: Rename and fix duplicated tests in `test_moni…
15ea3d5 
…toring` (pythonGH-109139)

(cherry picked from commit ea530f2)

Co-authored-by: Hugo van Kemenade <[email protected]>
hugovk added a commit that referenced this issue Oct 15, 2023
@hugovk
[3.12] gh-106193: Rename and fix duplicated tests in test_monitoring (
9c39149 
GH-109139) (#110897)
Glyphack pushed a commit to Glyphack/cpython that referenced this issue Sep 2, 2024
@hugovk @Glyphack
pythongh-106193: Rename and fix duplicated tests in test_monitoring (…
ace4b62 
…pythonGH-109139)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sobolevn sobolevn

Labels
3.12 only security fixes 3.13 bugs and security fixes tests Tests in the Lib/test dir type-bug An unexpected behavior, bug, or error
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

gh-106193: Rename and fix duplicated tests in test_monitoring hugovk/cpython
2 participants
@hugovk @sobolevn