Skip to content
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.

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

Fix pdb selftests on Python 3.13 #12885

Merged
merged 1 commit into from
Oct 13, 2024
Merged

Conversation

The-Compiler
Copy link
Member

Python 3.13 makes pdb break on the breakpoint() call, rather than on the next line:
https://docs.python.org/3/whatsnew/3.13.html#pdb

Also runs the pdb tests on Python 3.13 in CI.
See #12884 for a more proper solution for that.

Fixes #12497

@psf-chronographer psf-chronographer bot added the bot:chronographer:provided (automation) changelog entry is part of PR label Oct 13, 2024
@The-Compiler The-Compiler added the backport 8.3.x apply to PRs at any point; backports the changes to the 8.3.x branch label Oct 13, 2024
@@ -0,0 +1 @@
Fixed two failing pdb-related tests on Python 3.13.
Copy link
Member

Choose a reason for hiding this comment

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

Given this did not fix anything on pytest itself, only in the test suite, perhaps making this a misc changelog entry would be more appropriate?

Copy link
Member Author

Choose a reason for hiding this comment

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

We have contrib too, which is described as:

stuff that affects the contributor experience. e.g.
Running tests, building the docs, setting up the development
environment.

That seems like a good fit given the "running tests" example.

Copy link
Member

Choose a reason for hiding this comment

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

Agreed! 👍

Python 3.13 makes pdb break on the breakpoint() call,
rather than on the next line:
https://docs.python.org/3/whatsnew/3.13.html#pdb

Also runs the pdb tests on Python 3.13 in CI.
See pytest-dev#12884 for a more proper solution for that.

Fixes pytest-dev#12497
@The-Compiler The-Compiler merged commit a4e40bc into pytest-dev:main Oct 13, 2024
29 checks passed
Copy link

patchback bot commented Oct 13, 2024

Backport to 8.3.x: 💚 backport PR created

✅ Backport PR branch: patchback/backports/8.3.x/a4e40bcf77f29615a12a12758e5307d3f4ee0663/pr-12885

Backported as #12887

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Oct 13, 2024
Fix pdb selftests on Python 3.13

(cherry picked from commit a4e40bc)
nicoddemus pushed a commit that referenced this pull request Oct 13, 2024
Fix pdb selftests on Python 3.13

(cherry picked from commit a4e40bc)

Co-authored-by: Florian Bruhin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 8.3.x apply to PRs at any point; backports the changes to the 8.3.x branch bot:chronographer:provided (automation) changelog entry is part of PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Python 3.13.0b2] test_pdb_used_outside_test and test_pdb_used_in_generate_tests are failing with Python 3.13
3 participants