已合并
fix:modify testcase #2030
已合并
quyh创建于 5月27日
quyh
quyh
5月27日

相关的Issue

https://gitcode.com/openharmony/distributeddatamgr_pasteboard/issues/1632

原因(目的、解决的问题等)

修正用例

描述(做了什么,变更了什么)

  • Feature or Bugfix:
  • Binary Source:NO

验证结果(新增、改动、可能影响的功能,建议贴上验证报告)

  • TDD结果:Pass
    image.png
  • XTS结果:NA
  • Pretest结果:NA

日志规范自检:

安全编码自检:

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 quyh 的贡献)
openharmony_ciopenharmony_ci成员
5月27日 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
5月27日 评论:

感谢提交 Pull Requests!如果您提交的PR已经开发完毕,请评论 "start build" 触发门禁,更多交互操作,请访问OpenHarmony社区支持命令清单。如果需要调整订阅PR、Issue的变更状态,请访问订阅链接。


Thanks for submitting the pull request. If your Pull Request has already been developed, you can leave a "start build" comment to trigger the gated system. For more commands, please visit OpenHarmony Command List. If you need to change the subscription of a Pull Request or Issue, please visit the link.

likedislike
openharmony_ciopenharmony_ci成员
5月27日 添加了label:dco检查成功
yangxiaodong
yangxiaodong
5月28日 评论:

✅ 🤖 AI 代码检视报告 ✅

总体评估: OK

问题统计:

  • 总问题数: 2
  • 严重问题: 0
  • 高危问题: 0

摘要:
这是一个测试用例的简单修改,交换了PatternDetection::Detect函数的两个布尔参数,变更范围小且仅涉及测试代码。

📊 详细报告
查看完整的审查详情,包括具体的问题描述、建议和代码位置:
🔗 查看详细报告


此评论由 OpenHarmony Insight 代码审查系统自动生成

likedislike
yangxiaodong
yangxiaodong5月28日进行代码检视1
services/test/unittest/src/pasteboard_pattern_test.cpp
@@ -483,3 +482,4 @@
482+ std::set<Pattern> result = PatternDetection::Detect(patternsToCheck, pasteData, false, true);
483483 ASSERT_EQ(result.count(Pattern::URL), 1);
484484 ASSERT_EQ(result.count(Pattern::NUMBER), 1);
485485}
yangxiaodong
yangxiaodong5月28日评论:

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

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

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

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

建议在PR描述中添加:

  1. 相关Issue编号
  2. 修改原因(如API参数顺序调整或测试逻辑修正)
  3. 验证结果(TDD/XTS/Pretest)
likedislike
yangxiaodong
yangxiaodong5月28日进行代码检视1
services/test/unittest/src/pasteboard_pattern_test.cpp
@@ -482,3 +482,3 @@
482- std::set<Pattern> result = PatternDetection::Detect(patternsToCheck, pasteData, true, false);
482+ std::set<Pattern> result = PatternDetection::Detect(patternsToCheck, pasteData, false, true);
483483 ASSERT_EQ(result.count(Pattern::URL), 1);
484484 ASSERT_EQ(result.count(Pattern::NUMBER), 1);
yangxiaodong
yangxiaodong5月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
quyhquyh
5月29日 推送  1 个提交:b5188e76-fix:modify testcase
openharmony_ci
openharmony_ci成员
5月29日 评论:

感谢提交 Pull Requests!如果您提交的PR已经开发完毕,请评论 "start build" 触发门禁,更多交互操作,请访问OpenHarmony社区支持命令清单。如果需要调整订阅PR、Issue的变更状态,请访问订阅链接。


Thanks for submitting the pull request. If your Pull Request has already been developed, you can leave a "start build" comment to trigger the gated system. For more commands, please visit OpenHarmony Command List. If you need to change the subscription of a Pull Request or Issue, please visit the link.

likedislike
quyhquyh
5月30日 修改了pull request 的描述
quyh
quyh
5月30日 评论:

start build

likedislike
openharmony_ci
openharmony_ci成员
5月30日 评论:

首次触发
门禁构建开始,包含静态检查、代码编译和测试【dayu200-codearts编译, arm64_virt编译, ohos_mini_tdd编译, master_inner_build编译, ohos-mini编译, dayu200编译, ohos-host编译, dayu200_tdd编译, part_compile编译, dayu200测试】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a1a916c64650f998b848d9b/runlist

likedislike
openharmony_ciopenharmony_ci成员
5月30日 添加了label:编译成功
openharmony_ciopenharmony_ci成员
5月30日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
5月30日 添加了label:冒烟测试成功
openharmony_ciopenharmony_ci成员
5月30日 通过测试
openharmony_ci
openharmony_ci成员
5月30日 评论:

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a1a916c64650f998b848d9b/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 dayu200 success success >>>
2 dayu200_tdd success NA >>>
3 part_compile success(IGNORE) NA >>>
4 master_inner_build success(IGNORE) NA >>>
5 dayu200-codearts success NA >>>
6 ohos-host success NA >>>
7 ohos-mini success NA >>>
8 arm64_virt failed(IGNORE)(compile failed) NA >>>
9 ohos_mini_tdd success NA >>>

