-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Tests: Migrate PluginsBuildPlanTests to SwiftTesting and augment #9058
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
Tests: Migrate PluginsBuildPlanTests to SwiftTesting and augment #9058
Conversation
797f864
to
4b724e5
Compare
@swift-ci test self hosted |
4b724e5
to
cc31932
Compare
@swift-ci test self hosted |
cc31932
to
d05981c
Compare
@swift-ci test self hosted |
d05981c
to
107d103
Compare
@swift-ci test self hosted windows |
Retriggering build at the previous build failure was no longer available on Jenkins. @swift-ci test self hosted windows |
WIndows self hosted pipeline passed. Trigger all self hosted pipelines @swift-ci test self hosted |
107d103
to
2936bbc
Compare
@swift-ci test self hosted |
@swift-ci test self hosted windows |
@swift-ci smoke test |
@swift-ci smoke test |
Migrate the `PluginsBuildPlanTests` suite to Swift Testing and, where applicable, augment the test to run against both the Native and SwiftBuild build system, in addition to the `debug` and `release` build configurations. Depends on: swiftlang#9013 Relates to: swiftlang#8997 issue: rdar://157669245
2936bbc
to
c4b01ff
Compare
@swift-ci test |
@swift-ci test windows |
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.
LGTM, just a couple of nits!
swiftSDK: .hostSwiftSDK(environment: [:]), | ||
environment: [:] | ||
) | ||
// let hostTriple = try! hostToolchain.targetTriple.withoutVersion().tripleString |
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.
nit: was this accidentally left in?
localFileSystem.exists( | ||
fixturePath.appending(RelativePath(".build/\(hostTriple)/debug/plugintool-tool")) | ||
try await fixture(name: "Miscellaneous/Plugins/CommandPluginTestStub") { fixturePath in | ||
// let hostBinPath: AbsolutePath = fixturePath.appending(components: hostBinPathSegments) |
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.
nit: same as above
Migrate the
PluginsBuildPlanTests
suite to Swift Testing and, where applicable, augment the test to run against both the Native and SwiftBuild build system, in addition to thedebug
andrelease
build configurations.Depends on: #9013
Relates to: #8997
issue: rdar://157669245