Skip to content
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

Subliminal fully supports running tests on devices. #75

Merged
merged 2 commits into from
Aug 27, 2013

Conversation

wearhere
Copy link
Contributor

instruments will profile an application on a device only if that application is already installed on the device. This could and did run unnoticed if you already had the application you were testing installed on your device i.e. from Xcode. Now Subliminal (re)installs the application under test on the target device by using Apple's MobileDevice framework.

@wearhere
Copy link
Contributor Author

Rebased to pick up bb53b6b, which will fix the last build's failure.

Jeff Wear added 2 commits August 27, 2013 13:54
Using an interface to the private `MobileDevice.framework`.
`device-install` is modeled after the [`mobiledevice`](https://github.com/imkira/mobiledevice)
and [`fruitstrap`](https://github.com/ghughes/fruitstrap) libraries, but only supports
installing and uninstalling applications for simplicity and to limit its use of
`MobileDevice.framework`.
…eviously

having been installed.

`instruments` will install an application into the simulator, but not onto a device--
it will only profile an application that is already installed on a device.
`subliminal-test` now uses `device-install.m` to (re)install the application-under-test.
@wearhere
Copy link
Contributor Author

Updated, @aegolden.

@ghost ghost assigned aegolden Aug 27, 2013
aegolden added a commit that referenced this pull request Aug 27, 2013
Subliminal fully supports running tests on devices.
@aegolden aegolden merged commit 49deeb1 into master Aug 27, 2013
@aegolden aegolden deleted the jeff/device_support branch August 27, 2013 23:42
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.

2 participants