likedislike
openharmony_ciopenharmony_ci成员
5月30日 删除了label:waiting_on_author
openharmony_ciopenharmony_ci成员
5月30日 添加了label:waiting_for_review
openharmony_ci
openharmony_ci成员
5月30日 评论:

您好,Committer @vb6174 @zhusiyuan2 @verystone ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @vb6174 @zhusiyuan2 @verystone . Please assign someone to review the PR. You can assign a reviewer by using the command "assign [@someone_id]", or you can comment "assign" to review the PR by yourself.

likedislike
quyhquyh
6月2日 修改了pull request 的描述
quyh
quyh
6月2日 评论:

start build

likedislike
openharmony_ciopenharmony_ci成员
6月2日 删除了label:编译成功
openharmony_ciopenharmony_ci成员
6月2日 删除了label:静态检查成功
openharmony_ciopenharmony_ci成员
6月2日 删除了label:冒烟测试成功
openharmony_ci
openharmony_ci成员
6月2日 评论:

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译和测试【ohos-host编译, part_compile编译, ohos-mini编译, master_inner_build编译, dayu200-codearts编译, dayu200编译, dayu200测试, arm64_virt编译, ohos_mini_tdd编译, dayu200_tdd编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a1e9a2e64650f998b165e52/runlist

likedislike
openharmony_ciopenharmony_ci成员
6月2日 添加了label:编译成功
openharmony_ciopenharmony_ci成员
6月2日 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
6月2日 添加了label:冒烟测试失败
openharmony_ci
openharmony_ci成员
6月2日 评论:

代码门禁未通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a1e9a2e64650f998b165e52/runlist

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 dayu200 success failed([05209] upgrade success, but the smoke testing has a critical problem) >>>
2 dayu200_tdd success NA >>>
3 part_compile failed(IGNORE)(compile failed) NA >>>
4 master_inner_build success(IGNORE) NA >>>
5 dayu200-codearts success(IGNORE) NA >>>
6 ohos-host success NA >>>
7 ohos-mini success NA >>>
8 arm64_virt failed(IGNORE)(skip build) NA >>>
9 ohos_mini_tdd success NA >>>

likedislike
openharmony_ciopenharmony_ci成员
6月2日 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
6月2日 添加了label:waiting_on_author
quyh
quyh
6月2日 评论:

start build

likedislike
openharmony_ciopenharmony_ci成员
6月2日 删除了label:编译成功
openharmony_ciopenharmony_ci成员
6月2日 删除了label:冒烟测试失败
openharmony_ci
openharmony_ci成员
6月2日 评论:

部分构建失败,仅触发失败构建
门禁构建开始,包含代码编译和测试【dayu200测试, arm64_virt编译, part_compile编译, dayu200编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a1ebccc64650f998b3564a9/runlist

likedislike
openharmony_ciopenharmony_ci成员
6月2日 添加了label:编译成功
openharmony_ciopenharmony_ci成员
6月2日 添加了label:冒烟测试成功
openharmony_ciopenharmony_ci成员
6月2日 通过测试
openharmony_ci
openharmony_ci成员
6月2日 评论:

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a1ebccc64650f998b3564a9/runlist

# Device build result test result package
1 dayu200 success success >>>
2 part_compile failed(IGNORE)(compile failed) NA >>>
3 arm64_virt (IGNORE) NA >>>

likedislike
openharmony_ciopenharmony_ci成员
6月2日 删除了label:waiting_on_author
openharmony_ciopenharmony_ci成员
6月2日 添加了label:waiting_for_review
openharmony_ci
openharmony_ci成员
6月2日 评论:

您好,Committer @vb6174 @zhusiyuan2 @verystone ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @vb6174 @zhusiyuan2 @verystone . Please assign someone to review the PR. You can assign a reviewer by using the command "assign [@someone_id]", or you can comment "assign" to review the PR by yourself.

likedislike
朱思远朱思远成员
6月2日 通过审查
朱思远
朱思远成员6月2日进行代码检视1
services/test/unittest/src/pasteboard_pattern_test.cpp
@@ -480,3 +480,3 @@
480480 record2->AddEntry(utdId2, plainEntry2);
481481 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
openharmony_ci
openharmony_ci成员
6月2日 评论:

未查询到检视人,无法提交检视意见。请先联系committer分配

likedislike
openharmony_ciopenharmony_ci成员
6月2日 关闭了关联的issue
openharmony_ciopenharmony_ci成员
6月2日 合入了pull request,合并节点 SHA:5449afb4cdf28513455af3c32bfbcc3bacf4b48c
openharmony_ciopenharmony_ci成员
6月2日 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
6月2日 添加了label:merged