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

Add percentage calculation to analysis tests #163

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aufi
Copy link
Member

@aufi aufi commented Jul 19, 2024

Adding calculation on success-rate of Got vs. Expected Issues and Incidents for analysis test and minor update of error output.

Everything below 100% is still considered as failure, just providing more background in info how much given analysis failed. Passing tests output is unchanged.

Example output:

--- FAIL: TestApplicationAnalysis (105.15s)
    --- FAIL: TestApplicationAnalysis/Tackle_Testapp_public (105.15s)
        --- FAIL: TestApplicationAnalysis/Tackle_Testapp_public/Issues_only__50_%__ (0.00s)

Adding calculation on success-rate of Got vs. Expected Issues and
Incidents for analysis test.

Everything below 100% is still considered
as failure, just providing more background in info how much given
analysis failed.

Signed-off-by: Marek Aufart <maufart@redhat.com>
@aufi aufi added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant