Skip to content

[tests] Capture device state on device-test failure#11832

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.

Sign up for GitHub

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

Merged
jonathanpeppers merged 4 commits into from
Jul 1, 2026
Merged

[tests] Capture device state on device-test failure #11832

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
344fe90
[tests] Capture device state on device-test failure
simonrozsival Jul 1, 2026
60d9c61
[tests] Re-check device attachment before capturing state
simonrozsival Jul 1, 2026
a3a69e2
Fix CA1305 in CaptureDeviceState
simonrozsival Jul 1, 2026
fcf354f
Merge branch 'main' into dev/simonrozsival/device-state-diagnostics
simonrozsival Jul 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
45 changes: 45 additions & 0 deletions src/Xamarin.Android.Build.Tasks/Tests/Xamarin.Android.Build.Tests/Utilities/DeviceTest.cs
View file
Open in desktop
Original file line numberDiff line numberDiff line change
Expand Up@@ -6,6 +6,7 @@
using System.Linq;
using System.Diagnostics;
using System.IO;
using System.Text;
using System.Text.RegularExpressions;
using System.Threading;
using System.Threading.Tasks;
Expand DownExpand Up@@ -159,11 +160,55 @@ protected override void CleanupTest ()
} else {
TestContext.WriteLine ($"{localUi} did not exist!");
}

CaptureDeviceState (outputDir);
}

base.CleanupTest ();
}

// Best-effort device-state snapshot captured on test failure so that on-device
// install/deploy failures can be classified from CI artifacts instead of guessed:
// connectivity (adb devices/get-state), disk pressure (df, dumpsys diskstats),
// storage-service readiness (dumpsys storaged - the StorageStatsManager NPE seen
// during install-create), boot completion, and how many test apps have piled up.
// See dotnet/android#11830.
static void CaptureDeviceState (string outputDir)
{
// Re-check attachment (the cached value can be stale if the device
// disconnected mid-test); otherwise each adb command below would wait
// the full timeout against an unresponsive device.
if (!IsDeviceAttached (refreshCachedValue: true)) {
TestContext.WriteLine ("No device attached; skipping device-state capture.");
return;
}

var sb = new StringBuilder ();
foreach (var (title, command) in new [] {
("adb devices -l", "devices -l"),
("adb get-state", "get-state"),
("getprop sys.boot_completed", "shell getprop sys.boot_completed"),
("getprop dev.bootcomplete", "shell getprop dev.bootcomplete"),
("df /data", "shell df /data"),
("df /storage/emulated/0", "shell df /storage/emulated/0"),
("dumpsys diskstats", "shell dumpsys diskstats"),
("dumpsys storaged", "shell dumpsys storaged"),
("pm list packages -3", "shell pm list packages -3"),
}) {
sb.AppendLine (CultureInfo.InvariantCulture, $"===== {title} =====");
sb.AppendLine (RunAdbCommand (command));
sb.AppendLine ();
}

string localState = Path.Combine (outputDir, "device-state-failed.log");
File.WriteAllText (localState, sb.ToString ());
if (File.Exists (localState)) {
TestContext.AddTestAttachment (localState);
} else {
TestContext.WriteLine ($"{localState} did not exist!");
}
}

protected int GetSdkVersion ()
{
var command = $"shell getprop ro.build.version.sdk";
Expand Down