CPPAクラスの自動テストを追加する #1817
Merged
CPPAクラスの自動テストを追加する #1817
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
本体改修なしで実現できるテストを実装。
|
Kudos, SonarCloud Quality Gate passed!
|
|
✅ Build sakura 1.0.4084 completed (commit 0bbc2f2c07 by @berryzplus) |
kengoide
reviewed
Mar 9, 2022
| TEST(CPPA, GetDllNameImp) | ||
| { | ||
| CPPA cPpa; | ||
| EXPECT_STREQ(L"PPA.DLL", cPpa.GetDllNameImp(0)); |
There was a problem hiding this comment.
要件上は大小文字を区別しなくて良いので STRCASEEQ が適当に思えます。このままでももちろん問題はありません。
There was a problem hiding this comment.
やってみました。
EXPECT_STRCASEEQ に変えるとビルドが通りません。
無理です。
このアサーションは本来、char* でも wchar_t* でも使えるように作ってあるはずです。
が、ここを書き換えたら wchar_t* を char* に変換できない というビルドエラーになりました。
gtestの不具合と考えられるのでフィードバックしたいですが、googletestは現在クローズド開発でC++11未対応の絶滅危惧種的コンパイラでもビルドできるようにする研究をしているらしいので無理っぽいです。（とても大事なことだと思いますが、一般ユーザーには縁のない話です。。。）
|
レビューありがとうございます。マージしちゃいます。 |
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 の目的
CPPAクラスのテストを追加します。
自動テスト完成までの道のりが長そうなので、本体改修なしで実現できるテストを先に追加してしまいたいです。
カテゴリ
PR の背景
#1722 で報告された CPPA::stdError の問題（？）に対処するため、自動テストを導入しようとしています。
自動テストを完成させるためには、本体側を改修しなくてはならないことがわかっています。
一括で導入するのが難しく、
かといって「少しずつ入れる」粒度も人によって基準がわかれそうなので、
いったん「本体側の改修を一切せずに導入できるテスト」を提案してみる次第です。
PR のメリット
PR のデメリット (トレードオフとかあれば)
仕様・動作説明
既存処理の正当性を担保するテストを追加します。
正当性の担保が目的なので、エラーになるケースは書いていません。
対象
基底クラス CDllImpl のインターフェースで、読込対象のDLL名（パス）を返します。
CPPAの公開static関数で、PPA.DLLに登録する関数名をフォーマットします。
DLL読み込み失敗時の挙動（≒CDllImpl派生クラスの共通動作）については、
CPPAクラスのテストとは観点が異なるのでこのPRのテストとは別に準備します。
PR の影響範囲
自動テストのテスト件数が増えます。
テスト内容
関連 issue, PR
参考資料