ci: fix coverage badge wf and omit test files from unit test coverage

[Bobbins228]

Issue link

What changes have been made

Verification steps

Run coverage run --omit="src/**/test_*.py,src/codeflare_sdk/common/utils/unit_test_support.py" -m pytest and verify the unit test files are not included when you run coverage report

Checks

• I've made sure the tests are passing.
• Testing Strategy
  • Unit tests
  • Manual tests
  • Testing is not required for this change

[Ygnas]

Verified works as expected.

[varshaprasad96]

/lgtm
/approve

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: varshaprasad96, Ygnas

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [varshaprasad96]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[akram]

thanks @Bobbins228 , this is somehting I have noticed while trying to push my PR; and it was biasing the coverage results.
Having now the rule global will make things better.
Also, we could have a .coveragerc file that has this to make tests also run the same way locally.

[Bobbins228]

A .coveragerc file could be good! Would help rather than having to make sure 2 workflows and documentation are kept updated. I will look into this thanks Akram 👍

[akram]

here is the .coveragerc that I use:

[run]
omit =
    src/**/test_*.py,src/codeflare_sdk/common/utils/unit_test_support.py

[report]
exclude_lines =
    pragma: no cover

and since I put it, unit tests appears in vscode now