GeezerVison is showing github.com — Standalone coverage by jgebal · Pull Request #1079 · utPLSQL/utPLSQL
Conversation
jgebal
commented
Jun 13, 2020
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
Resolves #1025
Coverage can now be gathered outside of utPLSQL test run (without invoking ut.run)
Example:
begin
ut_runner.coverage_start('A6AA5B7361251CE6E053020011ACA078');
some_schema.some_package.procedure;
ut_runner.coverage_stop();
end;
/
In same session or even a separate session (once previous call was finished) run:
select *
from
table (
ut_coverage_cobertura_reporter( ).get_report(
ut_coverage_options(
coverage_run_id => 'A6AA5B7361251CE6E053020011ACA078',
include_objects => ut_varchar2_rows('SOME_PACKAGE'),
schema_names => ut_varchar2_rows('SOME_SCHEMA')
)
)
);
TODO
- Documentation
- Internal code cleanup and refactoring
- Test fixes
@ksawerykarwacki - feel free to give a try to the version from this feature branch.
Let me know if you encounter any issues or have any questions.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
…LSQL.
Enables multi-session coverage too.
Fixed failing tests for 11.2 and 12.1
Refactored some parts of tests
Added missing testcase for get_report_cursor
jgebal
commented
Jun 21, 2020
•
edited
Loading
Uh oh!
There was an error while loading. Please reload this page.
-
client_character_set needs to be added to get_report and get_report_cursor.
Currently it is missing.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
@pesse , @PhilippSalvisberg, @lwasylow
Any comments on the way the new API is built?
Can one of you review the documentation and provide some feedback before I merge?
I think it's a major addition to current API and would like to know that I got it right.
Also, it would be great to know that the documentation is sufficient.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
|
Merging as there are no review comments in 20 days.
|
Sorry, something went wrong.
Uh oh!
There was an error while loading. Please reload this page.
Reviewers
lwasylow
Awaiting requested review from lwasylow
pesse
Awaiting requested review from pesse
Uh oh!
There was an error while loading. Please reload this page.
Assignees
No one assigned
Development
Successfully merging this pull request may close these issues.
Use coverage without ut.run()
Uh oh!
There was an error while loading. Please reload this page.
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.
|