Skip Specs 功能的 `JH_related_spec` 问题
当前 Skip Specs 功能的 JH_related_spec
定位比较模糊,既可以写 JH 的替代测试也可以写描述:
但如果只写描述的话,这个名称看起来会有点奇怪,比如:
- file_path: 'ee/spec/features/google_analytics_datalayer_spec.rb'
JH_related_spec: 'skip google service in JH'
这里需要重新定一个清晰的规范:
- file_path: 'ee/spec/some_spec.rb'
jh_related_spec: 'jh/spec/some_spec.rb'
- file_path: 'ee/spec/some_no_need_spec.rb'
skip_reason: 'The function is not use in JH'
-
jh_related_spec
只写极狐的替代测试(支持数组) -
skip_reason
描述测试跳过的原因 -
jh_related_spec
和skip_reason
至少需要存在一个
具体任务
-
更新当前已存在的 JH_related_spec
不符合规范的用法 (jh/spec/config/skip_specs.yml
,jh/qa/qa/specs/config/*.yml
) -
更新文档:https://docs.gitlab.cn/jh/development/workflows/skip_upstream_test.html -
更换名称 JH_related_spec
为jh_related_spec
-
添加功能校验 jh_related_spec
必须是一个 jh 目录下存在的测试文件路径(支持数组)(jh/lib/jh/skip_specs.rb
) -
添加功能校验 jh_related_spec
和skip_reason
选项至少存在一个(jh/lib/jh/skip_specs.rb
) -
提醒 maintainer 该规则的更改,注意后续代码审查的问题
本地验证功能
- 现在配置中
jh/spec/config/skip_specs.yml
临时添加测试规则 - 触发两个功能校验,确认功能正常
- 移除测试的规则,提交功能代码到 MR
由 Song Huang 编辑于