zapstore-mirror/com.vitorpamplona.amethyst
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #3845 from vitorpamplona/claude/explained-filter-test-warnings-clpcv8

Browse Source 
Test: Type ExplainedFilter.copy() result as base Filter
This commit is contained in:
Vitor Pamplona
2026-08-01 14:08:56 -04:00
committed by GitHub
parent 31c3eaf2b3 0cc3f72f55
commit 7922f9a4d1
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
commons/src/jvmTest/kotlin/com/vitorpamplona/amethyst/commons/relayClient/subscriptions/ExplainedFilterTest.kt
+3 -1
View File
@@ -108,7 +108,9 @@ class ExplainedFilterTest {
*/ */
@Test @Test
fun `copy preserves the purpose`() { fun `copy preserves the purpose`() {
val advanced = explained().copy(since = 1_785_379_272) // Typed as the base Filter so the is-check below stays a runtime assertion — with the
// override's covariant return type inferred, the compiler would prove it true statically.
val advanced: Filter = explained().copy(since = 1_785_379_272)
assertTrue("copy() must stay an ExplainedFilter", advanced is ExplainedFilter) assertTrue("copy() must stay an ExplainedFilter", advanced is ExplainedFilter)
assertEquals(SubPurpose.NOTIFICATIONS, advanced.purposeOrNull()) assertEquals(SubPurpose.NOTIFICATIONS, advanced.purposeOrNull())