From 0bfd2f8bc25bab37b4f157d1dc7076ef1980e928 Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 11 Jun 2026 01:20:59 +0000 Subject: [PATCH] test: move MarmotManagerLeaveRejoinTest to jvmTest so CI actually runs it MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The test lived in commons androidHostTest, but no CI workflow or pre-push task runs :commons:testAndroidHostTest — and running it manually fails before reaching any assertion: quartz's android PlatformLog actual hits unmocked android.util.Log stubs (NoSuchMethodError), since the source set is not configured with returnDefaultValues. The end-to-end leave/rejoin coverage was therefore never executed anywhere. :commons:jvmTest runs in CI and in the pre-push hook, already has the secp256k1 JVM bindings the test needs, and uses quartz's JVM logger. Verified green there alongside MarmotManagerRestoreTest. --- .../amethyst/commons/marmot/MarmotManagerLeaveRejoinTest.kt | 0 1 file changed, 0 insertions(+), 0 deletions(-) rename commons/src/{androidHostTest => jvmTest}/kotlin/com/vitorpamplona/amethyst/commons/marmot/MarmotManagerLeaveRejoinTest.kt (100%) diff --git a/commons/src/androidHostTest/kotlin/com/vitorpamplona/amethyst/commons/marmot/MarmotManagerLeaveRejoinTest.kt b/commons/src/jvmTest/kotlin/com/vitorpamplona/amethyst/commons/marmot/MarmotManagerLeaveRejoinTest.kt similarity index 100% rename from commons/src/androidHostTest/kotlin/com/vitorpamplona/amethyst/commons/marmot/MarmotManagerLeaveRejoinTest.kt rename to commons/src/jvmTest/kotlin/com/vitorpamplona/amethyst/commons/marmot/MarmotManagerLeaveRejoinTest.kt