Add navigation test for TutorialPage3ViewModel #286

kazuhiro4949 · 2021-07-17T08:32:15Z

Issue 番号 / Issue ID

Close ユニットテストの拡充 (TutorialPage3ViewModel) #285

目的 / 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 · 2021-07-25T10:14:50Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

@@ -0,0 +1,108 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at https://mozilla.org/MPL/2.0/. */


ライセンス通知ﾖｼｯ

Covid19Radar/Tests/Covid19Radar.UnitTests/Covid19Radar.UnitTests.csproj

keiji · 2021-07-25T10:56:32Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

+            mockUserDataService.Reset();
+            mockUserDialogs.Reset();
+            mockNavigationService.Reset();
+            mockLoggerService.Reset();
+            mockTermsUpdateService.Reset();


nitpickですが、ここの並びはクラスのメンバ変数の宣言の並びと合わせておくと、将来的に記述漏れに気づきやすそうです。

keiji · 2021-07-25T11:24:49Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

+                .Returns(Task.Run(() =>
+                {
+                    return true;
+                }));


一行で書いてしまった方がシンプルかもしれません（意図がつかみにくくなる・他と記述を合わせたなどの理由がある場合はこのままで）

Suggested change

.Returns(Task.Run(() =>

{

return true;

}));

.Returns(Task.Run(() => true));

keiji · 2021-07-25T11:25:24Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

+                .Returns(Task.Run(() =>
+                {
+                    return false;
+                }));


ここも同様につぎのようにも書けます。

Suggested change

.Returns(Task.Run(() =>

{

return false;

}));

.Returns(Task.Run(() => false));

keiji · 2021-07-25T11:26:05Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
+
+using System;


使っていないようなので削除してください。

keiji · 2021-07-25T11:26:22Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

+using Covid19Radar.ViewModels;
+using Moq;
+using Prism.Navigation;
+using Xamarin.Essentials;


使っていないようなので削除してください。

kazuhiro4949 · 2021-08-07T11:45:27Z

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

ghost · 2022-04-18T23:47:36Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

+        }
+
+        [Fact]
+        public void Dispose()


UserDialogs.Instanceのnullクリアが必要です。
逆にほかは毎回初期化されるものなので、ある分には問題ありませんが無くても良いです。

ghost · 2022-04-18T23:49:03Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

+            mockTermsUpdateService = mockRepository.Create<ITermsUpdateService>();
+        }
+
+        [Fact]


テストコード（テスト用のメソッド）ではないので[Fact]は不要です。

ghost · 2022-04-18T23:49:26Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

+
+namespace Covid19Radar.UnitTests.ViewModels
+{
+    public class TutorialPage3ViewModelTests


IDisposableを継承する必要があります。（Dispose()が呼ばれません）

ghost · 2022-04-18T23:52:55Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

+
+            mockUserDataService
+                .Setup(product => product.RegisterUserAsync())
+                .Returns(Task.Run(() => { return true; }));


非同期メソッドの戻り値の設定にはReturnsAsync()が使えるかと思います。

ghost · 2022-04-18T23:53:55Z

Covid19Radar/Tests/Covid19Radar.UnitTests/ViewModels/Tutorial/TutorialPage3ViewModelTests.cs

+            mockUserDialogs.Verify(x => x.HideLoading(), Times.Once());
+            mockUserDialogs.Verify(x => x.AlertAsync(It.IsAny<string>(), It.IsAny<string>(), "OK", null), Times.Never());
+
+            mockNavigationService.Verify(x => x.NavigateAsync("PrivacyPolicyPage"), Times.Once);


Times.OnceはメソッドですのでTimes.Once()でお願いします。
※他も同様の箇所の対応をお願いします

keiji · 2022-04-20T03:48:36Z

まずはじめにこちらcocoa-mhlwのブランチにマージして作業を続けます。

kazuhiro4949 added 2 commits July 17, 2021 17:19

Add navigation test for TutorialPage3ViewModel

2c71e4e

Fix test case on TutorialPage3ViewModelTests

91ef53f

keiji assigned kazuhiro4949 Jul 18, 2021

keiji self-requested a review July 18, 2021 09:54

keiji reviewed Jul 25, 2021

View reviewed changes

レビュー指摘修正

439763d

kazuhiro4949 marked this pull request as draft August 7, 2021 11:45

ghost reviewed Apr 18, 2022

View reviewed changes

keiji self-assigned this Apr 20, 2022

keiji changed the base branch from develop to feature/kazuhiro4949_tutorialpage3viewmodel_naivation_test April 20, 2022 03:53

keiji marked this pull request as ready for review April 20, 2022 03:54

keiji merged commit 89b9604 into cocoa-mhlw:feature/kazuhiro4949_tutorialpage3viewmodel_naivation_test Apr 20, 2022

keiji mentioned this pull request Apr 20, 2022

Add navigation test for TutorialPage3ViewModel #983

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add navigation test for TutorialPage3ViewModel #286

Add navigation test for TutorialPage3ViewModel #286

kazuhiro4949 commented Jul 17, 2021 •

edited

Loading

keiji Jul 25, 2021

keiji Jul 25, 2021

keiji Jul 25, 2021

keiji Jul 25, 2021

keiji Jul 25, 2021

keiji Jul 25, 2021

kazuhiro4949 commented Aug 7, 2021

ghost Apr 18, 2022

ghost Apr 18, 2022 •

edited by ghost

Loading

ghost Apr 18, 2022

ghost Apr 18, 2022

ghost Apr 18, 2022

keiji commented Apr 20, 2022

Add navigation test for TutorialPage3ViewModel #286

Add navigation test for TutorialPage3ViewModel #286

Conversation

kazuhiro4949 commented Jul 17, 2021 • edited Loading

Issue 番号 / Issue ID

目的 / Purpose

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

Pull Request の種類 / Pull Request type

検証方法 / How to test

コードの入手 / Get the code

コードの検証 / Test the code

確認事項 / What to check

その他 / Other information

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kazuhiro4949 commented Aug 7, 2021

Choose a reason for hiding this comment

ghost Apr 18, 2022 • edited by ghost Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

keiji commented Apr 20, 2022

kazuhiro4949 commented Jul 17, 2021 •

edited

Loading

ghost Apr 18, 2022 •

edited by ghost

Loading