Skip to content
This repository has been archived by the owner on Apr 12, 2023. It is now read-only.

Add navigation test for TutorialPage3ViewModel #983

Merged

Conversation

keiji
Copy link
Collaborator

@keiji keiji commented Apr 20, 2022

#286 の引き継ぎ

Issue 番号 / Issue ID

目的 / Purpose

ライセンス同意画面ではライセンスの同意後に通信を行い、成功時と失敗時に表示を出し分けています。
ここの分岐部分の仕様に関してユニットテストを追加しました。

破壊的変更をもたらしますか / Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request の種類 / Pull Request type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other... Please describe: Unit Test

検証方法 / How to test

以下で今回作成したテストが通るのを確認しています

dotnet test Covid19Radar/Tests/Covid19Radar.UnitTests

コードの入手 / Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
dotnet restore

コードの検証 / Test the code

dotnet test Covid19Radar/Tests/Covid19Radar.UnitTests

確認事項 / What to check

  • Xamarinの開発環境にあまりなれていないため、コーディングスタイルやテストの実装方法、ユニットテストの検証項目に不備がないか確認いただけると嬉しいです

その他 / Other information

@keiji keiji self-assigned this Apr 20, 2022
@keiji keiji requested a review from a user April 20, 2022 03:59
@sonarcloud
Copy link

sonarcloud bot commented Apr 20, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

No Coverage information No Coverage information
No Duplication information No Duplication information

@keiji keiji marked this pull request as ready for review April 20, 2022 04:28
@ghost
Copy link

ghost commented Apr 20, 2022

@keiji
元のPRでコメントされていた

何回かに一回テストがコケることが発覚したので一旦draftに戻しますmm (原因不明...)
#286 (comment)

の件ですが、コードを見た限りは問題ないように思います。
何回かテストし不安定なことがなければ問題ないかと思います。

@ghost
Copy link

ghost commented Apr 20, 2022

@keiji 元のPRでコメントされていた

何回かに一回テストがコケることが発覚したので一旦draftに戻しますmm (原因不明...)
#286 (comment)

の件ですが、コードを見た限りは問題ないように思います。 何回かテストし不安定なことがなければ問題ないかと思います。

CIを何回か実行し直してみましたが、特に問題はないようです。
テストコードとしても、特別変わったことはしていませんしおそらく問題ないですね。

@ghost ghost merged commit e657405 into develop Apr 20, 2022
@keiji keiji deleted the feature/kazuhiro4949_tutorialpage3viewmodel_naivation_test branch April 20, 2022 13:16
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ユニットテストの拡充 (TutorialPage3ViewModel)
2 participants