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

[Test Report] Optimize the test report to include xfail results #4765

Merged
merged 1 commit into from
Nov 26, 2021

Conversation

StormLiangMS
Copy link
Collaborator

Description of PR

Summary:
Fixes # (issue)
Optimize the test report to include xfail results

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911

Approach

What is the motivation for this PR?

By adding xfail marker, can't tell a real success or a xfail success, this PR is to optimize the test report function to include xfail results.

How did you do it?

Add 4 new result types to test case result, xfail_success, xfail_failure, xfail_error, xfail_skipped. And add one new filed xfails in test report summary.

How did you verify/test it?

Run test to generate test report to have different combinations of xfail cases.

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Copy link
Collaborator

@kevinskwang kevinskwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@StormLiangMS StormLiangMS merged commit f432e55 into master Nov 26, 2021
@StormLiangMS StormLiangMS deleted the storm/surface_xfail branch November 29, 2021 02:23
AntonHryshchuk pushed a commit to AntonHryshchuk/sonic-mgmt that referenced this pull request Jan 4, 2022
What is the motivation for this PR?
By adding xfail marker, can't tell a real success or a xfail success, this PR is to optimize the test report function to include xfail results.

How did you do it?
Add 4 new result types to test case result, xfail_success, xfail_failure, xfail_error, xfail_skipped. And add one new filed xfails in test report summary.

How did you verify/test it?
Run test to generate test report to have different combinations of xfail cases.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants