Skip to content

build: Fix TestFoundation to build and run in Xcode #1038

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

Merged
merged 1 commit into from
Jun 13, 2017

Conversation

spevans
Copy link
Contributor

@spevans spevans commented Jun 12, 2017

  • The paths to the built frameworks needed updating so that
    TestFoundation will now correctly build in Xcode and link
    with XCTest.

  • Use the correct name to the xdgTestHelper which is different on
    Linux and macOS.

  • The test HTTP server now listens on 127.0.0.1 instead of 0.0.0.0
    so that the firewall dialog doesnt popup on macOS if the firewall
    is enabled.

This PR works in conjunction with swiftlang/swift-corelibs-xctest#192 for swift-corelibs-xctest

This will fix building SwiftFoundation and then running TestFoundation in Xcode, correctly linking to SwiftXCTest. It requires the xctest PR listed above to work. Some of the tests are still broken on macOS but this should make it easier to fix them now. Also one bug was found by the malloc debugger so that fix is included here.

Tested on Xcode 8.3.3 and 9.0 beta with 2017-06-11 nightly toolchain.

- The paths to the built frameworks needed updating so that
  TestFoundation will now correctly build in Xcode and link
  with XCTest.

- Use the correct name to the xdgTestHelper which is different on
  Linux and macOS.

- The test HTTP server now listens on 127.0.0.1 instead of 0.0.0.0
  so that the firewall dialog doesnt popup on macOS if the firewall
  is enabled.
@parkera
Copy link
Contributor

parkera commented Jun 13, 2017

@swift-ci test and merge

@swift-ci swift-ci merged commit d8f78f9 into swiftlang:master Jun 13, 2017
@ianpartridge ianpartridge mentioned this pull request Jul 28, 2017
@spevans spevans deleted the pr_osx_tests branch August 9, 2017 06:39
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