-
Notifications
You must be signed in to change notification settings - Fork 1.3k
test_os windows failed test handling #4764
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
Merged
Merged
Changes from all commits
Commits
Show all changes
80 commits
Select commit
Hold shift + click to select a range
82dfef6
move test_os to independant tests
derlih e051061
test_closerange
derlih 5080806
test_execve_invalid_env
derlih 3356fe1
test_execve_with_empty_path
derlih 01911f5
test_execvpe_with_bad_arglist
derlih 9880389
remove test_iter from platform-dependent tests
derlih dd4dd4b
_execvpe_mockup
derlih aa44ee9
test_dup_nul
derlih 5463487
test_get_set_inheritable
derlih e27c02f
test_get_set_inheritable_badf
derlih b6620ea
test_open
derlih 296a519
test_exist_ok_existing_directory, test_exist_ok_s_isgid_directory
derlih 1089205
check_waitpid
derlih 11950c8
test_waitstatus_to_exitcode_windows
derlih cee0ddf
test_access_denied
derlih 7e0e355
test_stat_block_device
derlih 4a9e554
test_inheritable
derlih 1291a5a
test_large_time
derlih f87d4cd
test_create_junction, test_unlink_removes_junction
derlih af17a33
test_kill_sigterm, test_kill_int
derlih a0b2f45
test_stat_unlink_race
derlih 89365e2
test_putenv_unsetenv_error
derlih 1dfc0a9
test_file_attributes
derlih 88051cb
test_fdopen
derlih c16a6ea
test_fstat, test_fsync
derlih d906641
test_ftruncate, test_lseek, test_read, test_write
derlih f5c7f32
test_empty_path
derlih 88a9907
test_removed_dir, test_removed_file
derlih 961a53e
test_urandom_subprocess, test_urandom_fd_closed, test_urandom_fd_reop…
derlih 69dec52
test_utime, test_utime_directory, test_utime_current, test_utime_curr…
derlih 0e68f97
test_utime_invalid_arguments
derlih ca2aab2
test_file_attributes -> expectedFailureIfWindows
derlih e6b6137
test_access_denied -> expectedFailureIfWindows
derlih 1ea5b58
test_stat_block_device -> expectedFailureIfWindows
derlih 408a545
test_utime -> expectedFailureIfWindows
derlih f3f813f
test_utime_directory -> expectedFailureIfWindows
derlih 9c07fac
test_utime_current -> expectedFailureIfWindows
derlih fda05ba
test_utime_current_old -> expectedFailureIfWindows
derlih 5e252c5
test_large_time -> expectedFailureIfWindows
derlih bce276e
test_utime_invalid_arguments -> expectedFailureIfWindows
derlih ae71c46
test_putenv_unsetenv_error -> expectedFailureIfWindows
derlih a7ef291
test_exist_ok_existing_directory -> expectedFailureIfWindows
derlih f5c4008
test_exist_ok_s_isgid_directory -> expectedFailureIfWindows
derlih ace1a5e
test_urandom_subprocess -> expectedFailureIfWindows
derlih 54c9811
test_urandom_fd_reopened -> expectedFailureIfWindows
derlih 5ee7e22
test_execvpe_with_bad_arglist -> expectedFailureIfWindows
derlih 0478b28
test_execve_invalid_env -> expectedFailureIfWindows
derlih 89ce647
test_execve_with_empty_path -> expectedFailureIfWindows
derlih 20d4ce3
test_fdopen -> expectedFailureIfWindows
derlih 96e7990
test_ftruncate -> expectedFailureIfWindows
derlih d605872
test_lseek -> expectedFailureIfWindows
derlih 8ce9105
test_read -> expectedFailureIfWindows
derlih bbac9bd
test_write -> expectedFailureIfWindows
derlih 3a3da4d
test_inheritable -> expectedFailureIfWindows
derlih 25336e9
test_kill_sigterm -> expectedFailureIfWindows
derlih f868751
test_kill_int -> expectedFailureIfWindows
derlih de530e2
test_create_junction -> expectedFailureIfWindows
derlih cd3a610
test_unlink_removes_junction -> expectedFailureIfWindows
derlih 5ba2f82
test_stat_unlink_race -> expectedFailureIfWindows
derlih 6e5a6f2
test_waitpid_windows -> expectedFailureIfWindows
derlih 8d4aa13
test_waitstatus_to_exitcode -> expectedFailureIfWindows
derlih 9e06414
test_waitpid -> expectedFailureIfWindows
derlih 612d9b7
test_urandom_fd_closed -> expectedFailureIfWindows
derlih a55b244
revert test_waitpid
derlih b8f8092
revert test_waitstatus_to_exitcode
derlih 1f96e52
test_waitstatus_to_exitcode_windows -> expectedFailureIfWindows
derlih 4c3d415
test_removed_dir -> expectedFailureIfWindows
derlih 0f162c8
test_removed_file -> expectedFailureIfWindows
derlih 25a452e
test_empty_path -> expectedFailureIfWindows
derlih c5bd446
test_get_set_inheritable -> expectedFailureIfWindows
derlih 0d1e40f
test_get_set_inheritable_badf -> expectedFailureIfWindows
derlih 4876af1
test_open -> expectedFailureIfWindows
derlih 955b480
test_dup_nul -> expectedFailureIfWindows
derlih 6ccb320
add NOTE to comment
derlih d87afe1
test_closerange -> skipIf
derlih 46b0813
test_fstat, test_fsync -> expectedFailureIfWindows
derlih 9ec3c6e
adjust comment `os.* not implemented` -> `os.* not implemented yet fo…
derlih a3baa13
add comment for generated tests in TestInvalidFD
derlih 746dec8
Revert adding test_os to platform independent tests
youknowone 0748b5d
revert skip test_os for windows
youknowone File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -292,7 +292,6 @@ jobs: | |
test_glob | ||
test_importlib | ||
test_io | ||
test_iter | ||
test_os | ||
test_pathlib | ||
test_posixpath | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is a big progress without regression. just failing on CI.
Because it is not easily fixed now, let's revert this like to skip test_os on windows, and merge it first.
And lets think about this fix later.