Skip to content

gh-132742: Improve tests for fcntl.ioctl() #132791

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

serhiy-storchaka
Copy link
Member

@serhiy-storchaka serhiy-storchaka commented Apr 22, 2025

  • Do not skip ALL ioctl() tests when /dev/tty is not available.
  • Use better tests for integer argument.
  • Add also parallel tests for tcflush() and tcflow().

* Do not skip ALL ioctl() tests when /dev/tty is not available.
* Use better tests for integer argument.
* Add also parallel tests for tcflush() and tcflow().
@bedevere-app bedevere-app bot added awaiting core review tests Tests in the Lib/test dir labels Apr 22, 2025
@serhiy-storchaka
Copy link
Member Author

For some reasons tcflush() and tcflow() work differently on Linux/OpenIndiana vs BSD/macOS.

@serhiy-storchaka serhiy-storchaka marked this pull request as ready for review April 24, 2025 09:57
@serhiy-storchaka serhiy-storchaka added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Apr 24, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @serhiy-storchaka for commit 7efcbf9 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F132791%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Apr 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants