generated from actions/typescript-action
-
Notifications
You must be signed in to change notification settings - Fork 16
Open
Description
There is a parsing issue when including skipped tests. For example if we add
@pytest.mark.skip(reason="Not implemented yet")
def test_not_yet_implemented():
passThen the result is the following:
----------- coverage: platform linux, python 3.9.4-final-0 -----------
| Name | Stmts | Miss | Cover |
|---|---|---|---|
| .github/actions/docker-test/docker.py | 79 | 15 | 81% |
| .github/actions/another-test/another.py | 62 | 28 | 55% |
| TOTAL | 141 | 43 | 70% |
| ======================== | 17 | passed, | 3 |
Where instead of having an extra line about the number of tests passed and the time take it mashes it into the table.
I suspect this is caused by this line:
pytest-coverage-commentator/src/main.ts
Line 40 in aadbcfc
| } else if (lineOfText[i].indexOf('passed in') >= 0) { |
As 'passed in' doesn't occur when any tests are skipped or warnings appear. The raw output from pytest looks like one of the following:
==================== 18 passed, 6 warnings in 0.53 seconds =====================
=============== 18 passed, 3 skipped, 6 warnings in 0.58 seconds ===============
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels