Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[ios, build] Expand iOS tests to more versions #10742

Closed
wants to merge 15 commits into from

Conversation

friedbunny
Copy link
Contributor

@friedbunny friedbunny commented Dec 19, 2017

/cc @akitchen @1ec5 @fabian-guerra

@friedbunny friedbunny added iOS Mapbox Maps SDK for iOS tests labels Dec 19, 2017
@friedbunny friedbunny self-assigned this Dec 19, 2017
@friedbunny friedbunny added the ⚠️ DO NOT MERGE Work in progress, proof of concept, or on hold label Dec 19, 2017
@friedbunny friedbunny force-pushed the fb-ios-beefier-tests branch 3 times, most recently from 9af5530 to 025d800 Compare December 19, 2017 03:12
@friedbunny
Copy link
Contributor Author

friedbunny commented Dec 19, 2017

Looks like CircleCI’s stock macOS image doesn’t have iOS 8 or 9 simulators installed, which is a bummer — I’ll look into ways to get them installed.

https://discuss.circleci.com/t/please-add-simulators-for-ios-9-10-to-xcode-9-image/16530

@friedbunny friedbunny force-pushed the fb-ios-beefier-tests branch 4 times, most recently from a79df2b to d83525d Compare December 21, 2017 17:57
@friedbunny friedbunny force-pushed the fb-ios-beefier-tests branch 2 times, most recently from 7d6fa15 to 12ecfc8 Compare January 5, 2018 01:03
@friedbunny
Copy link
Contributor Author

friedbunny commented Jan 5, 2018

OK, I’ve got this to a point where it’s running more or less properly. Remaining:

@friedbunny friedbunny force-pushed the fb-ios-beefier-tests branch 6 times, most recently from f659b97 to 0af0da2 Compare January 11, 2018 23:17
@friedbunny friedbunny force-pushed the fb-ios-beefier-tests branch from 0af0da2 to a53f963 Compare February 2, 2018 15:02
@friedbunny friedbunny force-pushed the fb-ios-beefier-tests branch 4 times, most recently from f0b30ef to d917d4b Compare February 14, 2018 01:55
@friedbunny friedbunny force-pushed the fb-ios-beefier-tests branch 2 times, most recently from 8c00a66 to 6dbffff Compare February 14, 2018 02:11
@friedbunny
Copy link
Contributor Author

friedbunny commented Feb 14, 2018

I rebased this to bring in the new iOS integration tests, but they’re not playing nicely with multiple simulators — even with a reduced set, most of the simulators fail to boot:

2018-02-14 10:08:27.057 xcodebuild[2641:24307]  iOSSimulator: task_name_for_pid(mach_task_self(), 3063, &task) returned 5
2018-02-14 10:08:27.061 xcodebuild[2641:24307]  iOSSimulator: Falling back to DISPATCH_SOURCE_TYPE_PROC
2018-02-14 10:08:27.067 xcodebuild[2641:23586]  iOSSimulator: task_name_for_pid(mach_task_self(), 3062, &task) returned 5
2018-02-14 10:08:27.068 xcodebuild[2641:23586]  iOSSimulator: Falling back to DISPATCH_SOURCE_TYPE_PROC
2018-02-14 10:08:27.729 xcodebuild[2641:19676] Error Domain=IDETestOperationsObserverErrorDomain Code=6 "Early unexpected exit, operation never finished bootstrapping - no restart will be attempted" UserInfo={NSLocalizedDescription=Early unexpected exit, operation never finished bootstrapping - no restart will be attempted}
2018-02-14 10:08:27.738 xcodebuild[2641:23596] Connection peer refused channel request for "dtxproxy:XCTestManager_IDEInterface:XCTestManager_DaemonConnectionInterface"; channel canceled <DTXChannel: 0x7f8ba6be05d0>
2018-02-14 10:08:27.747 xcodebuild[2641:19676] Error Domain=IDETestOperationsObserverErrorDomain Code=6 "Early unexpected exit, operation never finished bootstrapping - no restart will be attempted" UserInfo={NSLocalizedDescription=Early unexpected exit, operation never finished bootstrapping - no restart will be attempted}
2018-02-14 10:08:27.747 xcodebuild[2641:24306] Connection peer refused channel request for "dtxproxy:XCTestManager_IDEInterface:XCTestManager_DaemonConnectionInterface"; channel canceled <DTXChannel: 0x7f8ba728b310>

Testing failed:
	Test target integration encountered an error (Early unexpected exit, operation never finished bootstrapping - no restart will be attempted)
** TEST FAILED **

/cc @akitchen

@akitchen
Copy link
Contributor

I also encountered this failure locally on Xcode 9.2, on a late model MBP 13"

@friedbunny friedbunny force-pushed the fb-ios-beefier-tests branch from 0b59e93 to 755b57f Compare March 26, 2018 20:22
@friedbunny friedbunny changed the base branch from master to release-boba March 26, 2018 20:22
@friedbunny friedbunny changed the title [ios, build] Expand iOS tests to more versions, more sanitizers [ios, build] Expand iOS tests to more versions Mar 26, 2018
@friedbunny
Copy link
Contributor Author

Everything not related to getting multiple simulators to run is now moving in #11529.

@friedbunny friedbunny force-pushed the fb-ios-beefier-tests branch 2 times, most recently from c83516e to fc8f12c Compare March 26, 2018 22:44
@friedbunny friedbunny force-pushed the fb-ios-beefier-tests branch from fc8f12c to 91f13a8 Compare March 30, 2018 18:31
@1ec5
Copy link
Contributor

1ec5 commented Apr 27, 2018

#11776 removes support for deploying back to iOS 8.x, due in part to the challenges associated with this PR. Does this PR contain any work that’ll come in handy the next time we need to drop support for an iOS version?

@stale stale bot added the archived Archived because of inactivity label Dec 10, 2018
@stale
Copy link

stale bot commented Dec 10, 2018

This pull request has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this Dec 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
archived Archived because of inactivity ⚠️ DO NOT MERGE Work in progress, proof of concept, or on hold iOS Mapbox Maps SDK for iOS tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants