Skip to content

Conversation

@modocache
Copy link
Contributor

This is the second of six pull requests necessary to introduce asynchronous testing to swift-corelibs-xctest. @modocache will merge this once the CI passes.

#43 will add a dependency between swift-corelibs-xctest and swift-corelibs-foundation. On OS X, this will also necessitate an Xcode workspace that references both XCTest and Foundation .xcodeproj's.

To prevent CI from breaking when #43 is landed, preemptively add an Xcode workspace. Before landing #43, the Swift build script will be modified to build XCTest.xcworkspace, not XCTest.xcodeproj.

@modocache
Copy link
Contributor Author

Here's how the build script will need to change: modocache/swift@71f20f6

Notice that it builds the XCTest workspace.

swiftlang#43 will add a dependency
between swift-corelibs-xctest and swift-corelibs-foundation. On OS X, this
will also necessitate an Xcode workspace that references both XCTest
and Foundation .xcodeproj's.

To prevent CI from breaking when swiftlang#43 is landed, preemptively add an Xcode
workspace. Before landing swiftlang#43, the Swift build script will be modified to
build XCTest.xcworkspace, not XCTest.xcodeproj.
@modocache modocache force-pushed the xctestexpectation-workspace-migration branch from 889134b to a24d5fc Compare March 5, 2016 21:20
@briancroom
Copy link
Contributor

Cool!

There are a lot of steps to this dance.

@modocache modocache changed the title Preemptively add XCTest.xcworkspace Preemptively add XCTest.xcworkspace [AsyncXCTest 2/5] Mar 7, 2016
@modocache modocache self-assigned this Mar 7, 2016
@modocache modocache changed the title Preemptively add XCTest.xcworkspace [AsyncXCTest 2/5] Preemptively add XCTest.xcworkspace [AsyncXCTest 2/6] Mar 7, 2016
@mike-ferris
Copy link

@swift-ci please test

@mike-ferris
Copy link

@swift-ci Please test

modocache added a commit that referenced this pull request Mar 7, 2016
…ration

Preemptively add XCTest.xcworkspace [AsyncXCTest 2/6]
@modocache modocache merged commit 50d3823 into swiftlang:master Mar 7, 2016
@modocache modocache deleted the xctestexpectation-workspace-migration branch March 7, 2016 22:04
@modocache
Copy link
Contributor Author

The OS X failure is expected. I think @shahmishal is working on getting xcodebuild working with the latest Swift toolchain (which is necessary for #file and #line identifiers to compile).

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.

3 participants