[pytest] Adds test metadata to pytest xml report#1734
Merged
daall merged 1 commit intosonic-net:masterfrom Jun 8, 2020
Merged
Conversation
- Adds DUT data to test report - Adds testbed info to test report Signed-off-by: Danny Allen <daall@microsoft.com>
lguohan
approved these changes
Jun 5, 2020
tahmed-dev
approved these changes
Jun 5, 2020
Contributor
Author
|
retest this please |
Collaborator
|
Build triggered for merge commit. |
Collaborator
|
Build started for merge commit. |
kazinator-arista
pushed a commit
to kazinator-arista/sonic-mgmt
that referenced
this pull request
Mar 4, 2026
8b149a3 Load the database global_db only once for show cli (sonic-net#1712) cd0e560 [config][interface][speed] Fixed the config interface speed in multiasic issue (sonic-net#1739) b595ba6 [fast-reboot] revert the change of disabling counter polling before fast-reboot (sonic-net#1744) 8518820 [minigraph] Donot enable PFC watchdog for MgmtTsToR (sonic-net#1734) 2213774 [CLI][show][bgp] Fix the show ip bgp network command (sonic-net#1733) 3526507 [configlet] Python3 compatible syntax for extracting a key from the dict (sonic-net#1721) 5b56b97 [sonic_installer] don't print errors when installing an image not supporting app ext (sonic-net#1719) a581955 [LLDP] Fix lldpshow script to enable display multiple MAC addresses on the same remote physical interface (sonic-net#1657)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Danny Allen daall@microsoft.com
NOTE: Depends on #1732
Description of PR
Summary: Adds data about the test device and testbed setup to the XML test report.
Type of change
Approach
What is the motivation for this PR?
The default JUnit test report only includes the results of the individual test cases, so unless you can associate the test report with a specific test run using an external system like Jenkins there's a lot of missing context in the test report.
We also hope to use this data to make it possible to filter/query from a set of test results based on attributes like the SKU being tested, OS version, etc.
How did you do it?
I took advantage of the built-in
record_testsuite_propertyAPI in pytest to annotate the test report with this extra data.How did you verify/test it?
I ran a few test cases with the
--junit-xmlflag turned on and checked that the XML had the extra attributes included.Documentation