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

Core tests not running on iOS on CI #6423

Open
1ec5 opened this issue Sep 22, 2016 · 11 comments
Open

Core tests not running on iOS on CI #6423

1ec5 opened this issue Sep 22, 2016 · 11 comments
Labels
build iOS Mapbox Maps SDK for iOS tests
Milestone

Comments

@1ec5
Copy link
Contributor

1ec5 commented Sep 22, 2016

Along with removing the macOS SDK unit tests (#5991), #5359 also removed the core tests from the iOS test rule. On CI, we’re still installing ios-sim but not using it anywhere. We should be running the core tests on iOS, one of the officially supported platforms.

/cc @jfirebaugh @kkaefer

@1ec5 1ec5 added iOS Mapbox Maps SDK for iOS tests labels Sep 22, 2016
@1ec5 1ec5 added this to the ios-v3.4.0 milestone Sep 22, 2016
@1ec5 1ec5 self-assigned this Sep 22, 2016
@boundsj boundsj self-assigned this Sep 28, 2016
@1ec5
Copy link
Contributor Author

1ec5 commented Sep 28, 2016

This is apparently blocked by ios-control/ios-sim#196.

@boundsj
Copy link
Contributor

boundsj commented Sep 28, 2016

Unfortunately our ios-sim does not appear to work with Xcode 8. I'm getting the error: Failed to find class DTiPhoneSimulatorSystemRoot at runtime. I'm going to take this off the 3.4.0 milestone and we can come back to it when there is more time and/or something moves in one of the ios-sim projects (i.e. ios-control/ios-sim#196)

@boundsj boundsj modified the milestones: ios-v3.4.0, ios-future Sep 28, 2016
@boundsj
Copy link
Contributor

boundsj commented Sep 28, 2016

Additional context for when we pick this up again:

A previous version of the Makefile used to run ios-sim launch $(IOS_OUTPUT_PATH)/Build/Products/$(BUILDTYPE)-iphonesimulator/ios-test.app --verbose --devicetypeid 'com.apple.CoreSimulator.SimDeviceType.iPhone-6,9.3' which corresponds to mbgl-test. This issue is to add back a rule to run that app in the Makefile and also as a step in the ios bitrise configuration.

@jfirebaugh
Copy link
Contributor

Supposedly it's possible to accomplish what we do with ios-sim via xcrun simctl commands, but the thing is not documented in any meaningful way, and neither @1ec5 nor I could figure out the right incantations.

@jfirebaugh
Copy link
Contributor

For posterity, some back-and-forth on radar issues I filed about simctl:

image

image

@jfirebaugh
Copy link
Contributor

They're not being run on macOS either, which is why we didn't catch #6579 (comment).

@jfirebaugh
Copy link
Contributor

Restored to macOS in 388fa9c.

@boundsj boundsj removed their assignment Aug 21, 2017
@friedbunny
Copy link
Contributor

friedbunny commented Sep 26, 2017

ios-sim was vestigial since #5359 and is being removed in #10070.

@1ec5
Copy link
Contributor Author

1ec5 commented Oct 3, 2017

Supposedly it's possible to accomplish what we do with ios-sim via xcrun simctl commands, but the thing is not documented in any meaningful way, and neither @1ec5 nor I could figure out the right incantations.

This command sends the application’s stdout to the console, but the return value is always 0 as long as the application successfully launches, even if the exit code is nonzero:

xcrun simctl install booted /path/to/mapbox-gl-native/build/ios/Debug-iphonesimulator/core-test.app
xcrun simctl launch --console booted 'com.mapbox.core-test'

@1ec5
Copy link
Contributor Author

1ec5 commented Oct 3, 2017

I suppose we can pipe stdout to another command that checks for signs of failure, similar to how xctool and xcpretty work.

@stale
Copy link

stale bot commented May 13, 2019

This issue 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 as completed May 13, 2019
@friedbunny friedbunny reopened this May 13, 2019
@stale stale bot removed the archived Archived because of inactivity label May 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
build iOS Mapbox Maps SDK for iOS tests
Projects
None yet
Development

No branches or pull requests

5 participants