test(android): cover UTF-16 notification boundaries

steipete · steipete · commit c647e7a2a363 · 2026-07-09T07:36:28.000+01:00
diff --git a/apps/android/app/src/test/java/ai/openclaw/app/node/DeviceNotificationListenerServiceTest.kt b/apps/android/app/src/test/java/ai/openclaw/app/node/DeviceNotificationListenerServiceTest.kt
@@ -128,12 +128,4 @@ class DeviceNotificationListenerServiceTest {
     assertTrue(limiter.allow(nowEpochMs = nowEpochMs, maxEventsPerMinute = 2))
     assertFalse(limiter.allow(nowEpochMs = nowEpochMs, maxEventsPerMinute = 2))
   }
-
-  @Test
-  fun sanitizeNotificationText_preservesUtf16BoundariesAtLimit() {
-    val prefix = "a".repeat(511)
-    val result = sanitizeNotificationText("$prefix🚀 trailing text")
-
-    assertEquals(prefix, result)
-  }
 }
diff --git a/apps/android/app/src/test/java/ai/openclaw/app/node/NotificationsHandlerTest.kt b/apps/android/app/src/test/java/ai/openclaw/app/node/NotificationsHandlerTest.kt
@@ -265,6 +265,18 @@ class NotificationsHandlerTest {
     assertTrue((sanitized ?: "").all { it == 'x' })
   }
 
+  @Test
+  fun sanitizeNotificationTextPreservesUtf16BoundariesAtLimit() {
+    val splitPairPrefix = "a".repeat(511)
+    assertEquals(splitPairPrefix, sanitizeNotificationText("$splitPairPrefix🚀 trailing text"))
+
+    val completePairPrefix = "a".repeat(510)
+    assertEquals(
+      "$completePairPrefix🚀",
+      sanitizeNotificationText("$completePairPrefix🚀 trailing text"),
+    )
+  }
+
   @Test
   fun notificationsActionClearablePolicy_onlyRequiresClearableForDismiss() {
     assertTrue(actionRequiresClearableNotification(NotificationActionKind.Dismiss))

Original file line number	Diff line number	Diff line change
`@@ -128,12 +128,4 @@ class DeviceNotificationListenerServiceTest {`
`128`	`128`	`assertTrue(limiter.allow(nowEpochMs = nowEpochMs, maxEventsPerMinute = 2))`
`129`	`129`	`assertFalse(limiter.allow(nowEpochMs = nowEpochMs, maxEventsPerMinute = 2))`
`130`	`130`	`}`
`131`		`-`
`132`		`- @Test`
`133`		`- fun sanitizeNotificationText_preservesUtf16BoundariesAtLimit() {`
`134`		`- val prefix = "a".repeat(511)`
`135`		`- val result = sanitizeNotificationText("$prefix🚀 trailing text")`
`136`		`-`
`137`		`- assertEquals(prefix, result)`
`138`		`- }`
`139`	`131`	`}`