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

PeerConnectionのCreate/Closeでエラーが発生しても処理を継続するように変更 #9

Merged
merged 17 commits into from Dec 15, 2023

Conversation

kiyohome
Copy link
Contributor

@kiyohome kiyohome commented Nov 26, 2023

何の変更を加えましたか?

タイトルの通りです。
全ての変更はCommitsを参照してください。

何を確認しましたか?

実装

  • Frameworkの誤った使い方にすぐに気づけるように、無効な引数や無効なメソッド呼び出しに対するチェックが入っていることを確認しました
  • Framework実行時の動きが分かるように、ログ(Error/Warn/Info/Debug)を出力していることを確認しました
  • 静的解析で問題が見つからないことを確認しました
  • フレームワーク利用者が使うAPI(主にprivate以外)に C# ドキュメントを記述しました
    • 該当箇所なしのためスキップ

テスト

  • 全ての自動テストが成功することを確認しました
    • 自動テストは存在しないためスキップ
  • テストカバレッジが100%になることを確認しました
  • サンプルがあるものはサンプルが動作することを確認しました

変更影響

  • GuideのReleaseページに変更内容が追加されることを確認しました
  • GuideのModuleページ(機能ページ)に変更が反映されることを確認しました
  • GuideのLearningページに変更が反映されることを確認しました
  • Sample Applicationに変更が反映されることを確認しました

レビュアーへのメッセージ

  • 56761ff でカバレッジ通過不要だと思われる箇所へのコメント行追加を行いました
    - 今回追加した例外処理については、Extreal.Integration.Chat.WebRTCのPeerClientHookTestブランチで動作確認を行いました
  • 今回追記した例外処理については、Samples~/MVS/ClientControl/ClientControlScope.csで例外を投げるクラスを呼び出しているため、実行するとカバレッジ通過するようになっています

WebScripts~/src/PeerClient.ts Outdated Show resolved Hide resolved
WebScripts~/yarn.lock Show resolved Hide resolved
WebScripts~/yarn.lock Show resolved Hide resolved
Runtime/NativePeerClient.cs Outdated Show resolved Hide resolved
Samples~/MVS/ClientControl/ClientControlScope.cs Outdated Show resolved Hide resolved
Samples~/MVS/WebGLScripts~/src/FailureClient.ts Outdated Show resolved Hide resolved
Samples~/MVS/WebGLScripts~/src/index.ts Outdated Show resolved Hide resolved
WebScripts~/yarn.lock Show resolved Hide resolved
Samples~/MVS/WebGLScripts~/src/FailureClient.ts Outdated Show resolved Hide resolved
WebScripts~/yarn.lock Show resolved Hide resolved
@sus-taguchi-t sus-taguchi-t merged commit aebd0e1 into develop Dec 15, 2023
1 check passed
@sus-taguchi-t sus-taguchi-t deleted the add-error-handling branch December 15, 2023 01:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants