CShareDataのテストを追加する #1801
Merged
CShareDataのテストを追加する #1801
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.4046 completed (commit bf7eb00d8d by @berryzplus) |
|
レビューありがとうございます。マージしちゃいます。 |
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 の目的
テストカバレッジを向上させます。
カテゴリ
PR の背景
サクラエディタは日本語アプリです。
日本語アプリですが、UIを英語で表示する機能を持っています。
この状況は、たいへん紛らわしいので国際化したいと考えています。
#1782
プログラムの国際化に向けて、修正が必要なコードのテストカバレッジをあげたいです。
PR のメリット
PR のデメリット (トレードオフとかあれば)
仕様・動作説明
アプリの仕様・機能は変更しません。
言語変更時に共有メモリに保持しているローカライズ文字列を書き換える処理に対してテストを追加します。
この機構は、言語変更時に必要な処理を呼び忘れるリスクがあるので、国際化対応を騙るためには排除する必要があると考えています。排除する前に「ちゃんと動いてんだぜ？」を示すためにテストを導入しますい。
PR の影響範囲
テストコードを追加するのみなので、本体への影響はありません。
テスト内容
テスト1
手順
関連 issue, PR
#1782
参考資料