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

test を書く #26

Open
tsutaj opened this issue Mar 20, 2021 · 6 comments
Open

test を書く #26

tsutaj opened this issue Mar 20, 2021 · 6 comments
Labels
Priority: High Type: Testing Adding missing tests or correcting existing tests

Comments

@tsutaj
Copy link
Owner

tsutaj commented Mar 20, 2021

無いので

テスト方法

  • このリポジトリのどこかに、正解となる成果物を格納しておく
  • ss-manager run -o (md|html|pdf) で成果物を吐き出す
  • diff を取って一致するならテストが成功しているとみなす
    • difflib という標準モジュールがあるのでそれでできそう。

作業手順

@tsutaj
Copy link
Owner Author

tsutaj commented Apr 8, 2021

そろそろちゃんと書かないといけなくない?

手元の Linux でしか動かしてないけど Windows とか Mac で動く保証がまったくない

@tsutaj
Copy link
Owner Author

tsutaj commented Aug 28, 2021

バグをいろいろ埋め込んでしまった経験から test の重要性をまた感じた

せっかくだし pytest 使ってみたさはある。

@tsutaj tsutaj added Priority: Medium Type: Testing Adding missing tests or correcting existing tests and removed Priority: Low labels Aug 28, 2021
@tsutaj tsutaj added this to the v1.5.1 milestone Aug 28, 2021
@tsutaj
Copy link
Owner Author

tsutaj commented Aug 28, 2021

そもそもテストどうやって実行したらいいかな

  • このリポジトリのどこかに、正解となる成果物を格納しておく
  • ss-manager run -o (md|html|pdf) で成果物を吐き出す
  • diff を取って一致するならテストが成功しているとみなす

こんな感じになるか?

@tsutaj tsutaj modified the milestones: v1.7.1 or later, v2.0.0 or later Nov 23, 2022
@tsutaj
Copy link
Owner Author

tsutaj commented Jun 17, 2023

デグレの確認がそろそろ大変なので優先度を上げたい(すぐやるかと言われると・・・)

@tsutaj
Copy link
Owner Author

tsutaj commented Nov 3, 2023

PR を作ったり PR branch を push したときにカバレッジを出す:https://qiita.com/alkshmir/items/acd7ef40c0b9a622971d

@tsutaj
Copy link
Owner Author

tsutaj commented Nov 5, 2023

カバレッジの diff は diff-cover で見れるっぽい

カバレッジ周りの設定は、ある程度テストが用意できてからでよさそう

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: High Type: Testing Adding missing tests or correcting existing tests
Projects
None yet
Development

No branches or pull requests

1 participant