From c5bd671c5cf0c3c4ddf60a8ba90fad6b56b1fcc7 Mon Sep 17 00:00:00 2001 From: Claude Date: Sun, 6 Sep 2026 00:02:58 +0000 Subject: [PATCH] test(commons): drop commas from three commonTest names that break the iOS test compile Kotlin/Native rejects backtick identifiers containing "," ("Name contains illegal characters"), so `:commons:compileTestKotlinIosSimulatorArm64` has failed since #4059 moved these relay-group tests from the app's JVM-only test set into commonTest. JVM never minded, which is why the pre-push hook stayed green. Reworded the three names; behaviour unchanged. Co-Authored-By: Claude Fable 5.1 Claude-Session: https://claude.ai/code/session_01J836W9ZjSUJUQ1d23TpiJo --- .../channel/relayGroup/FilterRelayGroupsByAuthorsTest.kt | 2 +- .../channel/relayGroup/RelayGroupFilterBuildersTest.kt | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/commons/src/commonTest/kotlin/com/vitorpamplona/amethyst/commons/relayClient/channel/relayGroup/FilterRelayGroupsByAuthorsTest.kt b/commons/src/commonTest/kotlin/com/vitorpamplona/amethyst/commons/relayClient/channel/relayGroup/FilterRelayGroupsByAuthorsTest.kt index e69c49e62b..9f3ae2c7c1 100644 --- a/commons/src/commonTest/kotlin/com/vitorpamplona/amethyst/commons/relayClient/channel/relayGroup/FilterRelayGroupsByAuthorsTest.kt +++ b/commons/src/commonTest/kotlin/com/vitorpamplona/amethyst/commons/relayClient/channel/relayGroup/FilterRelayGroupsByAuthorsTest.kt @@ -52,7 +52,7 @@ class FilterRelayGroupsByAuthorsTest { } @Test - fun `builds the author-signed and p-tagged roster filters, authors sorted`() { + fun `builds the author-signed and p-tagged roster filters with authors sorted`() { val filters = filterRelayGroupsByAuthors(relayA, setOf(f2, f1), since = 7L, cachedChannels = emptyList()) assertEquals(2, filters.size) diff --git a/commons/src/commonTest/kotlin/com/vitorpamplona/amethyst/commons/relayClient/channel/relayGroup/RelayGroupFilterBuildersTest.kt b/commons/src/commonTest/kotlin/com/vitorpamplona/amethyst/commons/relayClient/channel/relayGroup/RelayGroupFilterBuildersTest.kt index 1f2dd0b17c..f7f85e742f 100644 --- a/commons/src/commonTest/kotlin/com/vitorpamplona/amethyst/commons/relayClient/channel/relayGroup/RelayGroupFilterBuildersTest.kt +++ b/commons/src/commonTest/kotlin/com/vitorpamplona/amethyst/commons/relayClient/channel/relayGroup/RelayGroupFilterBuildersTest.kt @@ -73,7 +73,7 @@ class RelayGroupFilterBuildersTest { // --- State (always-on): one #d filter per host relay, batching that relay's group ids --- @Test - fun `state batches d-filters per host relay, metadata and pins kept apart`() { + fun `state batches d-filters per host relay with metadata and pins kept apart`() { val filters = buildRelayGroupStateFilters(joined) { null } // two relays x (metadata + pins) assertEquals(4, filters.size) @@ -225,7 +225,7 @@ class RelayGroupFilterBuildersTest { // --- Open chat history (backward pager): only armed relays, each at its own until, all authors --- @Test - fun `history emits only armed relays at their until, all authors`() { + fun `history emits only armed relays at their until for all authors`() { val untilByRelay = mapOf(relayA to 200L) // relayB not armed → no cursor val filters = buildRelayGroupHistoryFilters(g1OnA, listOf(relayA, relayB), { untilByRelay[it] }, 50)