マイグレーション処理のタイムゾーン補正に起因する不具合を修正する

[keiji]

Issue 番号 / Issue ID

• マイグレーション処理を改善する #520
• Close TimeZoneInfo の取得方法を統一する #554

目的 / Purpose

• JSTの文字列からUTCのDateTimeへの変換を安全に行う手段を検討する #523
• マイグレーションのユニットテストを拡充する #524
• Migrator_1_2_2. MigrateTermAsyncに警告コメントを記述する #522

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

[x] Yes
[ ] No

Pull Request の種類 / Pull Request type

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

検証方法 / How to test

コードの入手 / Get the code

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

コードの検証 / Test the code

確認事項 / What to check

• 追加したテストでマイグレーション処理の問題を正しく検出できる（ Reproduce a critical bug that migration failed. #527 ）
  • 追加したテストを実行した結果、正しく失敗を検出できることを確認した。
  • https://github.com/cocoa-mhlw/cocoa/runs/4344513219?check_suite_focus=true

[xUnit.net 00:00:10.96]     Covid19Radar.UnitTests.Services.Migration.MigrationServiceTests.Migrate_100to130Async [FAIL]
  Failed Covid19Radar.UnitTests.Services.Migration.MigrationServiceTests.Migrate_100to130Async [1 s]
  Error Message:
   System.ArgumentOutOfRangeException : The added or subtracted value results in an un-representable DateTime. (Parameter 't')
  Stack Trace:
     at System.DateTime.op_Addition(DateTime d, TimeSpan t)
   at Covid19Radar.Services.Migration.Migrator_1_3_0.MigrateDateTimeToEpoch(String dateTimeKey, String epochKey, TimeSpan differential) in /home/runner/work/cocoa/cocoa/Covid19Radar/Covid19Radar/Services/Migration/Migrator_1_3_0.cs:line 67
   at Covid19Radar.Services.Migration.Migrator_1_3_0.ExecuteAsync() in /home/runner/work/cocoa/cocoa/Covid19Radar/Covid19Radar/Services/Migration/Migrator_1_3_0.cs:line 41
   at Covid19Radar.Services.Migration.MigrationService.MigrateAsync(Version fromVersion) in /home/runner/work/cocoa/cocoa/Covid19Radar/Covid19Radar/Services/Migration/MigrationService.cs:line 221
   at Covid19Radar.Services.Migration.MigrationService.MigrateAsync() in /home/runner/work/cocoa/cocoa/Covid19Radar/Covid19Radar/Services/Migration/MigrationService.cs:line 136
   at Covid19Radar.UnitTests.Services.Migration.MigrationServiceTests.Migrate_100to130Async() in /home/runner/work/cocoa/cocoa/Covid19Radar/Tests/Covid19Radar.UnitTests/Services/Migration/MigrationServiceTests.cs:line 884
--- End of stack trace from previous location where exception was thrown ---

Failed!  - Failed:     1, Passed:   246, Skipped:     0, Total:   247, Duration: 13 s - /home/runner/work/cocoa/cocoa/Covid19Radar/Tests/Covid19Radar.UnitTests/bin/Debug/netcoreapp3.1/Covid19Radar.UnitTests.dll (netcoreapp3.1)

• 修正でテストが通る（ Fix an issue that migration failed when DateTime is DateTime.MinValue. #525 ）
  • https://github.com/cocoa-mhlw/cocoa/runs/4345975331?check_suite_focus=true

Passed!  - Failed:     0, Passed:   247, Skipped:     0, Total:   247, Duration: 13 s - /home/runner/work/cocoa/cocoa/Covid19Radar/Tests/Covid19Radar.UnitTests/bin/Debug/netcoreapp3.1/Covid19Radar.UnitTests.dll (netcoreapp3.1)

その他 / Other information

[keiji]

このPull Requestの内容は変更の必要がある。

Hotfix #518 では 0001/01/01 00:00:00のまま、タイムゾーン補正時にエラーをcatchする方式を採用した。
この場合、v1.3.0のマイグレーションを終えても 0001/01/01 00:00:00 のままとなる。

一方、ここで提案している変更だとタイムゾーン補正時に 0001/01/01 00:00:00 が来た場合 2020/06/19 09:00:00に置き換える操作が加わり、現在のv1.3.0マイグレーションと結果が変わる。

すでにHotfixがリリースされて端末上でマイグレーション処理が実行されているので、v1.3.0にこのPull Requestの変更を適用すると、規約合意日時が0001/01/01 00:00:00になっている利用者がいるという事実を伝える手段がなくなってしまう。

• 0001/01/01 00:00:00のままという結果は変えない
• タイムゾーン補正の方法のみを変更する
• マイグレーションと2020/06/19 09:00:00への置き換えは、v1.4.x またはCOCOA2で再検討する

[ghost]

@keiji
すみませんが、Confrictの解消もお願いします。

[cocoa-dev005]

LGTM :)
実機でマイグレーション処理のテストを行うので、もしロジックに問題があってもそこで発見できると思います。