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

gh-104855: Update more tk tests for 8.7 #105047

Closed
wants to merge 2 commits into from

Conversation

terryjreedy
Copy link
Member

@terryjreedy terryjreedy commented May 29, 2023

Where applicable:

  • Add '' to the valid options in error messages.
  • Test that '' is valid.
  • Test that 'to' < 'from' is valid instead of invalid.

This PR also addresses gh-104856 .

Where applicable:
* Add '' to the valid options in error messages.
* Test that '' is valid.
* Test that 'to' < 'from' is valid instead of invalid.

This PR also fixes pythongh-104856.
@terryjreedy
Copy link
Member Author

As before, I will wait for Christopher to test on 8.7 and Serhiy to comment and possibly revise.

@terryjreedy
Copy link
Member Author

@serhiy-storchaka Please review

@serhiy-storchaka serhiy-storchaka added needs backport to 3.13 bugs and security fixes and removed needs backport to 3.11 only security fixes labels May 9, 2024
@serhiy-storchaka serhiy-storchaka self-assigned this May 31, 2024
@serhiy-storchaka
Copy link
Member

@terryjreedy, there were errors in these fixes, and there were also many other test failures (and even crashes, due to Tk bugs). I fixed them all (at least on Linux), and think that this is ready to merge.

If there are Windows or macOS specific failures, they can be fixed in a separate PR.

@serhiy-storchaka
Copy link
Member

I expected to only fix few errors in this PR by committing in the original branch, but it required much more changes. So I created a branch in my repository and a new PR #120824.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting core review DO-NOT-MERGE needs backport to 3.12 bug and security fixes needs backport to 3.13 bugs and security fixes skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants