Skip to content

Test: Enable most AsyncProcessTests #8500

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 2 commits into
base: main
Choose a base branch
from

Conversation

bkhouri
Copy link
Contributor

@bkhouri bkhouri commented Apr 15, 2025

Enable all but 3 tests.

Some AsyncProcessTests call to cat and echo commands. Update the
executable to be called correctly on Windows via the cmd.exe /c
command, where the cat equivalent is type, which displays the file
contents.

Also, many tests call a script. We created a batch file, which simply
calls the associated script invoked by python.

Only 3 tests remain skipped.

  • one failed assertion on Windows
  • two otheers appear to have caused swift test tp hang.

Related to: #8433
rdar://148248105

@bkhouri bkhouri added windows test suite improvements to SwiftPM test suite labels Apr 15, 2025
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 15, 2025

@swift-ci test self hosted windows

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 15, 2025

@swift-ci please test self hosted

@bkhouri bkhouri force-pushed the t/main/fix_echo_command_in_windows_tests_gh8433_rdar148248105 branch from 38a7f2f to a253651 Compare April 15, 2025 16:53
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 15, 2025

@swift-ci test self hosted windows

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 15, 2025

@swift-ci test windows

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 15, 2025

@swift-ci test

@bkhouri bkhouri force-pushed the t/main/fix_echo_command_in_windows_tests_gh8433_rdar148248105 branch from a253651 to 8312d6b Compare April 16, 2025 19:16
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 17, 2025

@swift-ci test

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 17, 2025

@swift-ci test self hosted windows

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 22, 2025

@swift-ci test

@bkhouri bkhouri force-pushed the t/main/fix_echo_command_in_windows_tests_gh8433_rdar148248105 branch from 8312d6b to c97ba8e Compare April 22, 2025 14:08
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 22, 2025

@swift-ci test

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 22, 2025

@swift-ci test macOS

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 22, 2025

@swift-ci test windows

Enable all but 3 tests.

Some AsyncProcessTests call to `cat` and `echo` commands.  Update the
executable to be called correctly on Windows via the `cmd.exe /c`
command, where the `cat` equivalent is `type`, which displays the file
contents.

Also, many tests call a script. We created a batch file, which simply
calls the associated script invoked by python.

Only 3 tests remain skipped.
- one failed assertion on Windows
- two otheers appear to have caused `swift test` tp hang.

Depends on: swiftlang#8495
Related to: swiftlang#8433
rdar://148248105
@bkhouri bkhouri force-pushed the t/main/fix_echo_command_in_windows_tests_gh8433_rdar148248105 branch from c97ba8e to f54edb2 Compare April 22, 2025 15:20
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 22, 2025

@swift-ci test

@bkhouri bkhouri force-pushed the t/main/fix_echo_command_in_windows_tests_gh8433_rdar148248105 branch from f54edb2 to e53fb23 Compare April 22, 2025 15:26
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 22, 2025

@swift-ci test

@bkhouri bkhouri enabled auto-merge (squash) April 22, 2025 15:52
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 22, 2025

@swift-ci test linux

@bkhouri bkhouri force-pushed the t/main/fix_echo_command_in_windows_tests_gh8433_rdar148248105 branch from e53fb23 to 909c874 Compare April 22, 2025 20:11
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 22, 2025

@swift-ci test

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 25, 2025

@swift-ci test self hosted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test suite improvements to SwiftPM test suite windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants