Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: support extra global params per testsuite #326

Closed
wants to merge 1 commit into from

Conversation

mmagi
Copy link
Contributor

@mmagi mmagi commented Mar 25, 2023

Whether this PR is eventually merged or not, Sonic will thank you very much for your contribution.

无论此PR最终是否合并,Sonic组织都非常感谢您的贡献。

Checklist

  • The title starts with fix, fea, or doc. | 标题为fix、feat或doc开头。
  • I have checked that there are no duplicate pull requests with this request. | 我已检查没有与此请求重复的拉取请求。
  • I have considered and confirmed that this submission is valuable to others. | 我已经考虑过,并确认这份呈件对其他人很有价值。
  • I accept that this submission may not be used. | 我接受此提交可能不会被使用。

Description

支持为每个测试套件单独设置全局变量,优先级高于项目整体的全局变量,以便测试用例的复用
前端部分见SonicCloudOrg/sonic-client-web#238

@mmagi
Copy link
Contributor Author

mmagi commented Mar 25, 2023

另外留意到之前也有过类似的pr,但是没被接受,我的使用场景是有多个渠道包,希望可以按需要分别测,按不同频率设置定时任务等,虽然可以复制测试用例,但是复制之后后续修改变的很麻烦。增加了这个功能就不局限于复用公共步骤了,而是能够复用测试用例。

@ZhouYixun
Copy link
Member

ZhouYixun commented Mar 25, 2023 via email

@mmagi mmagi force-pushed the feat_testsuite_global_params branch from 4264422 to ec924f5 Compare March 30, 2023 15:42
@ZhouYixun ZhouYixun closed this Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants