Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Support multiple expectation failures with teamcity reporter #1251

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

Merged
merged 4 commits into from
Apr 28, 2023

Conversation

jgebal
Copy link
Member

@jgebal jgebal commented Apr 26, 2023

When running unit tests with TeamCity reporter in JetBrains IDE like IntelliJ or DataGrip, only first failed expectation is reported for a test, even in there are multiple failing expectations.

That is changed with this PR and so now all the failing expectations will be shown when running utPLSQL unit tests with DataGrip or IntelliJ

jgebal added 3 commits April 12, 2023 19:54
…tBrains IDEs)

The reporter will now report all errors and all failed expectations.
The test description will be used instead of test name.
@github-actions
Copy link
Contributor

github-actions bot commented Apr 26, 2023

Unit Test Results

       1 files         2 suites   2m 1s ⏱️
1 035 tests 1 031 ✔️ 4 💤 0
1 039 runs  1 031 ✔️ 8 💤 0

Results for commit 73e9cf3.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Apr 26, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.19 ⚠️

Comparison is base (8b01a39) 94.78% compared to head (73e9cf3) 94.59%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1251      +/-   ##
===========================================
- Coverage    94.78%   94.59%   -0.19%     
===========================================
  Files          109      109              
  Lines         5173     5257      +84     
  Branches        24       33       +9     
===========================================
+ Hits          4903     4973      +70     
- Misses         262      277      +15     
+ Partials         8        7       -1     
Flag Coverage Δ
11XE 94.18% <100.00%> (+0.06%) ⬆️
12.1EE 94.22% <100.00%> (+0.08%) ⬆️
18XE 94.40% <100.00%> (+0.02%) ⬆️
21XE 94.57% <100.00%> (+0.02%) ⬆️
23free 94.20% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
source/reporters/ut_teamcity_reporter_helper.pkb 91.66% <ø> (-1.82%) ⬇️
source/reporters/ut_teamcity_reporter.tpb 94.73% <100.00%> (+0.29%) ⬆️

... and 21 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@sonarqubecloud
Copy link

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 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@jgebal jgebal merged commit 320b3b0 into develop Apr 28, 2023
@jgebal jgebal deleted the feature/improvements_to_teamcity_reporter branch April 28, 2023 09:10
@jgebal jgebal added this to the v3.1.14 milestone May 21, 2023
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.

1 participant