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

WinMainTestを改良して効率的なテストを実現する #1538

Merged

Conversation

berryzplus
Copy link
Contributor

PR の目的

WinMainTestを改良して実施するテストを改善します。

カテゴリ

  • その他の問題

PR の背景

#1535 であげている通り、単体テストがたまに失敗します。

原因が良く分かっていないのですが、
テスト方法を変更して実行結果を安定させることにより、
失敗の原因を調べるのに役立てることができないか試してみたいと思っています。

PR のメリット

  • プロファイル指定がある場合だけ実行されるコードを確実にテストできるようになります。
  • パラメータテスト化することにより、プロファイル指定がある場合とない場合のテスト条件を揃えることができます。
  • 従来実施できていなかった、INIファイル読み込みのテストができるようになります。

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

  • 根本的な問題(起動シーケンスが複雑すぎて安定したテストが実施できない)を解決するPRではありません。

仕様・動作説明

アプリコードは変更しません。

- 仕様1
従来 コントロールプロセスを起動してからエディタプロセスを起動。
今後 エディタプロセスを直接起動。
- 仕様2
従来 プロファイル指定が空の場合のテストのみ実施。
今後 プロファイル指定に値がある場合のテストも実施。
- 仕様3
従来 INIがない状態からINIを作成するテストのみ実施。
今後 作成したINIを読み取るテストも実施。

PR の影響範囲

単体テストのテスト結果に影響する変更です。
アプリコードを変更しないため、エディタの機能に映鏡はありません。

テスト内容

修正内容に関しては、変更したテストコードのみで担保できると考えています。
ときどきテスト実行が失敗することが分かっているので、一旦Draftで作成します。

関連 issue, PR

#1535

参考資料

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@berryzplus
Copy link
Contributor Author

とりあえずエラーにはならなかったのでDraft外します。

Code Smell A 2 Code Smells

👆については修正理由の説明がめんどくさいので、一旦放置します。

@berryzplus berryzplus marked this pull request as ready for review February 11, 2021 11:00
@AppVeyorBot
Copy link

@sanomari
Copy link
Contributor

Code Smell A 2 Code Smells

👆については修正理由の説明がめんどくさいので、一旦放置します。

これ対応して良いです?

@berryzplus
Copy link
Contributor Author

Code Smell A 2 Code Smells

👆については修正理由の説明がめんどくさいので、一旦放置します。

これ対応して良いです?

どうぞ。

@berryzplus
Copy link
Contributor Author

とりあえず、マージしちゃいます。

@berryzplus berryzplus merged commit 87dcee7 into sakura-editor:master Feb 11, 2021
@berryzplus berryzplus deleted the feature/refactoring_of_winmaintest branch February 11, 2021 12:36
@sanomari
Copy link
Contributor

Code Smell A 2 Code Smells

👆については修正理由の説明がめんどくさいので、一旦放置します。

これ対応して良いです?

どうぞ。

着手しかけましたがやめておきます。

@beru beru added the UnitTest label Mar 21, 2021
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