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

Reinstate instrumentation tests using MacOS test boxes as a platform #2034

Merged
merged 14 commits into from
Jun 6, 2024

Conversation

Cawllec
Copy link
Contributor

@Cawllec Cawllec commented May 24, 2024

Goal

We've had to remove our instrumentation tests from github actions for a variety of reasons. This change adds a script which should add Android Virtual Devices and SDKs to our MacOS test boxes as required, using them to run the instrumentation tests. The version of android we're testing against is determined by the API_LEVEL environment variable, which should allow us to run against several different API levels if we see the need to.

Testing

It's successfully completed several test runs, and a negative test run to ensure the script failed successfully.

@Cawllec Cawllec requested a review from lemnik May 24, 2024 11:12
@bugsnagbot
Copy link
Collaborator

bugsnagbot commented May 24, 2024

Android notifier sizes

Format Size impact of Bugsnag (kB) Size impact of Bugsnag when Minified (kB)
APK 1852.07 1675.96
arm64_v8a 631.04 450.82
armeabi_v7a 565.51 385.29
x86 704.75 528.62
x86_64 676.09 495.86

Generated by 🚫 Danger

@lemnik
Copy link
Contributor

lemnik commented May 24, 2024

It's probably worth using the compressed option in the BK artifact plugin so that we just get a single zip file for the test reports.

Copy link
Contributor

@lemnik lemnik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Cawllec Cawllec force-pushed the tests/instrumentation-reinstatement branch 2 times, most recently from 86ccfff to ef9a54b Compare May 31, 2024 08:32
@Cawllec Cawllec force-pushed the tests/instrumentation-reinstatement branch from ef9a54b to 7c16ed1 Compare June 4, 2024 10:32
@Cawllec Cawllec merged commit d5ac07d into next Jun 6, 2024
26 checks passed
@Cawllec Cawllec deleted the tests/instrumentation-reinstatement branch June 6, 2024 09:59
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