getsentry / getsentry/sentry-python

Unpin Pytest on Redis tests

オープン
#3,715 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
Component: DX good first issue Improvement Python Quality Improvement
主要言語
Python
スター
2.2k
フォーク
669
平均マージ
1日 40分
マージ済み PR(30日)
212

説明

We currently pin `pytest` for our Redis tests:

https://github.com/getsentry/sentry-python/blob/bf400904245c3809bad5f20fd637408f519e7a15/tox.ini#L585

Unpinning `pytest` causes many of the Redis tests to fail. See the status checks on https://github.com/getsentry/sentry-python/pull/3703 for details.

It seems that these test failures are occurring because data is leaking between tests. When running just one of the failing tests, the test passes.

Let's investigate what changed in Pytest in version 8.0.0 to cause this change in behavior, and see whether we can remove the version pin.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

tox.ini#L585 から始め、pull request 3703 のステータスチェックで報告されている Redis のテスト失敗を確認します。失敗しているテストを個別に実行した場合と、pytest 8.0.0 で Redis のテストセット全体を実行した場合を比較し、その後、報告されているデータ漏洩を調査します。pytest の pin を削除でき、Redis のテストがパスすれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
testing-qa
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。