已合并
[fix]: 支持按版本装饰器过滤测试用例 #45273
thickhair创建于 14 天前
[fix]: 支持按版本装饰器过滤测试用例 #45273
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 thickhair 的贡献）
atomgit-bot
14 天前 评论：
14 天前 评论：
变更摘要
本 PR 调整 CI 版本过滤逻辑：在 ci/access_control/version_filter.py 中，_file_should_run 对「未包含任何 @runIfVersion 装饰器」的测试文件的处理策略由「保留（在所有版本上运行）」改为「丢弃（仅执行带版本装饰器的用例）」，即无装饰器文件不再通过 CI 版本范围过滤。其余改动仅为 test/_inductor/test_abs.py、test/distributed/test_all_to_all_single.py 中的空行调整，以及 test/utils/version_mark.py 文档字符串中导入示例的写法修正，均无业务逻辑影响。
主要改动
- 过滤规则反转:
ci/access_control/version_filter.py的模块级文档中，将「无@runIfVersion装饰器 -> 保留（在所有版本运行）」改为「无装饰器 -> 丢弃（仅版本装饰的用例运行）」 _file_should_run返回值变更: 当文件内未发现任何@runIfVersion装饰器时，函数由返回not found_any（即True）改为直接返回False，使无装饰器测试文件被过滤掉，且行为与「装饰器均不命中 CI 版本范围」一致- 导入示例修正:
test/utils/version_mark.py文档字符串中的用法示例由from test.utils.version_mark import runIfVersion改为from .utils.version_mark import runIfVersion（仅注释层面调整） - 空行调整:
test/_inductor/test_abs.py与test/distributed/test_all_to_all_single.py仅补充空行，不涉及逻辑变更
不准确？
ascend-robot
14 天前 评论：
14 天前 评论：
atomgit-bot
14 天前 评论：
14 天前 评论：
14 天前 添加了label：ascend-cla/yes
此处折叠了291条消息 查看更多
Jingwei Huang
13 天前 评论：
13 天前 评论：
/lgtm
chenrayray
13 天前 评论：
13 天前 评论：
/approve
13 天前 添加了label：approvedlgtm
13 天前 合入了pull request
ascend-robot
13 天前 评论：
13 天前 评论：
流水线 pytorch_gitcode_PR_multiVersion#14544 [ commitID：794290b2 ] 已完成
【合入来源】
【修改方案】
【资料变更】
【接口变更】
【功能验证】
【CheckList】