Skip to content

Copy helpers internally - take 3 #8495

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bkhouri
Copy link
Contributor

@bkhouri bkhouri commented Apr 14, 2025

Until #8223 is fixed copy some helpers from
IntergrationTests/Source/IntegrationTests to
Test/_InternalTestSupport so we can re-use the functionality.

Related to: #8433
rdar://148248105
Requires: swiftlang/swift#80717

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 14, 2025

swiftlang/swift#80717

@swift-ci test

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 15, 2025

swiftlang/swift#80717

@swift-ci test windows

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 15, 2025

swiftlang/swift#80717

@swift-ci test macOS

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 15, 2025

@swift-ci test self hosted windows

bkhouri added a commit to bkhouri/swift-package-manager that referenced this pull request 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.

Depends on: swiftlang#8495
Related to: swiftlang#8433
rdar://148248105
@bkhouri bkhouri force-pushed the t/main/copy_helpers_2 branch from 1cc14bd to 3e145e2 Compare April 15, 2025 16:09
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 15, 2025

@swift-ci test

@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 added a commit to bkhouri/swift-package-manager that referenced this pull request 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.

Depends on: swiftlang#8495
Related to: swiftlang#8433
rdar://148248105
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 15, 2025

@swift-ci test

@bkhouri bkhouri changed the title Copy helpers internally Copy helpers internally - take 3 Apr 16, 2025
@bkhouri bkhouri force-pushed the t/main/copy_helpers_2 branch from 3e145e2 to ce35984 Compare April 16, 2025 16:57
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 16, 2025

@swift-ci test

Until swiftlang#8223 is fixed copy some helpers from
`IntergrationTests/Source/IntegrationTests` to
`Test/_InternalTestSupport` so we can re-use the functionality.

Related to: swiftlang#8433
rdar://148248105
@bkhouri bkhouri force-pushed the t/main/copy_helpers_2 branch from ce35984 to c0f4f8e Compare April 16, 2025 19:11
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 16, 2025

swiftlang/swift#80717

@swift-ci test

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 16, 2025

swiftlang/swift#80717

@swift-ci test self hosted windows

bkhouri added a commit to bkhouri/swift-package-manager that referenced this pull request Apr 16, 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.

Depends on: swiftlang#8495
Related to: swiftlang#8433
rdar://148248105
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 16, 2025

swiftlang/swift#80717

@swift-ci test windows

@bkhouri bkhouri assigned bkhouri and unassigned bkhouri Apr 17, 2025
bkhouri added a commit to bkhouri/swift-package-manager that referenced this pull request Apr 22, 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.

Depends on: swiftlang#8495
Related to: swiftlang#8433
rdar://148248105
bkhouri added a commit to bkhouri/swift-package-manager that referenced this pull request Apr 22, 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.

Depends on: swiftlang#8495
Related to: swiftlang#8433
rdar://148248105
@swiftlang swiftlang deleted a comment from CyberDigits-bot Apr 24, 2025
@swiftlang swiftlang deleted a comment from CyberDigits-bot Apr 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant