test: focus pytest summary on unexpected failures - #2203
Conversation
Use pytest's failure and error summary modes so expected-failure details do not obscure unexpected failures while retaining aggregate counts. Assisted-by: OpenAI Codex Signed-off-by: raisulchowdhury <34920788+raisulchowdhury@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Updates the project’s pytest default options to reduce verbose reporting of expected failures (xfail) so CI output makes unexpected failures easier to spot, addressing #1891.
Changes:
- Switch pytest “extra summary” flags from
-ra(all non-passing, including xfail details) to-rfE(failures + errors only). - Adjust the inline comment describing the summary mode.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| [tool.pytest.ini_options] | ||
| addopts = "--cov=git --cov-report=term -ra" | ||
| addopts = "--cov=git --cov-report=term -rfE" |
| # --cov-report html:path # html file at path | ||
| # --maxfail # number of errors before giving up | ||
| # -rfE # default test summary: list fail and error | ||
| # -rfE # test summary: list failures and errors, omitting expected failures |
Byron
left a comment
Byron
left a comment
There was a problem hiding this comment.
Thanks!
I do, however, think that the copilot comment should be looked into.
Include XPASS results while still omitting expected-failure detail, and document the exact report flags.\n\nAssisted-by: OpenAI Codex Signed-off-by: raisulchowdhury <34920788+raisulchowdhury@users.noreply.github.com>
|
Addressed the review request in commit 1d246f0:
Assisted-by: OpenAI Codex. The change and validation were reviewed before pushing. |
|
The Copilot points are addressed in |
Summary
Fixes #1891
Validation
The full GitPython suite was not run because its setup script mutates repository branches, tags, and reflogs; the focused reporter fixture directly exercises this configuration change.
Assisted-by: OpenAI Codex. AI assistance was used for investigation and implementation; the submitted diff and validation were reviewed before publication.