Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pytest のポリシーを考える #595

Open
meltingrabbit opened this issue May 24, 2023 · 6 comments
Open

pytest のポリシーを考える #595

meltingrabbit opened this issue May 24, 2023 · 6 comments
Assignees
Labels
documentation Improvements or additions to documentation priority::high priorityg high

Comments

@meltingrabbit
Copy link
Collaborator

概要

pytest のポリシーを考える

詳細

どういう思想,仮定で書くかをかんがえる

close条件

考えて,ドキュメントにしたら

@meltingrabbit meltingrabbit added documentation Improvements or additions to documentation priority::high priorityg high labels May 24, 2023
@meltingrabbit meltingrabbit self-assigned this May 24, 2023
@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented May 24, 2023

Draft

仮定

テスト全体スタート時

  • 起動直後の状態からスタートする

1 pytest スタート時

  • テスト全体は,起動直後の状態を仮定するが,その状態であることのアサーションは,冒頭で行う
    • 例:
      • 電源がOFFであること
      • 機能AがStop状態であること
      • など
  • これをいれないと,そもそもテストをすべき状態であることの保証や,前段の pytest が現在の pytest に悪影響を与えていないかのチェックができない

1 pytest 終了時

  • できるだけ pytest 開始時の状態を復元する(がmustでなはい)
  • 後続の pytest に悪影響のある状態は困るので,それは必ず復元する

@meltingrabbit
Copy link
Collaborator Author

FYI @sksat @chutaro pytestのポリシー考えてます.

@sksat
Copy link
Collaborator

sksat commented May 24, 2023

一連の pytest が一つの C2A のインスタンスを使い回していること自体をどうにかしたくはありつつ,一旦現状はそうなのでよいと思う

@sksat
Copy link
Collaborator

sksat commented May 24, 2023

それはそれとして,依存関係を持ったテストを書きたくなることもあったりはしないんだろうか,というのは気になる.あっても良いとは思っていて,その上である場合は命名などでのなんらかの明示をしたい,みたいな気持ち

@meltingrabbit
Copy link
Collaborator Author

依存関係を持ったテストを書きたくなることもあったりはしないんだろうか

それって 1 pytest にまとめるべき?(多分今そうなってる)

@sksat
Copy link
Collaborator

sksat commented May 24, 2023

まあそうか.少なくとも登場人物が1つの SILS な C2A だけならあくまで unit test に徹するべきだな.たぶんあるとうれしいかもしれないのは共通化した処理を切った上で「途中のここで落ちました~」みたいなやつで,別にそれはこまめに assert 書けばいいだけか.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation priority::high priorityg high
Projects
None yet
Development

No branches or pull requests

2 participants