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

Update TFS reporter to correct format #1270

Merged
merged 2 commits into from Feb 18, 2024

Conversation

lwasylow
Copy link
Member

@lwasylow lwasylow commented Sep 24, 2023

Fixes #1081
Fixes #1257
As per notes on 1081 the new code will roll up context into suites.
And report empty test suites only when number of tests is equal 0.

Still some more tests to go for complex nesting.
@github-actions
Copy link
Contributor

github-actions bot commented Sep 24, 2023

Unit Test Results

       1 files         2 suites   5m 10s ⏱️
1 056 tests 1 052 ✔️ 4 💤 0
1 060 runs  1 052 ✔️ 8 💤 0

Results for commit d974ce3.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Sep 24, 2023

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Files Coverage Δ
source/reporters/ut_tfs_junit_reporter.tpb 94.28% <92.10%> (+1.30%) ⬆️

... and 1 file with indirect coverage changes

📢 Thoughts on this report? Let us know!.

@lwasylow lwasylow changed the title First checking. Update TFS reporter to correct format Sep 24, 2023
@sonarcloud
Copy link

sonarcloud bot commented Sep 29, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@jgebal jgebal merged commit eb8bb06 into develop Feb 18, 2024
20 of 21 checks passed
@jgebal jgebal deleted the feature/1257_missing_testcases_tfs_reporter branch February 18, 2024 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants