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

[WIP] ユニットテストフレームワーク を googletest から Catch2 へ変更 #433

Closed
wants to merge 2 commits into from
Closed

[WIP] ユニットテストフレームワーク を googletest から Catch2 へ変更 #433

wants to merge 2 commits into from

Conversation

beru
Copy link
Contributor

@beru beru commented Sep 9, 2018

Catch2 に関しては、

https://github.com/catchorg/Catch2

を参照してください。

@m-tmatma
Copy link
Member

m-tmatma commented Sep 9, 2018

↑ まず乗り換えるメリットがあるのか、ないのかを調査するのが先だと思います

@beru
Copy link
Contributor Author

beru commented Sep 9, 2018

↑ まず乗り換えるメリットがあるのか、ないのかを調査するのが先だと思います

評価軸は色々とあると思いますが、 #427 では AppVeyor のビルドの実行時間を気にされていたようなのでまずはその観点で確認お願いできますか?

@m-tmatma
Copy link
Member

m-tmatma commented Sep 9, 2018

進め方の問題です。

#427 ではチケットの作成を依頼しました。
そのチケットの中で、現状の問題点と乗り換えることに対するメリット等を
考慮して乗り換えましょうという話になると思います。

その評価はまず言い出しっぺがやってほしいです。

それが難しいなら少なくとも、現状わかっていることだけでもチケットにまとめてほしいです。
いきなり PR しかも WIP なしはよろしくないです。

@beru
Copy link
Contributor Author

beru commented Sep 9, 2018

なるほど、そういう事だったんですね。説明ありがとうございます。

#434 を作成しました。

@beru beru changed the title ユニットテストフレームワーク を googletest から Catch2 へ変更 [WIP] ユニットテストフレームワーク を googletest から Catch2 へ変更 Sep 9, 2018
@beru beru added the UnitTest label Sep 10, 2018
@beru
Copy link
Contributor Author

beru commented Sep 17, 2018

#434 を Close したのでこちらも Close します。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants