Skip to content

runtests: log ignored but passed tests - #14457

Closed
vszakats wants to merge 2 commits into
curl:masterfrom
vszakats:tests-show-ignored-passed
Closed

vszakats wants to merge 2 commits into
curl:masterfrom
vszakats:tests-show-ignored-passed

Conversation

@vszakats

@vszakats vszakats commented Aug 8, 2024 •

Copy link
Copy Markdown
Member

List tests explicitly ignored, but passing. It can be useful to catch
ignore rules that are no longer necessary because the tests are not
failing anymore.

To not clutter the default view, hide this information under a GitHub
Actions foldable group.

Closes #14457

@vszakats
vszakats marked this pull request as draft August 8, 2024 12:43
@vszakats vszakats added the tests label Aug 8, 2024
@github-actions github-actions Bot added the script label Aug 8, 2024
@vszakats
vszakats force-pushed the tests-show-ignored-passed branch from 7e95675 to e1f8edd Compare August 8, 2024 12:54
@vszakats vszakats changed the title runtests: try to show ignored but passed tests runtests: show ignored but passed tests Aug 8, 2024
@vszakats

vszakats commented Aug 8, 2024

Copy link
Copy Markdown
Member Author

It's quite a bit of lines when ignoring by keywords like ~FTP, so I'm worried this is too verbose. Though the goal would be not have anything (unnecessarily) ignored, so ideally these lines should converge to zero.

@vszakats
vszakats marked this pull request as ready for review August 12, 2024 15:37
@vszakats vszakats changed the title runtests: show ignored but passed tests runtests: log ignored but passed tests Aug 13, 2024
@vszakats vszakats closed this in 1a444e3 Aug 13, 2024
@vszakats
vszakats deleted the tests-show-ignored-passed branch August 13, 2024 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant