已合并
fix:modify testcase #2030
已合并
quyh创建于 5月27日
共 2 个文件变更+2-1
@@ -50,6 +50,7 @@ int32_t PasteboardAbilityManager::CheckUIExtensionIsFocused(uint32_t tokenId, bo
50 MessageParcel data;50 MessageParcel data;
51 MessageParcel reply;51 MessageParcel reply;
52 MessageOption option;52 MessageOption option;
53+ 
53 bool ret = data.WriteInterfaceToken(ABILITY_MGR_DESCRIPTOR);54 bool ret = data.WriteInterfaceToken(ABILITY_MGR_DESCRIPTOR);
54 PASTEBOARD_CHECK_AND_RETURN_RET_LOGE(ret, ERR_INVALID_DATA, PASTEBOARD_MODULE_SERVICE,55 PASTEBOARD_CHECK_AND_RETURN_RET_LOGE(ret, ERR_INVALID_DATA, PASTEBOARD_MODULE_SERVICE,
55 "write interface token failed");56 "write interface token failed");
@@ -479,7 +479,7 @@ HWTEST_F(PasteboardPatternTest, DetectPlainTextTest001, TestSize.Level1)
479 );479 );
480 record2->AddEntry(utdId2, plainEntry2);480 record2->AddEntry(utdId2, plainEntry2);
481 pasteData.AddRecord(record2);481 pasteData.AddRecord(record2);
482- std::set<Pattern> result = PatternDetection::Detect(patternsToCheck, pasteData, true, false);482+ std::set<Pattern> result = PatternDetection::Detect(patternsToCheck, pasteData, false, true);
朱思远
朱思远朱思远6月2日

lgtm

likedislike
483 ASSERT_EQ(result.count(Pattern::URL), 1);483 ASSERT_EQ(result.count(Pattern::URL), 1);
484 ASSERT_EQ(result.count(Pattern::NUMBER), 1);484 ASSERT_EQ(result.count(Pattern::NUMBER), 1);
yangxiaodong
yangxiaodongyangxiaodong5月28日

🤖 AI 代码检视意见 | ✨ Quality | ✅ Low | 行号区间: L482-L484

问题描述: PatternDetection::Detect函数的参数从(true, false)变更为(false, true),但测试断言保持不变,仍期望检测到URL和NUMBER两种模式。需要确认:1) 这两个参数的具体含义;2) 参数变更后断言预期结果是否仍然正确。

💡 查看修复建议(点击展开)

修改建议:确认PatternDetection::Detect函数第3、4个参数的语义,验证参数交换后测试预期是否符合实际行为。如有必要,添加注释说明参数含义。

// 参数说明:第3个参数控制xxx,第4个参数控制yyy
std::set<Pattern> result = PatternDetection::Detect(patternsToCheck, pasteData, false, true);
ASSERT_EQ(result.count(Pattern::URL), 1);
likedislike
485}485}
yangxiaodong
yangxiaodongyangxiaodong5月28日

🤖 AI 代码检视意见 | ✨ Quality | ✅ Low | 行号区间: L479-L485

问题描述: 该PR的描述部分几乎为空,未说明修改测试用例的原因、目的或验证结果。参数从(true, false)改为(false, true)的逻辑变更没有文档说明,这会影响代码审查效率和后续维护。

💡 查看修复建议(点击展开)

修改建议:补充完整的PR描述,说明修改测试参数的原因,例如是否因为API接口变更或测试用例本身存在问题。

建议在PR描述中添加:

  1. 相关Issue编号
  2. 修改原因(如API参数顺序调整或测试逻辑修正)
  3. 验证结果(TDD/XTS/Pretest)
likedislike