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

MSVCのCMakeによる単体テストにリソースを埋め込む #1337

Conversation

berryzplus
Copy link
Contributor

@berryzplus berryzplus commented Jul 7, 2020

PR の目的

MSVCのCMakeによる単体テストにリソースを埋め込みます。

カテゴリ

  • プログラムの動作上の問題
    • ローカルビルド版
  • ビルド関連
    • ローカルビルド
  • その他の問題

PR の背景

#1275 単体テストで文字列リソースを利用できるようにする
 ↓
#1334 単体テストで x64 Debug で CDlgProfileMgr.TrySelectProfile_001 で assert になる

PR のメリット

PR のデメリット (トレードオフとかあれば)

  • 現状 build-and-test.bat はMinGWでしか使っておらず、MSVC版のテストプロジェクトはCMakeで生成しないので、 変更するメリットはそんなにないかもしれません。

仕様・動作説明

仕様変更はありません。

テストモジュールにリソースが埋め込まれていないとassertするコードを走らせているのに、リソースを埋め込まないテストプロジェクトが生成されてしまう余地があるので埋め込むように修正入れるだけです。

テスト内容

python.exeをインストールしてパスを通し、pipでopenpyxlを入れた端末で、build-and-test.bat でMSVC版のビルドとテスト実行を行います。(masterで#1334が再現する環境で、このPRをチェックアウトして再現しなくなることを確認します。)

テスト1

手順
tests/build-and-test.bat x64 Debug を実行する
 ↓
正常に実行できてassertダイアログが出なければOK。

PR の影響範囲

  • CMake経由でコンパイラにMSVCを使ったテストモジュールにリソースが埋め込まれます。

関連 issue, PR

#1275 単体テストで文字列リソースを利用できるようにする
#1334 単体テストで x64 Debug で CDlgProfileMgr.TrySelectProfile_001 で assert になる

参考資料

@AppVeyorBot
Copy link

@berryzplus
Copy link
Contributor Author

レビューありがとうございます。
とりあえずマージしときます。

@berryzplus berryzplus merged commit 851d652 into sakura-editor:master Jul 8, 2020
@berryzplus berryzplus deleted the feature/add_res_to_CMake_for_MSVC branch July 8, 2020 04:34
@beru beru added the UnitTest label Jul 18, 2020
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.

4 participants