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

Skip to content

Add COVERAGE_RUN_ID to utPLSQL context #1194

Merged
jgebal merged 1 commit intodevelopfrom
feature/coverage_run_id_into_context
Feb 26, 2022
Merged

Add COVERAGE_RUN_ID to utPLSQL context #1194
jgebal merged 1 commit intodevelopfrom
feature/coverage_run_id_into_context

Conversation

@jgebal
Copy link
Copy Markdown
Member

@jgebal jgebal commented Feb 26, 2022

This will allow for more accurate coverage reporting when running tests against the coverage module of the framework.

With this, in the test, we can get the current coverage_run_id and pass it to the JOB that runs utPLSQL in separate session.
After the JOB is finished, we can then copy the output from ut3_develop coverage tables into ut3 coverage tables.
That way, we will get a full picture of the coverage gathered in the course of utPLSQL test-run with coverage enabled.

Most of the setup is already in place in our tests. The only missing peace was the ability to tie the current ut3 test-run with the coverage gathered by jobs.
So now, after each test JOB, we can copy the coverage over to ut3.
I suggest using negative-id's when performing the copy from ut3_develop profiler/dbms_plsql_cov tables.
That way we can:

  • avoid conflicts on run_id
  • identify data from core run vs test-job run

…ate coverage reporting when runnign tests against the coverage module of the framework.
@jgebal jgebal added this to the 3.1.13 milestone Feb 26, 2022
@jgebal jgebal self-assigned this Feb 26, 2022
@jgebal jgebal changed the title Adding COVERAGE_RUN_ID to utPLSQL context Add COVERAGE_RUN_ID to utPLSQL context Feb 26, 2022
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 26, 2022

Unit Test Results

       1 files      2 suites   2m 20s ⏱️
   999 tests 995 ✔️ 4 💤 0
1 002 runs  994 ✔️ 8 💤 0

Results for commit b8a3fb6.

♻️ This comment has been updated with latest results.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 26, 2022

Codecov Report

Merging #1194 (b8a3fb6) into develop (b2d318c) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1194      +/-   ##
===========================================
+ Coverage    94.77%   94.79%   +0.02%     
===========================================
  Files          104      104              
  Lines         4879     4881       +2     
  Branches        22       22              
===========================================
+ Hits          4624     4627       +3     
  Misses         247      247              
+ Partials         8        7       -1     
Flag Coverage Δ
11XE 94.20% <100.00%> (+0.04%) ⬆️
12.1EE 94.27% <100.00%> (+0.11%) ⬆️
18XE 94.41% <100.00%> (+<0.01%) ⬆️
21XE 94.60% <100.00%> (+<0.01%) ⬆️

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

Impacted Files Coverage Δ
source/core/session_context/ut_session_info.tpb 98.71% <100.00%> (+0.03%) ⬆️
source/core/ut_utils.pkb 94.44% <0.00%> (+0.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9f229b8...b8a3fb6. Read the comment docs.

@sonarqubecloud
Copy link
Copy Markdown

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 db1c1d2 into develop Feb 26, 2022
@jgebal jgebal deleted the feature/coverage_run_id_into_context branch February 26, 2022 00:57
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.

1 participant