From b1dd2c81efa5eb15c51d0e607eb89a056d66bd37 Mon Sep 17 00:00:00 2001
From: Tim Perry <pimterry@gmail.com>
Date: Tue, 8 Jul 2025 15:03:42 +0200
Subject: [PATCH] Skip cert-but-no-unpinning test in Android <10

---
 test/android/test.ts | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/test/android/test.ts b/test/android/test.ts
index b1d1fde..40af9fd 100644
--- a/test/android/test.ts
+++ b/test/android/test.ts
@@ -227,7 +227,7 @@ describe('Test Android unpinning', function () {
 
     describe("given basic interception", () => {
 
-        beforeEach(async () => {
+        beforeEach(async function () {
             await launchFrida([
                 './test/android/tmp/config.js', // Our custom config
                 // Otherwise just the basic Android settings injection scripts to set the
@@ -235,6 +235,11 @@ describe('Test Android unpinning', function () {
                 './android/android-proxy-override.js',
                 './android/android-system-certificate-injection.js'
             ]);
+
+            // Android <10 uses X509TrustManager (not the cert stores hooked by system-certificate-injection)
+            // so this fails without the unpinning scripts - not really a problem in practice, but unhelpful
+            // for testing.
+            if (driver.capabilities['deviceApiLevel'] <= 28) return this.skip();
         });
 
         it("all unpinned requests should succeed, most others should fail", async () => {