DISABLED テストのサンプルを追加する #943
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
コード修正しても codefactor の結果に反映されない |
|
codefactor が参照してるコミットが古いままになってる |
berryzplus
reviewed
Jun 5, 2019
| */ | ||
| class DISABLED_TestsImcomplte1 : public ::testing::Test | ||
| { | ||
|
|
There was a problem hiding this comment.
メモ: テストクラスのサンプル。未実装の機能をテストするクラスを、コードより先に書いてみるときに使う…。中身があった方がイメージはしやすいんだけど、これで十分かな？と思っています。
There was a problem hiding this comment.
これは Test Fixture と呼ばれるものです。
#944 でサンプル追加しました。
以下参考サイト
http://opencv.jp/googletestdocs/primer.html#primer-test-fixtures
| TEST_F(DISABLED_TestsImcomplte1, Test1) | ||
| { | ||
| /* 意図的に失敗させる */ | ||
| EXPECT_EQ(1 + 2, 4); |
There was a problem hiding this comment.
せっかく書いてもらったのに注文付けて恐縮ですが、(expected, actual)がいいです。
正確には、現時点で成立しないアサーションを記述したテストにdisableをくっ付けるはずです。(失敗させてるわけではない)
Closed
通った。 |
m-tmatma
force-pushed
the
feature/disabled-test-sample
branch
from
e726690
to
d1cf029
Compare
|
レビュープリーズ |
berryzplus
reviewed
Jun 15, 2019
|
|
||
| https://github.com/google/googletest/blob/9d4cde44a4a3952cf21861f9370b3bed9265dfd7/googletest/docs/advanced.md#temporarily-disabling-tests | ||
| */ | ||
| class DISABLED_TestsImcomplte1 : public ::testing::Test |
There was a problem hiding this comment.
スペルに違和感があったのでググってしまいました。
https://ejje.weblio.jp/content/incomplete
HoppingTappy
pushed a commit
to HoppingTappy/sakura
that referenced
this pull request
Jun 16, 2020
…est-sample DISABLED テストのサンプルを追加する
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
#877 で入れたテストがパスできるようになったので、DISABLED テストを削除するが
サンプルとしては残せるようにする。
カテゴリ
PR の背景
#877 で入れたテストが失敗したテストを #905 で 無効化したが
パスできるようになったので、有効化したいが DISABLED テストのサンプルを残したい。
PR のメリット
#877 で入れたテストをパスさせるようにした後でも DISABLED テストのサンプルを残す
PR のデメリット (トレードオフとかあれば)
なし
PR の影響範囲
関連チケット
参考資料
https://github.com/google/googletest/blob/9d4cde44a4a3952cf21861f9370b3bed9265dfd7/googletest/docs/advanced.md#temporarily-disabling-tests