Suggestion: Using capturemanager.global_and_fixture_disabled
まだ誰も着手していません。
評価
調査の方向性
pytest_timeout.py の 511 行目付近から始め、global_and_fixture_disabled が定義されている pytest の capture.py の 819 行目付近と capturemanager の扱いを比較してください。ターミナルロギングの動作を再現し、そのうえで、その context manager への切り替えによってロジックが簡素化されるかどうかを判断してください。動作を理解し、回帰なしでロギングパスがカバーされれば完了です。
索引モデルが issue の本文から書いたものです。
説明
I came across some odd behavior when I was debugging something related to terminal logging in pytest.
And noticed pytest-timeout was disabling capturemanager and doing it's own printing
Ref: https://github.com/pytest-dev/pytest-timeout/blob/2.3.1/pytest_timeout.py#L511
I think the better approach here may be to use global_and_fixture_disabled which is a context manager to temporarily disable capturemanager and write your own terminal outputs while it is disabled
https://github.com/pytest-dev/pytest/blob/main/src/_pytest/capture.py#L819
Any thoughts on whether this would make the logging logic simpler here?
- 主要言語
- Python
- スター
- 258
- フォーク
- 69
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトの環境構築ファイルはまだ確認していません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
pytest-dev/pytest-timeout のほかの issue
-
PR #200 introduced breaking change - `timeout` value in config is incompatible with older versionsオープン
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
pytest-dev/pytest-timeout#203 · コメント 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
pytest-dev/pytest-timeout#190 · コメント 9 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
pytest-dev/pytest-timeout#189 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 35/100
pytest-dev/pytest-timeout#181 · リアクション 4 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 35/100
pytest-dev/pytest-timeout#178 · コメント 1 件 ·
pytest-dev/pytest-timeout の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
solana-foundation/pay-kit#341 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
nasa/python_cmr#123 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
EleutherAI/lm-evaluation-harness#4243 ·
メンテナーはふだん 1 日以内に返信
-
area: dashboard bug perceived difficulty: 3
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
Nitjsefnie-Harness-Commons/daedalus#1179 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
cusp-ai-oss/tojax#17 ·