Skip to content

[spytest] Support junit-xml#1806

Merged
lolyu merged 1 commit intosonic-net:masterfrom
lolyu:spytest_junit
Jun 24, 2020
Merged

[spytest] Support junit-xml#1806
lolyu merged 1 commit intosonic-net:masterfrom
lolyu:spytest_junit

Conversation

@lolyu
Copy link
Copy Markdown
Collaborator

@lolyu lolyu commented Jun 23, 2020

Description of PR

Currently, run Spytest together with junit-xml option will result into
ValueError. This is because that Spytest tries to add user properties
that stores results for testcases related to the test function.
Temporarily removing this to enable junit-xml.

Signed-off-by: lolyu lolv@microsoft.com

Summary:
Fixes # (issue)

Type of change

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

Approach

run spytest with --junit-xml

What is the motivation for this PR?

How did you do it?

How did you verify/test it?

Any platform specific information?

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

Documentation

@lolyu lolyu force-pushed the spytest_junit branch 2 times, most recently from 72161d9 to 6e08e57 Compare June 23, 2020 11:45
Currently, run Spytest together with `junit-xml` option will result into
ValueError. This is because that Spytest tries to add user properties
that stores results for testcases related to the test function.
Temporarily removing this to enable `junit-xml`.

Signed-off-by: lolyu <lolv@microsoft.com>
@lolyu lolyu marked this pull request as ready for review June 23, 2020 11:48
@lolyu lolyu requested a review from yxieca June 23, 2020 11:48
@lgtm-com
Copy link
Copy Markdown

lgtm-com bot commented Jun 23, 2020

This pull request introduces 2 alerts when merging 311408e into 662d25b - view on LGTM.com

new alerts:

  • 2 for Unused local variable

@lolyu lolyu merged commit bfea307 into sonic-net:master Jun 24, 2020
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…-net#8769)

Included Commits
92d0744 [config reload] Removed job-mode for sonic.target restart (sonic-net#1820)
c28fb22 [ci]: Support PR coverage (sonic-net#1806)
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
sonic-swss commits

5fbd113 [Flex-counters] Fix the delay of flex counters flow to prevent infinite loop (sonic-net#1899)
dc685b6 [portsorch] Add an extra check before setting oper speed to APPL_DB (sonic-net#1885)
ceef728 Update port_rates & rif_rates lua scripts to convert poll_interval to MS (sonic-net#1855)

sonic-utilities commits

3160753 [ci]: Support PR coverage (sonic-net#1806)
3316fdb fix wrong code indent in sfputil (sonic-net#1808)
c33e3a8 [config reload] Fix config reload failure due to sonic.target job cancellation (sonic-net#1814)
4f7e107 [portconfig] Validate duplicate speed value and interface type value (sonic-net#1745)
59817e2 [warm-reboot] Add new preboot health check: verify database integrity (sonic-net#1785)
bf2ff3c [portstat, intfstat] added rates and utilization (sonic-net#1750)
3bf962c [show][platform] Revise chassis info fallback to only fall back on pmon crash (sonic-net#1751)
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
901e2efb4f5c6a1179068b43e6a837dfff19d9d9 (HEAD -> 201911, origin/201911) [fpmsyncd] Skip routes to eth0 or docker0 (sonic-net#1606)
6c6f4fbbeb24b6e16a2bffdee760ba6a95e2ceb3 configure extra inc/lib directory for build (sonic-net#1247)
cccb59efa031b817c1936a74db969debbb9b2775 [201911][Cherry-pick] [acl mirror action] Mirror session ref count fix at acl rule attachment (sonic-net#1896)
62dc36d9da988807c7a397fd90ac2859a5a07699 [ci]: Support Azure pipelines on 201911 branch (sonic-net#1806)

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants