Skip to content

Tests: Enable some BuildTests/**/*.swift tests on Windows #8512

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 16, 2025

  • Enable the BuildPlanTests on Windows
  • Split a PrepareForIndexTests tests into two withthe passing, and failing on Windows
  • update/add a message on some skipped windows tests, with a reference to GitHub Issue

Related to: #8433
rdar://148248105

- Enable the BuildPlanTests on Windows
- Split a PrepareForIndexTests tests into two withthe passing, and
  failing on Windows
- update/add a message on some skipped windows tests, with a reference
  to GitHub Issue

Related to: swiftlang#8433
rdar://148248105
@bkhouri bkhouri added windows test suite improvements to SwiftPM test suite labels Apr 16, 2025
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 16, 2025

@swift-ci test

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 16, 2025

@swift-ci test self hosted windows

@bkhouri bkhouri changed the title Tests: Enabled some BuildTests on Windows Tests: Enable some BuildTests/**/*.swift tests on Windows Apr 16, 2025
@@ -0,0 +1,33 @@
import argparse
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this be done in pure batch? This adds a python dependency to the tests, which might not be present on the host system.

Copy link
Contributor Author

@bkhouri bkhouri Apr 17, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It can. Could you please this with "request changes" so we don't loose track of this request?

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.

2 participants