feat: Add Android+Skia+NativeAOT startup instrumentation test to CI by Copilot · Pull Request #1244 · unoplatform/Uno.Gallery

Copilot · 2026-03-25T16:36:35Z

Original prompt

This section details on the original issue you should resolve

<issue_title>Update CI to test Android+Skia+NativeAOT</issue_title>
<issue_description>PR unoplatform/uno#22676 broke Android+Native AOT startup, and the "easiest" way to detect this would be for something downstream of unoplatform/uno to run the test.

For an integration test, we need something which uses NuGet packages.

Update the Uno.Gallery to contain a new file: Uno.Gallery/Platforms/Android/MainInstrumentation.Android.cs

with the contents:
using Android.App;
using Android.OS;
using Android.Runtime;
using Android.Views;

namespace Uno.Gallery
{
	[Instrumentation(Name = "my.MainInstrumentation")]
	public class MainInstrumentation : Android.App.Instrumentation
	{
		public MainInstrumentation(IntPtr handle, JniHandleOwnership ownership)
		    : base(handle, ownership)
		{
		}

		public override void OnCreate(Bundle? savedInstanceState)
		{
			base.OnCreate(savedInstanceState);
			Finish(Result.Ok, null);
		}
	}
}
Update CI YAML scripts to:
Build Uno.Gallery for Skia + NativeAOT, by using:
dotnet publish -f net10.0-android -r android-x64 -p:TargetFrameworkOverride=net10.0-android -p:SkiaPublishAot=true '-p:ApplicationTitleVendorSuffix= (NAOT)' -p:ApplicationIdVendorSuffix=.naot -p:AndroidPackageFormat=apk -bl Uno.Gallery/*.csproj
Deploy the .apk from (1) to an Android emulator:
adb install Uno.Gallery/bin/Release/net10.0-android/android-x64/publish/com.nventive.uno.ui.demo.naot-Signed.apk
Launch the added instrumentation:
adb shell am instrument -w com.nventive.uno.ui.demo.naot/my.MainInstrumentation
Check the output of (3) contains:
INSTRUMENTATION_CODE: -1
What would you like to be added:

Why is this needed:

For which Platform:

iOS

Android

WebAssembly

WebAssembly renders for Xamarin.Forms

Windows

Build tasks

Anything else we need to know?

</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes Update CI to test Android+Skia+NativeAOT #1241

⚡ Quickly spin up Copilot coding agent tasks from anywhere on your macOS or Windows machine with Raycast.

Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com> Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/425ca194-89dd-45f6-83d0-750534424d9d

CLAassistant · 2026-03-25T16:46:52Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

jonpryor · 2026-03-25T18:22:15Z

+
+mkdir -p $BUILD_ARTIFACTSTAGINGDIRECTORY
+
+install_android_sdk() {


@copilot: this function is also defined in android-uitest-run.sh. You should split out this function into a common script, then source that script from both android-naot-instrumentation-run.sh and android-uitest-run.sh.

Done in d49f7d3. Extracted the SDK setup variables, cmdline-tools download, and install_android_sdk function into a new build/scripts/android-emulator-common.sh script. Both android-uitest-run.sh and android-naot-instrumentation-run.sh now source it before their respective emulator setup.

jonpryor · 2026-03-25T18:23:28Z

+# Wait for the emulator to finish booting
+source $BUILD_SOURCESDIRECTORY/build/scripts/android-uitest-wait-systemui.sh 500
+
+$ANDROID_HOME/platform-tools/adb devices


@copilot: everything involving environment variables should be always be quoted: "$ANDROID_HOME/platform-tools/adb"

Fixed in c0d1090 — all $ANDROID_HOME, $LATEST_CMDLINE_TOOLS_PATH, $BUILD_SOURCESDIRECTORY, $BUILD_ARTIFACTSTAGINGDIRECTORY, and other env-var-based paths are now quoted throughout the script.

Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com> Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/bb6f9c6d-ce75-4473-85e7-e5f1e07d3787

jonpryor · 2026-03-25T18:24:24Z

+IFS=$'\n\t'
+
+# echo commands
+set -x


@copilot: this set -x can be merged with the previous set command, as set -euox pipefail.

Fixed in c0d1090 — merged into set -euox pipefail and removed the separate set -x line and its comment.

jonpryor · 2026-03-25T18:26:16Z


 - job: Android_Tests_NativeAOT
-  displayName: 'Run Android+NativeAOT UI Tests'
+  displayName: 'Run Android+Skia+NativeAOT Instrumentation Test'


@copilot: stage-uitests-android.yml SHOULD NOT BE CHANGED.

New Instrumentation-related tests should be done in either a new .yml file or within azure-pipelines.yml.

Done in 81b7b5a. Reverted stage-uitests-android.yml to its original state. Created a new build/stage-uitests-android-naot.yml with the instrumentation build and run jobs, and added an Android_NativeAOT_Instrumentation_Tests stage in azure-pipelines.yml that references it.

…on-run.sh Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com> Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/6be1dffd-5c3e-4b2c-b8d7-99ee179cc921

…naot.yml and azure-pipelines.yml stage Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com> Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/d8bfa5f9-fb10-4120-b280-1e9ae6400392

unodevops · 2026-03-25T20:00:19Z

⚠️⚠️ The build 203748 has failed on Uno.Gallery - CI.

jonpryor · 2026-03-26T13:09:55Z

@copilot: rename build/stage-uitests-android-naot.yml to build/stage-tests-android.yml. They are not "UI" tests; they are on-device tests.

Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com> Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/a08f68de-fbcd-42a6-b122-fda1d1daf750

Copilot · 2026-03-26T13:12:00Z

@copilot: rename build/stage-uitests-android-naot.yml to build/stage-tests-android.yml. They are not "UI" tests; they are on-device tests.

Done in dec9239 — renamed to build/stage-tests-android.yml and updated the reference in azure-pipelines.yml.

unodevops · 2026-03-26T15:15:23Z

⚠️⚠️ The build 203946 has failed on Uno.Gallery - CI.

jonpryor · 2026-03-26T20:32:37Z

+	# Build the sample with Skia+NativeAOT enabled (no native rendering)
 	publish_extra+=("-m:1" "-p:SkiaPublishAot=true" "-p:ApplicationTitleVendorSuffix= (NAOT)" "-p:ApplicationIdVendorSuffix=.naot")
 	BINLOG_SUFFIX="-naot"
+	USE_NATIVE_RENDERING=false


@copilot: USE_NATIVE_RENDERING cannot be within this if [ "${NAOT:-0}" = "1" ] block, because we require Android native to do UI tests, and commit f31954e tests Android+Native+NativeAOT for UI tests.

Instead, the USE_NATIVE_RENDERING initialize, above, should instead provide an overridable default:

USE_NATIVE_RENDERING=${USE_NATIVE_RENDERING:-true}

Then build/stage-tests-android.yml can do:

NAOT=1 USE_NATIVE_RENDERING=false $(build.sourcesdirectory)/build/scripts/android-uitest-build.sh

Fixed in 7543269 — USE_NATIVE_RENDERING is now initialized with ${USE_NATIVE_RENDERING:-true} before the if block, and build/stage-tests-android.yml passes USE_NATIVE_RENDERING=false alongside NAOT=1.

Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/46e7cf37-9c66-476a-8156-5afc4e45d7d2 Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com>

jonpryor · 2026-03-26T20:49:47Z

+# Requires BUILD_SOURCESDIRECTORY to be set by the caller.
+
+# Override Android SDK tooling
+export ANDROID_HOME=$BUILD_SOURCESDIRECTORY/build/android-sdk


@copilot: as mentioned before, all use of environment variables must be quoted. This is so that if the environment variable value contains a space , it doesn't blow everything up.

export ANDROID_HOME="$BUILD_SOURCESDIRECTORY/build/android-sdk" export ANDROID_SDK_ROOT="$BUILD_SOURCESDIRECTORY/build/android-sdk" export LATEST_CMDLINE_TOOLS_PATH="$ANDROID_SDK_ROOT/cmdline-tools/latest" export CMDLINETOOLS=commandlinetools-mac-8512546_latest.zip mkdir -p "$ANDROID_HOME"

ALL ENVIRONMENT VARIABLE USAGE MUST BE QUOTED.

Fixed in 8d18e85 — all environment variable usages in android-emulator-common.sh are now quoted, including the export assignments, mkdir, wget, unzip, rm, mv, conditionals, touch, and sdkmanager invocations.

Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/7d1b4334-dbaa-4fbe-8112-7d800c642830 Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com>

jonpryor · 2026-04-01T19:19:03Z

Superseded by #1245.

Initial plan

686b608

Copilot AI assigned Copilot and jonpryor Mar 25, 2026

Copilot started work on behalf of jonpryor March 25, 2026 16:36 View session

feat: Update CI to test Android+Skia+NativeAOT with instrumentation

668a54c

Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com> Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/425ca194-89dd-45f6-83d0-750534424d9d

Copilot AI changed the title ~~[WIP] Update CI to test Android+Skia+NativeAOT~~ feat: Add Android+Skia+NativeAOT startup instrumentation test to CI Mar 25, 2026

Copilot AI requested a review from jonpryor March 25, 2026 16:45

Copilot finished work on behalf of jonpryor March 25, 2026 16:45

jonpryor reviewed Mar 25, 2026

View reviewed changes

Copilot started work on behalf of jonpryor March 25, 2026 18:22 View session

jonpryor reviewed Mar 25, 2026

View reviewed changes

refactor: extract install_android_sdk into android-emulator-common.sh

d49f7d3

Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com> Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/bb6f9c6d-ce75-4473-85e7-e5f1e07d3787

jonpryor reviewed Mar 25, 2026

View reviewed changes

Copilot AI requested a review from jonpryor March 25, 2026 18:25

Copilot finished work on behalf of jonpryor March 25, 2026 18:25

Copilot started work on behalf of jonpryor March 25, 2026 18:25 View session

jonpryor reviewed Mar 25, 2026

View reviewed changes

fix: quote env vars and merge set flags in android-naot-instrumentati…

c0d1090

…on-run.sh Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com> Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/6be1dffd-5c3e-4b2c-b8d7-99ee179cc921

Copilot AI requested a review from jonpryor March 25, 2026 18:29

Copilot finished work on behalf of jonpryor March 25, 2026 18:29

Copilot started work on behalf of jonpryor March 25, 2026 18:29 View session

Copilot finished work on behalf of jonpryor March 25, 2026 18:34

Copilot started work on behalf of jonpryor March 26, 2026 13:10 View session

fix: rename stage-uitests-android-naot.yml to stage-tests-android.yml

dec9239

Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com> Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/a08f68de-fbcd-42a6-b122-fda1d1daf750

Copilot finished work on behalf of jonpryor March 26, 2026 13:15

jonpryor reviewed Mar 26, 2026

View reviewed changes

Copilot started work on behalf of jonpryor March 26, 2026 20:32 View session

fix: make USE_NATIVE_RENDERING overridable in android-uitest-build.sh

7543269

Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/46e7cf37-9c66-476a-8156-5afc4e45d7d2 Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com>

Copilot finished work on behalf of jonpryor March 26, 2026 20:35

Copilot AI requested a review from jonpryor March 26, 2026 20:35

jonpryor reviewed Mar 26, 2026

View reviewed changes

Copilot started work on behalf of jonpryor March 26, 2026 20:49 View session

fix: quote all environment variable usages in android-emulator-common.sh

8d18e85

Agent-Logs-Url: https://github.com/unoplatform/Uno.Gallery/sessions/7d1b4334-dbaa-4fbe-8112-7d800c642830 Co-authored-by: jonpryor <155958+jonpryor@users.noreply.github.com>

Copilot AI requested a review from jonpryor March 26, 2026 20:51

Copilot finished work on behalf of jonpryor March 26, 2026 20:51

jonpryor closed this Apr 1, 2026


		mkdir -p $BUILD_ARTIFACTSTAGINGDIRECTORY

		install_android_sdk() {

Conversation

Copilot AI commented Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What would you like to be added:

Why is this needed:

For which Platform:

Anything else we need to know?

Comments on the Issue (you are @copilot in this section)

Uh oh!

CLAassistant commented Mar 25, 2026

Uh oh!

jonpryor Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

jonpryor Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

jonpryor Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

jonpryor Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 25, 2026

Choose a reason for hiding this comment

Uh oh!

unodevops commented Mar 25, 2026

Uh oh!

jonpryor commented Mar 26, 2026

Uh oh!

Copilot AI commented Mar 26, 2026

Uh oh!

unodevops commented Mar 26, 2026

Uh oh!

jonpryor Mar 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

Uh oh!

jonpryor Mar 26, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

Uh oh!

jonpryor commented Apr 1, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Copilot AI commented Mar 25, 2026 •

edited

Loading