-
-
Notifications
You must be signed in to change notification settings - Fork 199
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
Pre Emulator Launch Script #247
Pre Emulator Launch Script #247
Conversation
Would this same behavior be possible by separating the command to create the emulator, and the command to run the tests with a Also, this seems less efficient than making a post-create-avd script to edit the config after the AVD is created. If you do this once, post-create, then you won't need to run it again if the emulator is cached. Post-create script would be able to be run in a emulator caching step. With this solution, you would need to run it every time, even if the emulator is cached. |
@nilsreichardt I suppose I do see the benefit of being able to call a script if using one instance of this action in your .yml @ychescale9 Can we merge this in? |
Sorry for the late response, I'll take a look at the PR this weekend. |
Co-authored-by: Yang <reactivecircus@gmail.com>
Just need to address this failed check |
Is fixed :) |
Some of these emulators have been so flakey lately when booting, and I don't think we have a way of rerunning failed runs unless @ychescale9 kicks them off. I'm trying to address it in my PR with a mechanism to comment "run tests" on a PR. Failed job: https://github.com/ReactiveCircus/android-emulator-runner/actions/runs/3328847468/jobs/5506688370 |
Thanks! |
Description
I added a new parameter
pre-emulator-launcher-script
(if you have better names, please let me know). With this parameter, you can pass a custom script which will be executed before launching the emulator (see #246 for more context).Tests
I haven't written any unit tests, but added the
pre-emulator-launcher-script
to the CI of 1 job, so that we have 1 job with the parameter and 1 without the parameter. You can see here the ouput: https://github.com/nilsreichardt/android-emulator-runner/runs/6196378411?check_suite_focus=true#step:9:97Related Tickets
Closes #246