fix(share): 认下字符串形态的 EXTRA_STREAM,并在一条内容都没读进来时如实告知
为什么字符串形态必须认 真实世界里 EXTRA_STREAM 有三种形态:规范推荐的 Uri(多选时 ArrayList<Uri>)、String[]、 ArrayList<String>。Android 上确实有一部分发送方(历史遗留 / 跨平台壳应用)在用后两种,而本端 此前完全认不出:getParcelableExtra 遇到 String[] 只往 logcat 打一句 ClassCastException 再回 null,getParcelableArrayListExtra 同理。实测 am start ... --esa android.intent.extra.STREAM uri1,uri2 投两张图进来就是这个样子。 为什么静默是 bug 上面那次分享的结局是:后端零请求、库里零新增、界面连一句软告知都没有。用户点了分享,没有任何 东西告诉他"没读进来"还是"在路上"——这是最糟的那种失败形态,所以本次一并消灭:带着内容迹象却 一条可用内容都没提取出来时,走既有的柔和通道说一句"这次分享的内容没能读进来,没有发送任何内容"。 EXTRA_TEXT 与 EXTRA_STREAM 都为空(用户确实没分享任何东西)时仍然保持静默,那条边界有测试钉住。 做了什么 - ShareIntake 改成读 EXTRA_STREAM 的原始值再按形态分派(Bundle.get 不做类型检查),Uri / Uri 数组 / String / CharSequence / 数组 / 列表一并摊平;顺带不再往 logcat 丢那两行 ClassCastException。 - 字符串形态只认 content://:那是本端新开的一条读入口,送进来的只是一段文本,没有任何框架级担保。 file:// 等于让发送方替本端指定一个本地路径、由本端拿自己的权限去读它;http(s):// 是网络地址, 收下它等于让第三方的一行文本指定一个 URL 由本应用代取。被拒的条目逐条跳过,不让整次分享失败。 - 优先顺序与去重不变:EXTRA_STREAM 的规范形态 -> 字符串兼容形态 -> ClipData(仅前者全为空时兜底), trim / 去空 / 去重,先来的先算;重复项既不算截断,也不把真图挤到上限之外。 - 「带着内容迹象」的判据取 EXTRA_STREAM 的原始值(由调用方读了原始值判出来,含形态本端认不出的那种), 不是「认下来的地址」——否则「有值但一条都认不出」恰恰会被判成「什么都没分享」而继续静默。 - SharedContent 新增 contentUnreadable 与 worthDelivering:后者是唯一判据,ShareIntake 用它决定 要不要往下接,shareNoticeFor 用它守卫"什么都没有就一个字都不说"那条边界,两处不会走岔。 - 分类(图片 vs 文档)、落 cache、截断与拷贝失败的既有语义一概不动。 测试 - ShareExtractionTest:字符串形态的顺序与逐条跳过、两种形态并存去重、scheme 白名单表、 读不进来 / 形态认不出 / 静默三条边界。 - ShareStreamExtrasTest(新):按真实对象验形态分派(String[] / ArrayList<String> / String / CharSequence / 认不出的形态不猜但照样算带着内容),并接到提取层验一次端到端。 - ShareNoticeTest(新):"没能读进来"成句且不说成发送失败、正常落地一个字不说、空白分享仍然静默。 assembleDebug 与 testDebugUnitTest 全绿(1084 tests / 0 failures / 0 errors)。
This commit is contained in:
AskaEth committed
1 parent
ef257c3167
commit
6cdf111419
6 files changed
+663
-29
No files matched your search
@@ -2,6 +2,7 @@ package top.yeij.cyrene.domain.share
|
||||
|
||||
import org.junit.Assert.assertEquals
|
||||
import org.junit.Assert.assertFalse
|
||||
import org.junit.Assert.assertNull
|
||||
import org.junit.Assert.assertTrue
|
||||
import org.junit.Test
|
||||
import top.yeij.cyrene.domain.chat.ChatSendRouting
|
||||
@@ -212,6 +213,137 @@ class ShareExtractionTest {
|
||||
assertEquals(listOf("content://media/external/images/media/1"), share.images)
|
||||
}
|
||||
|
||||
// —— 字符串形态的 EXTRA_STREAM（兼容读取）——
|
||||
|
||||
// 形态分派（`String` / `String[]` / `ArrayList<String>` 怎么变成下面这些字符串）在
|
||||
// `app/ShareStreamExtrasTest`——那一步要按**真实对象**判形态，放不进这个纯函数。
|
||||
// 这里管的是「拿到这些字符串之后怎么用」。
|
||||
|
||||
@Test
|
||||
fun `string form content uris are delivered in the order the sender gave them`() {
|
||||
// 实测事故：`am start … --esa android.intent.extra.STREAM uri1,uri2` 投两张图，
|
||||
// EXTRA_STREAM 是 `String[]`，getParcelableExtra / getParcelableArrayListExtra 双双回 null，
|
||||
// 于是整次分享什么都没发生（后端零请求、库里零新增、界面一句话都没有）。
|
||||
val share = ShareRules.extract(
|
||||
action = ShareRules.ACTION_SEND_MULTIPLE,
|
||||
streamStrings = listOf(
|
||||
"content://media/external/images/media/1",
|
||||
"content://media/external/images/media/2",
|
||||
),
|
||||
mimeOf = imageMime,
|
||||
)
|
||||
|
||||
assertEquals(
|
||||
listOf(
|
||||
"content://media/external/images/media/1",
|
||||
"content://media/external/images/media/2",
|
||||
),
|
||||
share.images,
|
||||
)
|
||||
assertEquals(0, share.truncatedImages)
|
||||
assertFalse(share.contentUnreadable)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `string form items with an unusable scheme are skipped one by one`() {
|
||||
val share = ShareRules.extract(
|
||||
action = ShareRules.ACTION_SEND_MULTIPLE,
|
||||
streamStrings = listOf(
|
||||
"content://media/external/images/media/1",
|
||||
"",
|
||||
" ",
|
||||
"javascript:alert(1)",
|
||||
"intent://scan/#Intent;scheme=zxing;end",
|
||||
"这只是一段纯文本",
|
||||
"file:///sdcard/DCIM/Camera/IMG_0001.jpg",
|
||||
"https://example.com/a.png",
|
||||
"/sdcard/DCIM/Camera/IMG_0001.jpg",
|
||||
"content://media/external/images/media/2",
|
||||
),
|
||||
mimeOf = imageMime,
|
||||
)
|
||||
|
||||
// 一条坏的不能把整次分享废掉。被拒的也不计进丢件计数：那说的是「对方给的东西读不到」，
|
||||
// 与「本端不收这个 scheme」不是一回事（后者连读都没试过）。
|
||||
assertEquals(
|
||||
listOf(
|
||||
"content://media/external/images/media/1",
|
||||
"content://media/external/images/media/2",
|
||||
),
|
||||
share.images,
|
||||
)
|
||||
assertEquals(0, share.truncatedImages)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `string form files go to the document path`() {
|
||||
val share = ShareRules.extract(
|
||||
action = ShareRules.ACTION_SEND,
|
||||
streamStrings = listOf("content://com.android.providers.downloads.documents/document/42"),
|
||||
mimeOf = textMime,
|
||||
)
|
||||
|
||||
assertTrue(share.images.isEmpty())
|
||||
assertEquals(
|
||||
listOf("content://com.android.providers.downloads.documents/document/42"),
|
||||
share.documents.map { it.uri },
|
||||
)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `a uri given in both forms is counted once and the uri form comes first`() {
|
||||
val share = ShareRules.extract(
|
||||
action = ShareRules.ACTION_SEND,
|
||||
stream = "content://media/external/images/media/1",
|
||||
streamStrings = listOf(
|
||||
"content://media/external/images/media/1",
|
||||
"content://media/external/images/media/2",
|
||||
),
|
||||
mimeOf = imageMime,
|
||||
)
|
||||
|
||||
assertEquals(
|
||||
listOf(
|
||||
"content://media/external/images/media/1",
|
||||
"content://media/external/images/media/2",
|
||||
),
|
||||
share.images,
|
||||
)
|
||||
// 去重发生在截断之前：重复项不该把真图挤掉，也不该被算成「截掉了 1 张」。
|
||||
assertEquals(0, share.truncatedImages)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `the string form only accepts content uris`() {
|
||||
assertEquals(
|
||||
"content://media/external/images/media/1",
|
||||
ShareRules.stringStreamUri("content://media/external/images/media/1"),
|
||||
)
|
||||
// 两侧空白是传输噪声，trim 掉再用（认它比因为一个空格而静默安全）。
|
||||
assertEquals("content://m/1", ShareRules.stringStreamUri(" content://m/1 "))
|
||||
// scheme 大小写不敏感（`Uri.parse` 也会把它归一成小写）。
|
||||
assertEquals("CONTENT://m/1", ShareRules.stringStreamUri("CONTENT://m/1"))
|
||||
// 这里只判 scheme 形态，不解析路径：这条确实读不到，会由落盘那一步如实计入丢件。
|
||||
assertEquals("content:not//a/uri", ShareRules.stringStreamUri("content:not//a/uri"))
|
||||
|
||||
// 下面这些**不能**当地址读。`file://` 是别人指定的本地路径（本端会拿自己的权限去读它，
|
||||
// 本应用私有目录也在里面）；`http(s)://` 是网络地址（收下等于让第三方指定一个 URL、
|
||||
// 由本应用代它去取）；其余是任意 scheme / 没有 scheme 的裸路径 / 纯文本。
|
||||
assertNull(ShareRules.stringStreamUri("file:///sdcard/DCIM/Camera/IMG_0001.jpg"))
|
||||
assertNull(ShareRules.stringStreamUri("http://example.com/a.png"))
|
||||
assertNull(ShareRules.stringStreamUri("https://example.com/a.png"))
|
||||
assertNull(ShareRules.stringStreamUri("javascript:alert(1)"))
|
||||
assertNull(ShareRules.stringStreamUri("intent://x/#Intent;end"))
|
||||
assertNull(ShareRules.stringStreamUri("android.resource://com.example/drawable/icon"))
|
||||
assertNull(ShareRules.stringStreamUri("/sdcard/DCIM/Camera/IMG_0001.jpg"))
|
||||
assertNull(ShareRules.stringStreamUri("这只是一段纯文本"))
|
||||
// scheme 的形态按 RFC 3986：首字符必须是字母，其后不能出现 `/`。
|
||||
assertNull(ShareRules.stringStreamUri("1content://m/1"))
|
||||
assertNull(ShareRules.stringStreamUri("a/b:c"))
|
||||
assertNull(ShareRules.stringStreamUri(""))
|
||||
assertNull(ShareRules.stringStreamUri(" "))
|
||||
}
|
||||
|
||||
// —— 文件（非图片）分流 ——
|
||||
|
||||
@Test
|
||||
@@ -373,4 +505,96 @@ class ShareExtractionTest {
|
||||
assertFalse(name, name.contains(".."))
|
||||
assertTrue(name, name.startsWith("share-"))
|
||||
}
|
||||
|
||||
// —— 读不进来 / 静默那条边界 ——
|
||||
|
||||
@Test
|
||||
fun `stream values that are all unusable are reported as unreadable instead of vanishing`() {
|
||||
val share = ShareRules.extract(
|
||||
action = ShareRules.ACTION_SEND_MULTIPLE,
|
||||
streamStrings = listOf("javascript:alert(1)", "这只是一段纯文本"),
|
||||
mimeOf = imageMime,
|
||||
)
|
||||
|
||||
assertTrue(share.isEmpty)
|
||||
assertTrue("有内容迹象却一条都没读进来，必须能说出口", share.contentUnreadable)
|
||||
assertTrue(share.worthDelivering)
|
||||
// 它连丢件计数都留不下（本端一条都没接收），所以这件事不能靠 droppedImages 来说。
|
||||
assertEquals(0, share.droppedImages)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `clip items without a uri are an indication too`() {
|
||||
// ClipData 里有条目、但只有纯文本条目（uri == null）：本端**不读** ClipData 里的文本，
|
||||
// 所以这也是一次「读不进来」，而不是「什么都没分享」。
|
||||
val share = ShareRules.extract(
|
||||
action = ShareRules.ACTION_SEND,
|
||||
clipUris = listOf(null),
|
||||
)
|
||||
|
||||
assertTrue(share.isEmpty)
|
||||
assertTrue(share.contentUnreadable)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `a share that carries nothing at all stays silent`() {
|
||||
// 边界：EXTRA_TEXT 没有值、EXTRA_STREAM 也没有值、ClipData 也没有条目——用户确实没分享
|
||||
// 任何东西，这时候说一句才是噪声。（`ShareIntake` 靠 worthDelivering 判要不要往下走，
|
||||
// `shareNoticeFor` 靠同一条判要不要说。）
|
||||
val share = ShareRules.extract(action = ShareRules.ACTION_SEND)
|
||||
|
||||
assertTrue(share.isEmpty)
|
||||
assertFalse(share.contentUnreadable)
|
||||
assertFalse(share.worthDelivering)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `a blank text with blank stream entries is still nothing`() {
|
||||
val share = ShareRules.extract(
|
||||
action = ShareRules.ACTION_SEND,
|
||||
text = " \n\t ",
|
||||
streamStrings = listOf(" ", ""),
|
||||
)
|
||||
|
||||
assertTrue(share.isEmpty)
|
||||
assertFalse("占位空白不是内容迹象", share.contentUnreadable)
|
||||
assertFalse(share.worthDelivering)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `unusable streams next to a usable text are not a whole read failure`() {
|
||||
val share = ShareRules.extract(
|
||||
action = ShareRules.ACTION_SEND,
|
||||
text = "看这个",
|
||||
streamStrings = listOf("javascript:alert(1)"),
|
||||
)
|
||||
|
||||
assertEquals("看这个", share.text)
|
||||
// 正文照发：这次分享不是「什么都没读进来」，就不能说成那一句。
|
||||
assertFalse(share.contentUnreadable)
|
||||
assertTrue(share.worthDelivering)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `a stream value whose shape we cannot decode is still an indication`() {
|
||||
// 调用方读了 EXTRA_STREAM 的原始值：里面确实躺着东西，只是形态本端认不出（本端**不猜**）。
|
||||
// 这也不能闭嘴——它是「分享点下去什么都没发生」的另一条缝。默认 false 保住上面那条静默边界。
|
||||
val share = ShareRules.extract(action = ShareRules.ACTION_SEND, streamPresent = true)
|
||||
|
||||
assertTrue(share.isEmpty)
|
||||
assertTrue(share.contentUnreadable)
|
||||
assertTrue(share.worthDelivering)
|
||||
}
|
||||
|
||||
@Test
|
||||
fun `an unrelated intent with a stream in it still extracts to nothing`() {
|
||||
// 不是分享 action 就不是分享：哪怕里面照样躺着地址，也不该走这条路（更不该出一条告知）。
|
||||
val share = ShareRules.extract(
|
||||
action = "android.intent.action.VIEW",
|
||||
streamStrings = listOf("content://media/external/images/media/1"),
|
||||
)
|
||||
|
||||
assertFalse(share.contentUnreadable)
|
||||
assertFalse(share.worthDelivering)
|
||||
}
|
||||
}
|
||||
Reference in new issue
Block a user