Title: Detailed xfail output sometimes distracts from unexpected failures · Issue #1891 · gitpython-developers/GitPython · GitHub
Open Graph Title: Detailed xfail output sometimes distracts from unexpected failures · Issue #1891 · gitpython-developers/GitPython
X Title: Detailed xfail output sometimes distracts from unexpected failures · Issue #1891 · gitpython-developers/GitPython
Description: This issue is to track the problem noticed in #1853 and #1862 (see comments in both) where the specific details of new test failures are made less obvious in reading pytest output, in CI checks, due to the otherwise useful pytest 8 featu...
Open Graph Description: This issue is to track the problem noticed in #1853 and #1862 (see comments in both) where the specific details of new test failures are made less obvious in reading pytest output, in CI checks, du...
X Description: This issue is to track the problem noticed in #1853 and #1862 (see comments in both) where the specific details of new test failures are made less obvious in reading pytest output, in CI checks, du...
Opengraph URL: https://github.com/gitpython-developers/GitPython/issues/1891
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Detailed xfail output sometimes distracts from unexpected failures","articleBody":"This issue is to track the problem noticed in #1853 and #1862 (see comments in both) where the specific details of new test failures are made less obvious in reading `pytest` output, in CI checks, due to the otherwise useful pytest 8 feature to show more such details.\r\n\r\nFrom https://github.com/gitpython-developers/GitPython/pull/1853#issuecomment-1974005684:\r\n\r\n\u003e `pytest` output [now contains details](https://github.com/pytest-dev/pytest/issues/11233) of tests with xfail (expected failure) status, which are unrelated, and that may have distracted from the related test's failure. This [changed in major version 8 of `pytest`](https://docs.pytest.org/en/8.0.x/changelog.html#pytest-8-0-0rc2-2024-01-17) (which GitPython is using automatically except on Python 3.7 which doesn't support it). Overall, for most projects, it seems to me that this is an improvement, since it's useful to be able to see the specific details of xfailing tests without extra effort (and check if they really match up with the messages written in the xfail decorators). However, it may be that for GitPython's tests it makes common tasks more cumbersome. If so, then the `pytest` configuration can be changed (this does not require downgrading `pytest`).\r\n\r\nI'm unsure what the best solution is, but if necessary the `-ra` in\r\n\r\nhttps://github.com/gitpython-developers/GitPython/blob/53640535cf8314366a01da081947dd8504a299cd/pyproject.toml#L6\r\n\r\ncan be changed to something that does not imply `x` (xfail). The \"full list of available characters\" in the [Producing a detailed summary report](https://docs.pytest.org/en/8.0.x/how-to/output.html#producing-a-detailed-summary-report) section of the documentation documents how to specify which test statuses to emit detailed output for.\r\n\r\nI'm unsure of what should be done and I don't know of an easy way to eliminate the distracting effects while still having the benefit of seeing specific output of xfailing tests.","author":{"url":"https://github.com/EliahKagan","@type":"Person","name":"EliahKagan"},"datePublished":"2024-03-31T23:39:41.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":0},"url":"https://github.com/1891/GitPython/issues/1891"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:c01dbf45-b606-b69c-8d5f-e56dbc279223 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | E914:3F11A1:B5AF1B:F342D0:6968727F |
| html-safe-nonce | 13c8dd2d9b4dd95ecaf1bc0effa14aa1bbae33d521bb983c68147c881194004a |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJFOTE0OjNGMTFBMTpCNUFGMUI6RjM0MkQwOjY5Njg3MjdGIiwidmlzaXRvcl9pZCI6IjM5MjE5MzYwNzgwMDcyMDI0MzEiLCJyZWdpb25fZWRnZSI6ImlhZCIsInJlZ2lvbl9yZW5kZXIiOiJpYWQifQ== |
| visitor-hmac | 55dc2a9b486e6373db8bc1867892cd1c4943a74b3207bc668406973f5b2eb8a7 |
| hovercard-subject-tag | issue:2217249298 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/gitpython-developers/GitPython/1891/issue_layout |
| twitter:image | https://opengraph.githubassets.com/3a349de46016e01ba6605160b1977092184e11415eb5527da077140783a13c02/gitpython-developers/GitPython/issues/1891 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/3a349de46016e01ba6605160b1977092184e11415eb5527da077140783a13c02/gitpython-developers/GitPython/issues/1891 |
| og:image:alt | This issue is to track the problem noticed in #1853 and #1862 (see comments in both) where the specific details of new test failures are made less obvious in reading pytest output, in CI checks, du... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | EliahKagan |
| hostname | github.com |
| expected-hostname | github.com |
| None | 50f46dc2d6192249fd8ebf20e76c800f4f2596d4a5f3ab63dd63a754df154f54 |
| turbo-cache-control | no-preview |
| go-import | github.com/gitpython-developers/GitPython git https://github.com/gitpython-developers/GitPython.git |
| octolytics-dimension-user_id | 503709 |
| octolytics-dimension-user_login | gitpython-developers |
| octolytics-dimension-repository_id | 1126087 |
| octolytics-dimension-repository_nwo | gitpython-developers/GitPython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 1126087 |
| octolytics-dimension-repository_network_root_nwo | gitpython-developers/GitPython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | fef287f17234b4529a4b112a3d47fe8551e32ddd |
| ui-target | canary-2 |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width