chore: Utilize app-runner for Android#2548
Conversation
Semver Impact of This PR🟢 Patch (bug fixes) 📋 Changelog PreviewThis is how your changes will appear in the changelog. This PR will not appear in the changelog. 🤖 This preview updates automatically when you update the PR. |
![sentry[bot]](https://avatars.githubusercontent.com/in/12637?s=60&v=4){kind=small}
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
test/Scripts.Integration.Test/Scripts/IntegrationOptionsConfiguration.cs
Outdated
Show resolved
Hide resolved
src/Sentry.Unity.Editor/ConfigurationWindow/SentryEditorWindowInstrumentation.cs
Show resolved
Hide resolved
| # api-level: [30, 31, 34] | ||
| api-level: [30] |
There was a problem hiding this comment.
I'll check and update the API levels in a followup.
There was a problem hiding this comment.
We need a tiny bump on the .NET SDK here that hardcodes isAOT = false to fix some false positives on older Unity versions.
| PlayerSettings.SetIl2CppCodeGeneration(NamedBuildTarget.FromBuildTargetGroup(group), UnityEditor.Build.Il2CppCodeGeneration.OptimizeSpeed); | ||
| #elif UNITY_2021_2_OR_NEWER | ||
| EditorUserBuildSettings.il2CppCodeGeneration = UnityEditor.Build.Il2CppCodeGeneration.OptimizeSize; | ||
| EditorUserBuildSettings.il2CppCodeGeneration = UnityEditor.Build.Il2CppCodeGeneration.OptimizeSpeed; |
There was a problem hiding this comment.
A more likely setting for actual release builds.
| #if !UNITY_EDITOR | ||
| t.ExpectMessage(currentMessage, "'op':'awake','description':'Main Camera.SmokeTester'"); // auto instrumentation | ||
| #endif |
There was a problem hiding this comment.
The current change from having a TestLauncher instead of having the MonoBehaviour already available during scene load makes this unfeasible. We can add this back in after the migration.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
There was a problem hiding this comment.
We missed setting the Dist until now.
| if (args.TryGetValue("dsn", out value)) | ||
| { | ||
| Debug.LogFormat("{0}: Setting DSN", functionName); | ||
| options.Dsn = value; | ||
| } | ||
|
|
There was a problem hiding this comment.
We're overwriting the DSN now from CI to protect the project and for later test validation via Pester.
There was a problem hiding this comment.
Extracted since these apply to all test cases, capture-event, capture-crash e.t.c.
There was a problem hiding this comment.
This is a temporary solution so we can use the old SmokeTester and the new IntegrationTester in parallel.
2 participants
Goal
On Android, this moves the the
Smoke Testsover toIntegration Tests, utilizingapp-runnerto have actual end to end coverage.Implementation
We only want to move the tests for Android specifically. To make use of the already existing infrastructure I introduced a
TestLauncherthat depending on the test case provided instantiates the correctMonoBehaviour. This has the side effect that theIntegrationTesteralso already compiles for all platforms, giving a positive signal, even tho it is not being used right now.#skip-changelog